Timeline
02/22/07:
- 22:54 Changeset [22990] by
-
2007-02-22 Oliver Hunt < oliver@apple.com>
rs = adam
Replacing radar numbers from previous patch with one
the isn't fictitious < rdar://problem/5018591>
- platform/win/CursorWin.cpp: (WebCore::columnResizeCursor): (WebCore::rowResizeCursor): (WebCore::verticalTextCursor):
- 22:48 Changeset [19822] by
-
Reviewed by Adam Roben, who never sleeps.
To avoid code duplication, move notImplementedGdk() macro to a separate
header file. Remove existing notImplementedGdk() macros and replace
remaining notImplemented() to notImplementedGdk() in gdk code.
- loader/gdk/DocumentLoaderGdk.cpp: (WebCore::DocumentLoader::getResponseModifiedHeader):
- loader/gdk/FrameLoaderClientGdk.cpp: (WebCore::FrameLoaderClientGdk::hasWebView): (WebCore::FrameLoaderClientGdk::hasFrameView):
- page/gdk/ContextMenuClientGdk.cpp: (WebCore::ContextMenuClientGdk::contextMenuDestroyed): (WebCore::ContextMenuClientGdk::getCustomMenuFromDefaultItems): (WebCore::ContextMenuClientGdk::contextMenuItemSelected): (WebCore::ContextMenuClientGdk::downloadURL): (WebCore::ContextMenuClientGdk::copyImageToClipboard): (WebCore::ContextMenuClientGdk::searchWithGoogle): (WebCore::ContextMenuClientGdk::lookUpInDictionary): (WebCore::ContextMenuClientGdk::speak): (WebCore::ContextMenuClientGdk::stopSpeaking):
- page/gdk/EventHandlerGdk.cpp:
- platform/gdk/CursorGdk.cpp: (WebCore::Cursor::Cursor):
- platform/gdk/EditorClientGdk.cpp:
- platform/gdk/NotImplementedGdk.h: Added.
- platform/gdk/PopupMenuGdk.cpp: (WebCore::PopupMenu::PopupMenu): (WebCore::PopupMenu::~PopupMenu): (WebCore::PopupMenu::show): (WebCore::PopupMenu::hide): (WebCore::PopupMenu::updateFromElement):
- platform/gdk/RenderThemeGdk.cpp:
- platform/gdk/ScreenGdk.cpp: (WebCore::screenDepth): (WebCore::screenDepthPerComponent): (WebCore::screenIsMonochrome): (WebCore::screenRect): (WebCore::screenAvailableRect):
- platform/gdk/TemporaryLinkStubs.cpp:
- platform/graphics/gdk/ImageGdk.cpp: (WebCore::Image::drawPattern):
- platform/network/gdk/ResourceHandleCurl.cpp: (WebCore::ResourceHandle::setDefersLoading):
- platform/network/gdk/ResourceHandleManager.cpp: (WebCore::ResourceHandleManager::setupPUT): (WebCore::ResourceHandleManager::setupPOST):
- 22:30 Changeset [22989] by
-
2007-02-22 Oliver Hunt < oliver@apple.com>
Reviewed by Ada.
Implemented a couple more cursor methods
- platform/win/CursorWin.cpp: (WebCore::columnResizeCursor): (WebCore::rowResizeCursor): (WebCore::verticalTextCursor): Added radar bug reference < rdar://problem/5018452> (WebCore::progressCursor): (WebCore::noDropCursor): (WebCore::notAllowedCursor): Making them return the correct cursor
- 20:32 Changeset [19821] by
-
Reviewed by Tim H.
Fixes: http://bugs.webkit.org/show_bug.cgi?id=12852
Bug#12852: Drosera should select function name, not "function" keyword when selecting from function list pop-up
- Drosera/debugger.js: Fixed some logic errors resulting in generating double IDs and names.
- 19:48 Changeset [19820] by
-
LayoutTests:
Reviewed by Maciej.
- Test for http://bugs.webkit.org/show_bug.cgi?id=12850 Leaks >10k objects
- fast/events/remove-event-listener-expected.txt: Added.
- fast/events/remove-event-listener.html: Added.
WebCore:
Reviewed by Maciej.
- Patch for http://bugs.webkit.org/show_bug.cgi?id=12850 Leaks >10k objects
and
- http://bugs.webkit.org/show_bug.cgi?id=12853 add a EventListener leak counter
Problem: RemoveEventListener leaks memory if the listener is not
registered.
Fix: Added Window::findJSEventListener function w/o creating a
JSEventListener; Renamed getJSEventListener to findOrCreateJSEventListener;
As an enhancement, added a leak counter for EventListeners.
Added a test case, LayoutTests/fast/events/remove-event-listener.html.
- WebCore/bindings/js/kjs_dom.cpp:
- WebCore/bindings/js/kjs_window.h:
- WebCore/bindings/js/kjs_window.cpp:
- WebCore/bindings/js/kjs_event.cpp: Add a leak counter.
- WebCore/bindings/js/JSXMLHttpRequest.cpp:
- LayoutTests/fast/events/remove-event-listener.html:
- 19:38 Changeset [19819] by
-
Reviewed by Geoff.
< rdar://problem/4998203>
REGRESSION: Back-forward list gets messed up when navigating to error pages with back-forward cache turned off
- loader/FrameLoader.cpp: (WebCore::FrameLoader::load): Remove call to setPolicyDocumentLoader(), the policy document loader is set immediately after, and setting it before calling shouldReloadToHandleUnreachableURL causes us to get the wrong URL.
(WebCore::FrameLoader::shouldReloadToHandleUnreachableURL):
Check if the unreachable URL is equal to the document loader's request.
(WebCore::FrameLoader::checkLoadCompleteForThisFrame):
Check the provisional document loader's unreachable URL.
- 18:47 Changeset [22988] by
-
2007-02-22 Oliver Hunt < oliver@apple.com>
Reviewed by Anders.
Removing accessive COMPtr use from ClipboardWin constructors
Adding data verification to ClipoardWin::SetData
Small amount code tidying, and allowing for failure when writing an URL
- page/win/EventHandlerWin.cpp: (WebCore::EventHandler::createDraggingClipboard): Updated for new ClipboardWin constructor
- platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::ClipboardWin): (WebCore::writeURL): (WebCore::ClipboardWin::getData): Tidying code, and allowing failure (WebCore::ClipboardWin::setData): Verifying that we're writing a valid url
- platform/win/ClipboardWin.h:
- platform/win/EditorWin.cpp: (WebCore::Editor::newGeneralClipboard): Updated for new ClipboardWin constructor
- 17:13 Changeset [19818] by
-
Versioning.
- 16:35 Changeset [19817] by
-
Versioning.
- 16:32 Changeset [19816] by
-
Merge in r19806.
2007-02-22 Adele Peterson < adele@apple.com>
Reviewed by Darin.
Fix for < rdar://problem/5014970> 9A374: Tabs don't work in the message body
Test: fast/events/keypress-insert-tab.html
- page/EventHandler.cpp: (WebCore::EventHandler::defaultTextInputEventHandler): If the defaultTabEventHandler doesn't handle the event, then go on to the insertText case.
- 16:29 Changeset [19815] by
-
Merge in r19805.
2007-02-22 Geoffrey Garen < ggaren@apple.com>
Reviewed by Darin Adler.
Fixed < rdar://problem/4942528> Installer.app Stuck "Examining additional
volumes..."
The problem was that JavaScriptGlue held a JSObject in a static pointer
without protecting it from garbage collection.
sUserObjectPrototypeImp = new UserObjectPrototypeImp();
static ProtectedPtr<UserObjectPrototypeImp> protectPrototype; // valiant, brave, but useless
Through an interesting comedy of errors, this object would be collected
and then assigned as the prototype of another object allocated at the
same address, thus becoming its own prototype. (See http://www.ziplo.com/grandpa.htm
for further explanation of this phenomenon.) Hence the infinte loop in
property lookup.
The solution is equally humorous: all of this code was completely useless --
it only served to create an empty object in the prototype chain -- so
I just removed it.
- UserObjectImp.cpp: (UserObjectImp::UserObjectImp):
- UserObjectImp.h:
- 16:26 Changeset [19814] by
-
Merge in r19783.
2007-02-21 Brady Eidson < beidson@apple.com>
Reviewed by Dave Harrison
< rdar://problem/5014313> - Crash in FrameLoader::restoreScrollPositionAndViewState()
after regaining network connection
- loader/FrameLoader.cpp: (WebCore::FrameLoader::restoreScrollPositionAndViewState): Leave an assertion for debug build exploration, but gracefully handle the release build case
- 16:21 Changeset [19813] by
-
New tag.
- 16:12 Changeset [22987] by
-
Updating this file to match WebCore.
- WebKit.vcproj/textAreaResizeCorner.png:
- 16:03 Changeset [19812] by
-
Reviewed by John.
Updating this image to match the one in WebCore.
- WebKit.vcproj/textAreaResizeCorner.png:
- 15:59 Changeset [22986] by
-
Reviewed by Adam.
Fix for http://bugs.webkit.org/show_bug.cgi?id=12399 REGRESSION:
Unable to prevent default context menu from appearing. < rdar://
problem/5017416>
- WebView.cpp: (WebView::handleContextMenuEvent): Clear the controller's context menu before propagating a new context menu event through the DOM.
- 15:51 Changeset [19811] by
-
LayoutTests:
Reviewed by darin
< rdar://problem/5007143>
REGRESSION: WebKit doesn't remember horizontal position of caret when moving forward by line
Added, tests the removal of the use of the
recalc bool from xPosForVerticalArrowNavigation(...):
- editing/selection/5007143-2-expected.checksum: Added.
- editing/selection/5007143-2-expected.png: Added.
- editing/selection/5007143-2-expected.txt: Added.
- editing/selection/5007143-2.html: Added. Added, demonstrate the changes to modify:
- editing/selection/5007143-expected.checksum: Added.
- editing/selection/5007143-expected.png: Added.
- editing/selection/5007143-expected.txt: Added.
- editing/selection/5007143.html: Added. Demonstrates a bug (filed 5017312):
- editing/style/block-style-004-expected.txt:
- editing/style/block-style-005-expected.txt:
- editing/style/block-style-006-expected.txt: Fixed:
- editing/selection/3690703-2-expected.checksum:
- editing/selection/3690703-2-expected.png:
- editing/selection/3690703-2-expected.txt:
- editing/style/create-block-for-style-009-expected.checksum:
- editing/style/create-block-for-style-009-expected.png:
- editing/style/create-block-for-style-009-expected.txt:
- editing/style/create-block-for-style-011-expected.checksum:
- editing/style/create-block-for-style-011-expected.png:
- editing/style/create-block-for-style-011-expected.txt:
WebCore:
Reviewed by darin
< rdar://problem/5007143>
REGRESSION: WebKit doesn't remember horizontal position of caret when moving forward by line
- editing/SelectionController.cpp: (WebCore::SelectionController::SelectionController): Initialize m_xPosForVerticalArrowNavigation. (WebCore::SelectionController::setSelection): Call the new setter. (WebCore::SelectionController::modifyMovingRightForward): For line and paragraph granularities, use the saved xPosForVerticalArrowNavigation even if the old selection was a range. Use the x position of the *start*, not the end of the selection if there is no saved xPosForVerticalArrowNavigation. This matches TextEdit's behavior. (WebCore::SelectionController::modifyMovingLeftBackward): For line and paragraph granularities, use the saved xPosForVerticalArrowNavigation even if the old selection was a range. (WebCore::SelectionController::modify): Preserve the xPosForVerticalArrowNavigation for line and paragraph granularities. (WebCore::SelectionController::xPosForVerticalArrowNavigation): Removed the recalc bool because it is now always false. (WebCore::SelectionController::setXPosForVerticalArrowNavigation): Added.
- editing/SelectionController.h: Moved m_xPosForVerticalArrowNavigation from Frame.
- page/Frame.cpp: Moved m_xPosForVerticalArrowNavigation to SelectionController.
- page/Frame.h: Ditto.
- page/FramePrivate.h: Ditto.
- 15:49 Changeset [19810] by
-
WebCore:
Reviewed by Adam. Manual test added.
Fix for http://bugs.webkit.org/show_bug.cgi?id=12399 REGRESSION:
Unable to prevent default context menu from appearing. < rdar://
problem/5017416>
This problem appeared because of two facets of the current context
menu design. First, all context menu events are now considered to
be "swallowed" since we take care of building up the regular
context menu through the defaultEventHandler(). Second, the context
menu controller holds onto it's context menu until a new one is
created. There would be logistical problems changing this since
AppKit relies on the menu being around for as long as it is
visible on the screen and we don't get any notification once the
menu is popped-down.
This patch fixes the problem by giving WebKit a way to clear the
controller's context menu. WebKit now clears the menu before it
propagates a new context menu event through the DOM.
- WebCore.exp:
- manual-tests/DOMContextMenuEvent.html: Added.
- page/ContextMenuController.cpp: (WebCore::ContextMenuController::clearContextMenu):
- page/ContextMenuController.h:
WebKit:
Reviewed by Adam.
Fix for http://bugs.webkit.org/show_bug.cgi?id=12399 REGRESSION:
Unable to prevent default context menu from appearing. < rdar://
problem/5017416>
- WebView/WebHTMLView.mm: (-[WebHTMLView menuForEvent:]): Clear the controller's context menu before propagating a new context menu event through the DOM.
- 15:44 Changeset [22985] by
-
2007-02-22 Oliver Hunt < oliver@apple.com>
Reviewed by Ada, Lou, and Steve.
WebKit:
Last of the major changes for drag and drop
- Move COMPtr.h to WebCore
- Added WebDragSource, a modified version of SDataSource from the App that can notify a WebView's eventhandler of drag events
- Implement the last bits of the DragClient
- As part of drag client added code to draw the "doubled" text used by drag links. May be useful for other parts of WebKit/App
- COMPtr.h: Removed. Moving to WebCore
- WebDragClient.cpp: (WebDragClient::actionMaskForDrag): (WebDragClient::willPerformDragDestinationAction): (WebDragClient::dragSourceActionMaskForPoint): (WebDragClient::willPerformDragSourceAction):
- Removing WebCore
- as i'm now using the WebCore namespace
(WebDragClient::startDrag):
Implemented client method
(allocImage):
(dragLabelFont):
Helper functions for generating the link dragging image
(WebDragClient::createDragImageForLink):
Implemented client method
- WebDropSource.cpp: Added. (WebDropSource::createInstance): (WebDropSource::WebDropSource): (WebDropSource::QueryInterface): (WebDropSource::AddRef): (WebDropSource::Release): (generateMouseEvent): Generates a mouse event for the EventHandler (WebDropSource::QueryContinueDrag): (WebDropSource::GiveFeedback): Copied from SDataObject in the App. Minor changes per suggestions from Ada and Steve (error codes, potential leaks) Also includes logic to notify the source WebView of drag events (moving and drag end)
- WebDropSource.h: Added. Header for the above
- WebKit.vcproj/WebKit.vcproj: Removing COMPtr.h, and adding WebDropSource.{h,cpp}
- WebKitGraphics.cpp: (DrawDoubledTextAtPoint): Utility function for drawing double text, as used by link dragging image
- WebCore.vcproj/WebCore.vcproj: Adding new files
- page/win/DragControllerWin.cpp: (WebCore::DragController::maxDragImageSize): Maximum size of drag image to allow sharp edges is 208x208
- page/win/EventHandlerWin.cpp: (WebCore::EventHandler::createDraggingClipboard): Properly implemented now
- platform/win/ClipboardWin.cpp: (WebCore::): (WebCore::clipboardTypeFromMIMEType): (WebCore::ClipboardWin::ClipboardWin): (WebCore::ClipboardWin::~ClipboardWin): (WebCore::writeURL): Helper function to write a url to a WCDataObject (WebCore::ClipboardWin::clearData): (WebCore::ClipboardWin::clearAllData): (WebCore::ClipboardWin::getData): (WebCore::ClipboardWin::setData): (WebCore::addMimeTypesForFormat): Helper function for ClipboardWin::types (WebCore::ClipboardWin::types): (WebCore::ClipboardWin::setDragImage): (WebCore::ClipboardWin::setDragImageElement): General setters, match behaviour of WebKit/Mac (WebCore::ClipboardWin::createDragImage): Creates a drag image for a DOM specified Image is specified can't currently generate an image for an arbitrary element (WebCore::imageToMarkup): Wraps a url in an img tag (WebCore::ClipboardWin::declareAndWriteDragImage): (WebCore::ClipboardWin::writeURL): (WebCore::ClipboardWin::writeRange): (WebCore::ClipboardWin::hasData): Implement Clipboard required by DragController. Provides read/write DOM Clipboard implementation for dragging, read only when accessing system clipboard.
- platform/win/ClipboardWin.h: (WebCore::ClipboardWin::dataObject): Provide an accessor for the dataObject
- platform/win/DragDataWin.cpp: (WebCore::urlWFormat): (WebCore::urlFormat): (WebCore::plainTextFormat): (WebCore::plainTextWFormat): (WebCore::texthtmlFormat): (WebCore::htmlFormat): (WebCore::getURL): (WebCore::DragData::asURL): (WebCore::getPlainText): (WebCore::DragData::asPlainText): (WebCore::DragData::asFragment): Refactoring and removing of a few static modifiers to allow sharing with Clipboard
- platform/win/DragImageWin.cpp: (WebCore::dragImageSize): (WebCore::deleteDragImage): (WebCore::allocImage): (WebCore::createCgContextFromBitmap): (WebCore::scaleDragImage): (WebCore::dissolveDragImageToFraction): (WebCore::createDragImageFromImage): (WebCore::createDragImageIconForCachedImage): Implement DragImage functions
- platform/win/EditorWin.cpp: (WebCore::Editor::newGeneralClipboard): Creating a Clipboard that wraps the system clipboard
- platform/win/PasteboardWin.cpp: (WebCore::createGlobalData): (WebCore::markupToCF_HTML): (WebCore::urlToMarkup): (WebCore::replaceNBSP): removing 'static'
- platform/win/WCDataObject.cpp: Added. (WebCore::WCEnumFormatEtc::WCEnumFormatEtc): (WebCore::WCEnumFormatEtc::QueryInterface): (WebCore::WCEnumFormatEtc::AddRef): (WebCore::WCEnumFormatEtc::Release): (WebCore::WCEnumFormatEtc::Next): (WebCore::WCEnumFormatEtc::Skip): (WebCore::WCEnumFormatEtc::Reset): (WebCore::WCEnumFormatEtc::Clone): (WebCore::WCDataObject::createInstance): (WebCore::WCDataObject::WCDataObject): (WebCore::WCDataObject::~WCDataObject): (WebCore::WCDataObject::QueryInterface): (WebCore::WCDataObject::AddRef): (WebCore::WCDataObject::Release): (WebCore::WCDataObject::GetData): (WebCore::WCDataObject::GetDataHere): (WebCore::WCDataObject::QueryGetData): (WebCore::WCDataObject::GetCanonicalFormatEtc): (WebCore::WCDataObject::SetData): (WebCore::WCDataObject::CopyMedium): (WebCore::WCDataObject::EnumFormatEtc): (WebCore::WCDataObject::DAdvise): (WebCore::WCDataObject::DUnadvise): (WebCore::WCDataObject::EnumDAdvise): (WebCore::WCDataObject::clearData): Copied from SDataObject in the App createInstance and clearData are the only new functions. clearData removes any entries in the object with a specified CLIPFORMAT. Have also made a few error and memory handling changes suggested by Steve and Ada
- platform/win/WCDataObject.h: Added. Header for the above
- 15:42 Changeset [19809] by
-
Reviewed by Kevin and Adele
- fixed < rdar://problem/4129417> Need final art for textarea's resize corner
This probably broke some layout pixel tests since the image changed. I'm going to
track those down in a moment (non-trivial since there are a bunch failing for other
reasons).
- Resources/textAreaResizeCorner.tiff: new art
- Resources/deleteButton.tiff:
- Resources/deleteButtonPressed.tiff: these got smaller when I ran compress-tiffs on this directory
- 15:40 Changeset [19808] by
-
2007-02-22 Oliver Hunt < oliver@apple.com>
Reviewed by Ada, Lou and Steve.
Moving basic components of ClipboardMac to Clipboard,
last of the basic common code merge will come later (namely setting the
drag element/image).
Tidying up in DragController including a few bad PassRefPtr uses and
removing unnecessary accessors. Also initialise fields properly to
fix <rdar:/problems/5014889>
- dom/Clipboard.cpp: (WebCore::Clipboard::Clipboard): New constructor
- dom/Clipboard.h: (WebCore::Clipboard::isForDragging): (WebCore::Clipboard::dragLocation): (WebCore::Clipboard::dragImage): (WebCore::Clipboard::dragImageElement): Exposing common data
- page/DragController.cpp: (WebCore::DragController::DragController): Initialise m_isHandlingDrag (oops) (WebCore::documentFragmentFromDragData): (WebCore::DragController::tryDocumentDrag): Fixing PassRefPtr misuse (WebCore::DragController::canProcessDrag): (WebCore::DragController::startDrag): (WebCore::DragController::doSystemDrag): Make sure we're using the correct coordinate space
- page/EventHandler.cpp: (WebCore::EventHandler::handleDrag): Reset drag source correctly
- platform/mac/ClipboardMac.h:
- platform/mac/ClipboardMac.mm: (WebCore::ClipboardMac::ClipboardMac): update constructor to use new Clipboard constructor
- 15:07 Changeset [22984] by
-
Reviewed by Adele.
Make sure we always paint the right size search results icon.
- 15:02 Changeset [22983] by
-
Bump version for submit
- 15:00 Changeset [19807] by
-
New tag.
- 14:20 Changeset [19806] by
-
LayoutTests:
Reviewed by Darin.
Test for < rdar://problem/5014970> 9A374: Tabs don't work in the message body
- fast/events/keypress-insert-tab-expected.txt: Added.
- fast/events/keypress-insert-tab.html: Added.
WebCore:
Reviewed by Darin.
Fix for < rdar://problem/5014970> 9A374: Tabs don't work in the message body
Test: fast/events/keypress-insert-tab.html
- page/EventHandler.cpp: (WebCore::EventHandler::defaultTextInputEventHandler): If the defaultTabEventHandler doesn't handle the event, then go on to the insertText case.
WebKitTools:
Reviewed by Darin.
Added option in layoutTestController to setTabKeyCyclesThroughElements, so we
can test < rdar://problem/5014970> 9A374: Tabs don't work in the message body
- DumpRenderTree/DumpRenderTree.m: (+[LayoutTestController isSelectorExcludedFromWebScript:]): (+[LayoutTestController webScriptNameForSelector:]): (-[LayoutTestController setTabKeyCyclesThroughElements:]):
- 14:10 Changeset [19805] by
-
Reviewed by Darin Adler.
Fixed < rdar://problem/4942528> Installer.app Stuck "Examining additional
volumes..."
The problem was that JavaScriptGlue held a JSObject in a static pointer
without protecting it from garbage collection.
sUserObjectPrototypeImp = new UserObjectPrototypeImp();
static ProtectedPtr<UserObjectPrototypeImp> protectPrototype; // valiant, brave, but useless
Through an interesting comedy of errors, this object would be collected
and then assigned as the prototype of another object allocated at the
same address, thus becoming its own prototype. (See http://www.ziplo.com/grandpa.htm
for further explanation of this phenomenon.) Hence the infinte loop in
property lookup.
The solution is equally humorous: all of this code was completely useless --
it only served to create an empty object in the prototype chain -- so
I just removed it.
- UserObjectImp.cpp: (UserObjectImp::UserObjectImp):
- UserObjectImp.h:
- 13:55 Changeset [19804] by
-
naguo, reviewed by sachin
DESC: Open while downloading UI setting functionality is not working correctly SCHY-6YFQ66
http://bugs.webkit.org/show_bug.cgi?id=12846
- 13:25 Changeset [19803] by
-
hongzhao, reviewed by Zalan, merged by mbradley++
DESC: merge of r19802 to s60 trunk. Cache memory collector is not initialized in 3.2 HZHO-6YMQCK
Fix: Initialize the initial size of cache memory collector after create it.
http://bugs.webkit.org/show_bug.cgi?id=12844
- 13:22 Changeset [19802] by
-
hongzhao, Reviewed by Zalan
DESC: [S60] Cache memory collector is not initialized in 3.1m HZHO-6YMQCK
Fix: Initialze the initial size of cache mem collector after create it.
http://bugs.webkit.org/show_bug.cgi?id=12844
- 13:16 Changeset [19801] by
-
sareen by yongjun
DESC: Merge of r19655 to s60/3.1m. Selection list not shown correctly in www.elisa.fi HMNN-6XQJMX
http://bugs.webkit.org/show_bug.cgi?id=12719
- 13:00 Changeset [19800] by
-
Reviewed by Darin.
< rdar://problem/4948887> REGRESSION: Correct differences in public DOM API that inadvertently changed in Leopard
- bindings/objc/PublicDOMInterfaces.h: Changes return types for two size properties, adds deprecated DOMEventTarget methods.
- bindings/scripts/CodeGeneratorObjC.pm: Fixed public API checking for protocols, adds support for ConvertToString.
- dom/EventTarget.idl: Marks addEventListener and removeEventListener with the OldStyleObjC attribute.
- html/HTMLBaseFontElement.idl: Changes the size property to a DOMString for ObjC only.
- html/HTMLInputElement.idl: Changes the size property to a DOMString for ObjC only.
- html/HTMLOptionsCollection.idl: Changes the length property to be unsigned.
- 12:50 Changeset [19799] by
-
Reviewed by John.
Fix < rdar://problem/5016969> REGRESSION: "Check Spelling As You Type" context menu item is never checked
- platform/ContextMenu.cpp: (WebCore::ContextMenu::checkOrEnableIfNeeded): Set shouldCheck for ContextMenuItemTagCheckSpellingWhileTyping.
- 12:19 Changeset [19798] by
-
LayoutTests:
Used svn merge -r19786:19785 to roll out previous SVG results produced by
hack to work around SVG painting issue in DRT.
The new work-around preserves the old SVG results -- both text and pixel.
This should fix the ~400 new pixel failures introduced in r19786.
- svg/W3C-SVG-1.1/animate-elem-02-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-03-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-04-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-05-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-06-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-07-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-08-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-13-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-14-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-15-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-16-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-17-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-18-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-19-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-20-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-21-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-24-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-29-b-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-33-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-36-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-39-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-40-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-46-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-77-t-expected.txt:
- svg/W3C-SVG-1.1/filters-blend-01-b-expected.txt:
- svg/W3C-SVG-1.1/fonts-elem-02-t-expected.txt:
- svg/W3C-SVG-1.1/interact-cursor-01-f-expected.txt:
- svg/W3C-SVG-1.1/interact-events-01-b-expected.txt:
- svg/W3C-SVG-1.1/linking-uri-01-b-expected.txt:
- svg/W3C-SVG-1.1/linking-uri-02-b-expected.txt:
- svg/W3C-SVG-1.1/masking-mask-01-b-expected.txt:
- svg/W3C-SVG-1.1/masking-path-04-b-expected.txt:
- svg/W3C-SVG-1.1/metadata-example-01-b-expected.txt:
- svg/W3C-SVG-1.1/painting-fill-02-t-expected.txt:
- svg/W3C-SVG-1.1/painting-marker-03-f-expected.txt:
- svg/W3C-SVG-1.1/pservers-grad-05-b-expected.txt:
- svg/W3C-SVG-1.1/pservers-grad-08-b-expected.txt:
- svg/W3C-SVG-1.1/pservers-grad-11-b-expected.txt:
- svg/W3C-SVG-1.1/render-elems-06-t-expected.txt:
- svg/W3C-SVG-1.1/render-elems-07-t-expected.txt:
- svg/W3C-SVG-1.1/render-elems-08-t-expected.txt:
- svg/W3C-SVG-1.1/script-handle-01-b-expected.txt:
- svg/W3C-SVG-1.1/struct-frag-06-t-expected.txt:
- svg/W3C-SVG-1.1/struct-use-01-t-expected.txt:
- svg/W3C-SVG-1.1/text-align-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-02-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-03-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-04-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-05-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-06-b-expected.txt:
- svg/W3C-SVG-1.1/text-altglyph-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-deco-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-fonts-01-t-expected.txt:
- svg/W3C-SVG-1.1/text-fonts-02-t-expected.txt:
- svg/W3C-SVG-1.1/text-text-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-text-03-b-expected.txt:
- svg/W3C-SVG-1.1/text-text-08-b-expected.txt:
- svg/W3C-SVG-1.1/text-tref-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-tselect-02-f-expected.txt:
- svg/W3C-SVG-1.1/text-tspan-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-ws-01-t-expected.txt:
- svg/W3C-SVG-1.1/text-ws-02-t-expected.txt:
- svg/carto.net/button-expected.txt:
- svg/carto.net/colourpicker-expected.txt:
- svg/carto.net/combobox-expected.txt:
- svg/carto.net/scrollbar-expected.txt:
- svg/carto.net/selectionlist-expected.txt:
- svg/carto.net/slider-expected.txt:
- svg/carto.net/tabgroup-expected.txt:
- svg/carto.net/textbox-expected.txt:
- svg/carto.net/window-expected.txt:
- svg/custom/gradient-stop-style-change-expected.txt:
- svg/custom/js-late-clipPath-and-object-creation-expected.txt:
- svg/custom/js-late-clipPath-creation-expected.txt:
- svg/custom/js-late-gradient-and-object-creation-expected.txt:
- svg/custom/non-opaque-filters-expected.txt:
- svg/custom/pointer-events-text-expected.txt:
- svg/custom/text-clip-expected.txt:
- svg/custom/text-filter-expected.txt:
- svg/custom/text-image-opacity-expected.txt:
- svg/custom/text-repaint-including-stroke-expected.txt:
- svg/custom/use-elementInstance-event-target-expected.txt:
- svg/custom/use-on-text-expected.txt:
WebKitTools:
Used svn merge -r19786:19785 to roll out previous hack to work around
SVG painting issue in DRT.
Implemented new work-around, which makes --paint paint *after* dumping
the render tree instead of before, so that painting doesn't influence
the SVG render tree.
This should fix the ~400 new pixel failures introduced in r19786.
- DumpRenderTree/DumpRenderTree.m: (dump):
- Scripts/run-webkit-tests:
- 11:57 Changeset [19797] by
-
LayoutTests:
Reviewed by Darin.
- tests for http://bugs.webkit.org/show_bug.cgi?id=11447 REGRESSION(NativeListBox): List not scrolled to preselected option < rdar://problem/4957463>
- fast/forms/select-initial-position-expected.checksum: Added.
- fast/forms/select-initial-position-expected.png: Added.
- fast/forms/select-initial-position-expected.txt: Added.
- fast/forms/select-initial-position.html: Added.
WebCore:
Reviewed by Darin.
- fix http://bugs.webkit.org/show_bug.cgi?id=11447 REGRESSION(NativeListBox): List not scrolled to preselected option < rdar://problem/4957463>
Initiate scroll when selected <option> is added to <select>.
- html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::insertedIntoDocument):
- html/HTMLOptionElement.h:
- html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::notifyOptionSelected): (WebCore::HTMLSelectElement::updateListBoxSelection): (WebCore::HTMLSelectElement::scrollToSelection):
- html/HTMLSelectElement.h:
- 11:32 Changeset [19796] by
-
Fix sites with fixed positioned backgrounds. (my blog =) )
r=Lars,George
- 11:18 Changeset [19795] by
-
Resetting --leaks mode default to run 1000 tests at a time because setting it
to 750 didn't stop the stack logging related crashes, and running more tests
at a time is faster. The crashes seem to be a real bug in stack logging,
not an out of memory condition.
- Scripts/run-webkit-tests:
- 09:38 Changeset [19794] by
-
Reviewed by Darin
Removed some unused keyView-related code that I happened to run across.
Replacement code is now in WebChromeClient.
- WebCoreSupport/WebFrameBridge.h:
- WebCoreSupport/WebFrameBridge.mm: removed unused stuff
- 08:52 Changeset [19793] by
-
Fixing the default scroll offset, cleaning
up private methods and making the default
look not use any frames.
r=Lars
- 08:17 Changeset [19792] by
-
Properly implement canvas/scrollview's. Get
rid of ScrollViewCanvasQt and replace it with
a tailor abstractscrollview (now in qwebframe).
Scrolling of pages is finally completely smooth :)
r=Lars
- 07:59 Changeset [19791] by
-
Reviewed by Maciej.
Fix ARM-specific alignment problem in FastMalloc
http://bugs.webkit.org/show_bug.cgi?id=12841
- wtf/FastMalloc.cpp: Modify how pageheap_memory is declared to ensure proper alignment on architectures such as ARM
- 07:45 Changeset [19790] by
-
Reviewed by Maciej.
- fix http://bugs.webkit.org/show_bug.cgi?id=12804 REGRESSION (r19043-r19063): suggestion popup doesn't work after pressing Option+Esc
This patch also fixes a bug where using the arrow keys while
the suggestion popup is open moves the caret instead of changing the
selection in the popup (for up/down) or accepting the selection and
closing the popup (for left/right).
- WebView/WebHTMLView.mm: (-[WebHTMLView keyDown:]): Changed to close the popup only if it was open before the current event, so that the Option-Esc that opens the popup will not close it immediately. (-[WebHTMLView _interceptEditingKeyEvent:]): Give the completion popup a chance to intercept keydown events. (-[WebTextCompleteController popupWindowIsOpen]): Added. Returns whether the suggestion popup is open.
- 07:34 Changeset [19789] by
-
Reviewed by Mark (age 21).
- fix http://bugs.webkit.org/show_bug.cgi?id=12805 REGRESSION: suggestion popup has a disabled scroll bar
- WebView/WebHTMLView.mm: (-[WebTextCompleteController _buildUI]): Uncommented the call to the NSWindow SPI that forces the scroll bar to look active. Also replaced a call to the deprecated NSTableView method setAutoresizesAllColumnsToFit: with the new method setColumnAutoresizingStyle: to eliminate console spew.
02/21/07:
- 23:20 Changeset [19788] by
-
Reviewed by Adam Roben.
Improve notImplemented() macro for gdk. Rename to notImplementedGdk() to
give it a unique name (so that it doesn't clash with notImplemented()
for other platforms).
Make it print a given warning only once, so that frequently called
functions don't clutter the output.
Don't print the warning if DISABLE_NI_WARNING env variable is set.
- loader/gdk/FrameLoaderClientGdk.cpp: (WebCore::FrameLoaderClientGdk::dispatchDidReceiveAuthenticationChallenge): (WebCore::FrameLoaderClientGdk::dispatchDidCancelAuthenticationChallenge): (WebCore::FrameLoaderClientGdk::dispatchWillSendRequest): (WebCore::FrameLoaderClientGdk::assignIdentifierToInitialRequest): (WebCore::FrameLoaderClientGdk::createPlugin): (WebCore::FrameLoaderClientGdk::createFrame): (WebCore::FrameLoaderClientGdk::redirectDataToPlugin): (WebCore::FrameLoaderClientGdk::createJavaAppletWidget): (WebCore::FrameLoaderClientGdk::objectContentType): (WebCore::FrameLoaderClientGdk::overrideMediaType): (WebCore::FrameLoaderClientGdk::windowObjectCleared): (WebCore::FrameLoaderClientGdk::hasWebView): (WebCore::FrameLoaderClientGdk::hasFrameView): (WebCore::FrameLoaderClientGdk::frameLoadCompleted): (WebCore::FrameLoaderClientGdk::saveViewStateToItem): (WebCore::FrameLoaderClientGdk::restoreViewState): (WebCore::FrameLoaderClientGdk::privateBrowsingEnabled): (WebCore::FrameLoaderClientGdk::makeDocumentView): (WebCore::FrameLoaderClientGdk::makeRepresentation): (WebCore::FrameLoaderClientGdk::forceLayout): (WebCore::FrameLoaderClientGdk::forceLayoutForNonHTML): (WebCore::FrameLoaderClientGdk::setCopiesOnScroll): (WebCore::FrameLoaderClientGdk::detachedFromParent1): (WebCore::FrameLoaderClientGdk::detachedFromParent2): (WebCore::FrameLoaderClientGdk::detachedFromParent3): (WebCore::FrameLoaderClientGdk::detachedFromParent4): (WebCore::FrameLoaderClientGdk::loadedFromPageCache): (WebCore::FrameLoaderClientGdk::dispatchDidHandleOnloadEvents): (WebCore::FrameLoaderClientGdk::dispatchDidReceiveServerRedirectForProvisionalLoad): (WebCore::FrameLoaderClientGdk::dispatchDidCancelClientRedirect): (WebCore::FrameLoaderClientGdk::dispatchWillPerformClientRedirect): (WebCore::FrameLoaderClientGdk::dispatchDidChangeLocationWithinPage): (WebCore::FrameLoaderClientGdk::dispatchWillClose): (WebCore::FrameLoaderClientGdk::dispatchDidReceiveIcon): (WebCore::FrameLoaderClientGdk::dispatchDidStartProvisionalLoad): (WebCore::FrameLoaderClientGdk::dispatchDidReceiveTitle): (WebCore::FrameLoaderClientGdk::dispatchDidCommitLoad): (WebCore::FrameLoaderClientGdk::dispatchDidFinishDocumentLoad): (WebCore::FrameLoaderClientGdk::dispatchDidFirstLayout): (WebCore::FrameLoaderClientGdk::dispatchShow): (WebCore::FrameLoaderClientGdk::cancelPolicyCheck): (WebCore::FrameLoaderClientGdk::dispatchDidLoadMainResource): (WebCore::FrameLoaderClientGdk::revertToProvisionalState): (WebCore::FrameLoaderClientGdk::clearUnarchivingState): (WebCore::FrameLoaderClientGdk::willChangeTitle): (WebCore::FrameLoaderClientGdk::finishedLoading): (WebCore::FrameLoaderClientGdk::finalSetupForReplace): (WebCore::FrameLoaderClientGdk::setDefersLoading): (WebCore::FrameLoaderClientGdk::isArchiveLoadPending): (WebCore::FrameLoaderClientGdk::cancelPendingArchiveLoad): (WebCore::FrameLoaderClientGdk::clearArchivedResources): (WebCore::FrameLoaderClientGdk::canHandleRequest): (WebCore::FrameLoaderClientGdk::canShowMIMEType): (WebCore::FrameLoaderClientGdk::representationExistsForURLScheme): (WebCore::FrameLoaderClientGdk::generatedMIMETypeForURLScheme): (WebCore::FrameLoaderClientGdk::provisionalLoadStarted): (WebCore::FrameLoaderClientGdk::didFinishLoad): (WebCore::FrameLoaderClientGdk::prepareForDataSourceReplacement): (WebCore::FrameLoaderClientGdk::setTitle): (WebCore::FrameLoaderClientGdk::setDocumentViewFromPageCache): (WebCore::FrameLoaderClientGdk::dispatchDidReceiveContentLength): (WebCore::FrameLoaderClientGdk::dispatchDidFinishLoading): (WebCore::FrameLoaderClientGdk::dispatchDidFailLoading): (WebCore::FrameLoaderClientGdk::dispatchDidLoadResourceFromMemoryCache): (WebCore::FrameLoaderClientGdk::dispatchDidFailProvisionalLoad): (WebCore::FrameLoaderClientGdk::dispatchDidFailLoad): (WebCore::FrameLoaderClientGdk::download): (WebCore::FrameLoaderClientGdk::cancelledError): (WebCore::FrameLoaderClientGdk::cannotShowURLError): (WebCore::FrameLoaderClientGdk::interruptForPolicyChangeError): (WebCore::FrameLoaderClientGdk::cannotShowMIMETypeError): (WebCore::FrameLoaderClientGdk::fileDoesNotExistError): (WebCore::FrameLoaderClientGdk::shouldFallBack): (WebCore::FrameLoaderClientGdk::willUseArchive): (WebCore::FrameLoaderClientGdk::saveDocumentViewToPageCache): (WebCore::FrameLoaderClientGdk::canCachePage): (WebCore::FrameLoaderClientGdk::dispatchCreatePage): (WebCore::FrameLoaderClientGdk::dispatchUnableToImplementPolicy): (WebCore::FrameLoaderClientGdk::setMainDocumentError): (WebCore::FrameLoaderClientGdk::startDownload): (WebCore::FrameLoaderClientGdk::updateGlobalHistoryForStandardLoad): (WebCore::FrameLoaderClientGdk::updateGlobalHistoryForReload): (WebCore::FrameLoaderClientGdk::shouldGoToHistoryItem):
- page/gdk/EventHandlerGdk.cpp: (WebCore::EventHandler::passMouseDownEventToWidget): (WebCore::EventHandler::eventActivatedView): (WebCore::EventHandler::passSubframeEventToSubframe): (WebCore::EventHandler::passWheelEventToWidget): (WebCore::EventHandler::createDraggingClipboard):
- platform/gdk/EditorClientGdk.cpp: (WebCore::EditorClientGdk::shouldDeleteRange): (WebCore::EditorClientGdk::isContinuousSpellCheckingEnabled): (WebCore::EditorClientGdk::isGrammarCheckingEnabled): (WebCore::EditorClientGdk::spellCheckerDocumentTag): (WebCore::EditorClientGdk::shouldBeginEditing): (WebCore::EditorClientGdk::shouldEndEditing): (WebCore::EditorClientGdk::shouldInsertText): (WebCore::EditorClientGdk::shouldChangeSelectedRange): (WebCore::EditorClientGdk::shouldApplyStyle): (WebCore::EditorClientGdk::didBeginEditing): (WebCore::EditorClientGdk::respondToChangedContents): (WebCore::EditorClientGdk::didEndEditing): (WebCore::EditorClientGdk::didWriteSelectionToPasteboard): (WebCore::EditorClientGdk::didSetSelectionTypesForPasteboard): (WebCore::EditorClientGdk::selectWordBeforeMenuEvent): (WebCore::EditorClientGdk::isEditable): (WebCore::EditorClientGdk::registerCommandForUndo): (WebCore::EditorClientGdk::registerCommandForRedo): (WebCore::EditorClientGdk::clearUndoRedoOperations): (WebCore::EditorClientGdk::canUndo): (WebCore::EditorClientGdk::canRedo): (WebCore::EditorClientGdk::undo): (WebCore::EditorClientGdk::redo): (WebCore::EditorClientGdk::shouldInsertNode): (WebCore::EditorClientGdk::pageDestroyed): (WebCore::EditorClientGdk::smartInsertDeleteEnabled): (WebCore::EditorClientGdk::toggleContinuousSpellChecking): (WebCore::EditorClientGdk::textFieldDidBeginEditing): (WebCore::EditorClientGdk::textFieldDidEndEditing): (WebCore::EditorClientGdk::textDidChangeInTextField): (WebCore::EditorClientGdk::doTextFieldCommandFromEvent): (WebCore::EditorClientGdk::textWillBeDeletedInTextField): (WebCore::EditorClientGdk::textDidChangeInTextArea):
- platform/gdk/RenderThemeGdk.cpp: (WebCore::RenderThemeGdk::setCheckboxSize): (WebCore::RenderThemeGdk::setRadioSize): (WebCore::RenderThemeGdk::adjustTextFieldStyle):
- platform/gdk/TemporaryLinkStubs.cpp: (FrameView::updateBorder): (Widget::setEnabled): (Widget::isEnabled): (Widget::focusPolicy): (Widget::enableFlushDrawing): (Widget::disableFlushDrawing): (Widget::lockDrawingFocus): (Widget::unlockDrawingFocus): (Widget::removeFromParent): (Widget::paint): (Widget::setIsSelected): (Widget::invalidate): (Widget::invalidateRect): (PlatformMouseEvent::PlatformMouseEvent): (WebCore::findNextSentenceFromIndex): (WebCore::findSentenceBoundary): (WebCore::findNextWordFromIndex): (ChromeClientGdk::chromeDestroyed): (ChromeClientGdk::windowRect): (ChromeClientGdk::setWindowRect): (ChromeClientGdk::pageRect): (ChromeClientGdk::scaleFactor): (ChromeClientGdk::focus): (ChromeClientGdk::unfocus): (ChromeClientGdk::createWindow): (ChromeClientGdk::createModalDialog): (ChromeClientGdk::show): (ChromeClientGdk::canRunModal): (ChromeClientGdk::runModal): (ChromeClientGdk::setToolbarsVisible): (ChromeClientGdk::toolbarsVisible): (ChromeClientGdk::setStatusbarVisible): (ChromeClientGdk::statusbarVisible): (ChromeClientGdk::setScrollbarsVisible): (ChromeClientGdk::scrollbarsVisible): (ChromeClientGdk::setMenubarVisible): (ChromeClientGdk::menubarVisible): (ChromeClientGdk::setResizable): (ChromeClientGdk::closeWindowSoon): (ChromeClientGdk::canTakeFocus): (ChromeClientGdk::takeFocus): (ChromeClientGdk::canRunBeforeUnloadConfirmPanel): (ChromeClientGdk::addMessageToConsole): (ChromeClientGdk::runBeforeUnloadConfirmPanel): (ChromeClientGdk::runJavaScriptAlert): (ChromeClientGdk::runJavaScriptConfirm): (ChromeClientGdk::runJavaScriptPrompt): (ChromeClientGdk::setStatusbarText): (ChromeClientGdk::shouldInterruptJavaScript): (ChromeClientGdk::tabsToLinks): (ChromeClientGdk::windowResizerRect): (ChromeClientGdk::addToDirtyRegion): (ChromeClientGdk::scrollBackingStore): (ChromeClientGdk::updateBackingStore): (PlugInInfoStore::createPluginInfoForPluginAtIndex): (PlugInInfoStore::pluginCount): (WebCore::PlugInInfoStore::supportsMIMEType): (WebCore::refreshPlugins): (SearchPopupMenu::saveRecentSearches): (SearchPopupMenu::loadRecentSearches): (SearchPopupMenu::SearchPopupMenu): (PlatformScrollbar::PlatformScrollbar): (PlatformScrollbar::~PlatformScrollbar): (PlatformScrollbar::setEnabled): (PlatformScrollbar::paint): (PlatformScrollbar::updateThumbPosition): (PlatformScrollbar::updateThumbProportion): (PlatformScrollbar::setRect): (FileChooser::FileChooser): (FileChooser::~FileChooser): (FileChooser::openFileChooser): (FileChooser::basenameForWidth): (ResourceHandle::willLoadFromCache): (ResourceHandle::loadsBlocked): (ResourceHandle::loadResourceSynchronously): (Icon::Icon): (Icon::~Icon): (Icon::newIconForFile): (Icon::paint): (Font::drawComplexText): (Font::floatWidthForComplexText): (Font::offsetForPositionForComplexText): (PageCache::close): (Editor::ignoreSpelling): (Editor::learnSpelling): (Editor::isSelectionUngrammatical): (Editor::isSelectionMisspelled): (Editor::guessesForMisspelledSelection): (Editor::guessesForUngrammaticalSelection): (Editor::markMisspellingsAfterTypingToPosition): (Editor::newGeneralClipboard): (Pasteboard::generalPasteboard): (Pasteboard::writeSelection): (Pasteboard::writeURL): (Pasteboard::writeImage): (Pasteboard::clear): (Pasteboard::canSmartReplace): (Pasteboard::documentFragment): (Pasteboard::plainText): (Pasteboard::Pasteboard): (Pasteboard::~Pasteboard): (ContextMenu::ContextMenu): (ContextMenu::~ContextMenu): (ContextMenu::appendItem): (ContextMenuItem::ContextMenuItem): (ContextMenuItem::~ContextMenuItem): (ContextMenuItem::releasePlatformDescription): (ContextMenuItem::type): (ContextMenuItem::setType): (ContextMenuItem::action): (ContextMenuItem::setAction): (ContextMenuItem::title): (ContextMenuItem::setTitle): (ContextMenuItem::platformSubMenu): (ContextMenuItem::setSubMenu): (ContextMenuItem::setChecked): (ContextMenuItem::setEnabled): (WebCore::supportedKeySizes): (WebCore::systemBeep): (WebCore::userIdleTime):
- 22:40 Changeset [19787] by
-
Add missing files from last commit
- 22:17 Changeset [19786] by
-
LayoutTests:
Reviewed by Adam Roben.
Updated results to new, more correct values produced by painting SVGs
before dumping their render trees.
- svg/W3C-SVG-1.1/animate-elem-02-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-03-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-04-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-05-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-06-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-07-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-08-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-13-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-14-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-15-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-16-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-17-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-18-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-19-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-20-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-21-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-24-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-29-b-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-33-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-36-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-39-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-40-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-46-t-expected.txt:
- svg/W3C-SVG-1.1/animate-elem-77-t-expected.txt:
- svg/W3C-SVG-1.1/filters-blend-01-b-expected.txt:
- svg/W3C-SVG-1.1/fonts-elem-02-t-expected.txt:
- svg/W3C-SVG-1.1/interact-cursor-01-f-expected.txt:
- svg/W3C-SVG-1.1/interact-events-01-b-expected.txt:
- svg/W3C-SVG-1.1/linking-uri-01-b-expected.txt:
- svg/W3C-SVG-1.1/linking-uri-02-b-expected.txt:
- svg/W3C-SVG-1.1/masking-mask-01-b-expected.txt:
- svg/W3C-SVG-1.1/masking-path-04-b-expected.txt:
- svg/W3C-SVG-1.1/metadata-example-01-b-expected.txt:
- svg/W3C-SVG-1.1/painting-fill-02-t-expected.txt:
- svg/W3C-SVG-1.1/painting-marker-03-f-expected.txt:
- svg/W3C-SVG-1.1/pservers-grad-05-b-expected.txt:
- svg/W3C-SVG-1.1/pservers-grad-08-b-expected.txt:
- svg/W3C-SVG-1.1/pservers-grad-11-b-expected.txt:
- svg/W3C-SVG-1.1/render-elems-06-t-expected.txt:
- svg/W3C-SVG-1.1/render-elems-07-t-expected.txt:
- svg/W3C-SVG-1.1/render-elems-08-t-expected.txt:
- svg/W3C-SVG-1.1/script-handle-01-b-expected.txt:
- svg/W3C-SVG-1.1/struct-frag-06-t-expected.txt:
- svg/W3C-SVG-1.1/struct-use-01-t-expected.txt:
- svg/W3C-SVG-1.1/text-align-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-02-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-03-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-04-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-05-b-expected.txt:
- svg/W3C-SVG-1.1/text-align-06-b-expected.txt:
- svg/W3C-SVG-1.1/text-altglyph-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-deco-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-fonts-01-t-expected.txt:
- svg/W3C-SVG-1.1/text-fonts-02-t-expected.txt:
- svg/W3C-SVG-1.1/text-text-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-text-03-b-expected.txt:
- svg/W3C-SVG-1.1/text-text-08-b-expected.txt:
- svg/W3C-SVG-1.1/text-tref-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-tselect-02-f-expected.txt:
- svg/W3C-SVG-1.1/text-tspan-01-b-expected.txt:
- svg/W3C-SVG-1.1/text-ws-01-t-expected.txt:
- svg/W3C-SVG-1.1/text-ws-02-t-expected.txt:
- svg/carto.net/button-expected.txt:
- svg/carto.net/colourpicker-expected.txt:
- svg/carto.net/combobox-expected.txt:
- svg/carto.net/scrollbar-expected.txt:
- svg/carto.net/selectionlist-expected.txt:
- svg/carto.net/slider-expected.txt:
- svg/carto.net/tabgroup-expected.txt:
- svg/carto.net/textbox-expected.txt:
- svg/carto.net/window-expected.txt:
- svg/custom/gradient-stop-style-change-expected.txt:
- svg/custom/js-late-clipPath-and-object-creation-expected.txt:
- svg/custom/js-late-clipPath-creation-expected.txt:
- svg/custom/js-late-gradient-and-object-creation-expected.txt:
- svg/custom/non-opaque-filters-expected.txt:
- svg/custom/pointer-events-text-expected.txt:
- svg/custom/text-clip-expected.txt:
- svg/custom/text-filter-expected.txt:
- svg/custom/text-image-opacity-expected.txt:
- svg/custom/text-repaint-including-stroke-expected.txt:
- svg/custom/use-elementInstance-event-target-expected.txt:
- svg/custom/use-on-text-expected.txt:
WebKitTools:
Reviewed by Adam Roben.
Added hack to force painting when running an SVG test. This is a work-around
for http://bugs.webkit.org/show_bug.cgi?id=12849 SVG renderers update at
paint time instead of style resolution time.
We need this so that --leaks mode, which always paints, doesn't appear to
fail SVG tests. We also need this if we ever want to test style application
in SVG, since many SVGs don't apply style to their renderers until they paint.
- Scripts/run-webkit-tests:
- 20:31 Changeset [22982] by
-
Build fix
- 19:31 Changeset [22981] by
-
Reviewed by Anders' rubberstamp
- WebDownload.cpp: (WebDownload::initToResumeWithBundle): Make a copy of the string we truncate so it doesn't affect the original
- 19:12 Changeset [19785] by
-
LayoutTests:
Reviewed by Hyatt.
http://bugs.webkit.org/show_bug.cgi?id=12820
REGRESSION (r19706): Caret not painted after popup, failing editing/pasteboard/4641033
add test for caret before <select> case too
- editing/selection/caret-before-select-expected.checksum: Added.
- editing/selection/caret-before-select-expected.png: Added.
- editing/selection/caret-before-select-expected.txt: Added.
- editing/selection/caret-before-select.html: Added.
WebCore:
Reviewed by Hyatt.
- fix http://bugs.webkit.org/show_bug.cgi?id=12820 REGRESSION (r19706): Caret not painted after popup, failing editing/pasteboard/4641033
Yet another attempt to figure out exactly who should paint the caret.
- editing/SelectionController.cpp: (WebCore::SelectionController::isInsideNode):
- editing/SelectionController.h:
- rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintCaret):
- 19:07 Changeset [22980] by
-
Reviewed by Adam
< rdar://problem/4964763> - Hookup resume downloads on the engine side
NOTE: Resume downloads are broken in the current ChickenCat - this functionality requires
a custom ToT CFNetwork as of CFNetwork r3372, or will require CFNetwork 172 or later
- WebDownload.cpp: (WebDownload::initWithRequest): Changed some logging, plugged some potential leaks (WebDownload::initToResumeWithBundle): Added a real implementation - this now works! (WebDownload::didFinish): Added some logging and fixed up some lifetime issues (extractResumeDataFromBundle): Renamed to imply the modification of the file. Truncate the resume data from the file once it is extracted
- 18:58 Changeset [19784] by
-
LayoutTests:
Reviewed by Hyatt.
- test for http://bugs.webkit.org/show_bug.cgi?id=12818 REGRESSION (r19148): shacknews.com does not render completely
- fast/dynamic/staticY-marking-parents-regression-expected.checksum: Added.
- fast/dynamic/staticY-marking-parents-regression-expected.png: Added.
- fast/dynamic/staticY-marking-parents-regression-expected.txt: Added.
- fast/dynamic/staticY-marking-parents-regression.html: Added.
WebCore:
Reviewed by Hyatt.
- fix http://bugs.webkit.org/show_bug.cgi?id=12818 REGRESSION (r19148): shacknews.com does not render completely
Test: fast/dynamic/staticY-marking-parents-regression.html
Changed calls to setChildNeedsLayout(true) during layout to
setChildNeedsLayout(true, false). Prior to r19148, the default behavior
of marking containers did not matter because the caller was the container
and was already marked, but r19148 added marking of the parent of
objects with static Y, which can be different from the caller and not
necessarily marked already.
- rendering/RenderBlock.cpp: (WebCore::RenderBlock::collapseMargins): (WebCore::RenderBlock::clearFloatsIfNeeded): (WebCore::RenderBlock::layoutBlockChildren):
- rendering/RenderBox.cpp: (WebCore::RenderBox::setStaticY):
- rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::layoutVerticalBox):
- 18:54 Changeset [19783] by
-
Reviewed by Dave Harrison
< rdar://problem/5014313> - Crash in FrameLoader::restoreScrollPositionAndViewState()
after regaining network connection
- loader/FrameLoader.cpp: (WebCore::FrameLoader::restoreScrollPositionAndViewState): Leave an assertion for debug build exploration, but gracefully handle the release build case
- 17:14 Changeset [19782] by
-
Rubber-stamped by Darin.
- bindings/objc/PublicDOMInterfaces.h: Add missing classes that are really public. Also adds the deprecated selector for getComputedStyle: on DOMDocument.
- 16:30 Changeset [19781] by
-
Add new version with WKCGContextIsBitmapContext.
- WebKitSystemInterface.h:
- libWebKitSystemInterface.a:
- 16:10 Changeset [19780] by
-
LayoutTests:
Reviewed by john
< rdar://problem/5012665>
Removing indent from list moves the caret to the line below
No new layout tests needed because the new results
for these tests and the removed FIXMEs cover the bug fix:
- editing/execCommand/indent-list-item.html:
- editing/execCommand/indent-list-item-expected.checksum:
- editing/execCommand/indent-list-item-expected.png:
- editing/execCommand/indent-list-item-expected.txt:
- editing/execCommand/remove-list-1.html:
- editing/execCommand/remove-list-1-expected.checksum:
- editing/execCommand/remove-list-1-expected.png:
- editing/execCommand/remove-list-1-expected.txt:
WebCore:
Reviewed by john
< rdar://problem/5012665>
Removing indent from list moves the caret to the line below
Selection preservation during indent, outdent and list
operations uses rangeFromLocationAndLength. Ranges returned
by rangeFromLocationAndLength were incorrect for locations
just before the line breaks that are emitted after blocks.
This is because TextIterator emitted bad ranges for these line
breaks (ranges that started and ended *after* the block).
The fix corrects the start but not the end. This is acceptible
because there is code in rangeFromLocationAndLength that corrects
the ends of runs using the start of the run and VisiblePosition
creation.
- editing/TextIterator.cpp: (WebCore::TextIterator::exitNode): Emit a position *inside* the block, after its contents.
- 14:56 Changeset [19779] by
-
Reduced --leaks mode to running only 750 tests at a time, in the hopes
of fixing intermittent crasher that may be caused by out of memory
conditions.
- Scripts/run-webkit-tests:
- 14:32 Changeset [19778] by
-
Reviewed by Darin Adler.
Made DRT paint when running run-webkit-tests in --leaks mode, to check
for painting leaks.
- DumpRenderTree/DumpRenderTree.m: Removed some "NO" initializations, since that's the default for statics. (dumpRenderTree): Added --paint command line option, which specifies that DRT should paint at the end of every test. (displayWebView): New function, called by dump() and -[LayoutTestController display].
- Scripts/run-webkit-tests: Use the --paint command line option when checking for leaks.
- Scripts/run-webkit-tests: Added a new, painting leak to the ignore list. Removed stale comment about THRD leaks -- we now ignore them reliably.
- 14:06 Changeset [22979] by
-
WebCoreWin:
Reviewed by Adele.
Updated for SafariTheme changes.
- 13:52 Changeset [22978] by
-
Reviewed by Darin.
- WebCore.vcproj/WebCore.vcproj: Added JSHTMLSelectElementCustom.h
- 13:49 Changeset [19777] by
-
LayoutTests:
Reviewed by Darin.
Test for < rdar://problem/4999030> Indexing into select element to get to items in the options collection is broken
- fast/forms/select-index-setter-expected.txt: Added.
- fast/forms/select-index-setter.html: Added.
WebCore:
Reviewed by Darin.
Fix for < rdar://problem/4999030> Indexing into select element to get to items in the options collection is broken
Test: fast/forms/select-index-setter.html
Added indexSetter for HTMLSelectElement.
- WebCore.xcodeproj/project.pbxproj: Added JSHTMLSelectElementCustom.h
- bindings/js/JSHTMLOptionsCollectionCustom.cpp: (WebCore::JSHTMLOptionsCollection::indexSetter): Moved code to selectIndexSetter so it can be shared between these classes.
- bindings/js/JSHTMLSelectElementCustom.cpp: (WebCore::selectIndexSetter): Added. Moved from JSHTMLOptionsCollection::indexSetter. (WebCore::JSHTMLSelectElement::indexSetter): Calls selectIndexSetter.
- bindings/js/JSHTMLSelectElementCustom.h: Added.
- html/HTMLSelectElement.idl: Added HasCustomIndexSetter keyword.
- bindings/scripts/CodeGeneratorJS.pm: When generating "put" methods, if the propertyName is a number, call the indexSetter first, and then call the version of lookupPut that will also try to use the parent's lookup table. This matches the order of our old code, before we autogenerated the JSHTMLSelectElement class.
- 12:48 Changeset [19776] by
-
Tools tweak. No review necessary.
- Scripts/parse-malloc-history: Enabled multiple merge-regexp command line arguments. Changed command line argument variables from special implicit no-op values to specific empty values that we test explicitly.
- 12:00 Changeset [19775] by
-
Reviewed by Tim H.
Bug 12834: Drosera should remember scroll position of open files like Xcode
http://bugs.webkit.org/show_bug.cgi?id=12834
- Drosera/debugger.js: Implemented scrolling memory
- 10:55 Changeset [19774] by
-
brmorris, reviewed by Sachin.
DESC: r19404 caused some regresion, so I'm reverting it. TMCN-6XYSLY
This was a fix for Enhanced Input feature not working in 3.1
http://bugs.webkit.org/show_bug.cgi?id=12533
- 08:55 Changeset [19773] by
-
Add undo/redo support and more editing commands.
- 05:59 Changeset [19772] by
-
Implementing popups for the Qt platform.
r=Lars
- 04:00 Changeset [19771] by
-
Remove unneeded hack
- 03:18 Changeset [19770] by
-
Fix rendering of lineedits and remove the spurious
padding on push-buttons.
r=Lars
- 01:13 Changeset [19769] by
-
fixing compilation with Qt
02/20/07:
- 23:30 Changeset [19768] by
-
Preemptive build fix.
- platform/mac/FontCacheMac.mm: Remove unused (and non-existent) header.
- 22:20 Changeset [19767] by
-
Versioning.
- 22:11 Changeset [19766] by
-
New tag.
- 22:04 Changeset [22977] by
-
WebCoreWin:
Reviewed by Maciej.
Some changes needed to match moving writeImage() to the Pasteboard
on the Mac.
- platform/win/PasteboardWin.cpp: (WebCore::Pasteboard::writeURL): Now takes a bool indicating if the url if for an image. (WebCore::Pasteboard::writeImage): This is just a stub for now. I am putting it here instead of on TemporaryLinkStubs because I plan on implementing it very soon.
WebKitWin:
Reviewed by Maciej.
Move copyImage functionality from the ContextMenuClient to the
Pasteboard.
- WebContextMenuClient.cpp:
- WebContextMenuClient.h:
M OpenSourceWin/WebKitWin/WebContextMenuClient.cpp
M OpenSourceWin/WebKitWin/ChangeLog
M OpenSourceWin/WebKitWin/WebContextMenuClient.h
M OpenSourceWin/WebCoreWin/platform/win/PasteboardWin.cpp
M OpenSourceWin/WebCoreWin/ChangeLog
- 22:02 Changeset [19765] by
-
Fix the build the right way.
- platform/StringTruncator.cpp: (WebCore::truncateString): Keep all widths as floats.
- 22:00 Changeset [19764] by
-
WebCore:
Reviewed by Maciej.
Implement writeImage() on Pasteboard so that the context menus can
call into the editor for the "Copy image" command instead of
calling across the clients.
- WebCore.exp:
- WebCore.xcodeproj/project.pbxproj:
- editing/Editor.cpp: (WebCore::Editor::copyImage): Call into the Pasteboard.
- editing/Editor.h:
- loader/mac/LoaderNSURLExtras.h: A new NSURL extra needed inside WebCore.
- loader/mac/LoaderNSURLExtras.m: Made this file Objective-C++ (from Objective-C) since the call into the MIMETypeRegistry will return a Vector. (urlByRemovingComponent): Cast to build as Obj-C++ (urlWithDataRelativeToURL): Same. (vectorContainsString): Helper. (suggestedFilenameWithMIMEType): New function.
- page/ContextMenuClient.h: Remove copyImageToClipboard()
- page/ContextMenuController.cpp: (WebCore::ContextMenuController::contextMenuItemSelected): Call into the editor instead of the client.
- page/mac/WebCoreViewFactory.h: New localized String.
- platform/LocalizedStrings.h: Same.
- platform/MimeTypeRegistry.h: Expose two additional functions for MIMEType information from WebCore System Interface.
- platform/Pasteboard.h:
- platform/SharedBuffer.h: Make platformData() and platformDataSize () public since they are needed to write the image to the pasteboard.
- platform/mac/LocalizedStringsMac.mm: New localized string. (WebCore::copyImageUnknownFileLabel):
- platform/mac/MimeTypeRegistryMac.mm: (WebCore::MimeTypeRegistry::getExtensionsForMIMEType): (WebCore::MimeTypeRegistry::getPreferredExtensionForMIMEType):
- platform/mac/PasteboardMac.mm: (WebCore::writableTypesForImage): (WebCore::Pasteboard::writeURL): Write the correct types if the url is for an image. (WebCore::fileWrapperForImage): (WebCore::Pasteboard::writeFileWrapperAsRTFDAttachment): (WebCore::Pasteboard::writeImage):
- platform/mac/WebCoreNSStringExtras.h: Added. Some the the NSStringExtras that were in WebKit are now needed by WebCore. To avoid having two copies of these functions, I just moved the implementations to WebCore. The WebKit functions just call into WebCore.
- platform/mac/WebCoreNSStringExtras.mm: Added. (hasCaseInsensitiveSuffix): (hasCaseInsensitiveSubstring): (filenameByFixingIllegalCharacters):
- platform/mac/WebCoreSystemInterface.h: wkGetPreferredExtensionForMIMEType and wkGetExtensionsForMIMEType are now needed in WebCore as well as WebKit.
- platform/mac/WebCoreSystemInterface.mm:
WebKit:
Reviewed by Maciej.
WebKit changes needed to implement writeImage() in WebCore's
Pasteboard class.
- Misc/WebKitNSStringExtras.m: Call into WebCore for these implementations. (-[NSString _webkit_hasCaseInsensitiveSuffix:]): (-[NSString _webkit_hasCaseInsensitiveSubstring:]): (-[NSString _webkit_filenameByFixingIllegalCharacters]):
- Misc/WebNSURLExtras.m: Same. (-[NSURL _webkit_suggestedFilenameWithMIMEType:]):
- WebCoreSupport/WebContextMenuClient.h: Remove copyImageToClipboard()
- WebCoreSupport/WebContextMenuClient.mm:
- WebCoreSupport/WebSystemInterface.m: Expose GetExtensionsForMIMEType and GetPreferredExtensionForMIMEType to WebCore. (InitWebCoreSystemInterface):
- WebCoreSupport/WebViewFactory.mm: New localized string for WebCore. (-[WebViewFactory copyImageUnknownFileLabel]):
- 21:16 Changeset [19763] by
-
Build fix.
- platform/StringTruncator.cpp: (WebCore::truncateString): Added explicit casts.
- 21:06 Changeset [22976] by
-
WebCoreWin:
Reviewed by Darin and Anders.
- WebCore.vcproj/WebCore.vcproj: Added new files.
- platform/win/FileChooserWin.cpp: (WebCore::FileChooser::openFileChooser): (WebCore::FileChooser::basenameForWidth): Added truncation.
- platform/win/WebCoreTextRenderer.cpp: (WebCore::WebCoreDrawTextAtPoint): Added bottomAlign and centerTruncate parameters.
- platform/win/WebCoreTextRenderer.h: Ditto.
WebKitWin:
Reviewed by Darin and Anders.
- 21:03 Changeset [19762] by
-
WebCore:
Reviewed by Darin and Anders.
Fix < rdar://problem/4736215> Make WebCoreStringTruncator use WebCore types
- WebCore.exp: Updated symbols.
- WebCore.xcodeproj/project.pbxproj: Updated file names.
- bridge/mac/WebCoreStringTruncator.h: Removed.
- bridge/mac/WebCoreStringTruncator.mm: Removed.
- platform/StringTruncator.cpp: Added. Moved from bridge/mac/WebCoreStringTruncator.mm. (WebCore::textBreakAtOrPreceding): Added. (WebCore::boundedTextBreakFollowing): Added. (WebCore::centerTruncateToBuffer): (WebCore::rightTruncateToBuffer): (WebCore::stringWidth): Now takes a const Font& instead of a Font*. (WebCore::truncateString): (WebCore::StringTruncator::centerTruncate): (WebCore::StringTruncator::rightTruncate): (WebCore::StringTruncator::width):
- platform/StringTruncator.h: Added.
- platform/TextBreakIterator.h: Added isTextBreak declaration.
- platform/TextBreakIteratorICU.cpp: (WebCore::isTextBreak): Implemented.
- platform/mac/FileChooserMac.mm: (WebCore::FileChooser::basenameForWidth): Updated for changes to WebCoreStringTruncator.
- platform/qt/TextBreakIteratorQt.cpp: (WebCore::isTextBreak): Stubbed out.
WebKit:
Reviewed by Darin and Anders.
Update WebKit for WebCore fix for < rdar://problem/4736215> Make
WebCoreStringTruncator use WebCore types.
- Misc/WebStringTruncator.m: (defaultMenuFont): Moved from WebCoreStringTruncator.mm. (core): Added. (+[WebStringTruncator centerTruncateString:toWidth:]): (+[WebStringTruncator centerTruncateString:toWidth:withFont:]): (+[WebStringTruncator rightTruncateString:toWidth:withFont:]): (+[WebStringTruncator widthOfString:font:]):
- WebKit.xcodeproj/project.pbxproj: Changed WebStringTruncator to ObjC++.
- 19:58 Changeset [19761] by
-
LayoutTests:
Reviewed by harrison
< rdar://problem/5006779>
REGRESSION: Paste and Match Style of quoted text onto empty line yields quoted text
- editing/pasteboard/5006779-expected.checksum: Added.
- editing/pasteboard/5006779-expected.png: Added.
- editing/pasteboard/5006779-expected.txt: Added.
- editing/pasteboard/5006779.html: Added.
WebCore:
Reviewed by harrison
< rdar://problem/5006779>
REGRESSION: Paste and Match Style of quoted text onto empty line yields quoted text
We decided to change the copy/paste behavior in both
the Paste and the Paste and Match Style cases.
- editing/markup.cpp: (WebCore::createMarkup): Don't add mail blockquote wrappers unless the user selected one or more paragraphs.
- 17:48 Changeset [19760] by
-
Reviewed by Mark Rowe.
< rdar://problem/4912899> WebCore project ObjC @property cleanup
- bindings/objc/DOMObject.h: Add the new style @property
- bindings/objc/PublicDOMInterfaces.h: Update to the new style @property
- bindings/scripts/CodeGeneratorObjC.pm: Export both new and old style @properties
- 17:39 Changeset [19759] by
-
Fixes the build!
- 16:54 Changeset [19758] by
-
Build fix for the no-SVG build.
- 16:48 Changeset [19757] by
-
Better build fix, rolling out part of r19579.
- 16:40 Changeset [19756] by
-
Build fix.
- 16:36 Changeset [19755] by
-
Tools tweak. No review necessary.
Made parse-malloc-history executable. (Oops!)
Made parse-malloc-history parse 'leaks' output in addition to 'malloc_history"
output.
Added the ability to merge callstacks by regexp, not just depth, so you
can ask questions like, "How many of these allocations were due to that one
call to ...?"
- Scripts/parse-malloc-history:
- 16:12 Changeset [19754] by
-
Fix compile
- 16:07 Changeset [19753] by
-
Reviewed by Sam Weinig.
< rdar://problem/5007248> need to export symbols for DOM exception constants
- WebCore.exp: Adds DOMEventException, DOMException, DOMRangeException, DOMXPathException
- 15:56 Changeset [19752] by
-
Reviewed by John.
Fixes the version number returned when using a CFBundleVersion of "420+".
- WebView/WebView.mm: (-[WebView _userVisibleBundleVersionFromFullVersion:]): Check the length up to the first non-decimal digit, so this works with versions that have "." and "+".
- 15:38 Changeset [19751] by
-
Reviewed by Kevin Decker
- fixed < rdar://problem/4989344> REGRESSION: After clicking on page, Find won't find anything if all hits are before the clicked point
This was caused by a mismatch between WebCore's search code's notion of "selection" and
WebView's search code's notion of "selection". WebCore's search code was starting
just before or just after the "selection", which included collapsed, zero-length
selections. WebKit's search code was only considering non-zero-length selections,
and would not search all of the content when there was a zero-length selection.
The fix was to make WebKit ignore the selection. This has a side effect of increasing
the amount of redundantly-searched content in the case where no matches are found. To
compensate for that, I special-cased the most common case of WebViews with a single frame,
to avoid ever searching redundantly in those.
- WebView/WebView.mm: (-[WebView searchFor:direction:caseSensitive:wrap:startInSelection:]): remove startHasSelection ivar; special-case WebViews with only one frame; clarify the code that leads to redundant searching with comments.
- 15:26 Changeset [19750] by
-
Build fix.
- 15:26 Changeset [19749] by
-
- Scripts/parse-malloc-history: Added copyright info.
- 14:57 Changeset [19748] by
-
yadavall, reviewed by zalan
DESC: Merge of r19731 to s60/trunk: Browser crashes when trying to open rss link HMNN-6YDEAW
http://bugs.webkit.org/show_bug.cgi?id=12790
- 14:55 Changeset [22975] by
-
Bump version for submit
- 14:53 Changeset [19747] by
-
WebCore:
Reviewed by Darin.
< rdar://problem/4900579> WebKit -finalize methods are not thread-safe; design change needed
Call WebCoreObjCFinalizeOnMainThread from the initialize method of all the classes
that have a finalizer that needs called on the main thread.
- WebCore.exp:
- WebCore.xcodeproj/project.pbxproj:
- bindings/objc/WebScriptObject.mm: (+[WebScriptObject initialize]):
- bridge/mac/WebCoreScriptDebugger.mm: (+[WebCoreScriptDebugger initialize]):
- platform/mac/SharedBufferMac.mm: (+[SharedBufferData initialize]):
- platform/mac/WebCoreObjCExtras.c: Added. (WebCoreObjCFinalizeOnMainThread):
- platform/mac/WebCoreObjCExtras.h: Added.
WebKit:
Reviewed by Darin.
< rdar://problem/4900579> WebKit -finalize methods are not thread-safe; design change needed
Call WebCoreObjCFinalizeOnMainThread from the initialize method of all the classes
that have a finalizer that needs called on the main thread. Assert in finalize that we
are on the main thread.
- Carbon/CarbonWindowAdapter.m: (+[CarbonWindowAdapter initialize]): (-[CarbonWindowAdapter finalize]):
- History/WebBackForwardList.mm: (+[WebBackForwardList initialize]): (-[WebBackForwardList finalize]):
- History/WebHistoryItem.mm: (+[WebHistoryItem initialize]):
- Misc/WebElementDictionary.mm: (+[WebElementDictionary initialize]): (-[WebElementDictionary finalize]):
- Plugins/WebBaseNetscapePluginStream.m: (+[WebBaseNetscapePluginStream initialize]): (-[WebBaseNetscapePluginStream finalize]):
- Plugins/WebBaseNetscapePluginView.mm: (+[WebBaseNetscapePluginView initialize]): (-[WebBaseNetscapePluginView finalize]):
- Plugins/WebBasePluginPackage.m: (+[WebBasePluginPackage initialize]): (-[WebBasePluginPackage finalize]):
- Plugins/WebNetscapePluginStream.mm: (-[WebNetscapePluginStream finalize]):
- WebCoreSupport/WebEditorClient.mm: (+[WebEditCommand initialize]): (-[WebEditCommand finalize]):
- WebCoreSupport/WebFrameBridge.mm: (+[WebFrameBridge initialize]): (-[WebFrameBridge finalize]):
- WebCoreSupport/WebFrameLoaderClient.mm: (+[WebFramePolicyListener initialize]): (-[WebFramePolicyListener finalize]):
- WebView/WebHTMLView.mm: (+[WebHTMLView initialize]): (-[WebHTMLView finalize]):
- WebView/WebView.mm: (+[WebViewPrivate initialize]): (-[WebViewPrivate finalize]):
- 14:53 Changeset [19746] by
-
New tag.
- 14:44 Changeset [19745] by
-
Reviewed by darin
< rdar://problem/4838199>
Integrate Mail and WebKit paste operations
Mail overrides paste: because it has different
preferred pasteboard types, but it should use our
fragment creation code.
- WebView/WebHTMLView.mm: (-[WebHTMLView _documentFragmentFromPasteboard:inContext:allowPlainText:chosePlainText:]): Moved fragment creation code into a new method. (-[WebHTMLView _documentFragmentFromPasteboard:forType:inContext:]): Moved fragment creation code here.
- WebView/WebHTMLViewPrivate.h: Exposed _documentFragmentFromPasteboard:forType:inContext: as SPI.
- 14:32 Changeset [19744] by
-
Reviewed by Tim Hatcher
- Misc/WebKitNSStringExtras.h:
- Misc/WebKitNSStringExtras.m: removed _webkit_userVisibleBundleVersionFromFullVersion; we decided to do this without adding SPI for it.
- WebView/WebView.mm: (-[WebView _userVisibleBundleVersionFromFullVersion:]): new method, moved here from WebKitNSStringExtras, and is now a WebView method rather than an NSString method (-[WebView _computeUserAgent]): updated for method signature change
- 14:09 Changeset [19743] by
-
Reviewed by Darin Adler.
New script to parse the output from malloc_history, so we can determine
what's using memory in WebKit.
- Scripts/parse-malloc-history: Added.
- 13:32 Changeset [19742] by
-
Not not reviewed by Mitz Pettel.
Fixed http://bugs.webkit.org/show_bug.cgi?id=12827 | < rdar://problem/5006671>
REGRESSION: -[DOMCSSStyleDeclaration setValue:forKey:] doesn't have an
immediate effect
Let's just pretend this never happened.
I attempted a layout test but I couldn't get the old code to fail because
there's no obvious way to isolate the ObjC bindings in a test that's
controlled by JS.
- bindings/objc/WebScriptObject.mm: (_didExecute):
- 13:21 Changeset [19741] by
-
Reviewed by Maciej S.
Gdk build fixes for recent refactorings.
- Projects/gdk/webcore-gdk.bkl:
- WebCoreSources.bkl:
- page/gdk/DragControllerGdk.cpp: (WebCore::DragController::dragOperation): (WebCore::DragController::maxDragImageSize):
- page/gdk/EventHandlerGdk.cpp: (WebCore::EventHandler::createDraggingClipboard):
- platform/DragImage.h:
- platform/gdk/ChromeClientGdk.h:
- platform/gdk/CursorGdk.cpp: (WebCore::verticalTextCursor): (WebCore::cellCursor): (WebCore::contextMenuCursor): (WebCore::noDropCursor): (WebCore::copyCursor): (WebCore::progressCursor): (WebCore::aliasCursor): (WebCore::noneCursor): (WebCore::notAllowedCursor):
- platform/gdk/EditorClientGdk.cpp: (WebCore::EditorClientGdk::shouldChangeSelectedRange): (WebCore::EditorClientGdk::isEditable): (WebCore::EditorClientGdk::textFieldDidBeginEditing): (WebCore::EditorClientGdk::textFieldDidEndEditing): (WebCore::EditorClientGdk::textDidChangeInTextField): (WebCore::EditorClientGdk::doTextFieldCommandFromEvent): (WebCore::EditorClientGdk::textWillBeDeletedInTextField): (WebCore::EditorClientGdk::textDidChangeInTextArea):
- platform/gdk/EditorClientGdk.h:
- platform/gdk/FrameGdk.cpp: (WebCore::Frame::print): (WebCore::Frame::issueTransposeCommand): (WebCore::Frame::respondToChangedSelection): (WebCore::Frame::cleanupPlatformScriptObjects): (WebCore::Frame::isCharacterSmartReplaceExempt): (WebCore::Frame::dragImageForSelection):
- platform/gdk/FrameGdk.h:
- platform/gdk/MimeTypeRegistryGdk.cpp: Added.
- platform/gdk/TemporaryLinkStubs.cpp: (ChromeClientGdk::tabsToLinks): (ChromeClientGdk::windowResizerRect): (ChromeClientGdk::addToDirtyRegion): (ChromeClientGdk::scrollBackingStore): (ChromeClientGdk::updateBackingStore):
- webcore-base.bkl:
- 13:13 Changeset [19740] by
-
Reviewed by Darin.
Allow over-writting Product, ProductSub, Vendor and VendorSub
js properties via #define WEBCORE_NAVIGATOR_*
- bindings/js/kjs_navigator.cpp: (KJS::Navigator::getValueProperty):
- 12:41 Changeset [19739] by
-
Reviewed by John.
- Misc/WebKitNSStringExtras.h: Added _webkit_userVisibleBundleVersionFromFullVersion.
- Misc/WebKitNSStringExtras.m: (-[NSString _webkit_userVisibleBundleVersionFromFullVersion]): If the version is 4 digits long or longer, then the first digit represents the version of the OS. Our user agent string should not include this first digit, so strip it off and report the rest as the version.
- WebView/WebView.mm: (-[WebView _computeUserAgent]): Call _webkit_userVisibleBundleVersionFromFullVersion on the CFBundleVersion.
- 11:33 Changeset [22974] by
-
Build fix
- WebCore.vcproj/WebCore.vcproj: Added PositionIterator.cpp & h
- 11:27 Changeset [19738] by
- 10:56 Changeset [22973] by
-
2007-02-20 Steve Falkenburg < sfalken@apple.com>
B&I build fix
- WebCore.vcproj/WebCore.vcproj:
- 10:20 Changeset [22972] by
-
Reviewed by Anders.
- WebFrame.cpp: (WebFrame::redirectDataToPlugin): Update for changes to FrameLoader and DocumentLoader.
- 10:18 Changeset [19737] by
-
WebCore:
Reviewed by Anders.
- prepare to fix some "stop loading" bugs by moving state from the FrameLoader class to the DocumentLoader class
- WebCore.exp: Update since cancelMainResourceLoad is now on the document loader.
- html/HTMLDocument.h:
- html/HTMLDocument.cpp: (WebCore::HTMLDocument::HTMLDocument): Removed unused bodyElement and htmlElement data members. (WebCore::HTMLDocument::lastModified): Call getResponseModifiedHeader directly on the document loader.
- loader/DocumentLoader.h: Moved m_mainResourceLoader, m_subresourceLoaders, and m_plugInStreamLoaders here.
- loader/DocumentLoader.cpp: (WebCore::cancelAll): Moved here from FrameLoader. (WebCore::setAllDefersLoading): Ditto. (WebCore::DocumentLoader::mainResourceData): Moved the body of FrameLoader's mainResourceData function here. (WebCore::DocumentLoader::stopLoading): Replaced calls to hasMainResourceLoader and cancelMainResourceLoad with code that works directly on m_mainResourceLoader. Call the stop functions moved from FrameLoader here on ourselves. (WebCore::DocumentLoader::setupForReplaceByMIMEType): Ditto. (WebCore::DocumentLoader::setPrimaryLoadComplete): Moved the code to store the main resource data here; it doesn't require separate functions. (WebCore::DocumentLoader::isLoadingInAPISense): Removed call to isLoadingSubresources and instead just check if the m_subresourceLoaders set is empty. (WebCore::DocumentLoader::setDefersLoading): Moved code here from FrameLoader. (WebCore::DocumentLoader::stopLoadingPlugIns): Ditto. (WebCore::DocumentLoader::stopLoadingSubresources): Ditto. (WebCore::DocumentLoader::addSubresourceLoader): Ditto. (WebCore::DocumentLoader::removeSubresourceLoader): Ditto. (WebCore::DocumentLoader::addPlugInStreamLoader): Ditto. (WebCore::DocumentLoader::removePlugInStreamLoader): Ditto. (WebCore::DocumentLoader::isLoadingSubresources): Ditto. (WebCore::DocumentLoader::isLoadingPlugIns): Ditto. (WebCore::DocumentLoader::startLoadingMainResource): Ditto. (WebCore::DocumentLoader::cancelMainResourceLoad): Ditto.
- loader/FrameLoader.h: Removed stopLoadingPlugIns, stopLoadingSubresources, cancelMainResourceLoad, hasMainResourceLoader, isLoadingSubresources, mainResourceData, releaseMainResourceLoader, lastModified, startLoadingMainResource, isLoadingPlugIns, m_mainResourceLoader, m_subresourceLoaders, m_plugInStreamLoaders, m_responseRefreshHeader, and m_responseModifiedHeader.
- loader/FrameLoader.cpp: (WebCore::FrameLoader::setDefersLoading): Call setDefersLoading on all the document loaders. (WebCore::FrameLoader::stopLoading): Get rid of code to set m_responseRefreshHeader and m_responseModifiedHeader. (WebCore::FrameLoader::receivedFirstData): Call httpHeaderField on the response here to get the Refresh, instead of storing it in a data member. (WebCore::FrameLoader::addPlugInStreamLoader): Change to just call this on the document loader. (WebCore::FrameLoader::removePlugInStreamLoader): Ditto. (WebCore::FrameLoader::isLoading): Change to call isLoadingSubresources and isLoadingPlugIns on the document loader. (WebCore::FrameLoader::addSubresourceLoader): Change to just call this on the document loader. (WebCore::FrameLoader::removeSubresourceLoader): Ditto. (WebCore::FrameLoader::commitProvisionalLoad): Call stopLoadingSubresources and stopLoadingPlugIns on the document loader. Removed code to set m_responseModifiedHeader. (WebCore::FrameLoader::startLoading): Call isLoadingMainResource and startLoadingMainResource on the document loader.
- loader/ImageDocument.cpp: (WebCore::ImageTokenizer::finish): Call mainResourceData() on the document loader instead of the frame loader.
- other changes
- loader/DocLoader.cpp: (WebCore::DocLoader::checkCacheObjectStatus): Fixed indentation.
WebKit:
Reviewed by Anders.
- Plugins/WebPluginController.mm: (-[WebPluginController pluginView:receivedResponse:]): Call cancelMainResourceLoad on the document loader instead of the frame loader.
- 09:24 Changeset [19736] by
-
Reviewed by Mitz.
< rdar://problem/5009627> REGRESSION: Repro overrelease of WebView in failed load, seen in DumpRenderTree
- WebView/WebView.mm: (-[WebView _removeObjectForIdentifier:]): Return early if the identifier can't be found in the map.
- 09:19 Changeset [19735] by
-
Fix focus handling and make the forms
work as expected.
r=Lars
- 08:00 Changeset [19734] by
-
Reviewed by Darin and then Sam.
- fix http://bugs.webkit.org/show_bug.cgi?id=10735 Clicking in SVG results causes WebKit to consume 100% CPU for several minutes
No test since there is no change in functionality.
- CMakeLists.txt:
- WebCore.pro:
- WebCore.xcodeproj/project.pbxproj: Added PositionIterator.{cpp,h}.
- WebCoreSources.bkl:
- dom/Position.cpp: (WebCore::Position::Position): Added converting constructor from PositionIterator. (WebCore::Position::previous): Changed assert() to ASSERT(). (WebCore::Position::next): Ditto. (WebCore::isStreamer): Changed the argument to a PositionIterator. (WebCore::Position::upstream): Changed to use PositionIterator. (WebCore::Position::downstream): Ditto. (WebCore::Position::inRenderedText): Made public. (WebCore::Position::hasRenderedNonAnonymousDescendantsWithHeight): Changed into a static member function. (WebCore::Position::nodeIsUserSelectNone): Ditto.
- dom/Position.h:
- dom/PositionIterator.cpp: Added. A Position iterator with constant-time increment, decrement, and several predicates on the Position it is at. Conversion to/from Position is O(n) in the offset. (WebCore::PositionIterator::increment): (WebCore::PositionIterator::decrement): (WebCore::PositionIterator::atStart): (WebCore::PositionIterator::atEnd): (WebCore::PositionIterator::atStartOfNode): (WebCore::PositionIterator::atEndOfNode): (WebCore::PositionIterator::isCandidate):
- dom/PositionIterator.h: Added. (WebCore::PositionIterator::PositionIterator): (WebCore::PositionIterator::node): (WebCore::PositionIterator::offsetInLeafNode):
- editing/htmlediting.cpp: (WebCore::nextCandidate): Changed to use PositionIterator. (WebCore::previousCandidate): Changed to use PositionIterator.
- 07:43 Changeset [19733] by
-
hongzhao, reviewed by Zalan
DESC: [S60] - Browser NG, ReTe: Some call links on "cti_attribute.html " page are highlighted incorrectly ESZG-6TU9N9
Fix: Add hover sto a[cti].s
http://bugs.webkit.org/show_bug.cgi?id=12814
- 07:39 Changeset [19732] by
-
Reviewed by Darin.
http://bugs.webkit.org/show_bug.cgi?id=12439
SVG parser complains about points attribute in polygon and polyline element
Return true when the parsed points specification is valid.
- 07:07 Changeset [19731] by
-
yadavall, Reviewed by Zalan Bujtas ( zbujtas@gmail.com).
DESC: Browser crashes when trying to open rss link hs.fi/rss HMNN-6YDEAW
http://bugs.webkit.org/show_bug.cgi?id=12790
- 06:46 Changeset [19730] by
-
Make editing of forms work plus make sure that non-void methods
always return something.
r=Lars
- 04:30 Changeset [19729] by
-
Redo form-theming (aka. make Qt pretty, so very, very pretty)
r=Lars
- 02:33 Changeset [19728] by
-
2007-02-19 Lars Naesbye Christensen < lars@naesbye.dk>
Reviewed by Maciej.
http://bugs.webkit.org/show_bug.cgi?id=12815
Two image files not included in project file
- WebCore.xcodeproj/project.pbxproj: Added crossHairCursor.png and notAllowedCursor.png.
- 02:11 Changeset [19727] by
-
WebKitTools:
Reviewed by Darin Adler.
Fix for http://bugs.webkit.org/show_bug.cgi?id=12802
WebKit.framework is built with SVG_SUPPORT for No-SVG build
- Scripts/build-webkit: Pass the "FEATURE_DEFINES=" option when building WebKit as well.
02/19/07:
- 21:35 Changeset [22971] by
-
Reviewed by Adam Roben.
Fixed some leaks in RenderThemeSafari.
I just merged some of the code I wrote for RenderThemeMac.
- 21:33 Changeset [19726] by
-
Reviewed by Adam Roben.
Fixed some leaks in RenderThemeMac.
No test case because 'leaks' can't detect these leaks inside DRT, even if
DRT forces the WebView to paint. ('leaks' can detect these leaks inside
Safari. Weird.)
- rendering/RenderThemeMac.h:
- Renamed data members to m_*, to match our style guidelines.
- Renamed *Is* to is**, to match our style guidelines.
- Removed a few unnecessary data members.
- Made cached values like m_checkbox mutable, since they can change inside functions that don't change the logical state of a RenderTheme object. (The alternative was to make some const functions non-const.)
(WebCore::RenderThemeMac::paintMenuList): Took CFType pointers that just
used to leak and placed them in RetainPtrs.
(WebCore::RenderThemeMac::paintMenuListButtonGradients): ditto
Factored inline implementations of lazy initialization into these accessor
functions, for better encapsulation:
(WebCore::RenderThemeMac::checkbox):
(WebCore::RenderThemeMac::radio):
(WebCore::RenderThemeMac::button):
(WebCore::RenderThemeMac::popupButton):
(WebCore::RenderThemeMac::search):
(WebCore::RenderThemeMac::sliderThumbHorizontal): The inline version of
this initialization used to confuse the retained data member with a local
stack value, causing both a leak and a failure to initialize. Fixed now.
(WebCore::RenderThemeMac::sliderThumbVertical): ditto
(WebCore::RenderThemeMac::resizeCornerImage):
- rendering/RenderThemeMac.h:
- 21:19 Changeset [19725] by
-
Reviewed by Geoff.
< rdar://problem/4636301> Deprecate JavaScriptGlue
- JavaScriptGlue.h: adds DEPRECATED_IN_MAC_OS_X_VERSION_10_5_AND_LATER to everything
- JavaScriptGlue.xcodeproj/project.pbxproj: use -Wno-deprecated-declarations to keep the project building
- config.h: define DEPRECATED_IN_MAC_OS_X_VERSION_10_5_AND_LATER as empty on other systems
- 20:38 Changeset [19724] by
-
Versioning.
- 20:33 Changeset [19723] by
-
Merge in r19665.
- 20:31 Changeset [19722] by
-
Merge in r19662.
- 20:30 Changeset [19721] by
-
Merge in r19660.
- 20:24 Changeset [19720] by
-
New tag.
- 19:01 Changeset [19719] by
-
LayoutTests:
Reviewed by Hyatt.
- repaint test for http://bugs.webkit.org/show_bug.cgi?id=12817 REGRESSION (r18634): Table border-collapse problems with right margin
- fast/repaint/table-outer-border-expected.checksum: Added.
- fast/repaint/table-outer-border-expected.png: Added.
- fast/repaint/table-outer-border-expected.txt: Added.
- fast/repaint/table-outer-border.html: Added.
WebCore:
Reviewed by Hyatt.
- fix http://bugs.webkit.org/show_bug.cgi?id=12817 REGRESSION (r18634): Table border-collapse problems with right margin
Test: fast/repaint/table-outer-border.html
- rendering/RenderTable.cpp: (WebCore::RenderTable::layout): Reordered to initialize horizontal overflow after table sections have computed their outer horizontal borders.
- 18:39 Changeset [19718] by
-
Reviewed by Darin
- fixed < rdar://problem/4613701> REGRESSION: A line break in the source HTML of a link becomes visible after drag & drop
- page/DragController.cpp: (WebCore::DragController::startDrag): Call simplifyWhiteSpace() on the proposed link title to match what's displayed on the web page, instead of using the raw source HTML text.
- 17:41 Changeset [19717] by
-
Fix for Radar 4981605, regression where button grows randomly when a select is
opened. With the new form controls, the WinIE quirk for using line width
ended up being used for them more.
This patch removes the quirk after verifying that Firefox doesn't implement
the quirk at all. usesLineWidth is being renamed to shrinksToAvoidFloats to
make it clear that the new method only applies to block objects that avoid floats
and that have auto width.
Also fixed a regression where whitespace after a floating <select> at the start
of a block got incorrectly rendered. Make sure skipWhitespace always skips
through floating/positioned elements regardless of the whitespace setting (this
way the white-space:pre value on floating <select>s doesn't cause trouble).
Reviewed by darin
fast/block/float/float-avoidance.html
- rendering/RenderBlock.cpp: (WebCore::RenderBlock::collapseMargins): (WebCore::RenderBlock::clearFloatsIfNeeded): (WebCore::RenderBlock::layoutBlockChildren): (WebCore::RenderBlock::markAllDescendantsWithFloatsForLayout):
- rendering/RenderBox.cpp: (WebCore::RenderBox::containingBlockWidth):
- rendering/RenderObject.cpp: (WebCore::RenderObject::shrinkToAvoidFloats):
- rendering/RenderObject.h:
- rendering/bidi.cpp: (WebCore::RenderBlock::skipWhitespace):
- 17:22 Changeset [19716] by
-
LayoutTests:
Reviewed by Darin.
- Test for background image with relative url, when base url element is present. .html has a base url, which according to the html spec is supposed to be absolute, is actually a relative url. Absolute url would not work with file:// scheme.
- Test for base element and @import with relative url. http://bugs.webkit.org/show_bug.cgi?id=12214
- fast/css/background-image-with-baseurl-expected.checksum: Added.
- fast/css/background-image-with-baseurl-expected.png: Added.
- fast/css/background-image-with-baseurl-expected.txt: Added.
- fast/css/background-image-with-baseurl.html: Added.
- fast/css/import_with_baseurl-expected.checksum: Added.
- fast/css/import_with_baseurl-expected.png: Added.
- fast/css/import_with_baseurl-expected.txt: Added.
- fast/css/import_with_baseurl.html: Added.
- fast/css/resources/background_image.css: Added.
WebCore:
Reviewed by Darin.
Update stylesheet href, when document's baseUrl changes. It needs
update when the stylesheet is constructed before the parser hits the base tag.
http://bugs.webkit.org/show_bug.cgi?id=12214
- css/StyleSheet.h: (WebCore::StyleSheet::setHref):
- dom/Document.cpp: (WebCore::Document::setBaseURL):
- dom/Document.h:
- 17:05 Changeset [19715] by
-
Reviewed by Adam.
- fix http://bugs.webkit.org/show_bug.cgi?id=11518 REGRESSION (r14376): View Source not available after Back navigation from a non-HTML page
The FrameLoader's responseMIMEType was not being updated when a page was
brought back from the page cache.
Moved the line that updates the MIME type from commitProvisionalLoad() to
transitionToCommitted().
- loader/FrameLoader.cpp: (WebCore::FrameLoader::commitProvisionalLoad): (WebCore::FrameLoader::transitionToCommitted):
- 16:01 Changeset [19714] by
-
Build fix.
- Scripts/check-for-global-initializers: Ignore the global counter I added for SubresourceLoaders.
- 15:36 Changeset [22970] by
-
Reviewed by Darin.
Fix < rdar://problem/5007566> Crash in WebFrame::dataSource on dni.ru
The fix for the bug was to zero out WebFrame's Frame pointer in
WebFrame::frameLoaderDestroyed, and then to null-check or ASSERT
non-null as appropriate. All IWebFrame methods that depend on the Frame
pointer will return E_FAIL if it is null, and everywhere within
WebFrame that wasn't null-checking before will now explicitly ASSERT.
I also changed all uses of d->frame and WebFrame::impl() to use the
core() function for consistency, and so that there is only one code
path for getting from a WebFrame to a Frame.
Additional more specific comments below.
- WebChromeClient.cpp: (WebChromeClient::scrollBackingStore): (WebChromeClient::updateBackingStore):
- WebFrame.cpp: (kit): Null-check the frame parameter. (core): Added two overloads of this. (WebFrame::DOMDocument): Added argument checking. (WebFrame::currentForm): (WebFrame::loadRequest): (WebFrame::loadData): (WebFrame::dataSource): Added argument checking, and return E_FAIL if the dataSource is null. (WebFrame::provisionalDataSource): Ditto. (WebFrame::stopLoading): (WebFrame::reload): (WebFrame::parentFrame): (WebFrame::renderTreeAsExternalRepresentation): Added argument checking. (WebFrame::firstLayoutDone): Ditto. (WebFrame::loadType): Ditto. (WebFrame::invalidate): (WebFrame::setTextSizeMultiplier): (WebFrame::inViewSourceMode): Added argument checking, and changed to return an HRESULT. (WebFrame::setInViewSourceMode): Changed to return an HRESULT. (WebFrame::searchForLabelsBeforeElement): Added argument checking. (WebFrame::matchLabelsAgainstElement): Ditto. (WebFrame::frameLoaderDestroyed): Zero out the Frame pointer because the Frame is being destroyed. (WebFrame::createFrame): (WebFrame::loadURLIntoChild): Added some assertions. (WebFrame::dispatchWillSubmitForm): (WebFrame::frameLoadCompleted): (WebFrame::receivedData): (WebFrame::setUpPolicyListener): (WebFrame::receivedPolicyDecision): (WebFrame::dispatchDecidePolicyForMIMEType): (WebFrame::dispatchDecidePolicyForNewWindowAction): (WebFrame::dispatchDecidePolicyForNavigationAction): (WebFrame::dispatchCreatePage): (WebFrame::createPlugin): (WebFrame::redirectDataToPlugin): (WebFrame::createJavaAppletWidget): (WebFrame::windowObjectCleared): (WebFrame::setPrinting): (WebFrame::setInPrintingMode): (WebFrame::computePageRects): (WebFrame::getPrintedPageCount): Zero out the out parameter before proceeding. (WebFrame::spoolPages): (WebFrame::forceLayoutWithPageWidthRange):
- WebFrame.h:
- WebView.cpp: (WebView::paint): Silently fail on a null Frame, because this method could be called at any time. (WebView::mouseWheel): Ditto. (WebViewWndProc): Ditto. (WebView::inViewSourceMode): Return whatever WebFrame::setInViewSourceMode returns. (WebView::setInViewSourceMode): Ditto. (WebView::setCustomTextEncodingName): Replaced check for a null FrameLoader with a check for a null Frame, which is equivalent but, as a bonus, won't crash. (WebView::stringByEvaluatingJavaScriptFromString): (WebView::elementAtPoint): Added argument checking. (WebView::selectedText): Ditto. (WebView::centerSelectionInVisibleArea):
- 15:34 Changeset [22969] by
-
Reviewed by Oliver.
Fix build.
- page/win/DragControllerWin.cpp: (WebCore::DragController::maxDragImageSize):
- 15:32 Changeset [19713] by
-
Reviewed by Oliver.
Make DragController::maxDragImageSize a function to get rid of the global initializer.
- page/DragController.cpp: (WebCore::DragController::doImageDrag):
- page/DragController.h:
- page/mac/DragControllerMac.mm: (WebCore::DragController::maxDragImageSize):
- page/qt/DragControllerQt.cpp: (WebCore::DragController::maxDragImageSize):
- 15:19 Changeset [22968] by
-
WebCoreWin:
Reviewed by Adam.
Fix for < rdar://problem/5006882> PlatformScrollbarSafari doesn't support small scroll bars
- 15:19 Changeset [19712] by
-
Reviewed by Tim Hatcher.
- Scripts/check-for-global-initializers: Delete the linked executable if the check fails. Without this, you only see the global initializer error once, which makes it very easy to miss them.
- 15:09 Changeset [19711] by
-
Reviewed by Darin Adler.
< rdar://problem/4841078> Remove the Mail.app editable link clicking behavior workaround when it is no longer needed
- WebKit.xcodeproj/project.pbxproj:
- WebView/WebView.mm: (-[WebView setPreferences:]):
- 14:47 Changeset [22967] by
-
Reviewed by Adam.
< rdar://problem/5008424>
Regression: Crash loading mp3.com
- platform/network/cf/ResourceErrorCF.cpp: (WebCore::ResourceError::ResourceError): Handle null resource errrors.
- 13:44 Changeset [19710] by
-
Reviewed by Hyatt.
- < rdar://problem/5006414> REGRESSION: Crash occurs at WebCore::Frame::loader() when loading AFL Ladder widget
- html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::openURL): Add missing null check so this fails without crashing when called on an element that's no longer in a frame.
- 13:39 Changeset [19709] by
-
Reviewed by Adam.
< rdar://problem/4868242>
http://bugs.webkit.org/show_bug.cgi?id=12670
REGRESSION: Many 3rd Party Apps crash in WebCore::DocumentLoader::frameLoader() (12670)
Bring back the semantic we had that a web view should be retained for as long as something is loading. Use the identifier
to object hash map for this.
- WebView/WebView.mm: (-[WebView _addObject:forIdentifier:]): (-[WebView _removeObjectForIdentifier:]):
- 11:41 Changeset [19708] by
-
Remove two imports of FrameMac.h, fixes clean Mac builds.
- 07:43 Changeset [19707] by
-
LayoutTests:
Reviewed by Mitz.
- test for http://bugs.webkit.org/show_bug.cgi?id=10990 REGRESSION: Pressing a pop-up's access key doesn't focus it < rdar://problem/4823138>
- fast/forms/select-accesskey-expected.checksum: Added.
- fast/forms/select-accesskey-expected.png: Added.
- fast/forms/select-accesskey-expected.txt: Added.
- fast/forms/select-accesskey.html: Added.
WebCore:
Reviewed by Mitz.
- fix for http://bugs.webkit.org/show_bug.cgi?id=10990 REGRESSION: Pressing a pop-up's access key doesn't focus it < rdar://problem/4823138>
Focus the select element too in accessKeyAction.
- html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::accessKeyAction):
- 01:20 Changeset [19706] by
-
Reviewed by Hyatt.
- really fix http://bugs.webkit.org/show_bug.cgi?id=11974 REGRESSION: Caret drawn over input when smaller than font size on initial focus < rdar://problem/4960258>
Avoid double painting caret when it is on a block element.
Layout test for this already went in previous try (it only fails in pixel mode) in r19626.
- rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintCaret):
- 00:35 Changeset [22966] by
-
WebCoreWin:
Build fix.
- bridge/win/FrameWin.cpp: (WebCore::computePageRectsForFrame): This is no longer an instance method of Frame.
- bridge/win/FrameWin.h:
WebKitWin:
Build fix.
- WebFrame.cpp: (WebFrame::computePageRects):
- 00:13 Changeset [19705] by
-
Added a one line patch from Olliej to create the
clipboard on demand (to stop things from crashing),
and added lots of notImplemented() warnings all
over the place.
02/18/07:
- 23:58 Changeset [22965] by
-
WebCoreWin:
Reviewed by Oliver.
- Adapt for Lars's patch to clean up Frame
- bridge/win/FrameWin.cpp: (WebCore::Frame::cleanupPlatformScriptObjects): (WebCore::Frame::createScriptInstanceForWidget): (WebCore::Frame::computePageRects): (WebCore::drawRectIntoContext): (WebCore::imageFromSelection): (WebCore::Frame::isCharacterSmartReplaceExempt):
- bridge/win/FrameWin.h:
- page/win/EventHandlerWin.cpp:
- platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::windowResizerRect): (WebCore::ScrollView::addToDirtyRegion): (WebCore::ScrollView::scrollBackingStore): (WebCore::ScrollView::updateBackingStore):
- platform/win/TemporaryLinkStubs.cpp: (Frame::print): (Frame::issueTransposeCommand): (WebCore::Frame::respondToChangedSelection):
- plugins/win/PluginDatabaseWin.cpp: (WebCore::PluginDatabaseWin::createPluginView):
- plugins/win/PluginDatabaseWin.h:
- plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::createNullPluginView): (WebCore::PluginViewWin::PluginViewWin):
- plugins/win/PluginViewWin.h:
WebKitWin:
Reviewed by Oliver.
- Adapt for Lars's patch to clean up Frame
- WebChromeClient.cpp: (WebChromeClient::tabsToLinks): (WebChromeClient::windowResizerRect): (WebChromeClient::addToDirtyRegion): (WebChromeClient::scrollBackingStore): (WebChromeClient::updateBackingStore):
- WebChromeClient.h:
- WebEditorClient.cpp: (WebEditorClient::shouldChangeSelectedRange): (WebEditorClient::textFieldDidBeginEditing): (WebEditorClient::textFieldDidEndEditing): (WebEditorClient::textDidChangeInTextField): (WebEditorClient::doTextFieldCommandFromEvent): (WebEditorClient::textWillBeDeletedInTextField): (WebEditorClient::textDidChangeInTextArea):
- WebEditorClient.h:
- WebElementPropertyBag.cpp: (WebElementPropertyBag::Read):
- WebFrame.cpp: (kit): (WebFrame::initWithWebFrameView): (WebFrame::searchForLabelsBeforeElement): (WebFrame::matchLabelsAgainstElement): (WebFrame::createPlugin): (WebFrame::createJavaAppletWidget): (WebFrame::computePageRects): (WebFrame::getPrintedPageCount): (WebFrame::spoolPages):
- WebFrame.h:
- WebView.cpp: (WebView::handleEditingKeyboardEvent): (WebView::keyDown): (WebView::generateSelectionImage):
- WebView.h:
- 23:58 Changeset [19704] by
-
Reviewed by Oliver
< rdar://problem/4985321> - Can't edit templates for Web Gallery/Web Page Export in Aperture
- Misc/WebKitVersionChecks.h: Add a #define for this APERTURE quirk
- WebView/WebView.mm: (-[WebView _shouldChangeSelectedDOMRange:toDOMRange:affinity:stillSelecting:]): If the current app is Aperture and it was linked against Tiger WebKit, always allow selection change
- 23:57 Changeset [19703] by
-
fix compilation.
- 23:54 Changeset [19702] by
-
WebCore:
Reviewed by Maciej.
Additional coding by Maciej, additional review by Oliver.
Get rid of the FrameMac and FrameQt classes. Instead
move all methods into Frame directly, and implement
some platform specific methods in foo/FrameFoo.cpp.
Some general cleanup in Frame:
- Move some methods out of Frame, to the place where they belong.
- Unify the different ways of creating the JavaScript wrappers for <object>, <embed> and <applet>
- Some cleanup of the WebCoreFrameBridge
- ForwardingHeaders/bindings/NP_jsobject.h: Added.
- ForwardingHeaders/bindings/npruntime_impl.h: Added.
- ForwardingHeaders/bindings/runtime_root.h: Added.
- WebCore.exp:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/kjs_binding.cpp: (KJS::ScriptInterpreter::shouldInterruptScript):
- bindings/objc/DOM.mm: (-[DOMNode KJS::Bindings::]):
- bindings/objc/DOMInternal.mm: (-[WebScriptObject _initializeScriptDOMNodeImp]):
- bridge/EditorClient.h:
- bridge/mac/FrameViewMac.mm: (WebCore::FrameView::updateBorder): (WebCore::FrameView::updateDashboardRegions):
- bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject rendererForView:]):
- dom/Position.cpp: (WebCore::Position::next):
- editing/CommandByName.cpp: (WebCore::Frame::execCopy): (WebCore::Frame::execCut): (WebCore::Frame::execPaste):
- editing/Editor.cpp: (WebCore::Editor::canEdit): (WebCore::Editor::pasteWithPasteboard): (WebCore::Editor::paste): (WebCore::Editor::pasteAsPlainText):
- editing/Editor.h:
- editing/JSEditor.cpp:
- html/HTMLAppletElement.cpp: (WebCore::HTMLAppletElement::getInstance):
- html/HTMLEmbedElement.cpp: (WebCore::HTMLEmbedElement::getInstance):
- html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::formData):
- html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::getInstance):
- loader/FrameLoader.cpp: (WebCore::FrameLoader::clear): (WebCore::FrameLoader::detachFromParent):
- loader/mac/ImageDocumentMac.mm: (WebCore::finishImageLoad):
- loader/qt/DocumentLoaderQt.cpp:
- page/Chrome.cpp: (WebCore::Chrome::shouldInterruptJavaScript):
- page/Chrome.h:
- page/ChromeClient.h:
- page/Frame.cpp: (WebCore::Frame::~Frame): (WebCore::Frame::markedTextRange): (WebCore::Frame::shouldChangeSelection): (WebCore::Frame::shouldDeleteSelection): (WebCore::Frame::isContentEditable): (WebCore::Frame::setSecureKeyboardEntry): (WebCore::Frame::isSecureKeyboardEntry): (WebCore::Frame::bindingRootObject): (WebCore::Frame::createRootObject): (WebCore::Frame::windowScriptNPObject): (WebCore::Frame::cleanupScriptObjects): (WebCore::FramePrivate::FramePrivate):
- page/Frame.h:
- page/FramePrivate.h:
- page/FrameView.h:
- page/mac/EventHandlerMac.mm: (WebCore::EventHandler::tabsToLinks): (WebCore::EventHandler::tabsToAllControls): (WebCore::EventHandler::focusDocumentView): (WebCore::EventHandler::passMouseDownEventToWidget): (WebCore::EventHandler::handleDrag): (WebCore::EventHandler::sendFakeEventsAfterWidgetTracking): (WebCore::EventHandler::keyboardUIMode):
- page/mac/FrameMac.h: Removed.
- page/mac/FrameMac.mm: (WebCore::Frame::setBridge): (WebCore::Frame::bridge): (WebCore::Frame::searchForLabelsAboveCell): (WebCore::Frame::searchForLabelsBeforeElement): (WebCore::Frame::matchLabelsAgainstElement): (WebCore::Frame::focusWindow): (WebCore::Frame::unfocusWindow): (WebCore::Frame::imageFromRect): (WebCore::Frame::selectionImage): (WebCore::Frame::snapshotDragImage): (WebCore::Frame::fontAttributesForSelectionStart): (WebCore::Frame::baseWritingDirectionForSelectionStart): (WebCore::Frame::print): (WebCore::Frame::issuePasteCommand): (WebCore::Frame::issueTransposeCommand): (WebCore::Frame::respondToChangedSelection): (WebCore::Frame::textFieldDidBeginEditing): (WebCore::Frame::textFieldDidEndEditing): (WebCore::Frame::textDidChangeInTextField): (WebCore::Frame::textDidChangeInTextArea): (WebCore::Frame::doTextFieldCommandFromEvent): (WebCore::Frame::textWillBeDeletedInTextField): (WebCore::Frame::setSecureKeyboardEntry): (WebCore::Frame::isSecureKeyboardEntry): (WebCore::Frame::setMarkedTextRange): (WebCore::Frame::dashboardRegionsDictionary): (WebCore::Frame::dashboardRegionsChanged): (WebCore::Frame::willPopupMenu): (WebCore::Frame::isCharacterSmartReplaceExempt): (WebCore::Frame::setNeedsReapplyStyles): (WebCore::Frame::customHighlightLineRect): (WebCore::Frame::paintCustomHighlight): (WebCore::Frame::createScriptInstanceForWidget): (WebCore::Frame::windowScriptObject): (WebCore::Frame::cleanupPlatformScriptObjects):
- page/mac/WebCoreFrameBridge.h:
- page/mac/WebCoreFrameBridge.mm: (createRootObject): (bridge): (-[WebCoreFrameBridge _frame]):
- page/qt/EventHandlerQt.cpp:
- page/qt/FrameQt.cpp: (WebCore::Frame::unfocusWindow): (WebCore::Frame::focusWindow): (WebCore::Frame::issueTransposeCommand): (WebCore::Frame::respondToChangedSelection): (WebCore::Frame::print): (WebCore::Frame::createScriptInstanceForWidget): (WebCore::Frame::cleanupPlatformScriptObjects): (WebCore::Frame::isCharacterSmartReplaceExempt):
- page/qt/FrameQt.h: Removed.
- platform/MimeTypeRegistry.h:
- platform/graphics/svg/SVGImage.cpp: (WebCore::SVGImage::setData):
- platform/graphics/svg/SVGImageEmptyClients.h: (WebCore::SVGEmptyChromeClient::shouldInterruptJavaScript): (WebCore::SVGEmptyEditorClient::shouldChangeSelectedRange):
- platform/mac/ClipboardMac.h:
- platform/mac/ClipboardMac.mm: (WebCore::ClipboardMac::ClipboardMac):
- platform/mac/FileChooserMac.mm: (-[OpenPanelController beginSheetWithFrame:]):
- platform/mac/MimeTypeRegistryMac.mm: (WebCore::MimeTypeRegistry::getMIMETypeForPath):
- platform/mac/PopupMenuMac.mm: (WebCore::PopupMenu::show):
- platform/mac/WidgetMac.mm: (WebCore::Widget::setFocus): (WebCore::Widget::setIsSelected):
- platform/network/mac/ResourceHandleMac.mm:
- platform/network/qt/ResourceHandleManagerQt.cpp:
- platform/network/qt/ResourceHandleQt.cpp: (WebCore::ResourceHandle::start):
- platform/qt/MimeTypeRegistryQt.cpp: (WebCore::MimeTypeRegistry::getMIMETypeForPath):
- platform/qt/PopupMenuQt.cpp:
- platform/qt/ScrollViewCanvasQt.cpp:
- platform/qt/ScrollViewQt.cpp:
- platform/win/TemporaryLinkStubs.cpp:
- rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paintCustomHighlight):
- rendering/RenderBox.cpp: (WebCore::RenderBox::paintCustomHighlight):
- rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::addHighlightOverflow): (WebCore::RootInlineBox::paintCustomHighlight):
WebKit:
Reviewed by Maciej.
Additional coding by Maciej, additional review by Oliver.
Added implementations for the new callbacks in EditorClient
and ChromeClient (basically moved from WebFrameBridge).
Cleaned up some code paths that are not called anymore
and done fully inside WebCore now.
- DefaultDelegates/WebDefaultContextMenuDelegate.mm:
- Misc/WebElementDictionary.mm:
- Misc/WebNSAttributedStringExtras.mm:
- Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView getVariable:value:]):
- Plugins/WebNetscapePluginEmbeddedView.mm:
- Plugins/WebNetscapePluginStream.mm:
- Plugins/WebPluginContainerCheck.mm:
- WebCoreSupport/WebChromeClient.h:
- WebCoreSupport/WebChromeClient.mm: (WebChromeClient::shouldInterruptJavaScript):
- WebCoreSupport/WebEditorClient.h:
- WebCoreSupport/WebEditorClient.mm: (WebEditorClient::shouldChangeSelectedRange):
- WebCoreSupport/WebFrameBridge.mm: (-[WebFrameBridge finishInitializingWithPage:frameName:frameView:ownerElement:]): (-[WebFrameBridge fini]):
- WebCoreSupport/WebFrameLoaderClient.mm:
- WebView/WebArchiver.mm:
- WebView/WebFrame.mm: (core): (kit): (-[WebFrame _updateBackground]):
- WebView/WebFrameInternal.h:
- WebView/WebFrameView.mm:
- WebView/WebHTMLRepresentation.mm:
- WebView/WebHTMLView.mm: (-[WebHTMLView _updateMouseoverWithEvent:]): (-[WebHTMLView _isEditable]): (-[WebHTMLView validateUserInterfaceItem:]): (-[WebHTMLView maintainsInactiveSelection]): (-[WebHTMLView scrollWheel:]): (-[WebHTMLView acceptsFirstMouse:]): (-[WebHTMLView shouldDelayWindowOrderingForEvent:]): (-[WebHTMLView cut:]): (-[WebHTMLView paste:]): (-[WebHTMLView selectedAttributedString]):
- WebView/WebScriptDebugDelegate.mm:
- WebView/WebView.mm: (-[WebView _dashboardRegions]): (-[WebView setProhibitsMainFrameScrolling:]): (-[WebView _setInViewSourceMode:]): (-[WebView _inViewSourceMode]): (-[WebView shouldClose]): (-[WebView setEditable:]):
WebKitQt:
Reviewed by Maciej.
Additional coding by Maciej, additional review by Oliver.
Add stubs for the new methods in ChormeClient and EditorClient,
remove all references to FrameQt.
- Api/qwebframe.cpp: (QWebFrame::QWebFrame):
- Api/qwebframe_p.h:
- Api/qwebpage.cpp:
- WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::shouldInterruptJavaScript):
- WebCoreSupport/ChromeClientQt.h:
- WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::shouldChangeSelectedRange): (WebCore::EditorClientQt::isEditable):
- WebCoreSupport/EditorClientQt.h:
- WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::setFrame): (WebCore::FrameLoaderClientQt::webFrame):
- WebCoreSupport/FrameLoaderClientQt.h:
- 23:18 Changeset [19701] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by NOBODY (Buildfix).
Oops, missed a 'const'
- platform/qt/ClipboardQt.h:
- 23:04 Changeset [19700] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by Adam.
Adding stubbed out implementation of Clipboard for Qt to stop
it crashing whenever someone drags the mouse.
- WebCore.pro:
- page/qt/EventHandlerQt.cpp: (WebCore::EventHandler::createDraggingClipboard):
- platform/qt/ClipboardQt.cpp: Added. (WebCore::ClipboardQt::ClipboardQt): (WebCore::ClipboardQt::clearData): (WebCore::ClipboardQt::clearAllData): (WebCore::ClipboardQt::getData): (WebCore::ClipboardQt::setData): (WebCore::ClipboardQt::types): (WebCore::ClipboardQt::dragLocation): (WebCore::ClipboardQt::dragImage): (WebCore::ClipboardQt::setDragImage): (WebCore::ClipboardQt::dragImageElement): (WebCore::ClipboardQt::setDragImageElement): (WebCore::ClipboardQt::createDragImage): (WebCore::ClipboardQt::declareAndWriteDragImage): (WebCore::ClipboardQt::writeURL): (WebCore::ClipboardQt::writeRange): (WebCore::ClipboardQt::hasData):
- platform/qt/ClipboardQt.h: Added. (WebCore::ClipboardQt::~ClipboardQt): (WebCore::ClipboardQt::isForDragging):
- 23:00 Changeset [22964] by
-
2007-02-18 Ada Chan < adachan@apple.com>
WebCoreWin:
Reviewed by Adam.
Move app text controls to use WebView.
- added a method to let us render text using WebKit.
- WebCore.vcproj/WebCore.vcproj:
- platform/win/WebCoreTextRenderer.cpp: Added. (WebCore::WebCoreDrawTextAtPoint):
- platform/win/WebCoreTextRenderer.h: Added.
WebKitWin:
Reviewed by Adam.
Move app text controls to use webview
- 22:14 WebKit Team edited by
- I appear to be the DnD guy now (diff)
- 22:07 Changeset [19699] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by Adam.
Don't even attempt to do a document based drag if there isn't a document
Fixes rdar://problem/4960109
- page/DragController.cpp: (WebCore::DragController::tryDocumentDrag):
Null check m_document
(WebCore::DragController::tryDHTMLDrag):
ASSERT(m_document) -- tryDocumentDrag is the only
thing that will ever call us, but lets play it safe
- 16:42 Changeset [22963] by
-
Rubberstamped by Hyatt.
Fix < rdar://problem/5006918> All selected text is black, even if it is
some other color when not selected
- 16:28 Changeset [22962] by
-
Reviewed by Hyatt.
Fix < rdar://problem/5006872> <select> popups need Aqua scroll bars
PopupMenuWin now uses a PlatformScrollbar.
- 16:26 Changeset [19698] by
-
Reviewed by Hyatt.
A little preparation for < rdar://problem/5006872>.
- platform/PopupMenu.h:
- 13:07 Changeset [19697] by
-
Reviewed by Adele.
http://bugs.webkit.org/show_bug.cgi?id=12807
XPath incorrectly converts NaN to boolean
Test: fast/xpath/nan-to-boolean.html
- xml/XPathValue.cpp: (WebCore::XPath::Value::toBoolean): Convert NaN to false.
- xml/XPathFunctions.cpp: (WebCore::XPath::FunSubstringAfter::doEvaluate): Fix substring-after to actually work.
- 08:15 Changeset [19696] by
-
2007-02-18 Mitz Pettel < mitz@webkit.org>
Reviewed by Hyatt.
- fix http://bugs.webkit.org/show_bug.cgi?id=12123 REGRESSION: Incomplete repaint of floats' overflows
Tests: fast/repaint/float-overflow.html
fast/repaint/float-overflow-right.html
fast/repaint/table-cell-vertical-overflow.html
- fix http://bugs.webkit.org/show_bug.cgi?id=10116 REGRESSION: Menu item drawn 2 pixels short on WWDC 2006 Attendee Site
Reflected in existing test results.
Unified floats and overflow for the purposes of painting and hit-testing.
Overhanging and overflowing floats are now factored into a block's overflow
unless their painting has been propagated to an ancestor.
Changed table cells to no longer expand to enclose overflow, thus making
it purely "visual overflow", having no effect on layout in WebCore. It
still determines scrolling dimensions.
- rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeBoxesHorizontally): Include inline blocks' horizontal overflow in the inline box's dimensions.
- rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock): Removed the expand to enclose overflows behavior. Added code to add this block's floats (and their overflow) to its overflow rect if it is not in a block formatting context. Otherwise, the inclusion of floats in the overflow is deferred until this block's parent examines its floats and possibly adopts overhanging floats. (WebCore::RenderBlock::layoutBlockChildren): (WebCore::RenderBlock::paint): (WebCore::RenderBlock::floatRect): Made non-virtual and changed to return an empty rect if there are no floats or the floats are clipped, instead of returning the border box. (WebCore::RenderBlock::addOverhangingFloats): Any floats of the child that are not to be painted by the parent are added to the child's overflow rect. (WebCore::RenderBlock::addVisualOverflow): Added. Adjusts the overflow bounds to include the given rect. (WebCore::RenderBlock::nodeAtPoint):
- rendering/RenderBlock.h:
- rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::layoutBlock): (WebCore::RenderFlexibleBox::layoutHorizontalBox): After placing a normal child, add its floats to its overflow, since painting of floats does not propagate to flexible boxes. (WebCore::RenderFlexibleBox::layoutVerticalBox): Ditto.
- rendering/RenderForeignObject.cpp: (WebCore::RenderForeignObject::layout):
- rendering/RenderHTMLCanvas.cpp: (WebCore::RenderHTMLCanvas::layout):
- rendering/RenderImage.cpp: (WebCore::RenderImage::layout):
- rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPositions): (WebCore::RenderLayer::absoluteBoundingBox):
- rendering/RenderLayer.h:
- rendering/RenderObject.cpp: (WebCore::RenderObject::repaintAfterLayoutIfNeeded):
- rendering/RenderObject.h: (WebCore::RenderObject::expandsToEncloseOverhangingFloats):
- rendering/RenderPath.cpp: (WebCore::RenderPath::layout):
- rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::layout):
- rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::layout):
- rendering/RenderTable.h: Removed the override of overflowHeight() since now tables can have vertical overflow.
- rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::paint): Made sure that overflow is repainted.
- rendering/RenderTableCell.h: (WebCore::RenderTableCell::expandsToEncloseOverhangingFloats): Removed.
- rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::RenderTableSection): (WebCore::RenderTableSection::layoutRows): Factor in vertical overflow from cells. (WebCore::RenderTableSection::paint):
- rendering/RenderTableSection.h: (WebCore::RenderTableSection::overflowHeight): (WebCore::RenderTableSection::overflowTop):
- 06:23 Changeset [19695] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by NOBODY (Buildfix).
Build fix for the build fix -- can't call something that isn't a function
- platform/qt/CursorQt.cpp: (WebCore::notAllowedCursor):
- 05:48 Changeset [19694] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by NOBODY (Build fix).
Adding platform/DragImage.cpp and platform/qt/DragImageQt.cpp to the Qt
project files
Stubbed method for WebCore::notAllowedCursor -- not sure what it should return,
have returned blank cursor, possibly needs to return something else
- WebCore.pro:
- platform/qt/CursorQt.cpp: (WebCore::notAllowedCursor):
- 05:07 Changeset [19693] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by NOBODY (Buildfix).
Build fix -- adding DragClientQt method stubs
- WebCoreSupport/DragClientQt.cpp: (WebCore::DragClientQt::willPerformDragSourceAction): (WebCore::DragClientQt::startDrag): (WebCore::DragClientQt::createDragImageForLink):
- WebCoreSupport/DragClientQt.h:
- 04:59 Changeset [19692] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by NOBODY (Buildfix).
Sigh, obnoxious occasional auto conversion from float to int
- platform/mac/ClipboardMac.mm: (WebCore::ClipboardMac::createDragImage):
- 04:48 Changeset [19691] by
-
Reviewed by Hyatt.
http://bugs.webkit.org/show_bug.cgi?id=12799
REGRESSION: Webpage doesn't load correctly (www.cincinnati.com
redirects to the wrong URL)
Test: http/tests/misc/refresh-meta-with-newline.html
- platform/network/HTTPParsers.cpp: (WebCore::skipWhiteSpace): (WebCore::parseHTTPRefresh):
- platform/network/HTTPParsers.h: In HTML, all characters with codes <= 0x20 are whitespace, while in HTTP, only space and tab are such.
- dom/Document.cpp: (WebCore::Document::processHttpEquiv):
- loader/FrameLoader.cpp: (WebCore::FrameLoader::receivedFirstData): Pass parseHTTPRefresh() a flag telling about the source of data.
- 04:44 Changeset [19690] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by NOBODY (Build fix).
Buld fix -- surprisingly RetainPtr doesn't exist on Qt
- platform/DragImage.h:
- 04:35 Changeset [22961] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by Adam.
WebCore:
Updated to include stubs for drag and drop methods
Added DragImage[Win].cpp to project file
Now uses platform independent EventHandler::handleDrag
- WebCore.vcproj/WebCore.vcproj:
- bridge/win/FrameWin.cpp: (WebCore::Frame::dragImageForSelection): Just wrap the platform specific selectionImage method
- page/win/DragControllerWin.cpp: Defining constants for drag images -- basically copied the Mac definitions
- page/win/EventHandlerWin.cpp: (WebCore::EventHandler::createDraggingClipboard): Basic implementation -- needs to create an IDataObject to be useful but ClipboardWin is basically unimplemented anyway so not yet a problem
- platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::createDragImage): (WebCore::ClipboardWin::declareAndWriteDragImage): (WebCore::ClipboardWin::writeURL): (WebCore::ClipboardWin::writeRange): (WebCore::ClipboardWin::hasData): Stubs
- platform/win/ClipboardWin.h: Updated interface
- platform/win/DragImageWin.cpp: Added. (WebCore::dragImageSize): (WebCore::deleteDragImage): (WebCore::scaleDragImage): (WebCore::dissolveDragImageToFraction): (WebCore::createDragImageFromImage): (WebCore::createDragImageIconForCachedImage): Stubs
WebKit:
Stubs to keep the build going
- WebDragClient.cpp: (WebDragClient::willPerformDragSourceAction): (WebDragClient::startDrag): (WebDragClient::createDragImageForLink):
- WebDragClient.h:
- 04:30 Changeset [19689] by
-
2007-02-18 Oliver Hunt < oliver@apple.com>
Reviewed by Adam.
WebCore:
More drag and drop migration, now the entirety of the
drag initiation logic has been rendered platform independent
This has required a number of new interfaces, and a reasonable
amount of logic migration.
As a side effect, this patch also fixes rdar://problem/4945341
There are some basic Qt stubs that should stop the build from failing,
however the absence of ClipboardQt means any attempt to initiate a drag
may cause a crash.
- WebCore.exp:
Exporting new symbols
- WebCore.xcodeproj/project.pbxproj:
New files
- dom/Clipboard.cpp: (WebCore::Clipboard::canSaveAsWebArchive): Migrated from WebKit
- dom/Clipboard.h: Added more methods to allow Clipboard to be used as a platform independent container for drag and drop
- page/DragClient.h: (WebCore::DragClient::declareAndWriteDragImage): This is a mac only helper function, so i've made it have an empty implementation, that way we won't need a PLATFORM(MAC) block in SVGImageEmptyClients
- page/DragController.cpp: (WebCore::DragController::dragExited): (WebCore::DragController::performDrag): (WebCore::DragController::tryDocumentDrag): (WebCore::DragController::tryDHTMLDrag): Using RefPtrs now (WebCore::getCachedImage): (WebCore::getImage): Helper functions (WebCore::dragLocForDHTMLDrag): (WebCore::dragLocForSelectionDrag): (WebCore::DragController::startDrag): (WebCore::DragController::doImageDrag): (WebCore::DragController::doSystemDrag): Logic that generates drag images and clipboard content, and initiates the actual system drag operation
- page/DragController.h: Method and variable declarations
- page/EventHandler.cpp: (WebCore::EventHandler::handleDrag): handleDrag is now platform independent (WebCore::EventHandler::handleTextInputEvent):
formatting
- page/EventHandler.h: (WebCore::EventHandler::eventLoopHandleMouseDragged): Forgot to define this stub function
- page/Frame.h: Declaring dragImageForSelection to provide drag image for selected content
- page/mac/DragControllerMac.mm: Defining drag images control vars
- page/mac/EventHandlerMac.mm: (WebCore::EventHandler::createDraggingClipboard): Migrated old clipboard creation to here
- page/mac/FrameMac.mm: (WebCore::Frame::dragImageForSelection): Wrap FrameMac::selectionImage
- page/qt/DragControllerQt.cpp: Defining drag images control vars
- page/qt/EventHandlerQt.cpp: (WebCore::EventHandler::createDraggingClipboard): stub
- page/qt/FrameQt.cpp: (WebCore::Frame::dragImageForSelection): stub
- platform/DragImage.cpp: Added. (WebCore::fitDragImageToMaxSize): (WebCore::createDragImageForSelection):
Platform independent processing for drag images
- platform/DragImage.h: Added.
Declaring typedefs and wrapper functions to abstract the handling
of drag images
- platform/Pasteboard.h: Declaring extracted writeURL and writeSelection methods
- platform/graphics/svg/SVGImageEmptyClients.h: (WebCore::SVGEmptyDragClient::willPerformDragSourceAction): (WebCore::SVGEmptyDragClient::startDrag): (WebCore::SVGEmptyDragClient::createDragImageForLink): Stubs FTW!
- platform/mac/ClipboardMac.h: (WebCore::ClipboardMac::pasteboard): Provide accessor for underlying NSPasteboard
- platform/mac/ClipboardMac.mm: (WebCore::ClipboardMac::hasData): (WebCore::ClipboardMac::writeRange): (WebCore::ClipboardMac::writeURL): (WebCore::ClipboardMac::declareAndWriteDragImage): (WebCore::ClipboardMac::createDragImage):
Implemented new Clipboarid functionality.
(WebCore::ClipboardMac::dragNSImage):
Made this a const method
- platform/mac/DragImageMac.mm: Added. (WebCore::dragImageSize): (WebCore::deleteDragImage): (WebCore::scaleDragImage): (WebCore::dissolveDragImageToFraction): (WebCore::createDragImageFromImage): (WebCore::createDragImageIconForCachedImage):
Implemented platform specific DragImage functions
- platform/mac/PasteboardMac.mm: (WebCore::writeSelection): (WebCore::Pasteboard::writeSelection): (WebCore::writeURL): (WebCore::Pasteboard::writeURL):
Extracted member implementations of these functions, so that
Clipboard could also make use of this functionality.
Pasteboard methods now call the new non-member implementations.
Also fixed implementations to respect the list of requested types.
- platform/qt/DragImageQt.cpp: Added. (WebCore::dragImageSize): (WebCore::deleteDragImage): (WebCore::scaleDragImage): (WebCore::dissolveDragImageToFraction): (WebCore::createDragImageFromImage): (WebCore::createDragImageIconForCachedImage): Stubs
WebKit:
Moving the drag initiation logic to WebCore.
The redundant code in webkit will be moved out in a later patch.
- WebCoreSupport/WebDragClient.h:
- WebCoreSupport/WebDragClient.mm: (getTopHTMLView): Helper function (WebDragClient::willPerformDragSourceAction): (WebDragClient::startDrag): (WebDragClient::createDragImageForLink): Implemented new DragClient methods (WebDragClient::declareAndWriteDragImage): Helper function for the Mac to allow new drag and drop code to match behaviour
- WebView/WebHTMLView.mm: (-[WebHTMLView _dragImageForURL:withLabel:]): (-[WebHTMLView _dragImageForLinkElement:]): Refactoring old _dragImageForLinkElement function so that the link drag image can be created with just a URL and label, rather than requiring the original element (-[WebHTMLView dragImage:at:offset:event:pasteboard:source:slideBack:]): Removed logic that is no longer necessary (-[WebHTMLView _mouseDownEvent]): The WebDragClient may need the original mouseDownEvent of a drag when initiating a drag
- WebView/WebHTMLViewInternal.h: Declaring _mouseDownEvent
- WebView/WebHTMLViewPrivate.h: Declaring _dragImageForURL
- 00:48 Changeset [19688] by
-
2007-02-18 Kevin Ollivier < kevino@theolliviers.com>
Reviewed by Adam Roben.
Fix cases where MSVC-specific code was identified as Win32 platform
code. (as it should be compiled for e.g. wx port when using MSVC too)
- wtf/Assertions.h:
- wtf/MathExtras.h:
- wtf/StringExtras.h: changed PLATFORM(WIN) sections to COMPILER(MSVC) as necessary