Timeline


and

10/18/06:

23:17 Changeset [17130] by ggaren

Test for some window properties that WebCore previously didn't enumerate.

  • fast/js/window-properties2-expected.txt: Added.
  • fast/js/window-properties2.html: Added.
22:09 Changeset [17129] by thatcher

Reviewed by Tim H.

Improve the doubleclick behavior of breakpoints, and make breakpoints with no custom condition set appear blank instead of return [-1, 1] depending on enabled state.

  • Drosera/DebuggerDocument.m: (-[WebScriptObject getDoubleClickMillis]): Add an ObjC wrapper for GetDblTime() so that JS can call it
  • Drosera/Drosera.xcodeproj/project.pbxproj: link Carbon for GetDblTime()
  • Drosera/debugger.js:
22:02 Changeset [17128] by thatcher

Reviewed by Anders.

Bug 10851: Crash with Drosera
http://bugs.webkit.org/show_bug.cgi?id=10851

This crash results in an assert in debug builds.

assert(implementsCall());

The drosera_introspection propery was not callable. Now we just assign
this.
drosera_introspection in the evaluateWebScript call.
This change also removes one DO message.

  • Drosera/DebuggerDocument.m: (-[WebScriptObject webScriptAttributeKeysForScriptObject:]):
19:42 Changeset [17127] by mjs

JavaScriptCore:

Reviewed by Geoff.


  • remove vestiges of KXMLCore name (former name of WTF).
  • wtf/Assertions.h:
  • wtf/FastMalloc.h: (operator new): (operator delete): (operator new[]): (operator delete[]):
  • wtf/FastMallocInternal.h:
  • wtf/Forward.h:
  • wtf/GetPtr.h:
  • wtf/HashCountedSet.h:
  • wtf/HashFunctions.h:
  • wtf/HashMap.h:
  • wtf/HashSet.h:
  • wtf/HashTable.h:
  • wtf/HashTraits.h:
  • wtf/ListRefPtr.h:
  • wtf/MathExtras.h:
  • wtf/Noncopyable.h:
  • wtf/OwnArrayPtr.h:
  • wtf/OwnPtr.h:
  • wtf/PassRefPtr.h:
  • wtf/Platform.h:
  • wtf/RefPtr.h:
  • wtf/StringExtras.h: (snprintf):
  • wtf/UnusedParam.h:
  • wtf/Vector.h:
  • wtf/VectorTraits.h:

WebCore:

Reviewed by Geoff.

  • remove vestiges of KXMLCore name (former name of WTF).
  • config.h:
18:45 Changeset [22383] by mjs

Reviewed by Geoff.

  • remove remnants of KXMLCore name.
  • WebCore.vcproj/WebCore.vcproj:
  • config.h:
18:43 Changeset [17126] by ggaren

LayoutTests:

Reviewed by Beth.


Updated some results now that some global JS objects repeatably report their
statically declared properties.

  • fast/js/window-properties-expected.txt:
  • fast/js/window-properties.html:

WebCore:

Reviewed by Beth.


Fixed http://bugs.webkit.org/show_bug.cgi?id=11222

Some global JS objects lacked references to their property hashtables,
so for-in enumeration didn't work. I added the references, and then removed
a few bogus properties from the tables. The bogus ones got in there through
bit rot.


We should come up with some solution to these hand-written bindings bugs.
Maybe we could use a script to autogenerate them or something.

  • bindings/js/kjs_window.cpp: (KJS::): (KJS::History::getValueProperty):
18:05 Changeset [17125] by aroben

Reviewed by Maciej.

Fix: http://bugs.webkit.org/show_bug.cgi?id=11342
fast/repaint/4774354 is failing

It seems that this test got checked in with incorrect results, as it
has been failing since its introduction in r17093.

  • fast/repaint/4774354-expected.txt:
17:11 Changeset [17124] by aroben

LayoutTests:

Reviewed by Brady.

New test to make sure we're generating the correct keycodes for Tab
and Shift-Tab keypresses.

  • fast/events/js-keyboard-event-creation-expected.txt: Added.
  • fast/events/js-keyboard-event-creation.html: Added.

WebCore:

Reviewed by Brady.

Change Shift-Tab to generate keyIdentifier U+000009 (Tab) instead of
U+000019 (Backtab). This matches our windowsKeyCode for this key
combination.

  • platform/mac/KeyEventMac.mm: (WebCore::keyIdentifierForKeyEvent):
16:28 Changeset [17123] by sfalken

2006-10-18 Steve Falkenburg <sfalken@apple.com>

Reviewed by Adam, Brady.

Loader tweaks.

  • platform/cf/ResourceLoaderCFNet.cpp: (WebCore::didFinishLoading): (WebCore::didFail): (WebCore::didReceiveChallenge): (WebCore::runLoaderThread): (WebCore::ResourceLoader::start): (WebCore::ResourceLoader::cancel):
16:15 Changeset [17122] by aroben

Reviewed by Brady.

Fix: http://bugs.webkit.org/show_bug.cgi?id=11344
REGRESSION (r16741): webViewDidChangeSelection delegate not firing in
2 layout tests

The behavior for display-none-in-onchange-keyboard is still correct
(we don't crash when the element about to receive focus gets
display:none set), so I've just generated new results for it.

The behavior of tabbing-input-iframe has changed a little. iframe
elements are now in the tab order, so I've changed the test to take
that into account and generated new results.

  • fast/forms/display-none-in-onchange-keyboard-expected.txt:
  • fast/forms/tabbing-input-iframe-expected.txt:
  • fast/forms/tabbing-input-iframe.html:
15:56 Changeset [17121] by thatcher

Reviewed by Darin.

<rdar://problem/4752138> Manipulating popup in HTML page crashed Xcode

  • Store the Mac popup button cell as a RetainPtr to prevent GC collection.
  • Convert more ObjC local statics and member variables to use RetainPtr.
  • Convert more CFRetain/CFRelease to HardRetain/HardRelease.
  • bindings/objc/DOM.mm: (ObjCNodeFilterCondition::ObjCNodeFilterCondition): use HardRetain and not CFRetain (ObjCNodeFilterCondition::~ObjCNodeFilterCondition): use HardRelease and not CFRelease
  • bridge/mac/AXObjectCacheMac.mm: (WebCore::AXObjectCache::~AXObjectCache): use HardRelease and not CFRelease (WebCore::AXObjectCache::get): use HardRetain and not CFRetain (WebCore::AXObjectCache::remove): use HardRelease and not CFRelease
  • bridge/mac/WebCoreFrameBridge.mm: (+[WebCoreFrameBridge supportedImageResourceMIMETypes]): use RetainPtr (+[WebCoreFrameBridge supportedImageMIMETypes]): ditto
  • bridge/mac/WebCoreIconDatabaseBridge.mm: (+[WebCoreIconDatabaseBridge sharedInstance]): use RetainPtr
  • platform/PopupMenu.h: renamed popup to m_popup and switched to RetainPtr<NSPopUpButtonCell>
  • platform/cf/RetainPtr.h: (WTF::::operator): implement a missing operator= template function
  • platform/mac/PopupMenuMac.mm: (WebCore::PopupMenu::PopupMenu): remove nil initialization (WebCore::PopupMenu::~PopupMenu): use .get(), remove release message and rename to m_popup (WebCore::PopupMenu::clear): use .get() when sending a message and rename to m_popup (WebCore::PopupMenu::populate): ditto (WebCore::PopupMenu::show): ditto (WebCore::PopupMenu::hide): ditto (WebCore::PopupMenu::addSeparator): ditto (WebCore::PopupMenu::addGroupLabel): ditto (WebCore::PopupMenu::addOption): ditto
15:52 Changeset [22382] by sfalken

2006-10-18 Steve Falkenburg <sfalken@apple.com>

Fix build

  • platform/win/TemporaryLinkStubs.cpp: (BrowserExtensionWin::historyURL): (BrowserExtensionWin::BrowserExtensionWin):
15:45 Changeset [17120] by ggaren

Reviewed by Adele.

Fixed <rdar://problem/3673233> JavaScript timers do not fire if controls
are tracking or menus are down (run loop modes)

I spent a few hours trying to write an automated test, but DRT doesn't
seem to support it. Manual tests attached to related bugs.


  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::passMouseDownEventToWidget):
  • platform/mac/SharedTimerMac.cpp: (WebCore::setSharedTimerFireTime):
14:58 Changeset [22381] by aliceli1

Reviewed by Steve.

Any file that #includes WebView.h also needed to include
IWebURLResponse.h since IWebView.h needed it. Adding this
to the IDL file.


  • Interfaces/IWebDataSource.idl: re-arranged imports because not all were being generated due to weird IDL file compiler bug
  • Interfaces/IWebView.idl: added reference to IWebURLResponse.h


  • WebFrame.cpp:
  • WebKitClassFactory.cpp:
  • WebView.cpp: removed reference to IWebURLResponse.h
14:30 Changeset [17119] by bdash

2006-10-18 Mark Rowe <bdash@webkit.org>

Reviewed by Tim H.

http://bugs.webkit.org/show_bug.cgi?id=11304
Bug 11304: Drosera fails to link as universal binary on PowerPC machine

  • Drosera/Drosera.xcodeproj/project.pbxproj: Use -weak_framework to link against JavaScriptCore and WebCore directly when they are not part of the WebKit umbrella framework.
14:12 Changeset [17118] by weinig

WebCore:

Reviewed by Maciej.

More Win32 buildfix.

  • bridge/win/FrameWin.h:

WebKit:

Reviewed by Maciej.

Win32 build fix.

  • COM/WebFrame.cpp: (WebFrame::createNewWindow):
  • COM/WebFrame.h:
13:51 Changeset [17117] by andersca

2006-10-18 Anders Carlsson <acarlsson@apple.com>

Reviewed by Adam.

http://bugs.webkit.org/show_bug.cgi?id=11000
REGRESSION (r16101): css2.1/t0801-c412-hz-box-00-b-a is failing because the QuickTime plugin is taking over but not rendering the png


  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge determineObjectFromMIMEType:URL:]): Return ObjectElementFrame if the MIME type is one of the image ones we support.
13:17 Changeset [17116] by weinig

Reviewed by Mitz

Win32 build fix.

  • bridge/win/BrowserExtensionWin.cpp: (WebCore::BrowserExtensionWin::BrowserExtensionWin): (WebCore::BrowserExtensionWin::createNewWindow):
  • bridge/win/BrowserExtensionWin.h:
  • bridge/win/FrameWin.cpp: (WebCore::FrameWin::createNewWindow):
  • bridge/win/FrameWin.h:
  • platform/win/CursorWin.cpp:
  • platform/win/TemporaryLinkStubs.cpp: (Image::getHBITMAP):
12:35 Changeset [17115] by aroben

fixo el buildo II: Release's Pride.

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
12:27 Changeset [17114] by justing

LayoutTests:

Reviewed by harrison

<rdar://problem/4784823>
GMail Editor: Hang occurs when removing list styling on text in a rich text message

  • editing/execCommand/remove-list-1-expected.checksum: Added.
  • editing/execCommand/remove-list-1-expected.png: Added.
  • editing/execCommand/remove-list-1-expected.txt: Added.
  • editing/execCommand/remove-list-1.html: Added.

WebCore:

Reviewed by harrison


<rdar://problem/4784823>
GMail Editor: Hang occurs when removing list styling on text in a rich text message

List removal moves the contents of every list item out of the list it's
in. When the code tried to move the contents of an empty list item (an li
with no child nodes, not even a placeholder br), moveParagraph didn't prune
the li, like it would if the li had a placeholder inside it. So the list
removal code went into an infinite loop, continually attempting to de-list
an empty list item.

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphs): If the selection to move was empty and in an empty block that doesn't require a placeholder to prop itself open (like a bordered div or an li), remove it during the move.
12:22 Changeset [17113] by justing

LayoutTests:

Reviewed by harrison

<rdar://problem/4765600>
REGRESSION: Mail.app: smart deletion of words does not work

  • editing/deleting/smart-delete-003-expected.checksum: Added.
  • editing/deleting/smart-delete-003-expected.png: Added.
  • editing/deleting/smart-delete-003-expected.txt: Added.
  • editing/deleting/smart-delete-003.html: Added.
  • editing/deleting/smart-delete-004-expected.checksum: Added.
  • editing/deleting/smart-delete-004-expected.png: Added.
  • editing/deleting/smart-delete-004-expected.txt: Added.
  • editing/deleting/smart-delete-004.html: Added.

WebKit:

Reviewed by harrison


<rdar://problem/4765600>
REGRESSION: Mail.app: smart deletion of words does not work


Regressed when we pushed selecion expansion down into WebCore. It's OK
to try a smart delete from _deleteWithDirection:, which is called by
deleteFoward: and deleteBackward: if the current selection is a
range.


  • WebView/WebHTMLView.m: (-[NSArray _deleteWithDirection:granularity:killRing:isTypingAction:]):
11:59 Changeset [17112] by ggaren

fixo el buildo.


Work around #import of <PDFKit/PDFView.h>.

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
11:47 Changeset [17111] by adele

Reviewed by Beth.

When I moved the autoscroll code from FrameMac to Frame, I mistakingly changed the order of
a call to invalidateClick. This change moves it back to the sequence that existed before. This allows
the click count to get properly reset during a selection.

  • page/Frame.cpp: (WebCore::Frame::handleMouseMoveEvent):
11:16 Changeset [22380] by sfalken

2006-10-18 Steve Falkenburg <sfalken@apple.com>

Reviewed by Lou.

<rdar://4781998> Need API for setting the rendering modes


Added font smoothing preference
Fixed some COM interface typos
Removed use of IWebImage (we use HBITMAP instead)


10:54 Changeset [17110] by spadma

2006-10-18 zhalu <zhaosong.lu@nokia.com>

Reviewed by Sachin Padma.
DESC: Don't need to set the focus. The listbox itself will set the focus when receiving key event.
Otherwise, it will cause problem in input. Here is a little explaination: When the form data saving is
enabled, the CControlSelectWidget::FilterContentL is called whenever a 'letter' is committed. The function
calls the the listbox's setFocus(). This setFocus triggers a CommitInlineEditL action in CAknFepManager.
If users press the key in a normal speed, there is no problem because there is nothing for committing.
But if users press two keys(not two letters on the same key) very quickly, the second key will be commited
by the SetFocus() of the first key and the cursor in the editbox will move to the third key position.
The result is that it seems not letting you insert some key in the second position.
http://bugzilla.opendarwin.org/show_bug.cgi?id=11192

  • BrowserCore/Formcontrols/src/FControlSelectWidget.cpp: (CControlSelectWidget::FilterContentL):
10:42 Changeset [17109] by thatcher

Removing SVGZoomEvent.idl from Copy Resources.

10:37 Changeset [17108] by kmccullo

Reviewed by Maciej.


Adjust include paths

09:29 Changeset [17107] by thatcher

Reviewed by Tim H.

Add Xcode 3 style inline breakpoint editor. Credit to xenon for the CSS wizardry to get the appearance working properly.

  • Drosera/Drosera.xcodeproj/project.pbxproj:
  • Drosera/Images/breakpointeditor.png: Added.
  • Drosera/breakpointEditor.css: Removed.
  • Drosera/breakpointEditor.html: Removed.
  • Drosera/breakpointEditor.js: Removed.
  • Drosera/debugger.js:
  • Drosera/viewer.css:
08:25 Changeset [17106] by thatcher

Reviewed by Tim H.

Bug 11341: REGRESSION (r16760): editing/selection/editable-links is failing
http://bugs.webkit.org/show_bug.cgi?id=11341

Link editing behavior became a preference. DumpRenderTree needs to specify the
non-default behavior it wants (WebKitEditableLinkOnlyLiveWithShiftKey).

  • DumpRenderTree/DumpRenderTree.m: (main):
00:45 Changeset [17105] by aroben

Reviewed by Maciej.

Fix: http://bugs.webkit.org/show_bug.cgi?id=11340
editing/pasteboard/4641033 needs new results

Update results after r17071.

  • editing/pasteboard/4641033-expected.txt:

10/17/06:

23:04 Changeset [17104] by ggaren

Reviewed by Anders.


Factored common scaling code into a helper function.

  • platform/mac/ScreenMac.mm: (WebCore::scale): (WebCore::scaleScreenRectToPageCoordinates): (WebCore::scalePageRectToScreenCoordinates):
22:52 Changeset [22379] by sfalken

2006-10-17 Steve Falkenburg <sfalken@apple.com>

Reviewed by ggaren.

<rdar://4781999> History menu gets borked if you visit a page with no title.
<rdar://4782002> The History menu is frequently utterly corrupted, with incorrect favicons next to mismatched titles.
<rdar://4780252> Page titles in History menu are incorrect
<rdar://4760334> Pages without title show an old title instead of a default title


2006-10-17 Steve Falkenburg <sfalken@apple.com>

Reviewed by ggaren.

<rdar://4781999> History menu gets borked if you visit a page with no title.
<rdar://4782002> The History menu is frequently utterly corrupted, with incorrect favicons next to mismatched titles.
<rdar://4780252> Page titles in History menu are incorrect
<rdar://4760334> Pages without title show an old title instead of a default title

  • Interfaces/IWebHistoryItemPrivate.idl: added setTitle to set title for a page after we receive it
  • WebFrame.cpp: Removed unnecessary WebFramePrivate::title (WebFrame::goToItem): Get back/forward list via frame-aware getter (WebFrame::receivedRedirect): Use BString (WebFrame::receivedResponse): Use BString. Get back/forward list, history via frame-aware getters. (WebFrame::createFrame): Use BString (WebFrame::submitForm): Clean up up BSTR usage (WebFrame::setTitle): Set title in back/forward, history when received (WebFrame::backForwardList): Frame-aware getter for back/forward (WebFrame::webHistory): Frame-aware getter for history
  • WebFrame.h: Added frame-aware getters for history, back/forward
  • WebHistory.cpp: Reorder includes. (WebHistory::QueryInterface): Support QI to WebHistory via CLSID (avoids yucky static cast in other places)
  • WebHistoryItem.cpp: (WebHistoryItem::setTitle): Added
  • WebHistoryItem.h: Added setTitle
  • WebView.cpp: (WebView::QueryInterface): Support QI to WebView via CLSID

2006-10-17 Steve Falkenburg <sfalken@apple.com>

Build fix

  • platform/win/TemporaryLinkStubs.cpp: (BrowserExtensionWin::createNewWindow):
22:46 Changeset [17103] by sfalken

2006-10-17 Steve Falkenburg <sfalken@apple.com>

Build fixes

  • bridge/win/BrowserExtensionWin.cpp: (WebCore::BrowserExtensionWin::createNewWindow):
  • bridge/win/BrowserExtensionWin.h:
  • bridge/win/FrameWin.cpp: (WebCore::FrameWin::createNewWindow):
  • bridge/win/FrameWin.h:
  • platform/gdk/BrowserExtensionGdk.h:
  • platform/gdk/TemporaryLinkStubs.cpp: (BrowserExtensionGdk::createNewWindow):
  • platform/qt/BrowserExtensionQt.cpp: (WebCore::BrowserExtensionQt::createNewWindow):
  • platform/qt/BrowserExtensionQt.h:
22:16 Changeset [22378] by ggaren

Reviewed by Anders.


Added WindowFeatures.h to the project.

  • WebCore.vcproj/WebCore.vcproj:
22:13 Changeset [17102] by ggaren

Reviewed by Anders.


Renamed WindowArgs to WindowFeatures, and moved it into its own header.
("Features" is the term of art in IE documentation. "Args" is pirate
talk.)

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/kjs_window.cpp: (KJS::createNewWindow): (KJS::showModalDialog): (KJS::setWindowFeature): (KJS::parseWindowFeatures): (KJS::constrainToVisible): (KJS::WindowFunc::callAsFunction):
  • bridge/BrowserExtension.h:
  • bridge/WindowFeatures.h: Added.
  • bridge/mac/BrowserExtensionMac.h:
  • bridge/mac/BrowserExtensionMac.mm: (WebCore::BrowserExtensionMac::createNewWindow):
21:16 Changeset [22377] by sfalken
21:15 Changeset [22376] by sfalken

2006-10-17 Steve Falkenburg <sfalken@apple.com>

Reviewed by Maciej.

Turn off a very noisy warning (unknown pragma) caused by an Interfacer include

  • WebCore.vcproj/WebCore.vcproj:
21:09 Changeset [17101] by sfalken

2006-10-17 Steve Falkenburg <sfalken@apple.com>

Reviewed by Maciej.


Adjust include paths

20:04 Changeset [17100] by weinig

Reviewed by Tim H.

Patch for http://bugs.webkit.org/show_bug.cgi?id=11268
Implement the Objective-C DOM bindings for SVGNumber, SVGRect, and SVGPoint

  • Adds implementation for DOMSVGNumber, DOMSVGRect, and DOMSVGPoint.
  • Moves the contents of ksvg/bindings/js/ to bindings/js/.
  • Moves SVGZoomEvent.idl from ksvg/svg/ to ksvg/events/.
  • Adds isSVGColor() and isSVGPaint() methods to StyleBase, matching the other CSSValue classing, and allowing the bindings to allocate appropriately.
  • Removes #ifdefs around the getBBox() method in SVGLocatable.idl allowing generation for Objective-C.
  • Assorted style cleanups.
  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOMCSS.mm: (+[DOMCSSValue _CSSValueWith:WebCore::]):
  • bindings/objc/DOMEvents.mm: (+[DOMEvent _eventWith:WebCore::]):
  • bindings/objc/DOMSVG.h:
  • bindings/objc/DOMSVGNumber.mm: (-[DOMSVGNumber dealloc]): (-[DOMSVGNumber finalize]): (-[DOMSVGNumber value]): (-[DOMSVGNumber setValue:]): (-[DOMSVGNumber _SVGNumber]): (-[DOMSVGNumber _initWithFloat:]): (+[DOMSVGNumber _SVGNumberWith:]):
  • bindings/objc/DOMSVGPoint.mm: (-[DOMSVGPoint dealloc]): (-[DOMSVGPoint finalize]): (-[DOMSVGPoint x]): (-[DOMSVGPoint setX:]): (-[DOMSVGPoint y]): (-[DOMSVGPoint setY:]): (-[DOMSVGPoint WebCore::]): (-[DOMSVGPoint _initWithFloatPoint:WebCore::]): (+[DOMSVGPoint _SVGPointWith:WebCore::]):
  • bindings/objc/DOMSVGRect.mm: (-[DOMSVGRect dealloc]): (-[DOMSVGRect finalize]): (-[DOMSVGRect x]): (-[DOMSVGRect setX:]): (-[DOMSVGRect y]): (-[DOMSVGRect setY:]): (-[DOMSVGRect width]): (-[DOMSVGRect setWidth:]): (-[DOMSVGRect height]): (-[DOMSVGRect setHeight:]): (-[DOMSVGRect WebCore::]): (-[DOMSVGRect _initWithFloatRect:WebCore::]): (+[DOMSVGRect _SVGRectWith:WebCore::]):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • css/StyleBase.h: (WebCore::StyleBase::isSVGColor): (WebCore::StyleBase::isSVGPaint):
  • ksvg2/bindings/js/JSSVGElementWrapperFactory.cpp: Removed.
  • ksvg2/bindings/js/JSSVGElementWrapperFactory.h: Removed.
  • ksvg2/bindings/js/JSSVGNumber.cpp: Removed.
  • ksvg2/bindings/js/JSSVGNumber.h: Removed.
  • ksvg2/bindings/js/JSSVGPoint.cpp: Removed.
  • ksvg2/bindings/js/JSSVGPoint.h: Removed.
  • ksvg2/bindings/js/JSSVGRect.cpp: Removed.
  • ksvg2/bindings/js/JSSVGRect.h: Removed.
  • ksvg2/svg/SVGColor.h: (WebCore::SVGColor::isSVGColor):
  • ksvg2/svg/SVGFECompositeElement.idl:
  • ksvg2/svg/SVGFEFloodElement.idl:
  • ksvg2/svg/SVGLocatable.idl:
  • ksvg2/svg/SVGMarkerElement.idl:
  • ksvg2/svg/SVGPaint.h: (WebCore::SVGPaint::isSVGPaint):
  • ksvg2/svg/SVGRectElement.idl:
  • ksvg2/svg/SVGZoomEvent.idl: Removed.
19:12 Changeset [17099] by hyatt

Fix changelog file list. It wasn't right.

19:10 Changeset [17098] by hyatt

Prepare for the elimination of the RenderLayer for <html> elements. The layer is still present,
but this patch makes everything work without a root layer being around.

Reviewed by beth

  • WebCore.xcodeproj/project.pbxproj:
  • platform/KURL.cpp: Removed.
  • platform/KURL.h: Removed.
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paint): (WebCore::RenderBlock::paintChildren): (WebCore::RenderBlock::nodeAtPoint):
  • rendering/RenderLayer.cpp: (WebCore::frameVisibleRect): (WebCore::RenderLayer::hitTest): (WebCore::RenderLayer::hitTestLayer): (WebCore::RenderLayer::intersectsDamageRect):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::setInnerNode):
  • rendering/RenderView.cpp: (WebCore::RenderView::paint):
18:56 Changeset [17097] by hyatt

Back out my previous patch to the WebCore memory cache.

  • loader/Cache.cpp: (WebCore::Cache::flush): (WebCore::Cache::setSize): (WebCore::Cache::getLRUListFor): (WebCore::Cache::removeFromLRUList): (WebCore::Cache::insertInLRUList):
  • loader/Cache.h:
  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::deref):
  • loader/CachedCSSStyleSheet.h:
  • loader/CachedImage.cpp: (WebCore::CachedImage::deref):
  • loader/CachedImage.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::setRequest): (WebCore::CachedResource::deref):
  • loader/CachedResource.h: (WebCore::CachedResource::allowInLRUList):
  • loader/CachedScript.cpp: (WebCore::CachedScript::deref):
  • loader/CachedScript.h:
  • loader/CachedXBLDocument.cpp: (WebCore::CachedXBLDocument::deref):
  • loader/CachedXBLDocument.h:
  • loader/CachedXSLStyleSheet.cpp: (WebCore::CachedXSLStyleSheet::deref):
  • loader/CachedXSLStyleSheet.h:
18:31 Changeset [17096] by oliver

2006-10-17 Oliver Hunt <oliver@apple.com>

Reviewed by Anders.

Make north and east resize cursors use correct image

  • platform/win/CursorWin.cpp: (WebCore::eastResizeCursor): (WebCore::northResizeCursor):


17:39 Changeset [22375] by oliver

2006-10-17 Oliver Hunt <oliver@apple.com>

Reviewed by Brady.

Fixed alpha in favicons, added composite operation conversion
for cg


rdar://problem/4758934

  • platform/win/GraphicsContextWin.cpp: (WebCore::GraphicsContext::setCompositeOperation):
  • platform/win/ImageWin.cpp: (WebCore::Image::getHBITMAP):
  • platform/win/TemporaryLinkStubs.cpp: (GraphicsContext::drawLineForMisspelling):


Fix flickering cursor

  • WebView.cpp: (registerWebViewWindowClass):
17:30 Changeset [17095] by oliver

2006-10-17 Oliver Hunt <oliver@apple.com>

Reviewed by Brady

Various cursor fixes

  • page/FrameView.cpp: (WebCore::FrameView::handleMouseMoveEvent):
  • platform/Cursor.h:
  • platform/mac/CursorMac.mm: (WebCore::pointerCursor):
  • platform/win/CursorWin.cpp: (WebCore::supportsAlphaCursors): (WebCore::Cursor::Cursor): (WebCore::Cursor::~Cursor): (WebCore::pointerCursor):
17:28 Changeset [17094] by justing

LayoutTests:

Reviewed by harrison

<rdar://problem/4776665>
REGRESSION: In Mail, caret is visible when the message body isn't focused

  • editing/selection/4776665-expected.checksum: Added.
  • editing/selection/4776665-expected.png: Added.
  • editing/selection/4776665-expected.txt: Added.
  • editing/selection/4776665.html: Added.

WebCore:

Reviewed by harrison


<rdar://problem/4776665>
REGRESSION: In Mail, caret is visible when the message body isn't focused

  • page/Frame.cpp: (WebCore::Frame::paintCaret): If a caret blink timer is ever on in error (like when the frame that contains the caret isn't focused) then we don't want to paint the caret. We should eventually move to using a single bool for caret painting.
16:52 Changeset [17093] by justing

LayoutTests:

Reviewed by harrison


<rdar://problem/4774354>
REGRESSION: In Gmail, caret reappears when dragging a selection over some text in the message body

  • fast/repaint/4774354-expected.checksum: Added.
  • fast/repaint/4774354-expected.png: Added.
  • fast/repaint/4774354-expected.txt: Added.
  • fast/repaint/4774354.html: Added.
  • fast/repaint/resources/contenteditable-iframe-src.html: Added.

WebCore:

Reviewed by harrison


<rdar://problem/4774354>
REGRESSION: In Gmail, caret reappears when dragging a selection over some text in the message body

Functions call recomputCaretRect to see if the caret rect has changed,
and, if it has, they invalidate (repaint) it. recomputeCaretRect was
returning false if the caret turned into a range selection or was blown
away.

  • editing/SelectionController.cpp: (WebCore::SelectionController::recomputeCaretRect): Don't early return if !isCaret(). Obtain the FrameView* using the m_frame pointer, because m_sel may now be null.
16:37 Changeset [17092] by ggaren

Reviewed by Brady.

BrowserExtension cleanup.


  • Removed unnecessary #include of BrowserExtension in Frame.h, added #includes to compensate in other files.


  • Removed unused versions of createNewWindow.


  • bindings/js/kjs_window.cpp:
  • bridge/BrowserExtension.h:
  • bridge/mac/BrowserExtensionMac.h:
  • bridge/mac/BrowserExtensionMac.mm: (WebCore::BrowserExtensionMac::createNewWindow):
  • bridge/mac/WebCoreFrameBridge.mm:
  • html/HTMLFormElement.cpp:
  • loader/mac/LoaderFunctionsMac.mm:
  • page/Frame.h:
  • page/FramePrivate.h:
14:46 Changeset [17091] by kmccullo

Reviewed by Brady.

Fixes a JavaScriptCore math issue on win.

  • kjs/math_object.cpp: (MathFuncImp::callAsFunction):
  • wtf/MathExtras.h: (wtf_atan2):
14:20 Changeset [22374] by ggaren

Reviewed by Adam.

Added HTMLFrameElementBase to the project.


  • WebCore.vcproj/WebCore.vcproj:
14:17 Changeset [17090] by ggaren

Reviewed by Adam.


  • Fixed "FIXME: Violates Liskov Substitution principle."


IFRAME was never a true subclass of FRAME, since they disagreed on a
number of behaviors, which, confusingly, IFRAME just overrode and contradicted.

The fix here is to factor out things common to IFRAME and FRAME into a
common base class, HTMLFrameElementBase.


I also changed some " *" to "* " and added #includes in files that implicitly
assumed that HTMLIFrameElement.h #included HTMLFrameElement.h.

12:47 Changeset [17089] by justing

LayoutTests:

Reviewed by sullivan


<rdar://problem/4776765>
REGRESSION: Caret's ghost left behind after inserting a paragraph separator (11237)

  • fast/repaint/4776765-expected.checksum: Added.
  • fast/repaint/4776765-expected.png: Added.
  • fast/repaint/4776765-expected.txt: Added.
  • fast/repaint/4776765.html: Added.

WebCore:

Reviewed by sullivan


<rdar://problem/4776765>
REGRESSION: Caret's ghost left behind after inserting a paragraph separator (11237)

We set m_needsLayout to false and call caretRect() in the hopes that it will give us
the old caret rect. It in fact corrects the caret rect for an offset that it
believes is due to scrolling but which is actually due to a change in selection
without an accompanying layout. So it returns the new caret rect regardless of
what m_needsLayout is set to.


  • editing/SelectionController.cpp: (WebCore::repaintRectForCaret): Moved the code from caretRepaintRect that adds a one pixel slop to this new function. (WebCore::SelectionController::caretRepaintRect): Moved this code to repaintRectForCaret. (WebCore::SelectionController::recomputeCaretRect): Compare the old caret rect to the new one that's computed with a fresh layout. If they are different, invalidate both repaint rects.
12:11 Changeset [17088] by thatcher

Tag for WebKit 312.8.1 (part 2 of 2.)

12:11 Changeset [17087] by thatcher

Tag for WebKit 312.8.1 (part 1 of 2.)

12:11 Changeset [17086] by thatcher

Versioning to 312.8.1

12:08 Changeset [17085] by thatcher

Merged fix from Safari-2-0-branch to Safari-1-3-branch

2006-09-07 Beth Dakin <bdakin@apple.com>

Reviewed by Darin sort of and Maciej definitely.

Fix for <rdar://problem/4719149> Crash caused by malformed frame
setup. This is the same bug as 3853672, which has been fixed on TOT
for a while. We feel like this fix is much safer for the branch
though.

  • WebKit.xcodeproj/project.pbxproj: Version wars.
  • WebView.subproj/WebBaseResourceHandleDelegate.m: (-[NSURLProtocol releaseResources]): Added comment for assertion that will be hit. (-[NSURLProtocol didFailWithError:]): Added comment for assertion that will be hit.
  • WebView.subproj/WebDataSource.m: (-[WebDataSource _receivedMainResourceError:complete:]): Retain and release all of the pieces involved with sending the error.
12:06 Changeset [17084] by aliceli1

Reviewed by Tim Hatcher.

When nodes are removed, selections are cleared, and when http://bugs.webkit.org/show_bug.cgi?id=6498 was fixed, we started sending didChangeSelection notifications. Updating the test results fixes some of the tests mentioned in http://bugs.webkit.org/show_bug.cgi?id=10924

  • fast/dynamic/move-node-with-selection-expected.txt:
  • fast/events/dblclick-addEventListener-expected.txt:
  • fast/events/updateLayoutForHitTest-expected.txt:
11:58 Changeset [22373] by aroben

Reviewed by Geoff.

A couple of cleanups that Geoff suggested after my last checkin.

  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::setPositionAndSize):
11:47 Changeset [22372] by aroben

Reviewed by Adele.

Fix: <rdar://problem/4772506> <select> popup menu should not clip items
when width: is set.

Popups are now sized and positioned as follows:

  1. Popups are at least as wide as the <select> control on the page.
  2. Popups for LTR <select>s are right-aligned with the <select>, popups for RTL <select>s are left-aligned. This means the scrollbar in the popup will always line up with the dropdown button in the <select>.
  3. Popups try to be wide enough to fit their widest item, but will shrink to ensure they don't extend off the screen.
  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::setPositionAndSize):
11:07 Changeset [17083] by harrison

Reviewed by Adele.

<rdar://problem/4770453> VO not honoring secure edit fields in web pages

  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject isPasswordField]): New. Checks for input with inputType() of PASSWORD.


(-[WebCoreAXObject subrole]):
Returns NSAccessibilitySecureTextFieldSubrole if [self isPasswordField].


(-[WebCoreAXObject roleDescription]):
Pass [self subrole] instead of nil when calling NSAccessibilityRoleDescription.


(-[WebCoreAXObject value]):
Return nil if [self isPasswordField].


(-[WebCoreAXObject accessibilityAttributeNames]):
Return generic attributes if [self isPasswordField].

(-[WebCoreAXObject accessibilityParameterizedAttributeNames]):
Return empty array if [self isPasswordField].

09:51 Changeset [17082] by thatcher

Tag for WebKit 418.9.1 (part 2 of 2.)

09:50 Changeset [17081] by thatcher

Tag for WebKit 418.9.1 (part 1 of 2.)

09:49 Changeset [17080] by aliceli1

Reviewed by aroben.

updating test results after r17068 - http://bugs.webkit.org/show_bug.cgi?id=11320

  • fast/block/float/013-expected.checksum:
  • fast/block/float/013-expected.txt:
09:42 Changeset [17079] by thatcher

Versioning to 418.9.1

09:36 Changeset [17078] by aroben

Reviewed by Adele.

Fixes http://bugs.webkit.org/show_bug.cgi?id=11214
http/tests/xmlhttprequest/exceptions.html needs new results

Updating results after r16794.

  • http/tests/xmlhttprequest/exceptions-expected.txt:
  • http/tests/xmlhttprequest/exceptions.html: Add comments about why we don't follow the XHR spec.
07:59 Changeset [17077] by brmorris

2006-10-13 brmorris <thisisbradley@gmail.com>

Reviewed by me.
DESC: Updated RVCT supporting libs.
http://bugs.webkit.org/show_bug.cgi?id=9435

  • target/RVCT/S60LibsRVCT.zip: Toolbar.dll & recentstore.dll
01:52 Changeset [17076] by hyatt

Fix "flash to white' problem on yahoo.com as it finishes loading. Refine the FOUC suppression
code so that it will only run if the body still has no renderer.

  • dom/Document.cpp: (WebCore::Document::updateLayoutIgnorePendingStylesheets):
00:32 Changeset [17075] by hyatt

Fix a bug where the WebCore memory cache could grow without bound during extended browsing
sessions.

Reviewed by mjs

  • loader/Cache.cpp: (WebCore::Cache::flush): (WebCore::Cache::setSize): (WebCore::Cache::getLRUListFor): (WebCore::Cache::removeFromLRUList): (WebCore::Cache::insertInLRUList):
  • loader/Cache.h:
  • loader/CachedCSSStyleSheet.cpp:
  • loader/CachedCSSStyleSheet.h:
  • loader/CachedImage.cpp:
  • loader/CachedImage.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::setRequest): (WebCore::CachedResource::deref):
  • loader/CachedResource.h: (WebCore::CachedResource::allowInLRUList):
  • loader/CachedScript.cpp:
  • loader/CachedScript.h:
  • loader/CachedXBLDocument.cpp:
  • loader/CachedXBLDocument.h:
  • loader/CachedXSLStyleSheet.cpp:
  • loader/CachedXSLStyleSheet.h:

10/16/06:

17:51 Changeset [22371] by aroben

Reviewed by Kevin.

Fix: <rdar://problem/4758923> <select> menu text doesn't draw right
aligned on right-to-left sites

  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::show): Pass appropriate styles to CreateWindowEx so that we get a left-aligned scrollbar in RTL popups. (WebCore::PopupMenu::drawItem): Set text to be right-aligned and draw RTL when direction:rtl is set. (WebCore::PopupWndProc): Add 1px of vertical height to each menu item to match RenderListBox.
  • rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::paintMenuList): Draw the dropdown button on the left when in RTL. (WebCore::RenderThemeWin::adjustMenuListStyle): Set padding to allow for the dropdown button being on the left in RTL.
17:48 Changeset [17074] by justing

LayoutTests:

Reviewed by harrison


<rdar://problem/3655385>
Editing: -indent: method unimplemented

  • editing/execCommand/nsresponder-indent-expected.checksum: Added.
  • editing/execCommand/nsresponder-indent-expected.png: Added.
  • editing/execCommand/nsresponder-indent-expected.txt: Added.
  • editing/execCommand/nsresponder-indent.html: Added.
  • editing/execCommand/nsresponder-outdent-expected.checksum: Added.
  • editing/execCommand/nsresponder-outdent-expected.png: Added.
  • editing/execCommand/nsresponder-outdent-expected.txt: Added.
  • editing/execCommand/nsresponder-outdent.html: Added.

WebCore:

Reviewed by harrison


<rdar://problem/3655385>
Editing: -indent: method unimplemented

Added Frame and bridge methods.

  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge indent]): (-[WebCoreFrameBridge outdent]):
  • page/Frame.cpp: (WebCore::Frame::indent): (WebCore::Frame::outdent):
  • page/Frame.h:

WebKit:

Reviewed by harrison


<rdar://problem/3655385>
Editing: -indent: method unimplemented

  • WebView/WebHTMLView.m: (-[NSArray validateUserInterfaceItem:]): Only allow indent:/outdent: in richly editable areas. (-[NSArray indent:]): (-[NSArray outdent:]):
17:35 Changeset [17073] by kmccullo

Reviewed by Geof.

Removed unecessary global specifiers.

  • kjs/math_object.cpp: (MathFuncImp::callAsFunction):
17:10 Changeset [22370] by lamadio
16:12 Changeset [17072] by kmccullo

Reviewed by John.

Fixes a compile order issue for testkjs on win.

14:57 Changeset [22369] by aroben

Added forgotten <rdar://problem/4710456> reference to old entry.

14:54 Changeset [22368] by aroben

Reviewed by John.

Forgot to save these changes John suggested before checking in.

  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::drawItem):
14:52 Changeset [22367] by aroben

Reviewed by John.

Fix: <rdar://problem/4709480> Implement option groups
Fix: <rdar://problem/4785693> Use CG to draw popup menu items
Fix: <rdar://problem/4709487> Implement separators in <select> popups
Fix: <rdar://problem/4709358> Move CreateWindowEx calls out of PopupMenu constructor

  • platform/win/PopupMenuWin.cpp: Added some now-required #includes, moved some constants into the WebCore namespace. (WebCore::PopupMenu::PopupMenu): Moved CreateWindowEx calls into PopupMenu::show(). (WebCore::PopupMenu::show): Pass LBS_OWNERDRAWFIXED to CreateWindowEx to assume responsibility for drawing menu itesm. Removed call to set the font on the popup menu, as CG will take care of this. (WebCore::addString): Added to help with the PopupMenu::add* methods. (WebCore::PopupMenu::addOption): Use addString. (WebCore::PopupMenu::addGroupLabel): Ditto, and moved from TemporaryLinkStubs.cpp. (WebCore::PopupMenu::addSeparator): Ditto. (WebCore::PopupMenu::down): Skip items that are not enabled <option> elements, and tell the popup menu to set the selection to the element we end up on. (WebCore::PopupMenu::up): Ditto. (WebCore::PopupMenu::drawItem): Added. This is the method that is called to draw each item in the menu. (WebCore::PopupWndProc): Add cases for WM_MEASUREITEM and WM_DRAWITEM messages (required now that we're drawing the menu items ourselves). In the cases for arrow key presses, return -2 to signify that we're handling all aspects of changing the selection.
  • platform/win/TemporaryLinkStubs.cpp: Removed PopupMenu::addSeparator and PopupMenu::addGroupLabel.
14:44 Changeset [17071] by aroben

Reviewed by John.

More menulist cleanup/tweaking.

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::setSelectedIndex): Renamed 'index' parameter to 'optionIndex' for clarity. (WebCore::HTMLSelectElement::menuListDefaultEventHandler): Skip over items that are not enabled <option> elements when using arrow keys, and make sure we pass an option index to setSelectedIndex.
  • platform/PopupMenu.h: (WebCore::PopupMenu::setWasClicked): Specify default argument.
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::setTextFromOption): Strip whitespace from option text before setting the control text so that options within <optgroup>s don't appear indented in the actual control.
13:52 Changeset [17070] by weinig

Reviewed by Adam.

Win32 build fix.

  • WebCore.vcproj/WebCore/WebCore.vcproj:
  • bridge/win/BrowserExtensionWin.cpp: (WebCore::BrowserExtensionWin::createNewWindow):
  • bridge/win/FrameWin.cpp: (WebCore::FrameWin::urlSelected): (WebCore::FrameWin::submitForm): (WebCore::FrameWin::createNewWindow):
  • bridge/win/FrameWin.h:
  • loader/loader.cpp: (WebCore::Loader::receivedResponse):
  • platform/win/GraphicsContextWin.cpp: (WebCore::GraphicsContext::getWindowsContext): (WebCore::GraphicsContext::releaseWindowsContext):
  • platform/win/TemporaryLinkStubs.cpp: (FrameWin::openURLRequest): (WebCore::ServeSynchronousRequest): (WebCore::CheckIfReloading): (WebCore::IsResponseURLEqualToURL): (WebCore::ResponseMIMEType): (WebCore::ResponseIsMultipart): (WebCore::CacheObjectExpiresTime): (CachedResource::setResponse): (CachedResource::setAllData):
  • rendering/RenderThemeWin.cpp: (WebCore::doneDrawing): (WebCore::RenderThemeWin::paintButton): (WebCore::RenderThemeWin::paintTextField):
11:17 Changeset [17069] by brmorris

2006-10-16 brmorris <thisisbradley@gmail.com>

Reviewed by Zalan.
DESC: Fix build break in render_frames.cpp caused by r17066


  • kwq/KWQWidget.h: (QWidget::focusable): Is available when not RD_32_BROWSER
01:29 Changeset [17068] by hyatt

Fix for bug 9347, positioned elements have the wrong containing block. This is a change from CSS2 to
CSS2.1. In the absence of any other enclosing positioned elements, the containing block for a positioned
element is the initial containing block (our RenderView).

Reviewed by mjs

Test cases added to fast/block/positioning for quirks and strict modes.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::lowestPosition): (WebCore::RenderBlock::rightmostPosition): (WebCore::RenderBlock::leftmostPosition):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::containingBlockHeightForPositioned): (WebCore::RenderBox::calcAbsoluteVertical):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::containingBlock): (WebCore::RenderObject::container):
01:27 Changeset [17067] by hyatt

Add test cases for bug 9347.

10/15/06:

19:00 Changeset [17066] by zbujtas

2006-10-13 sareen <shyam.sareen@nokia.com>

Reviewed by bujtas <zbujtas@gmail.com>
DESC: New Enum need to add for finding the whether the plugin

is interactive or not.

http://bugs.webkit.org/show_bug.cgi?id=11281

WARNING: NO TEST CASES ADDED OR CHANGED

  • bridge/WebCoreWidget.h:
  • khtml/html/html_objectimpl.cpp: (HTMLEmbedElementImpl::isFocusable): (HTMLObjectElementImpl::isFocusable):
  • khtml/rendering/render_frames.cpp:
  • khtml/rendering/render_frames.h:
  • kwq/KWQObjectWidget.cpp:
  • kwq/KWQObjectWidget.h:
18:33 Changeset [17065] by bdash

2006-10-15 Krzysztof Kowalczyk <kkowalczyk@gmail.com>

Reviewed by Anders.

Remove junk (as gcc calls it) after #else clause.

  • wtf/FastMalloc.cpp: (WTF::do_free):

2006-10-15 Krzysztof Kowalczyk <kkowalczyk@gmail.com>

Reviewed by Anders.

Misc Gdk/Linux build fixes.

  • platform/gdk/BrowserExtensionGdk.h:
  • platform/gdk/FrameGdk.cpp: (WebCore::FrameGdk::submitForm): (WebCore::FrameGdk::urlSelected):
  • platform/gdk/FrameGdk.h:
  • platform/gdk/RenderPopupMenuGdk.cpp: (WebCore::PopupMenu::PopupMenu): (WebCore::PopupMenu::~PopupMenu): (WebCore::PopupMenu::clear): (WebCore::PopupMenu::populate): (WebCore::PopupMenu::show): (WebCore::PopupMenu::hide): (WebCore::PopupMenu::addSeparator): (WebCore::PopupMenu::addGroupLabel): (WebCore::PopupMenu::addOption):
  • platform/gdk/RenderPopupMenuGdk.h:
  • platform/gdk/RenderThemeGdk.cpp:
  • platform/gdk/RenderThemeGdk.h: (WebCore::RenderThemeGdk::RenderThemeGdk):
  • platform/gdk/TemporaryLinkStubs.cpp: (BrowserExtensionGdk::createNewWindow): (BrowserExtensionGdk::BrowserExtensionGdk): (BrowserExtensionGdk::setTypedIconURL): (BrowserExtensionGdk::setIconURL): (BrowserExtensionGdk::getHistoryLength): (WebCore::CheckIfReloading): (WebCore::CacheObjectExpiresTime): (WebCore::ResponseURL): (WebCore::IsResponseURLEqualToURL): (WebCore::ResponseIsMultipart): (WebCore::ResponseMIMEType): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData): (FileChooser::disconnectUploadControl):
  • rendering/RenderTheme.h:
  • rendering/RenderThemeMac.h:

2006-10-15 Krzysztof Kowalczyk <kkowalczyk@gmail.com>

Reviewed by Anders.

Make compiler not complain about unused gk. SpinneretWebHost was not setting
initial refcount upon creation, so it is completely bogus.

  • GdkLauncher/main.cpp: (main):
  • Spinneret/Spinneret/Spinneret.h: (SpinneretWebHost::SpinneretWebHost):
15:22 Changeset [17064] by weinig

Reviewed by Mitz.

Fix for http://bugs.webkit.org/show_bug.cgi?id=11294
Add getModifierState method to KeyboardEvent

  • bindings/objc/DOMEvents.h:
  • bindings/objc/DOMEvents.mm:
  • bindings/objc/PublicDOMInterfaces.h:
  • dom/KeyboardEvent.cpp: (WebCore::KeyboardEvent::getModifierState):
  • dom/KeyboardEvent.h:
  • dom/KeyboardEvent.idl:
14:44 Changeset [22366] by aroben

Reviewed by Geoff, Anders.

Fix crash on startup.

We were crashing somewhere inside Frame::setResourceRequest. I'm not
sure what change made this start crashing.

  • WebFrame.cpp: (WebFrame::receivedResponse): Remove useless calls to Frame::resourceRequest and Frame::setResourceRequest.
13:52 Changeset [22365] by aroben

Build fix.

Merge OpenSource r17056 to TemporaryLinkStubs.cpp.

  • platform/win/TemporaryLinkStubs.cpp: (WebCore::CacheObjectExpiresTime): (WebCore::ResponseIsMultipart): (WebCore::ResponseMIMEType): (WebCore::IsResponseURLEqualToURL): (WebCore::ResponseURL): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData):
12:20 Changeset [17063] by ap

2006-10-15 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed and landed by ap.

Changed PlatformResponse handling for Qt. Don't just store
a QString object containing the data but a new PlatformResponseQt struct
containing data and url, and store it as pointer. This is compatible
to the OS X handling and removes the Qt plaform specific hacks from
CachedCSSStyleSheet, CachedResource & loader.

Add new file LoaderFunctionsQt, similar to LoaderFunctionsMac, and move
all methods from TemporaryLinkStubs related to loading in there, and implement
some like ServeSynchronousRequest (yay for xslt handling!).

  • CMakeLists.txt:
  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::checkNotify):
  • loader/CachedResource.h: (WebCore::CachedResource::CachedResource):
  • loader/loader.cpp: (WebCore::Loader::receivedResponse):
  • platform/ResourceLoaderClient.h:
  • platform/ResourceLoaderInternal.h:
  • platform/qt/FrameQt.cpp: (WebCore::FrameQt::~FrameQt): (WebCore::FrameQt::createEmptyDocument):
  • platform/qt/FrameQt.h: (WebCore::QtFrame):
  • platform/qt/FrameQtClient.cpp: (WebCore::FrameQtClientDefault::setFrame): (WebCore::FrameQtClientDefault::openURL): (WebCore::FrameQtClientDefault::loadFinished): (WebCore::FrameQtClientDefault::receivedAllData):
  • platform/qt/FrameQtClient.h:
  • platform/qt/LoaderFunctionsQt.cpp: Added. (WebCore::ServeSynchronousRequest): (WebCore::NumberOfPendingOrLoadingRequests): (WebCore::CheckIfReloading): (WebCore::CheckCacheObjectStatus): (WebCore::IsResponseURLEqualToURL): (WebCore::ResponseURL): (WebCore::ResponseMIMEType): (WebCore::ResponseIsMultipart): (WebCore::CacheObjectExpiresTime): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData):
  • platform/qt/ResourceLoaderManager.cpp: (WebCore::ResourceLoaderManager::remove):
  • platform/qt/ResourceLoaderQt.cpp: (WebCore::ResourceLoader::~ResourceLoader): (WebCore::ResourceLoader::start): (WebCore::ResourceLoader::assembleResponseHeaders): (WebCore::ResourceLoader::receivedResponse):
  • platform/qt/TemporaryLinkStubs.cpp:
03:00 Changeset [17062] by rwlbuis

Reviewed by Mitz.

http://bugs.webkit.org/show_bug.cgi?id=11217
Cleanup svg coding style

Cleanup coding style in svg basic datastructure classes.

10/14/06:

17:31 Changeset [17061] by oliver

2006-10-15 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by Oliver.

Implement some functions in FrameQt / ScrollViewQt.

Added comments what needs to be done in quite some places,
marked some functions as no-ops.

  • CMakeLists.txt:
  • platform/qt/FrameQt.cpp: (WebCore::FrameQt::FrameQt): (WebCore::FrameQt::runJavaScriptConfirm): (WebCore::FrameQt::locationbarVisible): (WebCore::FrameQt::createFrame): (WebCore::FrameQt::passSubframeEventToSubframe): (WebCore::FrameQt::menubarVisible): (WebCore::FrameQt::personalbarVisible): (WebCore::FrameQt::statusbarVisible): (WebCore::FrameQt::toolbarVisible): (WebCore::FrameQt::markedTextRange): (WebCore::FrameQt::lastEventIsMouseUp): (WebCore::FrameQt::openURLRequest): (WebCore::FrameQt::scheduleClose): (WebCore::FrameQt::unfocusWindow): (WebCore::FrameQt::focusWindow): (WebCore::FrameQt::overrideMediaType): (WebCore::FrameQt::runJavaScriptPrompt): (WebCore::FrameQt::bindingRootObject): (WebCore::FrameQt::addPluginRootObject): (WebCore::FrameQt::registerCommandForUndo): (WebCore::FrameQt::registerCommandForRedo): (WebCore::FrameQt::clearUndoRedoOperations): (WebCore::FrameQt::respondToChangedSelection): (WebCore::FrameQt::respondToChangedContents): (WebCore::FrameQt::shouldChangeSelection): (WebCore::FrameQt::canGoBackOrForward): (WebCore::FrameQt::canPaste): (WebCore::FrameQt::canRedo): (WebCore::FrameQt::canUndo):
  • platform/qt/FrameQt.h:
  • platform/qt/FrameQtClient.cpp: (WebCore::FrameQtClientDefault::runJavaScriptAlert): (WebCore::FrameQtClientDefault::runJavaScriptConfirm): (WebCore::FrameQtClientDefault::runJavaScriptPrompt): (WebCore::FrameQtClientDefault::menubarVisible): (WebCore::FrameQtClientDefault::toolbarVisible): (WebCore::FrameQtClientDefault::statusbarVisible): (WebCore::FrameQtClientDefault::personalbarVisible): (WebCore::FrameQtClientDefault::locationbarVisible):
  • platform/qt/FrameQtClient.h:
  • platform/qt/ImageSourceQt.cpp:
  • platform/qt/RenderThemeQt.cpp:
  • platform/qt/ScrollViewQt.cpp: (WebCore::ScrollView::scrollPointRecursively):
  • platform/qt/TemporaryLinkStubs.cpp: (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData):
17:04 Changeset [17060] by oliver

2006-10-15 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by Oliver.

Add another hanging test, to the "to be skipped" list.

  • DumpRenderTree/DumpRenderTree.qtproj/tests-skipped.txt:
10:38 Changeset [17059] by harrison

Reviewed by Geoff.

<rdar://problem/4775213> REGRESSION: Can't from tab from last link on the page to URL address field
<rdar://problem/4783926> REGRESSION: Typing tab key enters no text

The problem is that EventTargetNode::defaultEventHandler() calls event->setDefaultHandled() regardless
of whether frame->view()->advanceFocus() actually changed the focus.

Trivial fix is to preventDefault() only if advanceFocus() returns true.

  • No test is possible because DumpRenderTree sets the WebView preference to tab to links, and that preference cannot be changed dynamically after the webview is initialized. Also, the "tab from last link" is not testable because DumpRenderTree uses the WebView for the whole window, so tabs can't shift focus out of the WebView regardless.
  • dom/EventTargetNode.cpp: (WebCore::EventTargetNode::defaultEventHandler):
10:21 Changeset [17058] by weinig

Reviewed by Anders.

Some fixes to get the Qt BuildSlave to run the LayoutTests.

  • Scripts/build-dumprendertree: No need to call cmake again.
  • Scripts/run-webkit-tests: Expose LD_LIBRARY_PATH.
05:53 Changeset [17057] by weinig

Reviewed by Mitz.

Fix Qt/Linux build & some little fixes:

  • Adapt to Maciej's FrameLoadRequest changes.
  • Fix response handling - don't try to compare with '0' - for us it's a non-ptr QString object.
  • Actually call setAllData from loader/loader.cpp (David Carson forgot that in his last commit)
  • Fix CacheObjectExpiresTime/ResponseIsMultipart signature.
  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::checkNotify):
  • loader/CachedResource.h: (WebCore::CachedResource::CachedResource):
  • loader/loader.cpp: (WebCore::Loader::receivedAllData): (WebCore::Loader::receivedResponse):
  • platform/qt/BrowserExtensionQt.cpp: (WebCore::BrowserExtensionQt::createNewWindow):
  • platform/qt/BrowserExtensionQt.h:
  • platform/qt/FrameQt.cpp: (WebCore::FrameQt::submitForm): (WebCore::FrameQt::urlSelected): (WebCore::FrameQt::openURLRequest):
  • platform/qt/FrameQt.h:
  • platform/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::setURLForRect):
  • platform/qt/TemporaryLinkStubs.cpp: (WebCore::CacheObjectExpiresTime): (WebCore::ResponseIsMultipart): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData):
02:42 Changeset [17056] by bdash

2006-10-14 David Carson <dacarson@gmail.com>

Reviewed by Maciej.

Fix: http://bugs.webkit.org/show_bug.cgi?id=11257
Removed Mac platform dependance, changed WebCore functions to
use PlatformResponse/PlatformData rather than NSURLResponse/
NSData.

  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::checkNotify):
  • loader/CachedImage.cpp: (WebCore::CachedImage::createImage):
  • loader/CachedResource.cpp: (WebCore::CachedResource::~CachedResource):
  • loader/CachedResource.h: (WebCore::CachedResource::CachedResource): (WebCore::CachedResource::response): (WebCore::CachedResource::allData):
  • loader/LoaderFunctions.h:
  • loader/loader.cpp: (WebCore::Loader::receivedResponse):
  • loader/mac/LoaderFunctionsMac.mm: (WebCore::ResponseMIMEType): (WebCore::ResponseIsMultipart): (WebCore::CacheObjectExpiresTime): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData):
  • platform/qt/TemporaryLinkStubs.cpp: (WebCore::CacheObjectExpiresTime): (WebCore::ResponseIsMultipart): (WebCore::ResponseMIMEType): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData):
  • platform/win/TemporaryLinkStubs.cpp: (WebCore::CacheObjectExpiresTime): (WebCore::ResponseIsMultipart): (WebCore::ResponseMIMEType): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::setAllData):
02:39 Changeset [17055] by bdash

2006-10-14 Krzysztof Kowalczyk <kkowalczyk@gmail.com>

Reviewed by Maciej.

Define KXMLCORE_USE_CURL for platforms that wish to use CURL as
networking, and set it for GDK build

  • wtf/Platform.h:

2006-10-14 Krzysztof Kowalczyk <kkowalczyk@gmail.com>

Reviewed by Maciej.

Fix GDK build by defining FrameGdkClient() constructor as non-virtual.
Pure virtual classes require non-pure destructor, who knew.

Make CURL networking dependent on USE(CURL) instead of PLATFORM(GDK),
so that other platforms can also use CURL, if they wish.

Silence compiler warning about unused variable in SharedTimerLinux.cpp

  • platform/ResourceLoaderInternal.h: (WebCore::ResourceLoaderInternal::ResourceLoaderInternal):
  • platform/gdk/FrameGdk.h: (WebCore::FrameGdkClient::~FrameGdkClient):
  • platform/gdk/ResourceLoaderManager.cpp: (WebCore::ResourceLoaderManager::downloadTimerCallback):
  • platform/gdk/SharedTimerLinux.cpp: (WebCore::stopSharedTimer):
Note: See TracTimeline for information about the timeline view.