mjs [Tue, 1 Aug 2006 18:38:33 +0000 (18:38 +0000)]
Reviewed by Geoff.
- some refactoring in preparation for moving more stuff to WebFrameLoader.
* WebView/WebFrame.m:
(-[WebFrame _clearDataSource]):
(-[WebFrame _detachFromParent]):
(-[WebFrame _commitProvisionalLoad]):
(-[WebFrame _transitionToCommitted:]):
(-[WebFrame _clearProvisionalLoad]):
(-[WebFrame _markLoadComplete]):
(-[WebFrame _checkLoadCompleteForThisFrame]):
(-[WebFrame _startProvisionalLoad:]):
(-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]):
(-[WebFrame stopLoading]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15738
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 1 Aug 2006 16:40:22 +0000 (16:40 +0000)]
LayoutTests:
- updated a test affected by changes to the DumpRenderTree tool
to return a current mouse position
* fast/inline/dirtyLinesForInline-expected.txt: Updated.
WebKitTools:
- fix a bug in my recent change where the mouse position at the end of
the last test would affect the results of the next test
* DumpRenderTree/DumpRenderTree.m: (dumpRenderTree): Set lastMousePosition to 0.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15735
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 1 Aug 2006 06:15:38 +0000 (06:15 +0000)]
LayoutTests:
Reviewed by Darin.
- tests changed by http://bugzilla.opendarwin.org/show_bug.cgi?id=9955
Colors scaled from [0, 1] range to [0, 255] incorrectly
* fast/css/hsl-color-expected.checksum:
* fast/css/hsl-color-expected.png:
* fast/css/hsl-color-expected.txt:
* fast/forms/input-disabled-color-expected.checksum:
* fast/forms/input-disabled-color-expected.png:
* fast/forms/input-disabled-color-expected.txt:
WebCore:
Reviewed by Darin.
- patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=9955
Colors scaled from [0, 1] range to [0, 255] incorrectly
Fixes scaling issue and fixes Color to follow style guidelines.
* css/cssstyleselector.cpp:
(WebCore::CSSStyleSelector::getColorFromPrimitiveValue):
* platform/Color.cpp:
(WebCore::calcHue):
(WebCore::makeRGBAFromHSLA):
(WebCore::Color::Color):
(WebCore::Color::name):
(WebCore::Color::setNamedColor):
(WebCore::Color::light):
(WebCore::Color::dark):
(WebCore::blend):
* platform/Color.h:
(WebCore::Color::Color):
(WebCore::Color::isValid):
(WebCore::Color::red):
(WebCore::Color::green):
(WebCore::Color::blue):
(WebCore::Color::alpha):
(WebCore::Color::rgb):
(WebCore::Color::setRGB):
(WebCore::operator==):
(WebCore::operator!=):
* rendering/RenderObject.cpp:
(WebCore::RenderObject::drawBorder):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15734
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 1 Aug 2006 06:13:20 +0000 (06:13 +0000)]
LayoutTests:
- test for http://bugzilla.opendarwin.org/show_bug.cgi?id=10158
REGRESSION: Selection highlight is dark and opaque when solid-color images are used
Reviewed by Darin.
* fast/backgrounds/solid-color-context-restore-expected.checksum: Added.
* fast/backgrounds/solid-color-context-restore-expected.png: Added.
* fast/backgrounds/solid-color-context-restore-expected.txt: Added.
* fast/backgrounds/solid-color-context-restore.html: Added.
WebCore:
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10158
REGRESSION: Selection highlight is dark and opaque when solid-color images are used
Reviewed by Darin.
Pixel test: fast/backgrounds/solid-color-context-restore.html
* platform/mac/ImageMac.mm:
(WebCore::Image::draw): Added calls to preserve the graphics context
around the painting of a solid color image.
(WebCore::Image::drawTiled): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15733
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 1 Aug 2006 06:10:04 +0000 (06:10 +0000)]
Reviewed by Maciej.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10182
Bug 10182: [Drosera] Evaluating expressions in Console is slow when current frame has many variables
* Drosera/console.js: String.indexOf returns -1 when the string is not found.
Correct the logic to not reload local variable list unless an '=' character is in the expression.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15732
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 1 Aug 2006 06:00:32 +0000 (06:00 +0000)]
LayoutTests:
Reviewed by Maciej.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10171
REGRESSION: failing layout test: fast/events/objc-event-api.html
* fast/events/objc-event-api-expected.txt: Updated test results for changes in the
DumpRenderTree tool.
WebKitTools:
Reviewed by Maciej.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10171
REGRESSION: failing layout test: fast/events/objc-event-api.html
* DumpRenderTree/DumpRenderTree.m:
(main): Add pose so we can implement +[NSEvent mouseLocation]. Put the window at a predictable
location in flipped coordinates, since those are the coordinates that we use.
(+[DumpRenderTreeEvent mouseLocation]): Implement this, since it's used for mouse event handling
inside WebHTMLView.
* DumpRenderTree/EventSendingController.m:
(+[EventSendingController initialize]): Removed unused webkitDomEventProperties array.
(-[EventSendingController init]): Removed, since the whole thing was a no-op.
(-[EventSendingController mouseMoveToX:Y:]): Convert the x,y pair to window coordinates. The old code
probably worked OK, but this is needed to be correct.
(-[EventSendingController handleEvent:]): Put clientX/Y before screenX/Y and removed the flipping
code from screenY. We didn't really need to flip screenY -- what was actually happening was that the
position that DumpRenderTree chose was in non-flipped coordinates, and with that fixed we can just
dump the screenY as-is.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15731
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mjs [Tue, 1 Aug 2006 05:15:40 +0000 (05:15 +0000)]
WebCore:
Rubber-stamped by Tim Hatcher.
- renamed TransferJob to ResourceLoader
* Projects/gdk/webcore-gdk.bkl:
* WebCore.vcproj/WebCore/WebCore.vcproj:
* WebCore.xcodeproj/project.pbxproj:
* WebCoreSources.bkl:
* bridge/mac/FrameMac.mm:
* bridge/mac/WebCoreResourceLoaderImp.h:
* bridge/mac/WebCoreResourceLoaderImp.mm:
(-[WebCoreResourceLoaderImp redirectedToURL:]):
(-[WebCoreResourceLoaderImp addData:]):
(-[WebCoreResourceLoaderImp finishJobAndHandle:]):
* bridge/win/FrameWin.cpp:
* dom/XMLTokenizer.cpp:
(WebCore::openFunc):
* loader/Cache.cpp:
* loader/LoaderFunctions.h:
* loader/loader.cpp:
(WebCore::Loader::servePendingRequests):
(WebCore::Loader::receivedAllData):
(WebCore::Loader::receivedResponse):
(WebCore::Loader::receivedData):
(WebCore::Loader::cancelRequests):
(WebCore::Loader::jobForRequest):
* loader/loader.h:
* loader/mac/LoaderFunctionsMac.mm:
(WebCore::ServeSynchronousRequest):
* platform/ResourceLoader.cpp: Added.
(WebCore::ResourceLoader::ResourceLoader):
(WebCore::ResourceLoader::isErrorPage):
(WebCore::ResourceLoader::error):
(WebCore::ResourceLoader::setError):
(WebCore::ResourceLoader::queryMetaData):
(WebCore::ResourceLoader::addMetaData):
(WebCore::ResourceLoader::kill):
(WebCore::ResourceLoader::url):
(WebCore::ResourceLoader::postData):
(WebCore::ResourceLoader::method):
(WebCore::ResourceLoader::client):
* platform/ResourceLoader.h: Added.
(WebCore::ResourceLoader::getInternal):
* platform/ResourceLoaderClient.h: Added.
(WebCore::ResourceLoaderClient::~ResourceLoaderClient):
(WebCore::ResourceLoaderClient::receivedRedirect):
(WebCore::ResourceLoaderClient::receivedResponse):
(WebCore::ResourceLoaderClient::receivedData):
(WebCore::ResourceLoaderClient::receivedAllData):
* platform/ResourceLoaderInternal.h: Added.
(WebCore::ResourceLoaderInternal::ResourceLoaderInternal):
* platform/TransferJob.cpp: Removed.
* platform/TransferJob.h: Removed.
* platform/TransferJobClient.h: Removed.
* platform/TransferJobInternal.h: Removed.
* platform/gdk/FrameGdk.cpp:
(WebCore::FrameGdk::openURL):
(WebCore::FrameGdk::urlSelected):
(WebCore::FrameGdk::receivedData):
(WebCore::FrameGdk::receivedAllData):
* platform/gdk/FrameGdk.h:
* platform/gdk/ResourceLoaderCurl.cpp: Added.
(WebCore::ResourceLoaderInternal::~ResourceLoaderInternal):
(WebCore::ResourceLoader::~ResourceLoader):
(WebCore::ResourceLoader::start):
(WebCore::ResourceLoader::cancel):
* platform/gdk/ResourceLoaderManager.cpp: Added.
(WebCore::ResourceLoaderManager::ResourceLoaderManager):
(WebCore::ResourceLoaderManager::get):
(WebCore::ResourceLoaderManager::useSimpleTransfer):
(WebCore::writeCallback):
(WebCore::ResourceLoaderManager::downloadTimerCallback):
(WebCore::ResourceLoaderManager::remove):
(WebCore::ResourceLoaderManager::add):
(WebCore::ResourceLoaderManager::cancel):
* platform/gdk/ResourceLoaderManager.h: Added.
* platform/gdk/TemporaryLinkStubs.cpp:
(WebCore::ServeSynchronousRequest):
(WebCore::ResourceLoader::assembleResponseHeaders):
(WebCore::ResourceLoader::retrieveCharset):
* platform/gdk/TransferJobCurl.cpp: Removed.
* platform/gdk/TransferJobManager.cpp: Removed.
* platform/gdk/TransferJobManager.h: Removed.
* platform/mac/ResourceLoaderMac.mm: Added.
(WebCore::ResourceLoaderInternal::~ResourceLoaderInternal):
(WebCore::ResourceLoader::~ResourceLoader):
(WebCore::ResourceLoader::start):
(WebCore::ResourceLoader::assembleResponseHeaders):
(WebCore::ResourceLoader::retrieveCharset):
(WebCore::ResourceLoader::setLoader):
(WebCore::ResourceLoader::receivedResponse):
(WebCore::ResourceLoader::cancel):
* platform/mac/TransferJobMac.mm: Removed.
* platform/win/ResourceLoaderWin.cpp: Added.
(WebCore::addToOutstandingJobs):
(WebCore::lookupResourceLoader):
(WebCore::ResourceLoaderWndProc):
(WebCore::initializeOffScreenResourceLoaderWindow):
(WebCore::ResourceLoaderInternal::~ResourceLoaderInternal):
(WebCore::ResourceLoader::~ResourceLoader):
(WebCore::ResourceLoader::start):
(WebCore::ResourceLoader::fileLoadTimer):
(WebCore::ResourceLoader::cancel):
* platform/win/ResourceLoaderWin.h: Added.
* platform/win/TemporaryLinkStubs.cpp:
(WebCore::ServeSynchronousRequest):
(WebCore::ResourceLoader::assembleResponseHeaders):
(WebCore::ResourceLoader::retrieveCharset):
* platform/win/TransferJobWin.cpp: Removed.
* platform/win/TransferJobWin.h: Removed.
* xml/XSLTProcessor.cpp:
(WebCore::docLoaderFunc):
* xml/xmlhttprequest.cpp:
(WebCore::XMLHttpRequest::send):
(WebCore::XMLHttpRequest::receivedAllData):
(WebCore::XMLHttpRequest::receivedRedirect):
(WebCore::XMLHttpRequest::receivedData):
* xml/xmlhttprequest.h:
WebKit:
Reviewed by Tim Hatcher.
- renamed TransferJob to ResourceLoader in WebCore
* COM/WebFrame.cpp:
(WebFrame::loadDataSource):
(WebFrame::receivedRedirect):
(WebFrame::receivedResponse):
(WebFrame::receivedData):
(WebFrame::receivedAllData):
(WebFrame::setStatusText):
* COM/WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15730
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 1 Aug 2006 04:54:55 +0000 (04:54 +0000)]
Reviewed by Maciej.
- omit the margin and padding boxes for display types where they are ignored
- use CSS instead of properties for table spacing and padding as suggested by Tim H.
* WebInspector/webInspector/inspector.css: Added rules for spacing and padding.
Added rules that hide the margin and padding boxes (borders and all but the center cell)
when the hide attribute is present.
* WebInspector/webInspector/inspector.html: Added classes for the rules above.
Removed cellpadding and cellspacing attributes.
* WebInspector/webInspector/inspector.js: Added code to hide/show the margin and
padding boxes based on the display type.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15729
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 1 Aug 2006 04:14:20 +0000 (04:14 +0000)]
Reviewed by Hyatt.
- removed obsolete cell margin hack
* rendering/RenderBox.cpp: (WebCore::RenderBox::calcVerticalMargins):
Set top and bottom margins to 0 for table cells instead of -16384.
No effect on layout (including layout tests).
* css/CSSComputedStyleDeclaration.cpp:
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
Added a few comments about differences between this class and the
what the CSS specification says about computed style.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15728
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Tue, 1 Aug 2006 04:00:57 +0000 (04:00 +0000)]
LayoutTests:
Reviewed by Maciej.
http://bugzilla.opendarwin.org/show_bug.cgi?id=9738
Bug 9738: Unqualified :hover selector ignored in strict parsing mode
* fast/selectors/unqualified-hover-quirks-expected.checksum: Added.
* fast/selectors/unqualified-hover-quirks-expected.png: Added.
* fast/selectors/unqualified-hover-quirks-expected.txt: Added.
* fast/selectors/unqualified-hover-quirks.html: Added.
* fast/selectors/unqualified-hover-strict-expected.checksum: Added.
* fast/selectors/unqualified-hover-strict-expected.png: Added.
* fast/selectors/unqualified-hover-strict-expected.txt: Added.
* fast/selectors/unqualified-hover-strict.html: Added.
WebCore:
Reviewed by Maciej.
http://bugzilla.opendarwin.org/show_bug.cgi?id=9738
Bug 9738: Unqualified :hover selector ignored in strict parsing mode
* css/cssstyleselector.cpp:
(WebCore::CSSStyleSelector::checkSelector): Restrict the :hover and :active
exclusion based on onlyHoverActive to quirks mode.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15727
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Tue, 1 Aug 2006 03:54:30 +0000 (03:54 +0000)]
Reviewed by Darin.
Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=10159>
"REGRESSION: delegate returning no menu elements crashes webkit"
No automated test, because there's no way to programmatically open a context menu,
no manual test because there's no way to customize the context menu delegate.
* WebView/WebView.m:
(-[WebView _menuForElement:defaultItems:]): Make sure the context menu returned
some menu items before accessing the first one.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15726
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Tue, 1 Aug 2006 03:51:45 +0000 (03:51 +0000)]
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10178
Bug 10178: [Drosera] Selecting a frame in the stack list should take you to that location in the source
* Drosera/debugger.js: Keep a stack that contains the source file and line number references for outer
frames. Use this stack to determine which file and line to highlight when a stack frame is selected.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15725
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Tue, 1 Aug 2006 03:48:39 +0000 (03:48 +0000)]
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10175
Bug 10175: [Drosera] Anonymous functions show up as "(global scope)" in stack list
* Drosera/DebuggerDocument.m:
(-[WebScriptObject currentFunctionStack]): If the frame has a caller it isn't the global scope.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15724
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt [Tue, 1 Aug 2006 02:27:57 +0000 (02:27 +0000)]
Add ScrollBar.cpp to fix the build.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15723
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt [Tue, 1 Aug 2006 02:25:43 +0000 (02:25 +0000)]
Fix for bug 10179, digg.com scrolls slowly. Improve fixed positioning
and fixed backgrounds so that a count of them is kept on the FrameView.
This allows us to switch slow scrolling on and off as these objects come
and go.
* css/cssstyleselector.cpp:
(WebCore::CSSStyleSelector::adjustRenderStyle):
(WebCore::CSSStyleSelector::applyProperty):
* page/FrameView.cpp:
(WebCore::FrameViewPrivate::reset):
(WebCore::FrameView::layout):
(WebCore::FrameView::useSlowRepaints):
(WebCore::FrameView::setUseSlowRepaints):
(WebCore::FrameView::addSlowRepaintObject):
(WebCore::FrameView::removeSlowRepaintObject):
* page/FrameView.h:
* rendering/RenderBox.cpp:
(WebCore::RenderBox::paintBackgroundExtended):
* rendering/RenderObject.cpp:
(WebCore::RenderObject::setStyle):
* rendering/RenderView.cpp:
(WebCore::RenderView::paintBoxDecorations):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15722
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt [Tue, 1 Aug 2006 01:50:45 +0000 (01:50 +0000)]
Begin disentangling of scrollbar logic in preparation for engine-implemented
scrollbars. Split ScrollBar into two classes, an abstract base (still named
ScrollBar) and a new PlatformScrollBar subclass. This subclass is used only
by platforms that want to continue to use a platform scrollbar (rather than
the engine one).
Reviewed by darin
* WebCore.xcodeproj/project.pbxproj:
* html/HTMLCanvasElement.h:
* page/Frame.cpp:
* platform/PlatformScrollBar.h: Added.
(WebCore::PlatformScrollBar::isWidget):
* platform/ScrollBar.h:
(WebCore::ScrollBarClient::~ScrollBarClient):
(WebCore::ScrollBar::~ScrollBar):
(WebCore::ScrollBar::orientation):
(WebCore::ScrollBar::value):
(WebCore::ScrollBar::client):
* platform/mac/PlatformScrollBarMac.mm: Added.
(-[WebCoreScrollBar initWithPlatformScrollBar:]):
(-[WebCoreScrollBar detachPlatformScrollBar]):
(-[WebCoreScrollBar scroll:]):
(-[WebCoreScrollBar widget]):
(-[WebCoreScrollBar mouseDown:]):
(WebCore::PlatformScrollBar::PlatformScrollBar):
(WebCore::PlatformScrollBar::~PlatformScrollBar):
(WebCore::PlatformScrollBar::setScrollBarValue):
(WebCore::PlatformScrollBar::setKnobProportion):
(WebCore::PlatformScrollBar::scrollbarHit):
(WebCore::PlatformScrollBar::width):
(WebCore::PlatformScrollBar::height):
(WebCore::PlatformScrollBar::setRect):
(WebCore::PlatformScrollBar::setEnabled):
(WebCore::PlatformScrollBar::paint):
* platform/mac/ScrollBarMac.mm: Removed.
* platform/mac/WebCoreTextArea.mm:
* rendering/RenderBlock.cpp:
(WebCore::RenderBlock::isPointInScrollbar):
* rendering/RenderFormElement.cpp:
* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::horizontalScrollbarWidget):
(WebCore::RenderLayer::verticalScrollbarWidget):
(WebCore::RenderLayer::valueChanged):
(WebCore::RenderLayer::setHasHorizontalScrollbar):
(WebCore::RenderLayer::setHasVerticalScrollbar):
(WebCore::RenderLayer::positionScrollbars):
* rendering/RenderLayer.h:
* rendering/RenderWidget.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15721
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adele [Tue, 1 Aug 2006 00:05:30 +0000 (00:05 +0000)]
LayoutTests:
Reviewed by John and Anders.
- Test for <rdar://problem/
4380576> onChange does not get called when field changed by return key
* fast/forms/onchange-enter-submit-expected.txt: Added.
* fast/forms/onchange-enter-submit.html: Added.
WebCore:
Reviewed by John and Anders.
- Fix for <rdar://problem/
4380576> onChange does not get called when field changed by return key
Test: fast/forms/onchange-enter-submit.html
Blur (which will fire onChange appropriately) before submitting a form when pressing Enter.
This matches the sequence of events fired in WinIE.
* html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler):
* html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::defaultEventHandler):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15720
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Mon, 31 Jul 2006 23:55:15 +0000 (23:55 +0000)]
Reviewed by John.
<rdar://problem/
4658194> REGRESSION: "Search in Google"
and "Search in Spotlight" fail to work on text selected in a frame
Use selectedFrame to get the frame with the text selection.
* WebView/WebView.m:
(-[WebView _searchWithGoogleFromMenu:]):
(-[WebView _searchWithSpotlightFromMenu:]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15719
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren [Mon, 31 Jul 2006 22:27:46 +0000 (22:27 +0000)]
Reviewed by Kevin.
Moved string debug function outside of the WebCore namespace so that it's
easier to call in the debugger. I will never doubt Darin again (x100).
* platform/String.cpp:
(string):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15718
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt [Mon, 31 Jul 2006 21:06:15 +0000 (21:06 +0000)]
Get rid of the -50000 nonsense on scrollbars when they're first added to overflow blocks.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15717
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt [Mon, 31 Jul 2006 20:59:48 +0000 (20:59 +0000)]
Partial fix for 5453. Improve overflow so that when it changes
dynamically from scroll/auto to hidden that we properly hide the
scrollbars. Also refine the dirty dashboard region updating so that
it only happens if the scrollbars genuinely come and go.
Also eliminate the extra repaint of the entire block. This should not be
necessary, since the scrollbars repaint themselves already and children
will repaint already if they move.
Reviewed by darin
Adding test case from bug into fast/overflow/dynamic-hidden.html
* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::setHasHorizontalScrollbar):
(WebCore::RenderLayer::setHasVerticalScrollbar):
(WebCore::RenderLayer::updateScrollInfoAfterLayout):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15716
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt [Mon, 31 Jul 2006 20:58:28 +0000 (20:58 +0000)]
Add layout test for bug 5453.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15715
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Mon, 31 Jul 2006 19:49:10 +0000 (19:49 +0000)]
LayoutTests:
Reviewed by justin
<http://bugzilla.opendarwin.org/show_bug.cgi?id=9507>
Empty style spans created in applyInlineStyle
<rdar://problem/
4515463>
REGRESSION: Blot and Mail both do a very poor job of pasting the main www.apple.com page
* editing/pasteboard/paste-
4039777-fix-expected.txt:
* editing/pasteboard/testcase-9507-expected.checksum: Added.
* editing/pasteboard/testcase-9507-expected.png: Added.
* editing/pasteboard/testcase-9507-expected.txt: Added.
* editing/pasteboard/testcase-9507.html: Added.
* editing/unsupported-content/table-delete-001-expected.txt:
* editing/selection/selectNode-expected.png:
* editing/selection/selectNode-expected.checksum:
* editing/selection/selectNode-expected.txt:
WebCore:
Reviewed by justin
<http://bugzilla.opendarwin.org/show_bug.cgi?id=9507>
Empty style spans created in applyInlineStyle
Improves paste fidelity because some of these empty font/style spans had a non-zero
size and were messing up the layout of pasted content:
<rdar://problem/
4515463>
REGRESSION: Blot and Mail both do a very poor job of pasting the main www.apple.com page
* editing/ApplyStyleCommand.cpp:
(WebCore::ApplyStyleCommand::applyInlineStyle): Use the adjusted start node instead
of start.node(). Don't do any application if the endpoints are swapped. Adjust
endNode if the start node is a descendant of it, so that the pre-order traversal will
terminate properly.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15714
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Mon, 31 Jul 2006 19:14:26 +0000 (19:14 +0000)]
Reviewed by Tim Hatcher.
- http://bugzilla.opendarwin.org/show_bug.cgi?id=10168
add a first cut at a Metrics pane to the inspector
* WebInspector/webInspector/inspector.css: Add styles for the new metrics pane.
* WebInspector/webInspector/inspector.html: Add the new metrics pane, starting with
the table to show the box model.
* WebInspector/webInspector/inspector.js: Add the new metrics pane. Add back some
"title" attributes so we have more tooltips. Removed the optional parameter to
getComputedStyle.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15713
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren [Mon, 31 Jul 2006 18:44:35 +0000 (18:44 +0000)]
Reviewed by Darin.
* platform/String.cpp:
(WebCore::string): Added a debugging function to create a String from a
const char*.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15712
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Mon, 31 Jul 2006 18:21:03 +0000 (18:21 +0000)]
Update ChangeLog with bug link.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15711
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca [Mon, 31 Jul 2006 16:13:34 +0000 (16:13 +0000)]
2006-07-31 Anders Carlsson <acarlsson@apple.com>
Reviewed by John.
* Plugins/WebPluginDatabase.m:
(-[WebPluginDatabase refresh]):
Create a mutable set instead of a mutable array.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15710
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca [Mon, 31 Jul 2006 15:43:14 +0000 (15:43 +0000)]
2006-07-31 Anders Carlsson <acarlsson@apple.com>
* css/CSSPrimitiveValue.cpp:
Fix windows build.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15709
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren [Mon, 31 Jul 2006 08:44:46 +0000 (08:44 +0000)]
Reinstating my last change. Everything builds now.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15708
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Mon, 31 Jul 2006 08:41:52 +0000 (08:41 +0000)]
LayoutTests:
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10167
Bug 10167: REGRESSION(r15688): ASSERTION FAILED: _private->mouseDownEvent != nil in layout tests
* fast/events/objc-event-api-expected.txt: Update test results.
WebKitTools:
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10167
Bug 10167: REGRESSION(r15688): ASSERTION FAILED: _private->mouseDownEvent != nil in layout tests
* DumpRenderTree/EventSendingController.m: Don't set the mouse down flag
in keyDown handler.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15707
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren [Mon, 31 Jul 2006 08:31:37 +0000 (08:31 +0000)]
Backing out my last change because it causes a build failure in some
configurations. This seems to be the cleanest way to get svn to cooperate.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15706
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Mon, 31 Jul 2006 07:26:14 +0000 (07:26 +0000)]
* editing/pasteboard/paste-table-002-expected.txt: Corrected one result
affected by my recent change to getComputedStyle.
* editing/pasteboard/paste-table-002.html: Converted to a text test, since
it dumps the serialized form of hte result.
* editing/pasteboard/paste-table-002-expected.checksum: Removed.
* editing/pasteboard/paste-table-002-expected.png: Removed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15705
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren [Mon, 31 Jul 2006 07:17:51 +0000 (07:17 +0000)]
Reviewed by Maciej.
Moved files around to make WebCore's structure clearer
and easier to explain.
DerivedSources/JS* -> bindings/js/DerivedSources/ (so JS* files
show up when you search inside bindings/js/, but are marked, at
the same time, as files that you should not edit manually)
WebCore+SVG/DOMList.h -> ksvg2/svg/ (only files inside ksvg2/svg/
use DOMList.h)
icon -> loader/icon
I tested that everything still builds.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15704
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Mon, 31 Jul 2006 07:07:12 +0000 (07:07 +0000)]
Reviewed by Tim Hatcher.
* WebInspector/webInspector/inspector.js: Fix bug where a null property value
leads to an empty style pane.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15703
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Mon, 31 Jul 2006 06:52:06 +0000 (06:52 +0000)]
LayoutTests:
- test that displays all computed style -- demonstrates bug fixes I just
made to the computed style class and also acts as a regression test for
the existence of all of them and the basic format for a default object
* fast/css/computed-style-expected.txt: Added.
* fast/css/computed-style.html: Added.
WebCore:
Reviewed by Tim Hatcher.
- some improvements for the benefit of the style pane of the inspector
* css/CSSComputedStyleDeclaration.cpp: Removed background-position
and border-spacing from the list of properties that show up in
computed style, because of background-position-x, background-position-y,
-webkit-border-horizontal-spacing and -webkit-border-vertical-spacing.
(WebCore::valueForLength): Added handling for undefinedLength, intrinsic,
and min-intrinsic.
(WebCore::primitiveValueFromLength): Removed code that would add a
space to the string for no good reason.
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added a
special case for a line clamp of -1, which should come back as "none"
rather than an actual "-1".
(WebCore::CSSComputedStyleDeclaration::length): Return 0 if the
declaration has no corresponding node or no renderer.
(WebCore::CSSComputedStyleDeclaration::item): Check against length()
so that the two stay consistent rather than using a constant.
* css/CSSPrimitiveValue.cpp:
(WebCore::isCSSTokenizerIdentifier): Added.
(WebCore::isCSSTokenizerURL): Added.
(WebCore::quoteString): Added.
(WebCore::quoteStringIfNeeded): Changed to quote strings in many more
cases -- any cases where they would not parse in the CSS parser otherwise.
The main case this affects is font names with spaces in them.
(WebCore::quoteURLIfNeeded): Added.
(WebCore::CSSPrimitiveValue::cssText): Use quoteURLIfNeeded in the case
where we're making the text form of a URI.
* css/tokenizer.flex: Whitespace tweaks to line things up better.
WebKit:
Reviewed by Tim Hatcher.
- some improvements for the inspector
* WebInspector/WebInspector.m:
(+[WebInspector sharedWebInspector:]): Fixed bug that could cause the inspector
to be garbage collected if used in an application with GC enabled.
(-[WebInspector dealloc]): Removed a call to a non-existent close method.
(-[WebInspector window]): Added a custom WebPreferences object and called
setPrivateBrowsingEnabled:YES so the inspector won't appear in the history menu.
Also call setProhibitsMainFrameScrolling:YES to try to get rid of trouble where
the inspector scrolls when dragging.
* WebInspector/webInspector/inspector.css: Added style for the new color swatch,
and JavaScript properties. More of the style should be shared between the panes,
but this should be OK for now.
* WebInspector/webInspector/inspector.html: Added a first cut at a JavaScript
properties pane. Needs work, but better than nothing.
* WebInspector/webInspector/inspector.js: Lots of improvements:
- Omit "typical" property values from computed style display, making it much shorter.
- Use the words "black", "white", and "transparent" when appropriate for color values.
- Refactored the loaded() function to get rid of repetitive scrollbar setup.
- Added a new scrollarea for the JavaScript properties pane.
- Simplified refreshScrollbars() -- we now refresh all scrollbars every time, which does no harm.
- Removed unused resultsWithXpathQuery().
- Use [] instead of "new Array()" and {} instead of "new Object()".
- Removed unused xpathForNode().
- Changed style pane to display the style for a text node's parent instead of saying
it can't display the style for text.
- Fixed regression I caused a while back by checking the length of a computed style
and not trying to display anything if its length is 0. Before this change and the
corresponding change in WebCore, we'd see a complete list of all styles with the
empty string as the value for each one.
- Changed the name of the computedStyle flag on the style rules array to isComputedStyle
to make it easier to understand it's a boolean.
- Fixed an error in the code that does !important scanning where it was trying to
do a special case for computed style, but was checking the computed style flag on
the wrong object.
- Added populateStyleListItem() function to factor out things in common between the
items in the top level list and the expanded tree for shorthand properties.
- Added code to make a color swatch next to the textual representation for any
property that contains a color.
- Implemented a first cut at a simple JavaScript properties pane.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15702
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Mon, 31 Jul 2006 06:15:05 +0000 (06:15 +0000)]
Reviewed by Maciej.
Bug 9686: [Drosera] Need the ability to break into Drosera on Javascript exceptions
http://bugzilla.opendarwin.org/show_bug.cgi?id=9686
* Drosera/DebuggerDocument.m:
(-[WebScriptObject switchToServerNamed:]): Remove as listener before resuming to
ensure that the server does not try and notify us of events related to the resumption.
(-[WebScriptObject webView:exceptionWasRaised:sourceId:line:forWebFrame:]): Delegate
call through to JavaScript.
* Drosera/debugger.js: Pause debugger when exception is raised.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15701
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren [Sun, 30 Jul 2006 23:02:09 +0000 (23:02 +0000)]
Reviewed by Beth.
- Tweaked the checkout and build instructions to make them simpler and
clearer.
* building/build.html:
* building/checkout.html:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15700
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
eseidel [Sun, 30 Jul 2006 04:31:53 +0000 (04:31 +0000)]
2006-07-30 Eric Seidel <eric@eseidel.com>
Reviewed by gramps!
Remove some unused SVG code.
* ksvg2/ecma/Ecma.cpp: Removed.
* ksvg2/ecma/Ecma.h: Removed.
* ksvg2/ecma/GlobalObject.cpp: remove a couple dead functions.
* ksvg2/ecma/GlobalObject.h: add a comment about this dead code.
* ksvg2/ecma/SVGLookup.h: Removed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15699
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sun, 30 Jul 2006 00:04:22 +0000 (00:04 +0000)]
JavaScriptCore:
- Removed tabs from these source files that still had them.
We don't use them; that way source files look fine in editors
that have tabs set to 8 spaces or to 4 spaces.
- Removed allow-tabs Subversion property from the files too.
* bindings/NP_jsobject.cpp:
* bindings/c/c_utility.cpp:
* bindings/jni/jni_runtime.cpp:
* bindings/jni/jni_utility.cpp:
* bindings/objc/objc_utility.mm:
* bindings/runtime.cpp:
* bindings/runtime_method.cpp:
* bindings/testbindings.cpp:
* bindings/testbindings.mm:
* kjs/date_object.cpp:
* kjs/function.cpp:
* kjs/list.cpp:
* kjs/nodes.cpp:
* kjs/nodes.h:
* kjs/string_object.cpp:
* kjs/ustring.cpp:
WebCore:
- Removed tabs from these source files that still had them.
We don't use them; that way source files look fine in editors
that have tabs set to 8 spaces or to 4 spaces.
- Removed allow-tabs Subversion property from the files too.
* bindings/objc/WebScriptObject.h:
* bindings/objc/WebScriptObject.mm:
* platform/FontData.cpp:
WebKit:
- Removed tabs from these source files that still had them.
We don't use them; that way source files look fine in editors
that have tabs set to 8 spaces or to 4 spaces.
- Removed allow-tabs Subversion property from the files too.
* DefaultDelegates/WebDefaultPolicyDelegate.m:
* History/WebHistory.m:
* Misc/WebDownload.m:
* Misc/WebIconDatabase.m:
* Misc/WebKitErrors.m:
* Misc/WebKitLogging.m:
* Misc/WebNSDataExtras.m:
* Misc/WebNSFileManagerExtras.m:
* Panels/WebPanelAuthenticationHandler.m:
* Plugins/WebBaseNetscapePluginView.m:
* Plugins/npfunctions.h:
* WebCoreSupport/WebSubresourceLoader.m:
* WebView/WebMainResourceLoader.m:
* WebView/WebView.h:
* WebView/WebView.m:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15698
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 23:41:40 +0000 (23:41 +0000)]
* tests/mozilla/expected.html: Update test results now that regress-185165.js
is succeeding. I suspect Anders fix for bug
4620655 is the reason.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15697
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig [Sat, 29 Jul 2006 23:15:25 +0000 (23:15 +0000)]
JavaScriptCore:
Reviewed by Darin.
- patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10080
Adopt pedantic changes from the Unity project to improve
cross-compiler compatibility
Changes include:
* Removing trailing semicolon from namespace braces.
* Removing trailing comma from last enum declaration.
* Updating to match style guidelines.
* Adding missing newline to the end of the file.
* Turning on gcc warning for missing newline at the end of a source file
(GCC_WARN_ABOUT_MISSING_NEWLINE in Xcode, -Wnewline in gcc).
* Alphabetical sorting of Xcode source list files.
* Replace use of non-portable variable-size array with Vector.
* Use C-style comments instead of C++ comments in files that might
be included by either C or C++ files.
* API/JSCallbackConstructor.cpp:
(KJS::JSCallbackConstructor::construct):
* API/JSCallbackFunction.cpp:
(KJS::JSCallbackFunction::callAsFunction):
* API/JSCallbackObject.cpp:
(KJS::JSCallbackObject::construct):
(KJS::JSCallbackObject::callAsFunction):
* JavaScriptCore.xcodeproj/project.pbxproj:
* JavaScriptCorePrefix.h:
* bindings/jni/jni_class.cpp:
(JavaClass::fieldNamed):
* bindings/jni/jni_class.h:
* bindings/jni/jni_instance.cpp:
(JavaInstance::JavaInstance):
(JavaInstance::valueOf):
* bindings/jni/jni_objc.mm:
(KJS::Bindings::dispatchJNICall):
* bindings/jni/jni_runtime.cpp:
(JavaParameter::JavaParameter):
(JavaArray::JavaArray):
* bindings/jni/jni_runtime.h:
* bindings/jni/jni_utility.h:
* bindings/objc/objc_instance.h:
* bindings/runtime_array.h:
* kjs/collector.h:
* kjs/config.h:
* kjs/ustring.cpp:
* wtf/Platform.h:
JavaScriptGlue:
Reviewed by Darin.
- patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10080
Adopt pedantic changes from the Unity project to improve
cross-compiler compatibility
Changes include:
* Turning on gcc warning for missing newline at the end of a source file
(GCC_WARN_ABOUT_MISSING_NEWLINE in Xcode, -Wnewline in gcc).
* JavaScriptGlue.xcodeproj/project.pbxproj:
WebCore:
Reviewed by Darin.
- patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10080
Adopt pedantic changes from the Unity project to improve
cross-compiler compatibility
Changes include:
* Removing trailing semicolon from namespace braces.
* Removing trailing comma from last enum declaration.
* Updating to match style guidelines.
* Adding missing newline to the end of the file.
* Turning on gcc warning for missing newline at the end of a source file
(GCC_WARN_ABOUT_MISSING_NEWLINE in Xcode, -Wnewline in gcc).
* Alphabetical sorting of Xcode source list files.
* Use abs() function from <math.h> instead of ABS() macro.
* Use C-style comments instead of C++ comments in files that might
be included by either C or C++ files.
* Use -numeric_limits<double>::infinity() instead of -HUGE_VAL.
* WebCore+SVG/DOMList.h:
* WebCore.xcodeproj/project.pbxproj:
* WebCorePrefix.h:
* bindings/js/JSCanvasRenderingContext2DCustom.cpp:
* bindings/js/JSXSLTProcessor.h:
* bindings/js/kjs_domnode.h:
(KJS::DOMNode::):
* bindings/objc/DOMCSS.h:
(-[DOMCSSValue enum]):
* bindings/objc/DOMCore.h:
(-[DOMImplementation createDocument:::]):
* bindings/objc/DOMEvents.h:
(-[DOMOverflowEvent enum]):
* bindings/objc/DOMRange.h:
* bindings/objc/DOMTraversal.h:
* bindings/objc/DOMXPath.h:
(-[DOMXPathNSResolver lookupNamespaceURI:]):
* bridge/mac/WebCoreFrameBridge.h:
* bridge/mac/WebCoreKeyboardAccess.h:
* css/makeprop:
* css/makevalues:
* dom/ChildNodeList.h:
* dom/DocPtr.h:
* dom/Document.h:
(WebCore::Document::):
* dom/DocumentMarker.h:
(WebCore::DocumentMarker::):
(WebCore::DocumentMarker::operator==):
(WebCore::DocumentMarker::operator!=):
* dom/EventTargetNode.h:
* dom/KeyboardEvent.h:
(WebCore::KeyboardEvent::):
* dom/NameNodeList.h:
(WebCore::NameNodeList::rootNodeChildrenChanged):
* dom/QualifiedName.cpp:
* editing/TypingCommand.h:
(WebCore::TypingCommand::):
* editing/UnlinkCommand.h:
(WebCore::UnlinkCommand::editingAction):
* html/FormDataList.h:
(WebCore::FormDataListItem::FormDataListItem):
(WebCore::FormDataList::appendData):
* html/HTMLBlockquoteElement.h:
* html/HTMLDivElement.h:
* html/HTMLFormElement.h:
* html/HTMLHRElement.h:
* html/HTMLHeadingElement.h:
* html/HTMLMarqueeElement.h:
* html/HTMLParagraphElement.h:
* html/HTMLPlugInElement.h:
* html/HTMLPreElement.h:
* html/HTMLTokenizer.h:
(WebCore::HTMLTokenizer::State::):
* icon/IconDatabase.cpp:
* icon/SQLStatement.cpp:
* kcanvas/KCanvasFilters.h:
(WebCore::):
(WebCore::KCanvasPoint3F::KCanvasPoint3F):
(WebCore::KCanvasFilter::KCanvasFilter):
(WebCore::KCanvasFilter::~KCanvasFilter):
(WebCore::KCanvasFilterEffect::~KCanvasFilterEffect):
(WebCore::KCComponentTransferFunction::KCComponentTransferFunction):
(WebCore::KCanvasFEConvolveMatrix::KCanvasFEConvolveMatrix):
(WebCore::KCLightSource::KCLightSource):
(WebCore::KCDistantLightSource::KCDistantLightSource):
(WebCore::KCPointLightSource::KCPointLightSource):
(WebCore::KCSpotLightSource::KCSpotLightSource):
(WebCore::KCanvasFEDiffuseLighting::KCanvasFEDiffuseLighting):
(WebCore::KCanvasFEDisplacementMap::KCanvasFEDisplacementMap):
(WebCore::KCanvasFEImage::KCanvasFEImage):
(WebCore::KCanvasFESpecularLighting::KCanvasFESpecularLighting):
* kcanvas/RenderSVGImage.h:
* kcanvas/device/quartz/KRenderingDeviceQuartz.h:
* ksvg2/css/SVGRenderStyle.h:
(WebCore::SVGRenderStyle::InheritedFlags::):
(WebCore::SVGRenderStyle::NonInheritedFlags::):
* ksvg2/css/SVGRenderStyleDefs.h:
(WebCore::):
* ksvg2/events/SVGZoomEvent.h:
* ksvg2/ksvg.h:
(WebCore::):
* ksvg2/misc/KCanvasRenderingStyle.h:
* ksvg2/misc/SVGImageLoader.h:
* ksvg2/scripts/make_names.pl:
* ksvg2/svg/SVGAElement.h:
* ksvg2/svg/SVGAngle.h:
* ksvg2/svg/SVGAnimateColorElement.h:
* ksvg2/svg/SVGAnimateElement.h:
* ksvg2/svg/SVGAnimateTransformElement.h:
* ksvg2/svg/SVGAnimatedAngle.h:
* ksvg2/svg/SVGAnimatedBoolean.h:
* ksvg2/svg/SVGAnimatedColor.h:
* ksvg2/svg/SVGAnimatedEnumeration.h:
* ksvg2/svg/SVGAnimatedInteger.h:
* ksvg2/svg/SVGAnimatedLength.h:
* ksvg2/svg/SVGAnimatedLengthList.h:
* ksvg2/svg/SVGAnimatedNumber.h:
* ksvg2/svg/SVGAnimatedNumberList.h:
* ksvg2/svg/SVGAnimatedPathData.h:
* ksvg2/svg/SVGAnimatedPoints.h:
* ksvg2/svg/SVGAnimatedPreserveAspectRatio.h:
* ksvg2/svg/SVGAnimatedRect.h:
* ksvg2/svg/SVGAnimatedString.h:
* ksvg2/svg/SVGAnimatedTemplate.h:
* ksvg2/svg/SVGAnimatedTransformList.h:
* ksvg2/svg/SVGAnimationElement.cpp:
(SVGAnimationElement::calculateCurrentValueItem):
(SVGAnimationElement::calculateRelativeTimePercentage):
* ksvg2/svg/SVGAnimationElement.h:
(WebCore::):
* ksvg2/svg/SVGCircleElement.h:
* ksvg2/svg/SVGClipPathElement.h:
* ksvg2/svg/SVGColor.h:
* ksvg2/svg/SVGComponentTransferFunctionElement.h:
* ksvg2/svg/SVGCursorElement.h:
* ksvg2/svg/SVGDOMImplementation.h:
* ksvg2/svg/SVGDefsElement.h:
* ksvg2/svg/SVGDescElement.h:
* ksvg2/svg/SVGDocument.h:
* ksvg2/svg/SVGElement.h:
(WebCore::SVGElement::rendererIsNeeded):
(WebCore::svg_dynamic_cast):
* ksvg2/svg/SVGElementInstance.h:
* ksvg2/svg/SVGElementInstanceList.h:
* ksvg2/svg/SVGEllipseElement.h:
* ksvg2/svg/SVGExternalResourcesRequired.h:
* ksvg2/svg/SVGFEBlendElement.h:
* ksvg2/svg/SVGFEColorMatrixElement.h:
* ksvg2/svg/SVGFEComponentTransferElement.h:
* ksvg2/svg/SVGFECompositeElement.h:
* ksvg2/svg/SVGFEDiffuseLightingElement.h:
* ksvg2/svg/SVGFEDisplacementMapElement.h:
* ksvg2/svg/SVGFEDistantLightElement.h:
* ksvg2/svg/SVGFEFloodElement.h:
* ksvg2/svg/SVGFEFuncAElement.h:
* ksvg2/svg/SVGFEFuncBElement.h:
* ksvg2/svg/SVGFEFuncGElement.h:
* ksvg2/svg/SVGFEFuncRElement.h:
* ksvg2/svg/SVGFEGaussianBlurElement.h:
* ksvg2/svg/SVGFEImageElement.h:
* ksvg2/svg/SVGFELightElement.h:
* ksvg2/svg/SVGFEMergeElement.h:
* ksvg2/svg/SVGFEMergeNodeElement.h:
* ksvg2/svg/SVGFEOffsetElement.h:
* ksvg2/svg/SVGFEPointLightElement.h:
* ksvg2/svg/SVGFESpecularLightingElement.h:
* ksvg2/svg/SVGFESpotLightElement.h:
* ksvg2/svg/SVGFETileElement.h:
* ksvg2/svg/SVGFETurbulenceElement.h:
* ksvg2/svg/SVGFilterElement.h:
* ksvg2/svg/SVGFilterPrimitiveStandardAttributes.h:
* ksvg2/svg/SVGFitToViewBox.h:
* ksvg2/svg/SVGForeignObjectElement.cpp:
* ksvg2/svg/SVGForeignObjectElement.h:
* ksvg2/svg/SVGGElement.h:
* ksvg2/svg/SVGGradientElement.h:
* ksvg2/svg/SVGHelper.h:
(WebCore::):
* ksvg2/svg/SVGImageElement.h:
* ksvg2/svg/SVGLangSpace.h:
* ksvg2/svg/SVGLength.h:
* ksvg2/svg/SVGLengthList.h:
* ksvg2/svg/SVGLineElement.h:
* ksvg2/svg/SVGLinearGradientElement.h:
* ksvg2/svg/SVGList.h:
* ksvg2/svg/SVGLocatable.h:
* ksvg2/svg/SVGMarkerElement.h:
* ksvg2/svg/SVGMaskElement.h:
* ksvg2/svg/SVGMatrix.h:
* ksvg2/svg/SVGNumber.h:
* ksvg2/svg/SVGNumberList.h:
* ksvg2/svg/SVGPaint.h:
* ksvg2/svg/SVGPathElement.h:
* ksvg2/svg/SVGPathSeg.h:
* ksvg2/svg/SVGPathSegArc.h:
* ksvg2/svg/SVGPathSegClosePath.h:
* ksvg2/svg/SVGPathSegCurvetoCubic.h:
* ksvg2/svg/SVGPathSegCurvetoCubicSmooth.h:
* ksvg2/svg/SVGPathSegCurvetoQuadratic.h:
* ksvg2/svg/SVGPathSegCurvetoQuadraticSmooth.h:
* ksvg2/svg/SVGPathSegLineto.h:
* ksvg2/svg/SVGPathSegLinetoHorizontal.h:
* ksvg2/svg/SVGPathSegLinetoVertical.h:
* ksvg2/svg/SVGPathSegList.h:
* ksvg2/svg/SVGPathSegMoveto.h:
* ksvg2/svg/SVGPatternElement.h:
* ksvg2/svg/SVGPoint.h:
* ksvg2/svg/SVGPointList.h:
* ksvg2/svg/SVGPolyElement.h:
* ksvg2/svg/SVGPolygonElement.h:
* ksvg2/svg/SVGPolylineElement.h:
* ksvg2/svg/SVGPreserveAspectRatio.h:
* ksvg2/svg/SVGRadialGradientElement.h:
* ksvg2/svg/SVGRect.h:
* ksvg2/svg/SVGRectElement.h:
* ksvg2/svg/SVGSVGElement.h:
* ksvg2/svg/SVGScriptElement.h:
* ksvg2/svg/SVGSetElement.h:
* ksvg2/svg/SVGStopElement.h:
* ksvg2/svg/SVGStringList.h:
* ksvg2/svg/SVGStylable.h:
* ksvg2/svg/SVGStyleElement.h:
* ksvg2/svg/SVGStyledElement.h:
(WebCore::SVGStyledElement::rendererIsNeeded):
(WebCore::SVGStyledElement::canvasResource):
* ksvg2/svg/SVGStyledLocatableElement.h:
* ksvg2/svg/SVGStyledTransformableElement.h:
* ksvg2/svg/SVGSwitchElement.h:
* ksvg2/svg/SVGSymbolElement.h:
* ksvg2/svg/SVGTRefElement.h:
* ksvg2/svg/SVGTSpanElement.h:
* ksvg2/svg/SVGTests.h:
* ksvg2/svg/SVGTextContentElement.h:
* ksvg2/svg/SVGTextElement.h:
* ksvg2/svg/SVGTextPositioningElement.h:
* ksvg2/svg/SVGTitleElement.h:
* ksvg2/svg/SVGTransform.h:
* ksvg2/svg/SVGTransformList.h:
* ksvg2/svg/SVGTransformable.h:
* ksvg2/svg/SVGURIReference.h:
* ksvg2/svg/SVGUseElement.h:
* ksvg2/svg/SVGViewElement.h:
* ksvg2/svg/SVGZoomAndPan.h:
* ksvg2/svg/svgpathparser.h:
* page/Frame.h:
(WebCore::):
* platform/AffineTransform.h:
* platform/FontCache.cpp:
(WebCore::FontPlatformDataCacheKey::FontPlatformDataCacheKey):
* platform/FontData.cpp:
(WebCore::FontData::FontData):
* platform/FontData.h:
* platform/TextBox.h:
(WebCore::TextBox::):
* platform/Timer.cpp:
(WebCore::TimerBase::heapPop):
* platform/mac/FontCacheMac.mm:
* platform/mac/GlyphMapMac.cpp:
* platform/mac/WebFontCache.mm:
(betterChoice):
* rendering/DeprecatedRenderSelect.cpp:
(WebCore::DeprecatedRenderSelect::setWidgetWritingDirection):
* rendering/EllipsisBox.h:
* rendering/RenderBR.h:
(WebCore::RenderBR::renderName):
(WebCore::RenderBR::width):
* rendering/RenderBlock.h:
(WebCore::):
* rendering/RenderFlexibleBox.h:
* rendering/RenderFlow.h:
(WebCore::RenderFlow::RenderFlow):
* rendering/RenderFrame.cpp:
* rendering/bidi.h:
* rendering/break_lines.cpp:
WebKit:
Reviewed by Darin.
- patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=10080
Adopt pedantic changes from the Unity project to improve
cross-compiler compatibility
Changes include:
* Adding missing newline to the end of the file.
* Turning on gcc warning for missing newline at the end of a source file
(GCC_WARN_ABOUT_MISSING_NEWLINE in Xcode, -Wnewline in gcc).
* WebKit.xcodeproj/project.pbxproj:
* WebView/WebResourcePrivate.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15696
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 23:09:11 +0000 (23:09 +0000)]
Clear stray executable bits.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15695
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 22:50:14 +0000 (22:50 +0000)]
Clear some stray executable bits.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15694
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
rwlbuis [Sat, 29 Jul 2006 21:05:40 +0000 (21:05 +0000)]
Remove the output produced by an alert, so
hopefully buildbot will not complain anymore :)
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15693
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
rwlbuis [Sat, 29 Jul 2006 20:25:31 +0000 (20:25 +0000)]
Reviewed by Geoff.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10151
ghostmap XHTML + SVG fails due to javascript undefined exception
Make sure document.URL works when called from a non-HTML document.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15692
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 15:44:20 +0000 (15:44 +0000)]
JavaScriptCore:
Reviewed by Darin.
- fixes for Linux build
* JavaScriptCoreSources.bkl: Added new files to build, kjs/PropertyNameArray.cpp
and kjs/testkjs.cpp, and removed old files.
WebCore:
Reviewed and tweaked by Darin.
- fixes for Linux build
(still might not compile, since Darin tweaked but was unable to test).
* WebCoreSources.bkl: Added some new sources, removed some obsolete ones.
* Projects/gdk/webcore-gdk.bkl: Added RenderPopupMenuGdk.cpp
* html/HTMLPlugInElement.h: Removed incorrect syntax with class name
before ::.
* platform/gdk/FrameGdk.h:
* platform/gdk/FrameGdk.cpp: Updated parameters of constructor. Moved a couple stubs
out of this file.
* platform/gdk/RenderPopupMenuGdk.h: Added.
* platform/gdk/RenderPopupMenuGdk.cpp: Added. Just a file full of empty stubs.
* platform/gdk/RenderThemeGdk.h:
* platform/gdk/RenderThemeGdk.cpp: (WebCore::RenderThemeGdk::createPopupMenu): Added.
* platform/gdk/TemporaryLinkStubs.cpp: Add some new stubs.
WebKitTools:
Reviewed by Darin.
- fixes for Linux build
* GdkLauncher/mk: Added call to bakefile_gen before calling make.
* GdkLauncher/gdklauncher.bkl: Tweak comment.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15691
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 15:43:11 +0000 (15:43 +0000)]
- changed this test to a plain-text one and added text explaining what it tests
* fast/forms/input-named-action-overrides-action-attribute.html: Added explanation text,
got rid of some incorrect HTML syntax, added a call to dumpAsText(), and used a paragraph
element instead of a span element for the result.
* fast/forms/input-named-action-overrides-action-attribute-expected.txt: Updated.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15690
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 15:21:41 +0000 (15:21 +0000)]
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10022
Bug 10022: REGRESSION: Crash in WebCore::XMLTokenizer::characters
* dom/XMLTokenizer.cpp:
(WebCore::XMLTokenizer::resumeParsing): If the write() call resulted in new callbacks being
added, don't call through to end() just yet.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15689
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 15:19:13 +0000 (15:19 +0000)]
WebCore:
Reviewed by John Sullivan.
- manual test for http://bugzilla.opendarwin.org/show_bug.cgi?id=9984
ASSERTION FAILURE: _private->mouseDownEvent != nil
(WebKit/WebView/WebHTMLView.m:4863 -[WebHTMLView(WebInternal) _delegateDragSourceActionMask])
* manual-tests/subview-click-assertion.html: Added.
WebKit:
Reviewed by John Sullivan.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9984
ASSERTION FAILURE: _private->mouseDownEvent != nil
(WebKit/WebView/WebHTMLView.m:4863 -[WebHTMLView(WebInternal) _delegateDragSourceActionMask])
* WebView/WebHTMLView.m:
(-[WebHTMLView _setMouseDownEvent:]): Moved into the WebHTMLViewFileInternal category and changed
to accept nil.
(-[WebHTMLView _startDraggingImage:at:operation:event:sourceIsDHTML:DHTMLWroteData:]):
Copy the hit HTMLView's mouse down event to the top HTMLView.
(-[WebHTMLView acceptsFirstMouse:]): Added a call to _setMouseDownEvent:nil before returning.
(-[WebHTMLView shouldDelayWindowOrderingForEvent:]): Added a call to _setMouseDownEvent:nil
before returning.
(-[WebHTMLView mouseUp:]): Added a call to _setMouseDownEvent:nil to clear the event set in
mouseDown: (and used during dragging).
(-[WebHTMLView _delegateDragSourceActionMask]): Copy the hit HTMLView's mouse down event to
the top HTMLView.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15688
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 15:06:04 +0000 (15:06 +0000)]
LayoutTests:
Reviewed by Darin.
Test for http://bugzilla.opendarwin.org/show_bug.cgi?id=8587 / <rdar://problem/
4631844>
REGRESSION: {display:list-item} items outside an ol or ul element don't number correctly
* fast/lists/ordered-list-with-no-ol-tag-expected.checksum: Added.
* fast/lists/ordered-list-with-no-ol-tag-expected.png: Added.
* fast/lists/ordered-list-with-no-ol-tag-expected.txt: Added.
* fast/lists/ordered-list-with-no-ol-tag.html: Added.
WebCore:
Reviewed by Darin.
Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=8587 / <rdar://problem/
4631844>
REGRESSION: {display:list-item} items outside an ol or ul element don't number correctly
* rendering/RenderListItem.cpp:
(WebCore::enclosingList): If no real list element is found, use parent instead.
(WebCore::previousListItem): Check list elements to see if they are list items -- the
old code assumed that a list would never be a list item but this is entirely possible
with CSS and especially with the new rule that means any DOM element could act as a list.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15687
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Sat, 29 Jul 2006 14:56:42 +0000 (14:56 +0000)]
Reviewed by Darin.
Fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10147
REGRESSION: custom attribute values set via javascript are not persistent
Test: fast/dom/Element/setAttribute-case-insensitivity.html
* dom/Element.cpp:
(WebCore::Element::setAttribute): Use the lowercased localName for retrieving the old value, too.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15686
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Sat, 29 Jul 2006 14:53:40 +0000 (14:53 +0000)]
- rolled out the test for bug 9753, since we rolled out the fix
we'll bring back the test when we bring back the fix
* svg/custom/width-full-percentage-expected.checksum: Removed.
* svg/custom/width-full-percentage-expected.png: Removed.
* svg/custom/width-full-percentage-expected.txt: Removed.
* svg/custom/width-full-percentage.svg: Removed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15685
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Sat, 29 Jul 2006 00:25:56 +0000 (00:25 +0000)]
Suggested by Darin.
Moving the implementation of isContentEditable to the
the WebPrivate category.
* bindings/objc/DOM.mm:
(-[DOMNode isContentEditable]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15684
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Sat, 29 Jul 2006 00:24:21 +0000 (00:24 +0000)]
LayoutTests:
Reviewed by levi
<rdar://problem/
4515463>
REGRESSION: Blot and Mail both do a very poor job of pasting the main www.apple.com page
Removed incorrect nesting:
* editing/pasteboard/paste-table-001-expected.checksum:
* editing/pasteboard/paste-table-001-expected.png:
* editing/pasteboard/paste-table-001-expected.txt:
* editing/pasteboard/paste-table-003-expected.txt:
WebCore:
Reviewed by levi
First part of:
<rdar://problem/
4515463>
REGRESSION: Blot and Mail both do a very poor job of pasting the main www.apple.com page
Migrate to isBlock and enclosingBlock.
Changed RendereringInfo::isBlock and ReplacementFragment::isBlock
to wasBlock so that they don't conflict with isBlock and because
it's more descriptive.
* editing/ReplaceSelectionCommand.cpp:
(WebCore::ReplacementFragment::mergeStartNode):
(WebCore::ReplacementFragment::wasBlock):
(WebCore::ReplacementFragment::saveRenderingInfo):
(WebCore::ReplacementFragment::renderedBlocks):
(WebCore::RenderingInfo::RenderingInfo):
(WebCore::ReplaceSelectionCommand::shouldMergeStart):
(WebCore::ReplaceSelectionCommand::doApply):
* editing/ReplaceSelectionCommand.h:
(WebCore::RenderingInfo::wasBlock):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15683
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Fri, 28 Jul 2006 22:01:54 +0000 (22:01 +0000)]
LayoutTests:
Reviewed by levi
<rdar://problem/
4242293>
Poor paste fidelity of http://www.google.com/
Removed one of the two extraneous newlines that are introduced on paste:
* editing/pasteboard/paste-table-003-expected.checksum:
* editing/pasteboard/paste-table-003-expected.png:
* editing/pasteboard/paste-table-003-expected.txt:
Fewer pastes remove nodes containing the selection b/c of
the changes to interchange newlines:
* editing/pasteboard/paste-line-endings-007-expected.txt:
* editing/pasteboard/paste-line-endings-008-expected.txt:
* editing/pasteboard/paste-line-endings-009-expected.txt:
Added:
* editing/pasteboard/
4242293-1-expected.checksum: Added.
* editing/pasteboard/
4242293-1-expected.png: Added.
* editing/pasteboard/
4242293-1-expected.txt: Added.
* editing/pasteboard/
4242293-1.html: Added.
* editing/pasteboard/
4242293-expected.checksum: Added.
* editing/pasteboard/
4242293-expected.png: Added.
* editing/pasteboard/
4242293-expected.txt: Added.
* editing/pasteboard/
4242293.html: Added.
WebCore:
Reviewed by levi
More changes for:
<rdar://problem/
4242293>
Poor paste fidelity of http://www.google.com/
* editing/markup.cpp:
(WebCore::needInterchangeNewlineAfter): Added.
(WebCore::createMarkup):
Only add an interchangeNewline if we're annotating for interchange. Can't test
this until we expose createMarkup in non-annotate mode (I'd expose Range::toHTML).
Remove the FIXME about prepending the interchangeNewline before creating the rest
of the markup. Its correct to surround the interchangeNewline in ancestors of the
adjusted startNode.
Fixed the check to see if an interchangeNewline is needed (!inSameBlock). This would
add an interchangeNewline for a selection that started before a table.
Removed the workaround for the bug where markup for a table was not included when
selecting the contents of a table, since this change fixes the general problem of
markup for ancestors of the startNode being left out.
Don't leave out ancestors of the startNode as we leave their subtrees. Execute the
code to include them even if we closed ancestors in the ancestorsToClose list.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15673
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Fri, 28 Jul 2006 19:59:37 +0000 (19:59 +0000)]
Reviewed by John.
<rdar://problem/
4657473> REGRESSION: Spell check not available from contextual menu in Mail
The context menu code should be checking isContentEditable
on DOMNode not just DOMElement. This is needed because DOMText
will be the node class of any text that is clicked.
* DefaultDelegates/WebDefaultContextMenuDelegate.m:
(-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15671
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Fri, 28 Jul 2006 19:58:30 +0000 (19:58 +0000)]
WebCore:
Reviewed by John.
Adding isContentEditable to DOMNode's private category.
* bindings/objc/DOMPrivate.h:
WebKit:
Reviewed by John.
The context menu code should be checking isContentEditable
on DOMNode not just DOMElement. This is needed because DOMText
will be the node class of any text that is clicked.
* DefaultDelegates/WebDefaultContextMenuDelegate.m:
(-[WebDefaultUIDelegate webView:contextMenuItemsForElement:defaultMenuItems:]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15670
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Fri, 28 Jul 2006 19:19:07 +0000 (19:19 +0000)]
Rolling out bugzilla.opendarwin.org/show_bug.cgi?id=6010 because of failing layout tests.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15669
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adele [Fri, 28 Jul 2006 19:01:18 +0000 (19:01 +0000)]
Rolling out bugzilla.opendarwin.org/show_bug.cgi?id=9753 because of failing layout tests.
SVG with width and height 100% leaves room for scrollbar on the right.
* kcanvas/RenderPath.cpp:
* kcanvas/RenderPath.h:
* kcanvas/RenderSVGContainer.cpp:
(WebCore::RenderSVGContainer::layout):
* ksvg2/svg/SVGCircleElement.cpp:
* ksvg2/svg/SVGCircleElement.h:
* ksvg2/svg/SVGEllipseElement.cpp:
* ksvg2/svg/SVGEllipseElement.h:
* ksvg2/svg/SVGLineElement.cpp:
* ksvg2/svg/SVGLineElement.h:
* ksvg2/svg/SVGRectElement.cpp:
* ksvg2/svg/SVGRectElement.h:
* ksvg2/svg/SVGStyledElement.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15668
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
eseidel [Fri, 28 Jul 2006 18:07:35 +0000 (18:07 +0000)]
2006-07-27 Eric Seidel <eric@eseidel.com>
Reviewed by darin.
Add better SVGLoad event support.
http://bugzilla.opendarwin.org/show_bug.cgi?id=6010
There will still need to be additional support added for <link> and <script> elements.
Not all of the error -> loaded -> error transition cases work yet.
* dom/EventTargetNode.cpp:
(WebCore::EventTargetNode::dispatchGenericEvent):
* html/HTMLImageLoader.cpp:
(WebCore::HTMLImageLoader::dispatchLoadEvent):
* html/HTMLImageLoader.h:
(WebCore::HTMLImageLoader::haveFiredLoadEvent):
(WebCore::HTMLImageLoader::setHaveFiredLoadEvent):
* ksvg2/misc/SVGImageLoader.cpp:
(WebCore::SVGImageLoader::SVGImageLoader):
(WebCore::SVGImageLoader::dispatchLoadEvent):
* ksvg2/misc/SVGImageLoader.h:
* ksvg2/svg/SVGAElement.cpp:
(WebCore::SVGAElement::parseMappedAttribute):
* ksvg2/svg/SVGAnimatedBoolean.h:
* ksvg2/svg/SVGElement.cpp:
(WebCore::SVGElement::SVGElement):
(WebCore::SVGElement::parseMappedAttribute):
(WebCore::SVGElement::haveLoadedRequiredResources):
(WebCore::SVGElement::sendSVGLoadEventIfPossible):
(WebCore::SVGElement::closeRenderer):
* ksvg2/svg/SVGElement.h:
(WebCore::SVGElement::rendererIsNeeded):
(WebCore::svg_dynamic_cast):
* ksvg2/svg/SVGExternalResourcesRequired.h:
* ksvg2/svg/SVGImageElement.cpp:
(SVGImageElement::SVGImageElement):
(SVGImageElement::haveLoadedRequiredResources):
* ksvg2/svg/SVGImageElement.h:
* ksvg2/svg/SVGSVGElement.cpp:
(WebCore::SVGSVGElement::parseMappedAttribute):
* ksvg2/svg/SVGScriptElement.cpp:
(WebCore::SVGScriptElement::SVGScriptElement):
(WebCore::SVGScriptElement::parseMappedAttribute):
(WebCore::SVGScriptElement::executeScript):
* xml/xmlhttprequest.cpp:
(WebCore::XMLHttpRequest::callReadyStateChangeListener): use new Event() instead of createEvent
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15667
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adele [Fri, 28 Jul 2006 05:13:24 +0000 (05:13 +0000)]
Reviewed by Anders.
- http://bugzilla.opendarwin.org/show_bug.cgi?id=10135
Cleanup: remove useless parameters from RenderObject::position()
No test possible (no functionality change)
* rendering/RenderBox.cpp:
(WebCore::RenderBox::position):
* rendering/RenderBox.h:
* rendering/RenderObject.h:
(WebCore::RenderObject::position):
* rendering/RenderTableRow.h:
(WebCore::RenderTableRow::position):
* rendering/RenderTableSection.h:
(WebCore::RenderTableSection::position):
* rendering/RenderText.cpp:
(WebCore::RenderText::position):
* rendering/RenderText.h:
* rendering/bidi.cpp:
(WebCore::RenderBlock::constructLine):
(WebCore::RenderBlock::computeVerticalPositionsForLine):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15666
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adele [Fri, 28 Jul 2006 04:53:11 +0000 (04:53 +0000)]
LayoutTests:
Reviewed by Maciej.
Test case for
<http://bugzilla.opendarwin.org/show_bug.cgi?id=9753>
SVG with width and height 100% leaves room for scrollbar on the right.
* svg/custom/width-full-percentage-expected.txt: Added.
* svg/custom/width-full-percentage-expected.checksum: Added.
* svg/custom/width-full-percentage-expected.png: Added.
* svg/custom/width-full-percentage.svg: Added.
WebCore:
Reviewed by Maciej.
http://bugzilla.opendarwin.org/show_bug.cgi?id=9753
SVG with width and height 100% leaves room for scrollbar on the right.
Relayout children of svg containers, except svg shapes that do not depend on percentages.
* kcanvas/RenderPath.cpp: (WebCore::RenderPath::hasPercentageValues):
* kcanvas/RenderPath.h:
* kcanvas/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::layout):
* ksvg2/svg/SVGCircleElement.cpp: (SVGCircleElement::hasPercentageValues):
* ksvg2/svg/SVGCircleElement.h:
* ksvg2/svg/SVGEllipseElement.cpp: (WebCore::SVGEllipseElement::hasPercentageValues):
* ksvg2/svg/SVGEllipseElement.h:
* ksvg2/svg/SVGLineElement.cpp: (SVGLineElement::hasPercentageValues):
* ksvg2/svg/SVGLineElement.h:
* ksvg2/svg/SVGRectElement.cpp: (WebCore::SVGRectElement::hasPercentageValues):
* ksvg2/svg/SVGRectElement.h:
* ksvg2/svg/SVGStyledElement.h: (WebCore::SVGStyledElement::hasPercentageValues):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15665
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Thu, 27 Jul 2006 23:50:51 +0000 (23:50 +0000)]
LayoutTests:
Reviewed by levi
<rdar://problem/
4242293>
Poor paste fidelity of http://www.google.com/
* editing/pasteboard/styled-element-markup-expected.checksum: Added.
* editing/pasteboard/styled-element-markup-expected.png: Added.
* editing/pasteboard/styled-element-markup-expected.txt: Added.
* editing/pasteboard/styled-element-markup.html: Added.
WebCore:
Reviewed by levi
<rdar://problem/
4242293>
Poor paste fidelity of http://www.google.com/
* editing/ApplyStyleCommand.cpp:
(WebCore::ApplyStyleCommand::applyInlineStyle): Fixed crasher with nil-check.
* editing/markup.cpp:
(WebCore::startMarkup): Style information wasn't added to an element if it had
attributes but no style attribute.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15656
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
lypanov [Thu, 27 Jul 2006 21:06:13 +0000 (21:06 +0000)]
2006-07-27 Alexander Kellett <lypanov@kde.org>
Reviewed by Darin.
- fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=9867
REGRESSION: code that modifies form element in Firefox modifies form attribute in WebKit
* bindings/scripts/CodeGeneratorJS.pm: add new attribute - HasOverridingNameGetter
* html/HTMLFormElement.idl:
2006-07-27 Alexander Kellett <lypanov@kde.org>
Reviewed by Darin.
Test for: http://bugzilla.opendarwin.org/show_bug.cgi?id=9867
REGRESSION: code that modifies form element in Firefox modifies form attribute in WebKit
* fast/forms/input-named-action-overrides-action-attribute-expected.txt: Added.
* fast/forms/input-named-action-overrides-action-attribute.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15654
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
rwlbuis [Thu, 27 Jul 2006 20:18:49 +0000 (20:18 +0000)]
Reviewed by Anders.
http://bugzilla.opendarwin.org/show_bug.cgi?id=8992
Some HTML DOM attributes have incorrect types
Adjust idl and dom source code for HTMLImageElement and
HTMLBaseFontElement to choose correct type for
HTMLImageElement::border and HTMLBaseFontElement::size
attributes.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15653
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Thu, 27 Jul 2006 20:01:56 +0000 (20:01 +0000)]
LayoutTests:
Reviewed by levi
<rdar://problem/
4628409>
Safari crashes in CompositeEditCommand::removeBlockPlaceholder entering text in a web page form
* fast/forms/
4628409-expected.txt: Added.
* fast/forms/
4628409.html: Added.
WebCore:
Reviewed by levi
<rdar://problem/
4628409>
Safari crashes in CompositeEditCommand::removeBlockPlaceholder entering text in a web page form
* editing/CompositeEditCommand.cpp:
(WebCore::CompositeEditCommand::removeBlockPlaceholder):
A script on the page shrinks the textarea down to zero rows just
before insertion. That gives the shadow div a zero height, removing
[div, 0] as a candidate for a VisiblePosition. Then
InsertTextCommand::input() tries to create a VisiblePosition from
[div, 0] and fails. A nil check is a quick fix. Another might
be to make positions where there is a caret selection candidates.
Another might be to make any [div, 0] where div is a shadow div inside
a form element a candidate.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15652
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Thu, 27 Jul 2006 18:58:16 +0000 (18:58 +0000)]
Fix by Graham Dennis, reviewed by Darin.
- fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=10094
Japanese characters improperly rendering in TOT
Test: fast/encoding/denormalised-voiced-japanese-chars.html
* platform/Font.cpp:
(WebCore::WidthIterator::advance): pass currentCharacter to normalizeVoicingMarks
(WebCore::WidthIterator::normalizeVoicingMarks): turn the currentCharacter variable
into an argument as WidthIterator::advance doesn't update m_currentCharacter while
looping
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15651
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
eseidel [Thu, 27 Jul 2006 09:22:16 +0000 (09:22 +0000)]
2006-07-27 Eric Seidel <eric@eseidel.com>
Reviewed by ap.
Complex entities seem to fail on TOT
http://bugzilla.opendarwin.org/show_bug.cgi?id=6390
Use an alternative work-around to an XML Entity parsing bug.
* dom/XMLTokenizer.cpp:
(WebCore::XMLTokenizer::write):
(WebCore::hackAroundLibXMLEntityBug): added.
(WebCore::startElementNsHandler):
(WebCore::endElementNsHandler):
(WebCore::charactersHandler):
(WebCore::processingInstructionHandler):
(WebCore::cdataBlockHandler):
(WebCore::commentHandler):
(WebCore::getEntityHandler):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15648
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
rwlbuis [Thu, 27 Jul 2006 07:46:00 +0000 (07:46 +0000)]
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=9938
Fix viewBox issues in Hixie's test
Do a more accurate and quicker job of parsing the viewBox
attribute. Handle properly on negative width/height and
invalid syntax in general.
* ksvg2/svg/SVGFitToViewBox.cpp:
(SVGFitToViewBox::parseViewBox):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15647
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
eseidel [Thu, 27 Jul 2006 05:26:00 +0000 (05:26 +0000)]
2006-07-27 Eric Seidel <eric@eseidel.com>
Reviewed by ap.
Enables getSVGDocument() support for HTMLObjectElement and HTMLEmbedElement
This patch does not try to autogenerate anything, but enables the SVG functionality.
Also fix HTMLObjectElement vspace and hspace to be numbers instead of strings to match IE & Firefox.
* bindings/js/kjs_dom.cpp:
(KJS::toJS):
* bindings/js/kjs_html.cpp:
(KJS::JSHTMLElement::objectGetter):
(KJS::HTMLElementFunction::callAsFunction):
(KJS::JSHTMLElement::objectSetter):
* bindings/js/kjs_html.h:
(KJS::JSHTMLElement::):
* dom/Document.h:
(WebCore::Document::isSVGDocument):
* html/HTMLEmbedElement.cpp:
(WebCore::HTMLEmbedElement::contentDocument):
(WebCore::HTMLEmbedElement::getSVGDocument):
* html/HTMLEmbedElement.h:
* html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::HTMLObjectElement):
(WebCore::HTMLObjectElement::getInstance):
(WebCore::HTMLObjectElement::parseMappedAttribute):
(WebCore::HTMLObjectElement::contentDocument):
(WebCore::HTMLObjectElement::rendererIsNeeded):
(WebCore::HTMLObjectElement::createRenderer):
(WebCore::HTMLObjectElement::setCode):
(WebCore::HTMLObjectElement::setArchive):
(WebCore::HTMLObjectElement::setBorder):
(WebCore::HTMLObjectElement::setCodeBase):
(WebCore::HTMLObjectElement::setCodeType):
(WebCore::HTMLObjectElement::setData):
(WebCore::HTMLObjectElement::hspace):
(WebCore::HTMLObjectElement::setHspace):
(WebCore::HTMLObjectElement::setStandby):
(WebCore::HTMLObjectElement::setType):
(WebCore::HTMLObjectElement::setUseMap):
(WebCore::HTMLObjectElement::vspace):
(WebCore::HTMLObjectElement::setVspace):
(WebCore::HTMLObjectElement::getSVGDocument):
* html/HTMLObjectElement.h:
* ksvg2/svg/SVGDocument.h:
(WebCore::SVGDocument::isSVGDocument):
* rendering/RenderWidget.h:
(WebCore::RenderWidget::frameView):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15646
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Thu, 27 Jul 2006 04:31:29 +0000 (04:31 +0000)]
Reviewed by Darin.
Fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9901
XSLTProcessor cannot parse text nodes for INPUT ELEMENTS in an XSLT stylesheet
Test: fast/js/xhtml-serialize.html
* editing/markup.cpp:
(WebCore::endMarkup): If the element has child nodes, write a closing tag even if it
is not permitted in HTML, to make it valid XML.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15645
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aliceli1 [Thu, 27 Jul 2006 01:43:55 +0000 (01:43 +0000)]
Reviewed by John Sullivan (and Kevin and Timo).
landing this patch for John et al.
Fixed <rdar://problem/
4651380> REGRESSION (9A232->9A234): existing web clips don't render if you restart Dashboard
* page/FrameView.cpp:
(WebCore::FrameView::layout):
moved the call to didFirstLayout to above the check for needsLayout
because we need to call didFirstLayout for every webview, not just
ones we think need layout.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15644
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Thu, 27 Jul 2006 00:42:44 +0000 (00:42 +0000)]
LayoutTests:
Reviewed by darin
<rdar://problem/
4652788>
REGRESSION: Can't change size or style of any text in message body
* editing/style/designmode-expected.checksum: Added.
* editing/style/designmode-expected.png: Added.
* editing/style/designmode-expected.txt: Added.
* editing/style/designmode.html: Added.
WebCore:
Reviewed by darin
<rdar://problem/
4652788>
REGRESSION: Can't change size or style of any text in message body
* dom/Node.cpp:
(WebCore::Node::isContentRichlyEditable): Mimic Node::isContentEditable.
* html/HTMLElement.cpp:
(WebCore::HTMLElement::isContentRichlyEditable): First check
the frame for editability, then userModify, like what
HTMLElement::isContentEditable() does.
* html/HTMLElement.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15643
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Wed, 26 Jul 2006 19:33:46 +0000 (19:33 +0000)]
Fix by Mitz Pettel, reviewed by ap.
- fix debug-only information() string for table cells
* rendering/RenderObject.cpp:
(WebCore::RenderObject::information):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15639
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren [Wed, 26 Jul 2006 03:04:51 +0000 (03:04 +0000)]
Reviewed by Maciej, inspired by John.
- Fixed <rdar://problem/
4651931> 1% REGRESSION on iBench HTML due to
repeated requests for non-existent favicon
An optimization to avoid serializing favicon data for missing icons had stomped an
optimization to avoid GETing a missing favicon more than once. The solution
is a happy marriage of optimizations, ensuring that we *retain* the missing
favicon's "i am missing" data without posting a notification or saving it to disk.
* Misc/WebIconDatabase.m:
(-[WebIconDatabase _setIconURL:forURL:]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15636
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
harrison [Wed, 26 Jul 2006 00:30:06 +0000 (00:30 +0000)]
WebCore:
Reviewed by timo and Darin.
<rdar://problem/
4618584> "Paste and Match Style" is not working in Mail (add SPI)
* WebCore.xcodeproj/project.pbxproj:
* bridge/mac/WebCoreFrameBridge.h:
* bridge/mac/WebCoreFrameBridge.mm:
(-[WebCoreFrameBridge replaceSelectionWithNode:selectReplacement:smartReplace:matchStyle:]):
Add matchStyle parameter for use by WebKit.
WebKit:
Reviewed by timo and Darin.
<rdar://problem/
4618584> "Paste and Match Style" is not working in Mail (add SPI)
* WebKit.xcodeproj/project.pbxproj:
* WebView/WebView.m:
(-[WebView replaceSelectionWithNode:]):
(-[WebView _replaceSelectionWithNode:matchStyle:]):
* WebView/WebViewPrivate.h:
(-[WebView _replaceSelectionWithNode:matchStyle::]):
New SPI that is same as replaceSelectionWithNode: with added parameter whether to match existing style.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15633
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Tue, 25 Jul 2006 23:13:20 +0000 (23:13 +0000)]
LayoutTests:
Reviewed by adele
<rdar://problem/
4649560>
REGRESSION: Can't Select All when the body is contentEditable="true"
* editing/selection/select-all-005-expected.checksum: Added.
* editing/selection/select-all-005-expected.png: Added.
* editing/selection/select-all-005-expected.txt: Added.
* editing/selection/select-all-005.html: Added.
* editing/selection/select-all-006-expected.checksum: Added.
* editing/selection/select-all-006-expected.png: Added.
* editing/selection/select-all-006-expected.txt: Added.
* editing/selection/select-all-006.html: Added.
WebCore:
Reviewed by adele
<rdar://problem/
4649560>
REGRESSION: Can't Select All when the body is contentEditable="true"
* editing/VisiblePosition.cpp:
(WebCore::VisiblePosition::canonicalPosition): Allow descent from a non-editable html
element into an editable body as a convenience.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15627
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justing [Tue, 25 Jul 2006 21:06:25 +0000 (21:06 +0000)]
Reviewed by darin
<rdar://problem/
4468458> Incoming HTML message is editable <body contenteditable="true">
* dom/Document.cpp:
(WebCore::Document::inDesignMode): Removed some spaces.
* page/Frame.cpp:
(WebCore::Frame::applyEditingStyleToElement): Just add CSS properties needed for editing.
(WebCore::Frame::removeEditingStyleFromElement): Made this a no-op, we'll remove
it eventually.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15624
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ddkilzer [Tue, 25 Jul 2006 15:35:24 +0000 (15:35 +0000)]
WebCore:
Reviewed by Darin.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9717
Replace IDL ConvertUndefinedToTrue parameter attribute with Optional attribute
No test added because there is no change in functionality.
* bindings/js/kjs_binding.cpp: Removed valueToBooleanTreatUndefinedAsTrue() method.
* bindings/js/kjs_binding.h: Ditto.
* bindings/scripts/CodeGeneratorJS.pm: Removed support for ConvertUndefinedToTrue parameter attribute.
* dom/Element.h: Set default value to true for scrollIntoView() and scrollIntoViewIfNeeded() arguments.
* dom/Element.idl: Replaced ConvertUndefinedToTrue attribute with Optional attribute.
Fixed name of scrollIntoViewIfNeeded() argument to match implementation.
* page/DOMWindow.idl: Replaced ConvertUndefinedToTrue attribute with Optional attribute.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15619
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 25 Jul 2006 05:05:09 +0000 (05:05 +0000)]
JavaScriptCore:
Reviewed and tweaked a bit by Darin.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9902
jsNull and NSNull not properly converted between JS and ObjC
* bindings/objc/objc_utility.mm: (KJS::Bindings::convertObjcValueToValue):
Added case for converting NSNull to jsNull.
LayoutTests:
Reviewed and tweaked a bit by Darin.
- added some new tests that use "echo" including one for JavaScript null
* plugins/bindings-test-expected.txt: Added results.
* plugins/bindings-test.html: Added tests.
WebCore:
Reviewed and tweaked a bunch by Darin.
Test: plugins/bindings-test.html
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9902
jsNull and NSNull not properly converted between JS and ObjC
* bindings/objc/WebScriptObject.mm:
(+[WebScriptObject _convertValueToObjcValue:originExecutionContext:executionContext:]):
Added case that converts jsNull to NSNull.
WebKitTools:
Reviewed and tweaked a bit by Darin.
* DumpRenderTree/ObjCPlugin.m:
(+[ObjCPlugin isSelectorExcludedFromWebScript:]): Added "echo:".
(+[ObjCPlugin webScriptNameForSelector:]): Use the name "echo" for
"echo:" so it's nice to call from JavaScript.
(-[ObjCPlugin echo:]): Just returns the same object -- can be used
to test a round trip through Objective-C types.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15618
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 25 Jul 2006 04:39:25 +0000 (04:39 +0000)]
LayoutTests:
Reviewed by Adele and Justin.
- test for http://bugzilla.opendarwin.org/show_bug.cgi?id=9630
REGRESSION: some spaces typed in <textarea> are posted as non-breaking spaces
* fast/forms/textarea-type-spaces-expected.txt: Added.
* fast/forms/textarea-type-spaces.html: Added.
- test for http://bugzilla.opendarwin.org/show_bug.cgi?id=9939
REGRESSION: Pasting text into native text area with newline at end does not preserve newline
* fast/forms/textarea-paste-newline-expected.txt: Added.
* fast/forms/textarea-paste-newline.html: Added.
- updated tests affected by above changes, results are equivalent or better
* editing/pasteboard/
4641033-expected.checksum:
* editing/pasteboard/
4641033-expected.png:
* editing/pasteboard/
4641033-expected.txt:
* editing/pasteboard/paste-table-003-expected.checksum:
* editing/pasteboard/paste-table-003-expected.png:
* editing/pasteboard/paste-table-003-expected.txt:
* editing/pasteboard/paste-text-016-expected.txt:
* editing/pasteboard/quirks-mode-br-1-expected.txt:
* editing/pasteboard/quirks-mode-br-2-expected.txt:
* fast/forms/textarea-scrolled-type-expected.checksum:
* fast/forms/textarea-scrolled-type-expected.png:
* fast/forms/textarea-scrolled-type-expected.txt:
- changed a test to be a "dump as text" test
* fast/forms/paste-into-textarea-expected.txt:
* fast/forms/paste-into-textarea.html:
* fast/forms/paste-into-textarea-expected.checksum: Removed.
* fast/forms/paste-into-textarea-expected.png: Removed.
- changed a test to be a "dump as text" test, improved test a bit and also checked in new results
* fast/forms/textarea-scrolled-endline-caret-expected.txt:
* fast/forms/textarea-scrolled-endline-caret.html:
- fixed a test that was raising an exception every time it ran
* fast/forms/attributed-strings-expected.txt:
* fast/forms/attributed-strings.html:
- corrected checksums on a bunch of tests (pngs were right, but checksums wrong, apparently)
* editing/deleting/delete-to-select-table-expected.checksum:
* editing/selection/
3690719-expected.checksum:
* editing/selection/clear-selection-expected.checksum:
* editing/undo/undo-misspellings-expected.checksum:
- added some missing pixel test results
* editing/pasteboard/nested-blocks-with-text-area-expected.checksum: Added.
* editing/pasteboard/nested-blocks-with-text-area-expected.png: Added.
* editing/pasteboard/nested-blocks-with-text-field-expected.checksum: Added.
* editing/pasteboard/nested-blocks-with-text-field-expected.png: Added.
WebCore:
Reviewed by Adele and Justin.
- fix <rdar://problem/
4613616> REGRESSION: some spaces typed in <textarea> are posted as non-breaking spaces (9630)
http://bugzilla.opendarwin.org/show_bug.cgi?id=9630
- also fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=9939
REGRESSION: Pasting text into native text area with newline at end does not preserve newline
- removed some uses of DeprecatedPtrList in the markup code
Test: fast/forms/textarea-type-spaces.html
Test: fast/forms/textarea-paste-newline.html
* bindings/objc/DOMHTML.mm: (-[DOMHTMLDocument createDocumentFragmentWithText:]):
Updated call to pass a range -- in this case it is the range of the entire document,
so this will not handle the whitespace properly.
* bridge/mac/WebCoreFrameBridge.h: Added range context parameter to the
documentFragmentWithText: method, so we can handle whitespace properly.
* bridge/mac/WebCoreFrameBridge.mm:
(-[WebCoreFrameBridge nodesFromList:]): Changed from DeprecatedPtrList to Vector.
(-[WebCoreFrameBridge markupStringFromNode:nodes:]): Ditto.
(-[WebCoreFrameBridge markupStringFromRange:nodes:]): Ditto.
(-[WebCoreFrameBridge documentFragmentWithText:inContext:]): Added range context
parameter -- pass it on to createFragmentFromText.
(-[WebCoreFrameBridge documentFragmentWithNodesAsParagraphs:]): Changed from
DeprecatedPtrList to Vector.
(-[WebCoreFrameBridge replaceSelectionWithText:selectReplacement:smartReplace:]):
Pass the range of the current selection as context when creating the fragment.
* dom/Position.cpp: (WebCore::Position::inRenderedText): Replace range check with
a call to the new containsCaretOffset function -- helps make the caret work right when
it is past the end of the last line in a textarea.
* editing/CompositeEditCommand.cpp:
(WebCore::CompositeEditCommand::rebalanceWhitespaceAt): Don't do anything if the
style does not call for collapsing whitespace.
(WebCore::CompositeEditCommand::rebalanceWhitespace): Call replaceWhitespaceAt
to share code, including the new logic mentioned above.
* editing/InsertLineBreakCommand.cpp: (WebCore::InsertLineBreakCommand::doApply):
Use a text node instead of a break element when inserting and the style is preserveNewline.
* editing/JSEditor.cpp: (WebCore::execRemoveFormat): Pass the selection range
to createFragmentFromText.
* editing/RebalanceWhitespaceCommand.cpp: (WebCore::RebalanceWhitespaceCommand::doApply):
Assert that we're in a style that collapses whitespace. It's the caller's responsibility
not to call otherwise.
* editing/ReplaceSelectionCommand.h: Removed unused destructor, type, isSingleTextNode,
isTreeFragment, m_type, and added a context parameter to inertFragmentForTestRendering.
Also changed the constructor to take a selection rather than a pointer to the root
editable element, replaced removeEndBRIfNeeded with shouldRemoveEndBR and removed an
unused parameter from shouldMergeEnd.
* editing/ReplaceSelectionCommand.cpp:
(WebCore::ReplacementFragment::ReplacementFragment): Removed code to set up m_type.
Compute root editable element from passed-in selection. Used the start of the selection
as a base node for style purposes for the test rendering. Removed the special case
"single text node" alternative to createFragmentFromText in the plain text case, since
createFragmentFromText now handles that correctly.
(WebCore::ReplacementFragment::insertFragmentForTestRendering): Copy the whitespace
property from the source location when creating a temporary element for test rendering.
(WebCore::ReplacementFragment::shouldMergeEnd): Removed unneeded boolean
parameter fragmentHadInterchangeNewlineAtEnd, which is always false.
(WebCore::ReplaceSelectionCommand::doApply): Update for ReplacementFragment changes,
change code to not remove end BR when it can be re-used instead, don't call the
paragraph separator insertion when the position is at the start of a paragraph already,
removed redundant computation of identical "next" value, removed unneeded boolean
parameter to shouldMergeEnd, add case for merging when all we need to do is to delete
a newline character, removed unneeded code to set insertionPos after all code that uses
it, and use spaces instead of non-breaking spaces when doing smart paste if the
context is one where we do not collapse white space.
(WebCore::ReplaceSelectionCommand::shouldRemoveEndBR): Renamed and changed to return
a boolean instead of doing the removal.
* editing/markup.h: Use Vector instead of DeprecatedPtrList. Change the
createFragmentFromText function to take a range for context instead of a document.
* editing/markup.cpp:
(WebCore::markup): Use Vector instead of DeprecatedPtrList.
(WebCore::createMarkup): Ditto.
(WebCore::createParagraphContentsFromString): Remove unneeded document parameter
and changed a couple places to use isEmpty instead of comparing with "".
(WebCore::createFragmentFromText): Given the new context parameter, if the context
is one that preserves newlines, then use "\n" instead of <br> elements.
(WebCore::createFragmentFromNodes): Use Vector instead of DeprecatedPtrList.
* html/HTMLElement.cpp: (WebCore::HTMLElement::setInnerText): Do not use <br>
elements if the context of this node is one where we preserve newlines.
* rendering/InlineTextBox.h:
* rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::containsCaretOffset):
Added. Implements the appropriate rule for determining if a caret position is
in this line or not. Returns true for both one line and the next in cases where
affinity must be considered to determine which line the caret is on.
* rendering/RenderText.h: Make atLineWrap no longer be a member function.
* rendering/RenderText.cpp:
(WebCore::atLineWrap): Remove special rule about preserveNewline and isLineBreak,
which will no longer apply due to the new containsCaretOffset function logic.
(WebCore::RenderText::caretRect): Use containsCaretOffset.
(WebCore::RenderText::inlineBox): Ditto.
* rendering/RenderTextControl.cpp:
(WebCore::RenderTextControl::updateFromElement): Make a placeholder <br> element
after calling setInnerText so that the last newline in the string has the effect
we expect outside the HTML world (an additional line).
(WebCore::RenderTextControl::setSelectionRange): Set granularity of the selection
too. The layout tests caught this problem, which needs a better solution long term.
(WebCore::RenderTextControl::text): Call textContent with the parameter false
so it will not include newlines for <br> elements. Now the only <br> element
that will ever be in the shadow DOM tree is the one to prevent collapsing, and
that one should not show up in the text value.
* rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak): Took a rule
that specifically called out the pre whitespace mode and made it work for all
the modes that preserve newlines. This makes sure we get a last line box for
text after the last "\n" even in cases where there is no <br> afterward.
* editing/DeleteSelectionCommand.cpp:
(WebCore::DeleteSelectionCommand::fixupWhitespace):
* editing/InsertParagraphSeparatorCommand.cpp:
(WebCore::InsertParagraphSeparatorCommand::doApply):
Added assertions to make sure we don't do anything when we're not collapsing
whitespace.
* html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::setDefaultValue):
Changed to use Vector instead of DeprecatedPtrList.
* editing/HTMLInterchange.cpp: Removed obsolete comment.
* loader/Cache.h: Removed a stray include.
WebKit:
Reviewed by Adele and Justin.
- update for change to require context when creating fragments from text
(needed to handle whitespace properly)
* WebView/WebHTMLView.m:
(-[WebHTMLView _documentFragmentFromPasteboard:inContext:allowPlainText:chosePlainText:]):
Added context parameter, pass through to bridge.
(-[WebHTMLView _pasteWithPasteboard:allowPlainText:]): Pass selection range as context
when calling above method.
(-[WebHTMLView concludeDragForDraggingInfo:actionMask:]): Pass drag caret as context when
calling above method.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15617
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adele [Tue, 25 Jul 2006 03:40:28 +0000 (03:40 +0000)]
LayoutTests:
Reviewed by Justin.
- Test for <rdar://problem/
4632132> Changing style of content with mixed editability fails
* editing/style/apple-style-editable-mix-expected.checksum: Added.
* editing/style/apple-style-editable-mix-expected.png: Added.
* editing/style/apple-style-editable-mix-expected.txt: Added.
* editing/style/apple-style-editable-mix.html: Added.
WebCore:
Reviewed by Justin.
- Fix for <rdar://problem/
4632132> Changing style of content with mixed editability fails
Test: * editing/style/apple-style-editable-mix.html
* dom/Node.cpp: (WebCore::Node::isContentRichlyEditable): Doesn't need to check the editable root. It now just
checks the user modify property of its own renderer, since that is inherited.
* editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyInlineStyle): If the end position is in a table,
Adjust the end node to the last descendant of the table, so we don't skip over any runs.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15616
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mjs [Tue, 25 Jul 2006 03:37:14 +0000 (03:37 +0000)]
Reviewed by Geoff.
- fix <rdar://problem/
4609195> Help Viewer loads empty window (not getting didFailLoadingWithError: callback)
(without re-introducing http://bugzilla.opendarwin.org/show_bug.cgi?id=10062 )
* WebView/WebLoader.h:
* WebView/WebMainResourceLoader.m:
(-[WebMainResourceLoader receivedError:]): Copy in some code from the base class to do it in the proper
order, surrounding the call to [ds _receivedMainResourceError:error complete:YES].
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15615
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin [Tue, 25 Jul 2006 01:11:12 +0000 (01:11 +0000)]
Reviewed by Hyatt.
- implement String::createCFString
* WebCore.xcodeproj/project.pbxproj: Added StringImplMac.mm and StringMac.mm.
* bindings/objc/DOMInternal.mm: Moved NSString conversion functions out of here.
* platform/mac/StringImplMac.mm: Added. Moved NSString conversion members here
and added the CFStringRef ones.
* platform/mac/StringMac.mm: Added. Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15614
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt [Mon, 24 Jul 2006 23:16:15 +0000 (23:16 +0000)]
Two bug fixes.
(1) Make sure to mark positioned objects that depend on line position to
mark themselves as needing layout when their line box placeholder gets
repositioned.
* rendering/RenderBox.cpp:
(WebCore::RenderBox::position):
(2) Improve highlighting to factor the inflation into overflow so that
invalidation and repainting will work properly.
* rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::addHighlightOverflow):
* rendering/RootInlineBox.h:
* rendering/bidi.cpp:
(WebCore::RenderBlock::layoutInlineChildren):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15612
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca [Mon, 24 Jul 2006 22:50:32 +0000 (22:50 +0000)]
2006-07-24 Anders Carlsson <acarlsson@apple.com>
Reviewed by Tim O.
* Misc/WebIconDatabase.m:
(-[WebIconDatabase removeAllIcons]):
Make an array of the keys and iterate through it to avoid modifying the
dictionary while enumerating it.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15610
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca [Mon, 24 Jul 2006 20:55:41 +0000 (20:55 +0000)]
Add a newline
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15608
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca [Mon, 24 Jul 2006 20:48:28 +0000 (20:48 +0000)]
2006-07-24 Anders Carlsson <acarlsson@apple.com>
Reviewed by Adele.
<rdar://problem/
4641824> REGRESSION: Wikipedia widget fails to complete a search after pressing return key
* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::defaultEventHandler):
Don't eat enter events for search fields that have no enclosing form; they need to be passed to AppKit so
that onsearch can be invoked.
* manual-tests/onsearch-enter.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15607
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Mon, 24 Jul 2006 19:12:56 +0000 (19:12 +0000)]
Reviewed by John and Darin.
<rdar://problem/
4634290> Cannot selectively install a custom
scroller that differs from the default Aqua frame size.
Adds two new private methods to WebFrameView that allows
an application to set a custom scroll view class. This is needed
if the application wants to install a custom scroller that is wider
than the typical scroller, because NSScrollView does the content
rect calculations in a class method (ignoring custom scrollers.)
The _setScrollViewClass method requires the class to be a subclass
of WebDynamicScrollBarView, or nil can be passed to reset to the default class.
A new scroll view of the specified class will then replace the previous
one without the need to reload content of the frame.
* WebView/WebFrameView.m:
(-[WebFrameView _customScrollViewClass]):
(-[WebFrameView _setCustomScrollViewClass:]):
* WebView/WebFrameViewPrivate.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15605
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Mon, 24 Jul 2006 17:26:52 +0000 (17:26 +0000)]
Fix by Dave MacLachlan, reviewed by Darin and Alexey.
Fix for: <http://bugzilla.opendarwin.org/show_bug.cgi?id=8425>
and <http://bugzilla.opendarwin.org/show_bug.cgi?id=6947>
Test: svg/custom/non-opaque-filters.svg
* kcanvas/device/quartz/KCanvasFilterQuartz.mm:
(WebCore::KCanvasFilterQuartz::prepareFilter):
We create an autorelease pool so we can control the deallocation of the
CIContext that we're creating. The CIContext retains the CGContext that
you pass it internally so when the CIContext is released, the
CGContext is released as well.
This is all fine and dandy unless you wrap the creation of the CIFilter
with a pair of CGBegin/EndTransparencyLayer calls which swap the context
out from underneath you. So if you start with context A,
CGBeginTransparencyLayer swaps it out and gives you B. You create a CIFilter
with it and add a reference to B. CGEndTransparencyLayer swaps out B and
gives you back A. Autorelease pool comes and cleans up the Filter, and calls
release on A, but A never got the refcount in the first place. B did. BOOM!
So we create a pool, then do a retain, then release the pool so that we
don't have to worry about the pool releasing it at a later time.
See <rdar://problem/
4647735> for reduction of CGEndTransparencyLayer case
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15603
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Mon, 24 Jul 2006 16:55:02 +0000 (16:55 +0000)]
Fix by Mitz Pettel, reviewed by Hyatt.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9669
Incomplete repaint when changing an inline's border
Test: fast/repaint/line-overflow.html
* rendering/RenderBlock.h:
* rendering/bidi.cpp:
(WebCore::RenderBlock::layoutInlineChildren): Added repaintTop and repaintBottom
variables to track the vertical edges of the area that changed, accounting for
lines that were deleted, inserted or moved. Removed unnecessary updating of
m_overflowHeight.
(WebCore::RenderBlock::determineStartPosition): Removed unnecessary updating of
m_overflowHeight.
(WebCore::RenderBlock::determineEndPosition):
(WebCore::RenderBlock::matchedEndLine): Added repaintTop and repaintBottom
arguments, which this method updates to account for deleted lines.
(WebCore::RenderBlock::checkLinesForOverflow): Removed outdated FIXME.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15602
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Mon, 24 Jul 2006 16:36:26 +0000 (16:36 +0000)]
Land the ChangeLogs, too.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15601
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Mon, 24 Jul 2006 16:21:36 +0000 (16:21 +0000)]
Fix by Mitz Pettel, reviewed by Darin.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10075
REGRESSION: Reversed pop-up text in visually-ordered Hebrew pages
Test: fast/forms/select-visual-hebrew.html
* css/html4.css: Set "-webkit-rtl-ordering: logical" on select elements.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15600
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Mon, 24 Jul 2006 16:03:57 +0000 (16:03 +0000)]
Reviewed by Darin.
Fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10060
Improve iExploder results parsing
* Scripts/run-iexploder-tests: Fix Apach logs parsing to produce a correct results
in random mode, too.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15599
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap [Mon, 24 Jul 2006 15:58:48 +0000 (15:58 +0000)]
Reviewed by Darin.
Fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10009
REGRESSION: Schubert-IT PDF Plug-in not working for full page (works in frames)
No test case possible.
WebCore:
* page/Frame.cpp:
(WebCore::Frame::begin): Give PDF plugins a chance to handle frame content, before ImageDocument
claims it.
WebKit:
* WebView/WebView.m:
(+[WebView _viewClass:andRepresentationClass:forMIMEType:]): If we've got a type supported by WebPDFView,
make sure to initialize the plugin database, in case a plugin wants to handle it.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15598
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Mon, 24 Jul 2006 15:44:46 +0000 (15:44 +0000)]
Reviewed by Darin.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=10083
REGRESSION (r15584): editing/selection/select-from-textfield-outwards failing
* rendering/RenderBlock.cpp:
(WebCore::RenderBlock::positionForCoordinates): Added back change from the fix
for bug 9312: For coordinates outside a replaced object, return the position
just before (after) the element if the coordinates are above or to the left (below
or to the right).
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15597
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
rwlbuis [Mon, 24 Jul 2006 13:13:42 +0000 (13:13 +0000)]
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=4258
Date().toString() only includes GMT offset, not timezone string
Use the info in tm_zone to append timezone abbreviation
to Date().toString().
* kjs/date_object.cpp:
(KJS::formatTime):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15595
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
rwlbuis [Mon, 24 Jul 2006 11:45:34 +0000 (11:45 +0000)]
Reviewed by Darin.
http://bugzilla.opendarwin.org/show_bug.cgi?id=5257
setYear() does not match FireFox/IE behavior
Make sure the right values end up in tm_year.
* kjs/date_object.cpp:
(KJS::formatTime):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15594
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
thatcher [Mon, 24 Jul 2006 06:06:30 +0000 (06:06 +0000)]
JavaScriptCore:
Reviewed by Maciej.
Bug 9686: [Drosera] Need the ability to break into Drosera on Javascript exceptions
http://bugzilla.opendarwin.org/show_bug.cgi?id=9686
JavaScriptCore portion of the fix.
* JavaScriptCore.exp: Update symbol for change in argument type.
* kjs/debugger.cpp:
(Debugger::detach): Clear map of recent exceptions.
(Debugger::hasHandledException): Track the most recent exception
thrown by an interpreter.
(Debugger::exception): Change exception argument to a JSValue.
* kjs/debugger.h:
* kjs/nodes.cpp:
(Node::debugExceptionIfNeeded): Notify the debugger of an exception
if it hasn't seen it before.
(ThrowNode::execute): Notify the debugger that an exception is being thrown.
* kjs/nodes.h:
2006-07-23 Geoffrey Garen <ggaren@apple.com>
Patch by Eric Albert, reviewed by Darin and me.
- Fixed <rdar://problem/
4645931> JavaScriptCore stack-scanning code
crashes (Collector::markStackObjectsConservatively)
* bindings/jni/jni_jsobject.cpp: On 64bit systems, jint is a long, not an
int.
(JavaJSObject::getSlot):
(JavaJSObject::setSlot):
* kjs/collector.cpp:
(KJS::Collector::markCurrentThreadConservatively): Use a pointer instead of
an int as 'dummy,' because on LP64 systems, an int is not pointer-aligned,
and we want to scan the stack for pointers.
* JavaScriptCore.xcodeproj/project.pbxproj: After a tense cease-fire, the
XCode war has started up again!
WebCore:
Reviewed by maciej.
Bug 9686: [Drosera] Need the ability to break into Drosera on Javascript exceptions
http://bugzilla.opendarwin.org/show_bug.cgi?id=9686
WebCore portion of the fix.
* bridge/mac/WebCoreScriptDebugger.h:
(-[WebScriptDebugger exceptionRaised:sourceId:line::]): Add delegate method.
* bridge/mac/WebCoreScriptDebugger.mm:
(WebCoreScriptDebuggerImp::exception): Call delegate method when an exception is raised.
WebKit:
Reviewed by Maciej.
Bug 9686: [Drosera] Need the ability to break into Drosera on Javascript exceptions
http://bugzilla.opendarwin.org/show_bug.cgi?id=9686
WebKit portion of the fix.
* DefaultDelegates/WebDefaultScriptDebugDelegate.m:
(-[WebDefaultScriptDebugDelegate webView:exceptionWasRaised:sourceId:line:forWebFrame:]):
* DefaultDelegates/WebScriptDebugServer.h:
* DefaultDelegates/WebScriptDebugServer.m:
(-[WebScriptDebugServer webView:exceptionWasRaised:sourceId:line:forWebFrame:]): Notify
listeners that an exception has been raised.
* WebView/WebScriptDebugDelegate.h:
* WebView/WebScriptDebugDelegate.m:
(-[WebScriptCallFrame exceptionRaised:sourceId:line:]): Dispatch through to delegate and
WebScriptDebugServer.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15593
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aliceli1 [Mon, 24 Jul 2006 02:53:03 +0000 (02:53 +0000)]
Reviewed by Adele.
Actually Adele figured out how to fix the layout test failures. Landing this patch for her. Layout test failures exposed but not caused by r15584 (my patch earlier today).
* editing/visible_units.cpp:
(WebCore::previousLinePosition):
When setting selection endpoints, don't traverse down into nodes where editing would ignore its contents. Use the parent node instead.
(WebCore::nextLinePosition):
same as above
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15591
268f45cc-cd09-0410-ab3c-
d52691b4dbfc