WebKit-https.git
13 years ago Reviewed by Mark Rowe.
thatcher [Fri, 6 Apr 2007 16:29:19 +0000 (16:29 +0000)]
    Reviewed by Mark Rowe.

        Adds a build phase script that ensures WebKit's version dosen't end in a 4.
        If our version ends in 4, some sites might think we are Netscape 4 in their
        user agent checks.

        * Configurations/Version.xcconfig:
        * WebKit.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20755 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-06 Mitz Pettel <mitz@webkit.org>
bdash [Fri, 6 Apr 2007 14:58:41 +0000 (14:58 +0000)]
2007-04-06  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Oliver.

        Eliminate PlatformMouseEvent::currentEvent.

        No test possible (no change to functionality).

        * page/EventHandler.cpp:
        (WebCore::EventHandler::handleDrag):
        * platform/PlatformMouseEvent.h:
        * platform/gdk/TemporaryLinkStubs.cpp:
        * platform/mac/PlatformMouseEventMac.mm:
        (WebCore::PlatformMouseEvent::PlatformMouseEvent):
        * platform/qt/PlatformMouseEventQt.cpp:
        * platform/win/TemporaryLinkStubs.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20753 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoFix typo in ChangeLog.
bdash [Fri, 6 Apr 2007 14:56:25 +0000 (14:56 +0000)]
Fix typo in ChangeLog.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20752 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-05 Oliver Hunt <oliver@apple.com>
oliver [Fri, 6 Apr 2007 06:45:53 +0000 (06:45 +0000)]
2007-04-05  Oliver Hunt  <oliver@apple.com>

        Reviewed by Maciej.
WebCore:
        Fix for rdar://problem/4849948 -- JSCanvasRenderingContext2D::drawImage
        crashes when given invalid arguments.

        JSCanvasRenderingContext2D frequently casts from JSValue* to JSObject*
        and then checks isObject *after* the cast.  JSObject::isObject is unsafe
        if applied to a JSImmediate value (null, undefined, etc).  This patch
        corrects the logic in a number of places by performing the isObject check
        before casting to JSObject.

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        (WebCore::JSCanvasRenderingContext2D::drawImage):
        (WebCore::JSCanvasRenderingContext2D::drawImageFromRect):
        (WebCore::JSCanvasRenderingContext2D::createPattern):

LayoutTests:
        Layout tests for rdar://problem/4849948
        Make sure we don't crash when passing invalid args to Canvas::drawImage

        * fast/canvas/drawImage-with-invalid-args-expected.checksum: Added.
        * fast/canvas/drawImage-with-invalid-args-expected.png: Added.
        * fast/canvas/drawImage-with-invalid-args-expected.txt: Added.
        * fast/canvas/drawImage-with-invalid-args.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20746 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
adele [Fri, 6 Apr 2007 05:25:10 +0000 (05:25 +0000)]
LayoutTests:

        Reviewed by Oliver.

        Test for <rdar://problem/4965597> REGRESSION: mouseup doesn't fire for slider (causes it to always default to max)

        * fast/forms/slider-mouse-events-expected.txt: Added.
        * fast/forms/slider-mouse-events.html: Added.

WebCore:

        Reviewed by Oliver.

        Fix for <rdar://problem/4965597> REGRESSION: mouseup doesn't fire for slider (causes it to always default to max)

        Test: fast/forms/slider-mouse-events.html

        * page/EventHandler.cpp: (WebCore::EventHandler::dispatchMouseEvent):
        Since the actual target node of the event can't be a shadow node, set the target node to the capturing node before adjusting it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20743 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
andersca [Fri, 6 Apr 2007 03:05:48 +0000 (03:05 +0000)]
LayoutTests:

        Reviewed by Adam.

        <rdar://problem/5083023>
        REGRESSION: In Real Player (10.1.0), video continues to play after closing window

        Add a test where a window containing a plugin is opened and then closed. The window is closed
        without calling -[WebView close].

        * plugins/open-and-close-window-with-plugin-expected.txt: Added.
        * plugins/open-and-close-window-with-plugin.html: Added.
        * plugins/resources/open-and-close-window-with-plugin.html: Added.

WebKit:

        Reviewed by Adam.

        <rdar://problem/5083023>
        REGRESSION: In Real Player (10.1.0), video continues to play after closing window

        This broke in revision 18422 because now the plugin isn't stopped when the window is closed. Since the window is retained
        by the plugin view for as long as it is running (so that removeTrackingRect works even though the window has been closed),
        we would end up with a reference cycle (NSWindow -> WebView -> PluginView -> NSWindow) and stopping the plug-in when the window
        was closed would break that cycle.

        Applications that call -[WebView close] when closing aren't affected, but RealPlayer doesn't do this.

        The bug that 18422 was supposed to fix was fixed by 19275, which is why it's safe to add back the check.

        * Plugins/WebBaseNetscapePluginView.mm:
        (-[WebBaseNetscapePluginView addWindowObservers]):
        (-[WebBaseNetscapePluginView removeWindowObservers]):
        (-[WebBaseNetscapePluginView windowWillClose:]):

WebKitTools:

        Reviewed by Adam.

        Add support for opening new windows in DumpRenderTree.

        (+[LayoutTestController isSelectorExcludedFromWebScript:]):
        (+[LayoutTestController webScriptNameForSelector:]):
        (-[LayoutTestController setCallCloseOnWebViews:]):
        (-[LayoutTestController setCanOpenWindows]):
        Add two new methods callable from JavaScript. setCanOpenWindows controls whether a test
        can open new windows, and setCallCloseOnWebViews controls whether -[WebView close] should be called on
        web views that are about to be closed.

        (runTest):
        Make sure that only the main window is around when a test has finished running.

        (-[DumpRenderTreeWindow initWithContentRect:styleMask:backing:defer:]):
        (-[DumpRenderTreeWindow dealloc]):
        Manage the set of windows.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c:
        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
        * DumpRenderTree/TestNetscapePlugIn.subproj/main.c:
        (NPP_Destroy):
        Add a "logDestroy" property which controls whether plugins should print when they are destroyed or not.

        * DumpRenderTree/UIDelegate.m:
        (-[UIDelegate webView:createWebViewWithRequest:]):
        Create new windows.

        (-[UIDelegate webViewClose:]):
        Close windows.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20742 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Maciej.
andersca [Fri, 6 Apr 2007 01:46:42 +0000 (01:46 +0000)]
    Reviewed by Maciej.

        Move WebView and NSWindow creation to a separate function in preparation of supporting opening new
        windows in DRT.

        * DumpRenderTree/DumpRenderTree.m:
        (createWebView):
        (dumpRenderTree):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20741 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
kmccullo [Fri, 6 Apr 2007 01:05:58 +0000 (01:05 +0000)]
LayoutTests:

        Reviewed by Darin.

        - Now we add a console message when local resources are blocked.

        * http/tests/security/local-CSS-from-remote-expected.txt:
        * http/tests/security/local-JavaScript-from-remote-expected.txt:
        * http/tests/security/local-iFrame-from-remote-expected.txt:
        * http/tests/security/local-image-from-remote-expected.txt:

WebCore:

        Reviewed by Darin.

        - Added reportLocalLoadFailed to FrameLoader to send an error to the console when
        calls to canLoad fail.

        * loader/Cache.cpp:
        (WebCore::Cache::requestResource):
        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::loadSubframe):
        (WebCore::FrameLoader::loadPlugin):
        (WebCore::FrameLoader::load):
        (WebCore::FrameLoader::reportLocalLoadFailed):
        * loader/FrameLoader.h:
        * loader/SubresourceLoader.cpp:
        (WebCore::SubresourceLoader::create):

WebKit:

        Reviewed by Darin.

        - Moved registerURLSchemeAsLocal to the public API.

        * WebView/WebView.h:
        * WebView/WebView.mm:
        (+[WebView registerURLSchemeAsLocal:]):
        * WebView/WebViewPrivate.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20740 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Maciej.
adele [Fri, 6 Apr 2007 00:42:03 +0000 (00:42 +0000)]
    Reviewed by Maciej.

        Add text-align: -webkit-auto for form controls.  Buttons already override this property to center their text.

        * css/html4.css:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20739 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by darin.
kjk [Thu, 5 Apr 2007 22:52:38 +0000 (22:52 +0000)]
    Reviewed by darin.

        Coverity fix. Coverity says:
        "Event check_after_deref: Pointer "dateString" dereferenced before NULL check"

        * kjs/date_object.cpp:
        (KJS::parseDate):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20738 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by darin.
kjk [Thu, 5 Apr 2007 22:50:02 +0000 (22:50 +0000)]
    Reviewed by darin.

        Coverity fix. Coverity says:
        "Event check_after_deref: Pointer "re" dereferenced before NULL check"

        * pcre/pcre_study.c:
        (pcre_study):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20737 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by darin.
kjk [Thu, 5 Apr 2007 22:45:24 +0000 (22:45 +0000)]
    Reviewed by darin.

        Coverity fixes. Coverity says:
        "Event leaked_storage: Returned without freeing storage "buffer""
        and:
        "Event leaked_storage: Returned without freeing storage "script""

        * kjs/testkjs.cpp:
        (doIt):
        (createStringWithContentsOfFile):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20736 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by rwlbuis.
kjk [Thu, 5 Apr 2007 22:42:37 +0000 (22:42 +0000)]
    Reviewed by rwlbuis.

        Coverity fix. Coverity says:
        "Event var_deref_model: Variable "page" tracked as NULL was passed to a function that dereferences it."

        * platform/Font.cpp:
        (WebCore::Font::glyphDataForCharacter):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20735 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by darin.
kjk [Thu, 5 Apr 2007 22:38:20 +0000 (22:38 +0000)]
    Reviewed by darin.

        Coverity fix. Coverity says:
        "Event var_deref_model: Variable "unicode" tracked as NULL was passed to a
        function that dereferences it."

        * platform/DeprecatedString.cpp:
        (WebCore::DeprecatedString::DeprecatedString):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20734 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by darin.
kjk [Thu, 5 Apr 2007 22:33:48 +0000 (22:33 +0000)]
    Reviewed by darin.

        Coverity fix: in single-threaded case currentThreadIsMainThread is always true
        so the code in if (!currentThreadIsMainThread) cannot possibly be reached
        and Coverity complains about dead code.

        * kjs/collector.cpp:
        (KJS::Collector::collect):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20733 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Hyatt.
adele [Thu, 5 Apr 2007 21:26:09 +0000 (21:26 +0000)]
    Reviewed by Hyatt.

        Fix for <rdar://problem/5113593> REGRESSION: Search fields in RSS pages don't show ibeam cursor

        * css/html4.css: Set the cursor values explicitly for input, textarea (to auto), & select (to default).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20732 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Fix for <rdar://problem/5078866> tables should only reset text-align
hyatt [Thu, 5 Apr 2007 21:21:14 +0000 (21:21 +0000)]
    Fix for <rdar://problem/5078866> tables should only reset text-align
        in quirks mode.

        Reviewed by beth

        * css/html4.css:
        * css/quirks.css:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20731 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Fixing typo that's crashing.
adele [Thu, 5 Apr 2007 20:36:33 +0000 (20:36 +0000)]
    Fixing typo that's crashing.

        * html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::defaultEventHandler):
          Use evt instead of null variable k.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20729 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Hyatt.
adele [Thu, 5 Apr 2007 20:30:52 +0000 (20:30 +0000)]
    Reviewed by Hyatt.

        Fix for <rdar://problem/5021500> Cursor should stay as pointer when slider is being dragged

        Don't set the selection ibeam cursor when there's a node that's capturing mouse events.

        * page/EventHandler.cpp:
        (WebCore::selectCursor):
        (WebCore::EventHandler::handleMouseMoveEvent):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20728 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Beth Dakin.
ggaren [Thu, 5 Apr 2007 19:33:52 +0000 (19:33 +0000)]
    Reviewed by Beth Dakin.

        More clean-up while fixing <rdar://problem/5112273> REGRESSION(TOT):
        Reproducible crash loading an old version of amazon.com as a web archive

        Layout tests pass.

        Removed more superfluous document NULL checks. (Node::document() never
        returns NULL, and it asserts as much.)

        Merged some duplicate editing code into a static inline function.

        * page/mac/WebCoreFrameBridge.mm:
        (updateRenderingForBindings): Removed superfluous static_cast.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20725 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Darin.
adele [Thu, 5 Apr 2007 18:41:04 +0000 (18:41 +0000)]
    Reviewed by Darin.

        A little cleanup from my last checkin.  Always check if an Event is really a MouseEvent or a KeyboardEvent
        before casting it.

        * html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::defaultEventHandler):
        * html/HTMLSelectElement.cpp:
        (WebCore::HTMLSelectElement::menuListDefaultEventHandler):
        (WebCore::HTMLSelectElement::listBoxDefaultEventHandler):
        * html/HTMLTextFieldInnerElement.cpp:
        (WebCore::HTMLSearchFieldResultsButtonElement::defaultEventHandler):
        (WebCore::HTMLSearchFieldCancelButtonElement::defaultEventHandler):
        * ksvg2/svg/SVGAElement.cpp: (WebCore::SVGAElement::defaultEventHandler):
        * page/ContextMenuController.cpp: (WebCore::ContextMenuController::handleContextMenuEvent):
        * rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::userResize):
        * rendering/RenderSlider.cpp: (WebCore::HTMLSliderThumbElement::defaultEventHandler):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20724 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Beth Dakin.
ggaren [Thu, 5 Apr 2007 17:48:23 +0000 (17:48 +0000)]
    Reviewed by Beth Dakin.

        Some clean-up while fixing <rdar://problem/5112273> REGRESSION(TOT):
        Reproducible crash loading an old version of amazon.com as a web archive

        Removed superfluous document NULL checks. (Node::document() never returns
        NULL, and it asserts as much.)

        Removed duplicate implementation of displayString.

        Layout tests pass.

        * bindings/objc/DOMInternal.h:
        * bindings/objc/DOMInternal.mm:
        * html/HTMLGenericFormElement.cpp:
        (WebCore::HTMLGenericFormElement::supportsFocus):
        * html/HTMLPlugInElement.cpp:
        (WebCore::HTMLPlugInElement::createNPObject):
        * platform/ContextMenu.cpp:
        (WebCore::ContextMenu::checkOrEnableIfNeeded):
        * rendering/HitTestResult.cpp:
        (WebCore::displayString):
        * rendering/HitTestResult.h:
        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::updateHoverActiveState):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20723 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-05 Oliver Hunt <oliver@apple.com>
oliver [Thu, 5 Apr 2007 10:20:22 +0000 (10:20 +0000)]
2007-04-05  Oliver Hunt  <oliver@apple.com>

        Reviewed by Adam.

WebCore:
        Fix rdar://problem/5030934 -- landscape PDFs embedded as an image are cut off
        also fixes scaling of pdf images

        PDFDocumentImage was not accounting for page rotation in a number of places

        * platform/graphics/cg/PDFDocumentImage.cpp:
        (WebCore::PDFDocumentImage::size):
          we need to account for rotation of the pdf image effecting our bounds
        (WebCore::PDFDocumentImage::draw):
          When compensating for flipped coords we need to use the correct rect for the
          coordinate space.  We can't pass size() to CGContextDrawPDFDocument as it
          returns the rotated bounds now -- this is also save the old
          FloatSize -> IntSize -> FloatSize conversions.
LayoutTests:
        Testcase for landscape pdfs as image

        rdar://problem/5030934 -- landscape PDFs embedded as an image are cut off

        * fast/images/pdf-as-image-landscape-expected.txt: Added.
        * fast/images/pdf-as-image-landscape.html: Added.
        * fast/images/resources/pdf_test_landscape.pdf: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20720 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-05 Mark Malone <markmalone@mac.com>
bdash [Thu, 5 Apr 2007 05:24:58 +0000 (05:24 +0000)]
2007-04-05  Mark Malone  <markmalone@mac.com>

        Reviewed by Darin.

        http://bugs.webkit.org/show_bug.cgi?id=13273
        Bug 13273: DOM Interfaces Nav link should go somewhere more useful

        * nav.inc: Link directly to the hierarchy.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20716 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Darin.
ap [Thu, 5 Apr 2007 04:40:03 +0000 (04:40 +0000)]
    Reviewed by Darin.

        http://bugs.webkit.org/show_bug.cgi?id=13278
        REGRESSION: cannot change SELECT option

        Test: fast/forms/option-change-single-selected.html

        * html/HTMLSelectElement.cpp:
        (WebCore::HTMLSelectElement::setSelectedIndex): If multiple selection is not allowed,
        be sure to deselect the previous option.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20714 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Oliver.
adele [Thu, 5 Apr 2007 01:05:49 +0000 (01:05 +0000)]
    Reviewed by Oliver.

        Fix for <rdar://5105009> Prepare for fix to send mousedown and mouseup (but not click) events when right-clicking

        * page/EventHandler.cpp: Don't ever send click events for right-clicks.
        (WebCore::EventHandler::handleMouseDoubleClickEvent):
        (WebCore::EventHandler::handleMouseReleaseEvent):

        Check that the mousedown and mouseup are not with the right button before performing default behavior.
        * html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::defaultEventHandler):
        * html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler):
        * html/HTMLSelectElement.cpp:
        (WebCore::HTMLSelectElement::menuListDefaultEventHandler):
        (WebCore::HTMLSelectElement::listBoxDefaultEventHandler):
        * html/HTMLTextFieldInnerElement.cpp:
        (WebCore::HTMLSearchFieldResultsButtonElement::defaultEventHandler):
        (WebCore::HTMLSearchFieldCancelButtonElement::defaultEventHandler):
        * ksvg2/svg/SVGAElement.cpp:(WebCore::SVGAElement::defaultEventHandler):
        * rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::userResize):
        * rendering/RenderSlider.cpp: (WebCore::HTMLSliderThumbElement::defaultEventHandler):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20709 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
justing [Wed, 4 Apr 2007 20:49:52 +0000 (20:49 +0000)]
LayoutTests:

        Reviewed by darin

        <rdar://problem/5026848> Can cut/delete To Do subunits

        This tests to make sure that non-editable content
        that is in both the start and end editable roots
        is removed:
        * editing/deleting/5026848-1-expected.checksum: Added.
        * editing/deleting/5026848-1-expected.png: Added.
        * editing/deleting/5026848-1-expected.txt: Added.
        * editing/deleting/5026848-1.html: Added.
        This tests to make sure that non-editable content
        that is not in both the start and end editable roots
        is not removed:
        * editing/deleting/5026848-2-expected.checksum: Added.
        * editing/deleting/5026848-2-expected.png: Added.
        * editing/deleting/5026848-2-expected.txt: Added.
        * editing/deleting/5026848-2.html: Added.
        This tests to make sure that editable regions
        that are not in both the start and end editable roots
        are cleared (not removed, unless they are in another
        editable region):
        * editing/deleting/5026848-3-expected.checksum: Added.
        * editing/deleting/5026848-3-expected.png: Added.
        * editing/deleting/5026848-3-expected.txt: Added.
        * editing/deleting/5026848-3.html: Added.

WebCore:

        Reviewed by darin

        <rdar://problem/5026848> Can cut/delete ToDo subunits

        We were removing non-editable content during a delete.

        * editing/DeleteSelectionCommand.cpp:
        (WebCore::DeleteSelectionCommand::initializePositionData):
        Set m_startRoot and m_endRoot.
        (WebCore::DeleteSelectionCommand::removeNode): If a node
        is not inside both the start and end roots, remove it only
        if it is in editable content.
        * editing/DeleteSelectionCommand.h: Added m_startRoot and
        m_endRoot, the editable roots that contain the start and end
        of the selection, respectively.  We consult these every
        time we remove a node, and don't want to recompute them
        on every call to removeNode().

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20707 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by John.
andersca [Wed, 4 Apr 2007 17:30:16 +0000 (17:30 +0000)]
    Reviewed by John.

        <rdar://problem/5107536>
        http://bugs.webkit.org/show_bug.cgi?id=13264
        REGRESSION: Crash when canceling about:blank in Activity viewer

        * WebView/WebFrame.mm:
        (-[WebFrame stopLoading]):
        Add a null check for the frame loader - it can be null when the frame has been disconnected
        from the web page.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20703 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoTurn's out Brady's not the only one who can rewrite history!
andersca [Wed, 4 Apr 2007 16:47:10 +0000 (16:47 +0000)]
Turn's out Brady's not the only one who can rewrite history!

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20699 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Adam.
kmccullo [Wed, 4 Apr 2007 02:12:39 +0000 (02:12 +0000)]
    Reviewed by Adam.

        - Testing a post-commit hook.

        * JavaScriptCore.vcproj/testkjs/testkjs.vcproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20696 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoJavaScriptCore:
andersca [Wed, 4 Apr 2007 01:32:00 +0000 (01:32 +0000)]
JavaScriptCore:

        Reviewed by Adam.

        <rdar://problem/5107534>
        http://bugs.webkit.org/show_bug.cgi?id=13265
        REGRESSION: Crash in KJS::Bindings::convertValueToNPVariant

        * bindings/NP_jsobject.cpp:
        (_NPN_InvokeDefault):
        Return false if the object isn't a function.

LayoutTests:

        Reviewed by Adam.

        <rdar://problem/5107534>
        http://bugs.webkit.org/show_bug.cgi?id=13265
        REGRESSION: Crash in KJS::Bindings::convertValueToNPVariant

        * plugins/netscape-invoke-default.html:
        Test calling NPN_InvokeDefault on an object that isn't a function.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20695 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
andersca [Wed, 4 Apr 2007 00:22:06 +0000 (00:22 +0000)]
LayoutTests:

        Reviewed by Darin.

        <rdar://problem/5028178>
        Crash occurs at WebCore::FrameLoader::activeDocumentLoader() after loading Froggster widget

        * plugins/get-url-that-the-resource-load-delegate-will-disallow-expected.txt: Added.
        * plugins/get-url-that-the-resource-load-delegate-will-disallow.html: Added.

WebKit:

        Reviewed by Darin.

        <rdar://problem/5028178>
        Crash occurs at WebCore::FrameLoader::activeDocumentLoader() after loading Froggster widget

        * Plugins/WebNetscapePluginStream.mm:
        (-[WebNetscapePluginStream start]):
        If load returns no the plugin loader has already been removed by the didFail callback.

WebKitTools:

        Reviewed by Darin.

        Add the notion of a "disallowed URL", which the resource loader won't allow to be loaded.

        * DumpRenderTree/DumpRenderTree.h:
        * DumpRenderTree/DumpRenderTree.m:
        (dumpRenderTree):
        (+[LayoutTestController isSelectorExcludedFromWebScript:]):
        (+[LayoutTestController webScriptNameForSelector:]):
        (-[LayoutTestController addDisallowedURL:]):
        (runTest):
        * DumpRenderTree/ResourceLoadDelegate.m:
        (-[ResourceLoadDelegate webView:resource:willSendRequest:redirectResponse:fromDataSource:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20694 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Fix for <rdar://problem/5108896>, make sure the assert for setEncodedSize
hyatt [Tue, 3 Apr 2007 23:31:15 +0000 (23:31 +0000)]
    Fix for <rdar://problem/5108896>, make sure the assert for setEncodedSize
        deals with the error case.

        Reviewed by andersca

        * loader/CachedResource.cpp:
        (WebCore::CachedResource::setEncodedSize):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20693 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Dave Hyatt.
andersca [Tue, 3 Apr 2007 23:22:45 +0000 (23:22 +0000)]
    Reviewed by Dave Hyatt.

        * loader/ImageDocument.cpp:
        Don't call error() in stopParsing.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20692 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by darin
justing [Tue, 3 Apr 2007 22:50:34 +0000 (22:50 +0000)]
    Reviewed by darin

        <http://bugs.webkit.org/show_bug.cgi?id=13268>
        Make editing/selection/drag-in-iframe.html valid and deterministic

        * editing/resources/select-and-drag-iframe.html: Removed.
        * editing/selection/drag-in-iframe-expected.checksum:
        * editing/selection/drag-in-iframe-expected.png:
        * editing/selection/drag-in-iframe-expected.txt:
        * editing/selection/drag-in-iframe.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20691 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Updated results for recent change to always lighten disabled text color for...
adele [Tue, 3 Apr 2007 22:07:43 +0000 (22:07 +0000)]
    Updated results for recent change to always lighten disabled text color for black text.

        * fast/forms/input-disabled-color-expected.checksum:
        * fast/forms/input-disabled-color-expected.png:
        * fast/forms/input-disabled-color-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20690 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
justing [Tue, 3 Apr 2007 21:57:54 +0000 (21:57 +0000)]
LayoutTests:

        Reviewed by darin

        <rdar://problem/5107422>
        TOT REGRESSION: Delete key fails to delete text, and cursor disappears in Mail.app

        moveParagraphs no longer calls shouldInsert:
        * editing/deleting/delete-3857753-fix-expected.txt:
        * editing/deleting/delete-at-paragraph-boundaries-003-expected.txt:
        * editing/deleting/delete-at-paragraph-boundaries-007-expected.txt:
        * editing/deleting/delete-at-paragraph-boundaries-009-expected.txt:
        * editing/deleting/delete-block-merge-contents-001-expected.txt:
        * editing/deleting/delete-block-merge-contents-002-expected.txt:
        * editing/deleting/delete-block-merge-contents-003-expected.txt:
        * editing/deleting/delete-block-merge-contents-004-expected.txt:
        * editing/deleting/delete-block-merge-contents-005-expected.txt:
        * editing/deleting/delete-block-merge-contents-006-expected.txt:
        * editing/deleting/delete-block-merge-contents-007-expected.txt:
        * editing/deleting/delete-block-merge-contents-008-expected.txt:
        * editing/deleting/delete-block-merge-contents-009-expected.txt:
        * editing/deleting/delete-block-merge-contents-010-expected.txt:
        * editing/deleting/delete-block-merge-contents-012-expected.txt:
        * editing/deleting/delete-block-merge-contents-013-expected.txt:
        * editing/deleting/delete-block-merge-contents-014-expected.txt:
        * editing/deleting/delete-block-merge-contents-015-expected.txt:
        * editing/deleting/delete-block-merge-contents-016-expected.txt:
        * editing/deleting/delete-block-merge-contents-017-expected.txt:
        * editing/deleting/delete-block-merge-contents-018-expected.txt:
        * editing/deleting/delete-block-merge-contents-019-expected.txt:
        * editing/deleting/delete-block-merge-contents-020-expected.txt:
        * editing/deleting/delete-block-merge-contents-021-expected.txt:
        * editing/deleting/delete-block-merge-contents-022-expected.txt:
        * editing/deleting/delete-block-table-expected.txt:
        * editing/deleting/delete-br-010-expected.txt:
        * editing/deleting/delete-line-002-expected.txt:
        * editing/deleting/delete-line-004-expected.txt:
        * editing/deleting/delete-line-006-expected.txt:
        * editing/deleting/delete-line-007-expected.txt:
        * editing/deleting/delete-line-008-expected.txt:
        * editing/deleting/delete-line-011-expected.txt:
        * editing/deleting/delete-line-012-expected.txt:
        * editing/deleting/delete-listitem-001-expected.txt:
        * editing/deleting/delete-to-end-of-paragraph-expected.txt:
        * editing/deleting/merge-different-styles-expected.txt:
        * editing/deleting/merge-no-br-expected.txt:
        * editing/deleting/pruning-after-merge-1-expected.txt:
        * editing/deleting/pruning-after-merge-2-expected.txt:
        * editing/execCommand/create-list-from-range-selection-expected.txt:
        * editing/execCommand/create-list-with-hr-expected.txt:
        * editing/execCommand/format-block-expected.txt:
        * editing/execCommand/format-block-from-range-selection-expected.txt:
        * editing/execCommand/format-block-with-braces-expected.txt:
        * editing/execCommand/format-block-with-trailing-br-expected.txt:
        * editing/execCommand/indent-list-item-expected.txt:
        * editing/execCommand/indent-selection-expected.txt:
        * editing/execCommand/insert-list-and-stitch-expected.txt:
        * editing/execCommand/insert-list-with-id-expected.txt:
        * editing/execCommand/nsresponder-indent-expected.txt:
        * editing/execCommand/outdent-selection-expected.txt:
        * editing/execCommand/queryCommandState-01-expected.txt:
        * editing/execCommand/remove-list-1-expected.txt:
        * editing/execCommand/remove-list-from-range-selection-expected.txt:
        * editing/execCommand/remove-list-item-1-expected.txt:
        * editing/execCommand/remove-list-items-expected.txt:
        * editing/execCommand/switch-list-type-expected.txt:
        * editing/inserting/insert-3907422-fix-expected.txt:
        * editing/inserting/insert-div-026-expected.txt:
        * editing/inserting/multiple-lines-selected-expected.txt:
        * editing/inserting/return-key-with-selection-001-expected.txt:
        * editing/inserting/return-key-with-selection-002-expected.txt:
        * editing/pasteboard/4076267-2-expected.txt:
        * editing/pasteboard/4700297-expected.txt:
        * editing/pasteboard/8145-3-expected.txt:
        * editing/pasteboard/display-block-on-spans-expected.txt:
        * editing/pasteboard/emacs-ctrl-k-y-001-expected.txt:
        * editing/pasteboard/interchange-newline-1-expected.txt:
        * editing/pasteboard/interchange-newline-2-expected.txt:
        * editing/pasteboard/merge-after-delete-1-expected.txt:
        * editing/pasteboard/merge-after-delete-2-expected.txt:
        * editing/pasteboard/merge-after-delete-expected.txt:
        * editing/pasteboard/merge-end-1-expected.txt:
        * editing/pasteboard/merge-end-2-expected.txt:
        * editing/pasteboard/merge-end-3-expected.txt:
        * editing/pasteboard/merge-end-4-expected.txt:
        * editing/pasteboard/merge-end-5-expected.txt:
        * editing/pasteboard/merge-end-blockquote-expected.txt:
        * editing/pasteboard/merge-end-borders-expected.txt:
        * editing/pasteboard/merge-end-list-expected.txt:
        * editing/pasteboard/merge-end-table-expected.txt:
        * editing/pasteboard/paste-line-endings-007-expected.txt:
        * editing/pasteboard/paste-line-endings-008-expected.txt:
        * editing/pasteboard/paste-line-endings-009-expected.txt:
        * editing/pasteboard/paste-match-style-001-expected.txt:
        * editing/pasteboard/paste-text-002-expected.txt:
        * editing/pasteboard/paste-text-003-expected.txt:
        * editing/pasteboard/paste-text-004-expected.txt:
        * editing/pasteboard/paste-text-005-expected.txt:
        * editing/pasteboard/paste-text-006-expected.txt:
        * editing/pasteboard/paste-text-008-expected.txt:
        * editing/pasteboard/paste-text-009-expected.txt:
        * editing/pasteboard/paste-text-010-expected.txt:
        * editing/pasteboard/smart-paste-001-expected.txt:
        * editing/pasteboard/smart-paste-002-expected.txt:
        * editing/pasteboard/smart-paste-003-expected.txt:
        * editing/pasteboard/smart-paste-004-expected.txt:
        * editing/pasteboard/smart-paste-005-expected.txt:
        * editing/pasteboard/smart-paste-006-expected.txt:
        * editing/pasteboard/smart-paste-007-expected.txt:
        * editing/pasteboard/smart-paste-008-expected.txt:
        * editing/style/create-block-for-style-001-expected.txt:
        * editing/style/create-block-for-style-002-expected.txt:
        * editing/style/create-block-for-style-003-expected.txt:
        * editing/style/create-block-for-style-004-expected.txt:
        * editing/style/create-block-for-style-007-expected.txt:
        * editing/style/create-block-for-style-009-expected.txt:
        * editing/style/create-block-for-style-010-expected.txt:
        * editing/style/create-block-for-style-011-expected.txt:
        * editing/style/create-block-for-style-012-expected.txt:
        * editing/style/create-block-for-style-013-expected.txt:
        * editing/style/smoosh-styles-001-expected.txt:
        * editing/style/smoosh-styles-002-expected.txt:
        * editing/style/style-boundary-005-expected.txt:
        * editing/unsupported-content/list-delete-001-expected.txt:

WebCore:

        Reviewed by darin

        <rdar://problem/5107422>
        TOT REGRESSION: Delete key fails to delete text, and cursor disappears in Mail.app
        <rdar://problem/5107413>
        TOT REGRESSION: Crash deleting text in email (infinite recursion in shouldInsertNode)

        * editing/CompositeEditCommand.cpp:
        (WebCore::CompositeEditCommand::moveParagraphs):
        Back out the call to shouldInsert until we
        add WebViewInsertActionMoved, because Mail returns
        NO from shouldInsert for WebViewInsertActionPasted
        to override our Paste operation.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20689 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
andersca [Tue, 3 Apr 2007 21:39:53 +0000 (21:39 +0000)]
LayoutTests:

        Reviewed by Mitz.

        Add tests for innerWidth/innerHeight and clientWidth/clientHeight in quirks mode.

        Modify the strict clientWidth/clientHeight test to test the behavior with scrollbars visible.

        * fast/dom/client-width-height-quirks-expected.txt: Added.
        * fast/dom/client-width-height-quirks.html: Added.
        * fast/dom/client-width-height.html:
        * fast/dom/inner-width-height-expected.txt: Added.
        * fast/dom/inner-width-height.html: Added.

WebCore:

        Reviewed by Mitz.

        Make window.innerWidth/innerHeight include the size of any present scrollbar to match Firefox.

        Make document.body.clientWidth/clientHeight return the width/height of the visible content when in
        quirks mode to match WinIE.

        * bindings/js/kjs_window.cpp:
        (KJS::Window::getValueProperty):
        Return the width and height of the scroll view, not the visible width and height.

        * dom/Element.cpp:
        (WebCore::Element::clientWidth):
        (WebCore::Element::clientHeight):
        Return the visible size if the element is the body element and we're in quirks mode.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20688 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Darin.
adele [Tue, 3 Apr 2007 19:22:22 +0000 (19:22 +0000)]
    Reviewed by Darin.

        Updated fix for <rdar://problem/5107095> REGRESSION: disabled text color calculations are slowing down launchtime measurements

        * platform/graphics/Color.cpp: Moved optimization for lightening black and darkening white into the Color class.
        (WebCore::Color::light):
        (WebCore::Color::dark):
        * rendering/RenderTextControl.cpp: (WebCore::disabledTextColor): Skip the difference calculation for black text, which will always be lightened.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20685 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Tim.
andersca [Tue, 3 Apr 2007 18:35:35 +0000 (18:35 +0000)]
    Reviewed by Tim.

        <rdar://problem/4649516>
        Turn off slow script dialog or crank up time that makes it come up

        * bindings/js/kjs_binding.cpp:
        (KJS::ScriptInterpreter::ScriptInterpreter):
        Increase the timeout to 10 seconds to match Firefox.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20684 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-03 Mitz Pettel <mitz@webkit.org>
bdash [Tue, 3 Apr 2007 12:56:28 +0000 (12:56 +0000)]
2007-04-03  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13251
          editing/style/fontsize-1.html fails intermittently when run multiple times on Intel, generates incorrect DOM

        Covered by existing editing tests.

        * css/CSSPrimitiveValue.cpp:
        (WebCore::CSSPrimitiveValue::getFloatValue): Added ASSERTs that the requested conversion
        is from a scalar type to a scalar type. The intermittent failures in the test were
        the result of trying to convert an identifier type to a scalar type.
        * editing/ApplyStyleCommand.cpp:
        (WebCore::StyleChange::checkForLegacyHTMLStyleChange): Added a check that the
        font-size property value is an absolute length. If it is not (such as in the case
        of relative units or keywords like "x-large"), then no corresponding 'size' attribute
        will be generated for legacy HTML.

2007-04-03  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - updated results for http://bugs.webkit.org/show_bug.cgi?id=13251
          editing/style/fontsize-1.html fails intermittently when run multiple times on Intel, generates incorrect DOM

        * editing/style/block-style-004-expected.txt:
        * editing/style/block-style-005-expected.txt:
        * editing/style/block-style-006-expected.txt:
        * editing/style/fontsize-1-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20678 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-03 Mitz Pettel <mitz@webkit.org>
bdash [Tue, 3 Apr 2007 12:51:55 +0000 (12:51 +0000)]
2007-04-03  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13218
          <rdar://problem/5095977>
          Reproducible crash after call to window.close()

        * manual-tests/stale-currentEvent.html: Added.
        * page/EventHandler.cpp:
        (WebCore::EventHandler::hoverTimerFired): Instead of faking a mouse event,
        just hit-test the last mouse event coordinates and update for hover state
        changes. This eliminates the use of currentEvent, which can be a non-mouse
        event or even a stale event.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20677 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-03 Mark Rowe <mrowe@apple.com>
bdash [Tue, 3 Apr 2007 11:24:41 +0000 (11:24 +0000)]
2007-04-03  Mark Rowe  <mrowe@apple.com>

        Reviewed by Antti.

        * BuildSlaveSupport/build-launcher-dmg: Use bzip2 compression in nightly build disk images rather than gzip.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20676 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Maciej.
adele [Tue, 3 Apr 2007 05:54:33 +0000 (05:54 +0000)]
    Reviewed by Maciej.

        Fix for <rdar://problem/5107095> REGRESSION: disabled text color calculations are slowing down launchtime measurements

        Hardcode disabled text colors for the most common cases- black and white.

        * rendering/RenderTextControl.cpp: (WebCore::disabledTextColor):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20675 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Fix for <rdar://problem/5105784>, assertion failure in cache. Encoded sizes...
hyatt [Tue, 3 Apr 2007 01:11:35 +0000 (01:11 +0000)]
    Fix for <rdar://problem/5105784>, assertion failure in cache.  Encoded sizes were being set to
        garbage by accident because of a mismatch between derived class and base class constructors.  This
        patch cleans all of this up and gets rid of expiration date as a concept separate from the response.

        Reviewed by anders

        * dom/Document.cpp:
        (WebCore::Document::processHttpEquiv):
        * html/HTMLImageLoader.cpp:
        (WebCore::HTMLImageLoader::updateFromElement):
        * loader/Cache.cpp:
        (WebCore::createResource):
        (WebCore::Cache::requestResource):
        * loader/Cache.h:
        * loader/CachedCSSStyleSheet.cpp:
        (WebCore::CachedCSSStyleSheet::CachedCSSStyleSheet):
        * loader/CachedCSSStyleSheet.h:
        * loader/CachedImage.cpp:
        (WebCore::CachedImage::CachedImage):
        * loader/CachedImage.h:
        * loader/CachedResource.cpp:
        (WebCore::CachedResource::CachedResource):
        (WebCore::CachedResource::finish):
        * loader/CachedResource.h:
        * loader/CachedScript.cpp:
        (WebCore::CachedScript::CachedScript):
        * loader/CachedScript.h:
        * loader/CachedXBLDocument.cpp:
        (WebCore::CachedXBLDocument::CachedXBLDocument):
        * loader/CachedXBLDocument.h:
        * loader/CachedXSLStyleSheet.cpp:
        (WebCore::CachedXSLStyleSheet::CachedXSLStyleSheet):
        * loader/CachedXSLStyleSheet.h:
        * loader/DocLoader.cpp:
        (WebCore::DocLoader::DocLoader):
        (WebCore::DocLoader::requestResource):
        * loader/DocLoader.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20674 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Hyatt.
antti [Tue, 3 Apr 2007 01:01:09 +0000 (01:01 +0000)]
    Reviewed by Hyatt.

        Render tree memory savings, part 3

        These changes shrink all RenderBlocks by additional 28 bytes. Together with parts 1 and 2
        RenderBlock goes from 172 bytes to 128 bytes, a saving of 26%.

        - Pack bitfields together in RenderFlow
        - Move top/bottom min/max margin values to a struct that is only allocated in case these
          variables have non default values. Usually this is <5% of all blocks.
        - Move rarely used column variables to hash

        * rendering/RenderBlock.cpp:
        (WebCore::ColumnInfo::ColumnInfo):
        (WebCore::RenderBlock::RenderBlock):
        (WebCore::RenderBlock::~RenderBlock):
        (WebCore::RenderBlock::layoutBlock):
        (WebCore::RenderBlock::collapseMargins):
        (WebCore::RenderBlock::clearFloatsIfNeeded):
        (WebCore::RenderBlock::setCollapsedBottomMargin):
        (WebCore::RenderBlock::layoutBlockChildren):
        (WebCore::RenderBlock::paintColumns):
        (WebCore::RenderBlock::paintObject):
        (WebCore::RenderBlock::fillSelectionGaps):
        (WebCore::RenderBlock::lowestPosition):
        (WebCore::RenderBlock::rightmostPosition):
        (WebCore::RenderBlock::leftmostPosition):
        (WebCore::RenderBlock::nodeAtPoint):
        (WebCore::RenderBlock::hitTestColumns):
        (WebCore::RenderBlock::positionForCoordinates):
        (WebCore::RenderBlock::availableWidth):
        (WebCore::RenderBlock::calcColumnWidth):
        (WebCore::RenderBlock::setDesiredColumnCountAndWidth):
        (WebCore::RenderBlock::desiredColumnWidth):
        (WebCore::RenderBlock::desiredColumnCount):
        (WebCore::RenderBlock::columnRects):
        (WebCore::RenderBlock::layoutColumns):
        (WebCore::RenderBlock::adjustPointToColumnContents):
        (WebCore::RenderBlock::adjustRectForColumns):
        (WebCore::RenderBlock::setMaxTopMargins):
        (WebCore::RenderBlock::setMaxBottomMargins):
        * rendering/RenderBlock.h:
        (WebCore::RenderBlock::maxTopMargin):
        (WebCore::RenderBlock::maxBottomMargin):
        (WebCore::RenderBlock::maxTopPosMargin):
        (WebCore::RenderBlock::maxTopNegMargin):
        (WebCore::RenderBlock::maxBottomPosMargin):
        (WebCore::RenderBlock::maxBottomNegMargin):
        (WebCore::RenderBlock::initMaxMarginValues):
        (WebCore::RenderBlock::MaxMargin::MaxMargin):
        (WebCore::RenderBlock::MaxMargin::topPosDefault):
        (WebCore::RenderBlock::MaxMargin::topNegDefault):
        (WebCore::RenderBlock::MaxMargin::bottomPosDefault):
        (WebCore::RenderBlock::MaxMargin::bottomNegDefault):
        * rendering/RenderFlexibleBox.cpp:
        (WebCore::RenderFlexibleBox::layoutBlock):
        * rendering/RenderFlow.h:
        (WebCore::RenderFlow::RenderFlow):
        (WebCore::RenderFlow::hasColumns):
        * rendering/RenderInline.cpp:
        (WebCore::RenderInline::RenderInline):
        * rendering/RenderInline.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20673 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
antti [Tue, 3 Apr 2007 00:22:30 +0000 (00:22 +0000)]
LayoutTests:

        Reviewed by Hyatt.

        Test you don't crash when you have abs positioned elements inside rel inline.

        * fast/block/positioning/abs-inside-inline-rel-expected.checksum: Added.
        * fast/block/positioning/abs-inside-inline-rel-expected.png: Added.
        * fast/block/positioning/abs-inside-inline-rel-expected.txt: Added.
        * fast/block/positioning/abs-inside-inline-rel.html: Added.

WebCore:

        Reviewed by Hyatt.

        Fix asserts, these may also be called for rel positioned elements

        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::setStaticX):
        (WebCore::RenderBox::setStaticY):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20672 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Geoff.
andersca [Mon, 2 Apr 2007 23:49:03 +0000 (23:49 +0000)]
    Reviewed by Geoff.

        Fix crash when running plugins/destroy-stream-twice.html under GuardMalloc

        * Plugins/WebBaseNetscapePluginStream.h:
        * Plugins/WebBaseNetscapePluginStream.mm:
        (+[WebBaseNetscapePluginStream ownerForStream:]):
        (-[WebBaseNetscapePluginStream initWithRequestURL:plugin:notifyData:sendNotification:]):
        (-[WebBaseNetscapePluginStream dealloc]):
        (-[WebBaseNetscapePluginStream finalize]):
        Change the streams hash map to contain an NPStream*, and change ownerForStream to take an NPStream*.

        * Plugins/WebBaseNetscapePluginView.mm:
        (-[WebBaseNetscapePluginView destroyStream:reason:]):
        Check that the NPStream pointer is valid before accessing stream->ndata.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20671 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Throw in an assert to try to track down a problem where an image is being...
hyatt [Mon, 2 Apr 2007 23:41:57 +0000 (23:41 +0000)]
    Throw in an assert to try to track down a problem where an image is being set to a smaller encoded
        size.

        * loader/CachedResource.cpp:
        (WebCore::CachedResource::setEncodedSize):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20670 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-04-02 Ada Chan <adachan@apple.com>
adachan [Mon, 2 Apr 2007 22:29:45 +0000 (22:29 +0000)]
2007-04-02  Ada Chan  <adachan@apple.com>

        Reviewed by Adele.

        <rdar://5105331> Missing null check in Editor::execCommand
        Added null check.

        * editing/Editor.cpp:
        (WebCore::Editor::execCommand):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20669 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Geoff.
darin [Mon, 2 Apr 2007 21:15:34 +0000 (21:15 +0000)]
    Reviewed by Geoff.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13026
          <rdar://problem/5061026> incomplete render of menu
          (assertion failing in -[WebBaseNetscapePluginView saveAndSetNewPortStateForUpdate:])

        - fix http://bugs.webkit.org/show_bug.cgi?id=13120
          <rdar://problem/5080339> Plug-ins that draw through the QuickDraw interface may crash
          by hanging onto old GWorlds.

        - set clip path for CoreGraphics plug-ins in the same way we do for QuickDraw plug-ins
          this is a better fix for <rdar://problem/4939511> WebKit should set the the CG clip
          path for plug-ins that draw using Core Graphics

        Incorporates changes from a patch by Mark Ambachtsheer.

        Here are the changes:

            1) Don't try to use the offscreen code path if GGBitmapContextGetData returns 0.
            2) Handle kCGBitmapByteOrderDefault when computing the QD pixel format, even though
               we don't have any evidence that this happens in practice.
            3) Keep the GWorld around until we create a new one or the plug-in is destroyed.
            4) Use the GWorld pointer itself as a flag to indicate whether we are using an
               offscreen GWorld.
            5) Set up clipping for CoreGraphics in the same way we do for QuickDraw; remove an
               earlier attempt that handled CoreGraphics differently.

        * Plugins/WebBaseNetscapePluginView.h: Added a field named offscreenGWorld to hold
        the GWorld until it's needed.
        * Plugins/WebBaseNetscapePluginView.mm:
        (getQDPixelFormatForBitmapContext): Replaced QDPixelFormatFromCGBitmapInfo. Used the
        "get" prefix so we don't intrude on the QD namespace. Added code to handle the
        kCGBitmapByteOrderDefault case, although I'm not sure it will really come up in
        practice -- it wasn't really coming up in the buggy case.
        (getNPRect): Added helper functions. Used to make the code below clearer.
        (-[WebBaseNetscapePluginView saveAndSetNewPortStateForUpdate:]): Use getNPRect to
        streamline code. Use GetGWorld to save the port since we use SetGWorld to restore
        it later. Store the GWorld we create in the offscreenGWorld field and dispose the
        existing one. Don't treat the CGBitmapContext as an offscreen bitmap if it
        has a data pointer of 0. Set up the clip based on the result of
        -[NSView getRectsBeingDrawn:count] when setting up the port for CoreGraphics
        (after saving the port state).
        (-[WebBaseNetscapePluginView restorePortState:]): Remove now-unneeded code to
        destroy the offscreen GWorld, and simplified the code that restores the port so we
        don't need a separate case for offscreen.
        (-[WebBaseNetscapePluginView fini]): Renamed from freeAttributeKeysAndValues, since
        this method now does more than just the attributes. This is the shared method that
        does things needed in both dealloc and finalize. Added a call to DisposeGWorld here.
        (-[WebBaseNetscapePluginView dealloc]): Updated for name change.
        (-[WebBaseNetscapePluginView finalize]): Ditto.
        (-[WebBaseNetscapePluginView drawRect:]): Removed code to set clip. This is done in
        the saveAndSetNewPortStateForUpdate: method instead.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20668 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
andersca [Mon, 2 Apr 2007 19:33:50 +0000 (19:33 +0000)]
    * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
        Don't use #import in a header included by C files.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20664 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
justing [Mon, 2 Apr 2007 18:23:42 +0000 (18:23 +0000)]
LayoutTests:

        Reviewed by darin

        <rdar://problem/5058163>
        Hitting return inserts a new line between two To Dos, but caret is position after second ToDo

        * editing/inserting/5058163-1-expected.checksum: Added.
        * editing/inserting/5058163-1-expected.png: Added.
        * editing/inserting/5058163-1-expected.txt: Added.
        * editing/inserting/5058163-1.html: Added.
        * editing/inserting/5058163-2-expected.checksum: Added.
        * editing/inserting/5058163-2-expected.png: Added.
        * editing/inserting/5058163-2-expected.txt: Added.
        * editing/inserting/5058163-2.html: Added.

WebCore:

        Reviewed by darin

        <rdar://problem/5058163>
        Hitting return inserts a new line between two To Dos, but caret is position after second ToDo

        * editing/InsertParagraphSeparatorCommand.cpp:
        (WebCore::InsertParagraphSeparatorCommand::doApply):
        Removed refPos, it was unused.
        Don't assume that upstream() moves to a different
        node.  This assumption caused the line break to
        be inserted *after* the caret and a bad position
        to be set (added a testcase).
        Use isStartOfParagraph instead of checking
        for the existence of a br because the line might
        be broken by a newline character in text that
        preserves newlines.  Paragraphs that are created
        in other ways, like with blocks, are handled
        above (added a testcase).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20660 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago 2007-04-02 David Harrison <harrison@apple.com>
harrison [Mon, 2 Apr 2007 17:36:20 +0000 (17:36 +0000)]
 2007-04-02  David Harrison  <harrison@apple.com>

        Reviewed by Darin.

        <rdar://problem/4906488> text attributes are not working for frame based web page

        Attribute handling now treats each WebArea as its own world, rather than sometimes
        referring back up to the top WebArea. e.g. getting/setting the selection now
        operates on the WebArea containing the WebCoreAXObject that is being messaged (self).
        The methods that sit above this and are able to go into arbitrary WebAreas
        are accessibilityFocusedUIElement and doAXTextMarkerForPosition, allowing VoiceOver
        to navigate between frames.

        * bridge/mac/WebCoreAXObject.mm:
        (-[WebCoreAXObject topRenderer]):
        (-[WebCoreAXObject frameView]):
        (-[WebCoreAXObject topFrameView]):
        Removed topDocument, renamed topView to topFrameView, added frameView.

        (-[WebCoreAXObject accessibilityAttributeValue:]):
        AXSelectedTextMarkerRange, AXStartTextMarker, and AXEndTextMarker now use
        self's WebArea, rather than the top WebArea.

        (-[WebCoreAXObject doAXTextMarkerRangeForLine:]):
        Use self's WebArea, rather than the top WebArea.

        (-[WebCoreAXObject doAXTextMarkerForPosition:]):
        Use renamed methods for accessing top WebArea.

        (-[WebCoreAXObject doAXBoundsForTextMarkerRange:]):
        Use renamed methods for accessing self's WebArea.

        (AXAttributeStringSetColor):
        (AXAttributeStringSetNumber):
        (AXAttributeStringSetFont):
        (AXAttributeStringSetStyle):
        (AXAttributeStringSetElement):
        (-[WebCoreAXObject rendererForView:]):
        Clean up uses of nil and NULL.

        (-[WebCoreAXObject accessibilityFocusedUIElement]):
        Find the focused node on the whole page. If there is no focused node,
        return the WebArea for the focused document.

        (-[WebCoreAXObject doSetAXSelectedTextMarkerRange:]):
        Use self's WebArea, rather than the top WebArea.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20656 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Fix build. This one was not supposed to be checked in.
antti [Mon, 2 Apr 2007 10:06:12 +0000 (10:06 +0000)]
    Fix build. This one was not supposed to be checked in.

        * rendering/RenderFlexibleBox.cpp:
        (WebCore::RenderFlexibleBox::layoutBlock):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20654 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Hyatt.
antti [Mon, 2 Apr 2007 09:43:08 +0000 (09:43 +0000)]
    Reviewed by Hyatt.

        Render tree memory savings, part 2

        - Move staticX and staticY variables to RenderLayer since they are only
          applicable to positioned objects
        - Move very rarely used overrideSize variable to hash

        These changes shrink all box types by 12 bytes

        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::RenderBox):
        (WebCore::RenderBox::destroy):
        (WebCore::RenderBox::overrideSize):
        (WebCore::RenderBox::setOverrideSize):
        (WebCore::RenderBox::overrideWidth):
        (WebCore::RenderBox::overrideHeight):
        (WebCore::RenderBox::position):
        (WebCore::RenderBox::calcWidth):
        (WebCore::RenderBox::calcHeight):
        (WebCore::RenderBox::staticX):
        (WebCore::RenderBox::staticY):
        (WebCore::RenderBox::setStaticX):
        (WebCore::RenderBox::setStaticY):
        (WebCore::RenderBox::calcAbsoluteHorizontal):
        (WebCore::RenderBox::calcAbsoluteVertical):
        (WebCore::RenderBox::calcAbsoluteHorizontalReplaced):
        (WebCore::RenderBox::calcAbsoluteVerticalReplaced):
        * rendering/RenderBox.h:
        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::RenderLayer):
        * rendering/RenderLayer.h:
        (WebCore::RenderLayer::staticX):
        (WebCore::RenderLayer::staticY):
        (WebCore::RenderLayer::setStaticX):
        (WebCore::RenderLayer::setStaticY):
        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::RenderObject):
        * rendering/RenderObject.h:
        (WebCore::):
        (WebCore::RenderObject::hasOverrideSize):
        (WebCore::RenderObject::setHasOverrideSize):
        * rendering/RenderWidget.cpp:
        (WebCore::RenderWidget::destroy):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20653 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Not reviewed - gdk build fix.
kjk [Sun, 1 Apr 2007 23:01:35 +0000 (23:01 +0000)]
    Not reviewed - gdk build fix.

        * platform/gdk/TemporaryLinkStubs.cpp:
        (Font::selectionRectForComplexText):
        (Font::drawComplexText):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20649 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Darin.
ap [Sun, 1 Apr 2007 05:41:30 +0000 (05:41 +0000)]
    Reviewed by Darin.

        http://bugs.webkit.org/show_bug.cgi?id=12778
        REGRESSION: Nightly won't load Curl plugin for full page use

        * dom/DOMImplementation.cpp:
        (WebCore::DOMImplementation::createDocument): Re-order cases to give more priority to plugins.
        This also fixes Adobe SVG Viewer and any plugins that register XML MIME types.
        Added a case for XHTML to avoid enumerating plugins.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20648 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agogit-svn-id: https://svn.webkit.org/repository/webkit/trunk@20647 268f45cc-cd09-0410...
hyatt [Sun, 1 Apr 2007 05:21:33 +0000 (05:21 +0000)]
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20647 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Fix for bug 13234, layout of selected justified text is broken. Rework TextR...
hyatt [Sun, 1 Apr 2007 05:20:58 +0000 (05:20 +0000)]
    Fix for bug 13234, layout of selected justified text is broken.  Rework TextRuns to avoid having
        from/to members.  Let those be passed in only by the functions that need them (drawing and selection rect).

        Reviewed by mitz

        fast/text/justified-selection-at-edge.html

        * platform/Font.cpp:
        (WebCore::WidthIterator::WidthIterator):
        (WebCore::WidthIterator::advance):
        (WebCore::Font::canUseGlyphCache):
        (WebCore::Font::drawSimpleText):
        (WebCore::Font::drawText):
        (WebCore::Font::floatWidth):
        (WebCore::Font::floatWidthForSimpleText):
        (WebCore::Font::selectionRectForText):
        (WebCore::Font::selectionRectForSimpleText):
        (WebCore::Font::offsetForPositionForSimpleText):
        * platform/Font.h:
        (WebCore::TextRun::m_len):
        * platform/graphics/GraphicsContext.cpp:
        (WebCore::GraphicsContext::drawText):
        (WebCore::GraphicsContext::drawHighlightForText):
        * platform/graphics/GraphicsContext.h:
        * platform/mac/FontMac.mm:
        (WebCore::addDirectionalOverride):
        (WebCore::overrideLayoutOperation):
        (WebCore::ATSULayoutParameters::initialize):
        (WebCore::Font::selectionRectForComplexText):
        (WebCore::Font::drawComplexText):
        (WebCore::Font::floatWidthForComplexText):
        (WebCore::Font::offsetForPositionForComplexText):
        * platform/qt/FontQt.cpp:
        (WebCore::Font::drawComplexText):
        (WebCore::Font::floatWidthForComplexText):
        * platform/qt/TemporaryLinkStubs.cpp:
        (Font::selectionRectForComplexText):
        * rendering/EllipsisBox.cpp:
        (WebCore::EllipsisBox::paint):
        * rendering/InlineTextBox.cpp:
        (WebCore::InlineTextBox::selectionRect):
        (WebCore::InlineTextBox::paint):
        (WebCore::InlineTextBox::paintSelection):
        (WebCore::InlineTextBox::paintMarkedTextBackground):
        (WebCore::InlineTextBox::paintSpellingOrGrammarMarker):
        (WebCore::InlineTextBox::paintTextMatchMarker):
        (WebCore::InlineTextBox::offsetForPosition):
        (WebCore::InlineTextBox::positionForOffset):
        * rendering/RenderListMarker.cpp:
        (WebCore::RenderListMarker::paint):
        (WebCore::RenderListMarker::calcMinMaxWidth):
        (WebCore::RenderListMarker::getRelativeMarkerRect):
        * rendering/RenderMenuList.cpp:
        (WebCore::RenderMenuList::updateOptionsWidth):
        * rendering/RenderText.cpp:
        (WebCore::RenderText::widthFromCache):
        (WebCore::RenderText::width):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20646 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mitz Pettel <mitz@webkit.org>
bdash [Sun, 1 Apr 2007 05:04:47 +0000 (05:04 +0000)]
2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13243
              <rdar://problem/5103226>
          REGRESSION (r20506): Repro crash/assert when using scroll wheel on a list box taller than its contents

        * platform/ScrollBar.cpp:
        (WebCore::Scrollbar::setValue): Cleaned up a little.
        (WebCore::Scrollbar::scroll): Changed to apply the minimum and maximum constraints
        in the right order, and cleaned up a little.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20645 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mitz Pettel <mitz@webkit.org>
andrew [Sun, 1 Apr 2007 04:59:52 +0000 (04:59 +0000)]
2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - test for http://bugs.webkit.org/show_bug.cgi?id=13242
          REGRESSION: Repro crash when specifying the content property for an image

        * fast/images/text-content-crash-expected.txt: Added.
        * fast/images/text-content-crash.html: Added.

2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13242
          REGRESSION: Repro crash when specifying the content property for an image

        Test: fast/images/text-content-crash.html

        * html/HTMLImageElement.cpp:
        (WebCore::HTMLImageElement::parseMappedAttribute): Added a check that the
        renderer is an image.
        (WebCore::HTMLImageElement::attach): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20644 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mark Rowe <mrowe@apple.com>
bdash [Sun, 1 Apr 2007 04:50:11 +0000 (04:50 +0000)]
2007-03-31  Mark Rowe  <mrowe@apple.com>

        Reviewed by Mitz.

        Fix http://bugs.webkit.org/show_bug.cgi?id=13239
        Bug 13239: REGRESSION (r20343): Drosera hits exception trying to call "count" cross-process

        Use -[WebScriptObject valueForKey:@"length"] to retrieve the length of a JavaScript array
        rather than -[WebScriptObject count].

        * Drosera/DebuggerDocument.m:
        (-[DebuggerDocument webScriptAttributeKeysForScriptObject:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20643 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mark Rowe <mrowe@apple.com>
bdash [Sun, 1 Apr 2007 04:48:17 +0000 (04:48 +0000)]
2007-03-31  Mark Rowe  <mrowe@apple.com>

        Reviewed by Darin.

        A more correct fix for http://bugs.webkit.org/show_bug.cgi?id=13129
        Bug 13129: Democracy Player dies in NSException from WebScriptObject on startup

        The previous attempt at a fix in r20343 attempted to conditionally expose the 'count'
        method.  It did not work which resulted in 'count' always being hidden.  This change
        disables -[WebScriptObject count] completely until it can be determined whether it is
        needed and how to support it without breaking third-party applications.

        * bindings/objc/WebScriptObject.mm:
        (-[WebScriptObject count]):
        * bindings/objc/WebScriptObjectPrivate.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20642 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoReplaced bogus 'text/plain' svn property with svn:mime-type property set to 'text...
ddkilzer [Sat, 31 Mar 2007 15:32:27 +0000 (15:32 +0000)]
Replaced bogus 'text/plain' svn property with svn:mime-type property set to 'text/javascript'.
Fixed one instance where 'allow-tabs' property and 'true' value were reversed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20641 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoReplaced bogus 'text/plain' svn property with svn:mime-type property set to 'text...
ddkilzer [Sat, 31 Mar 2007 11:30:37 +0000 (11:30 +0000)]
Replaced bogus 'text/plain' svn property with svn:mime-type property set to 'text/javascript'.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20640 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mitz Pettel <mitz@webkit.org>
ap [Sat, 31 Mar 2007 10:25:21 +0000 (10:25 +0000)]
2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Sam Weinig.

        - pixel test for partially selected justified text

        * fast/text/justified-selection-expected.checksum: Added.
        * fast/text/justified-selection-expected.png: Added.
        * fast/text/justified-selection-expected.txt: Added.
        * fast/text/justified-selection.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20639 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mitz Pettel <mitz@webkit.org>
ap [Sat, 31 Mar 2007 10:18:33 +0000 (10:18 +0000)]
2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

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

        A layout test documenting shrink-to-fit width calculation for floating elements.

        * fast/block/float/shrink-to-fit-width-expected.checksum: Added.
        * fast/block/float/shrink-to-fit-width-expected.png: Added.
        * fast/block/float/shrink-to-fit-width-expected.txt: Added.
        * fast/block/float/shrink-to-fit-width.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20638 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Nicholas Shanks <webkit@nickshanks.com>
ap [Sat, 31 Mar 2007 10:07:32 +0000 (10:07 +0000)]
2007-03-31  Nicholas Shanks  <webkit@nickshanks.com>

        Reviewed by Adele.

        http://bugs.webkit.org/show_bug.cgi?id=13184
        HTMLImageElement longdesc should be complete resolved URL

        Test: fast/dom/HTMLImageElement/image-longdesc-absolute-url.html

        * html/HTMLImageElement.cpp:
        (WebCore::HTMLImageElement::longDesc): Return absolute URL.
        Behaviour now matches Firefox and Opera.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20637 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mitz Pettel <mitz@webkit.org>
ap [Sat, 31 Mar 2007 08:26:24 +0000 (08:26 +0000)]
2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Hyatt.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13209
          REGRESSION (r18756-18765): Incomplete list marker repaint when resizing list item

        Test: fast/repaint/list-marker-2.html

        A list marker can increase the horizontal overflow after layoutInlineChildren(),
        so made that function return only the vertical bounds of the repaint rect. As the
        horizontal bounds, made layoutBlock() just use the overflow bounds after layout.

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::layoutBlock):
        * rendering/RenderBlock.h:
        * rendering/bidi.cpp:
        (WebCore::RenderBlock::layoutInlineChildren):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20636 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mitz Pettel <mitz@webkit.org>
ap [Sat, 31 Mar 2007 07:59:22 +0000 (07:59 +0000)]
2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Hyatt.

        - http://bugs.webkit.org/show_bug.cgi?id=13178
          Helvetica zero-width space has non-zero width

        Test: fast/text/wide-zero-width-space.html

        * platform/FontData.cpp:
        (WebCore::FontData::FontData): Force the ZERO WIDTH SPACE glyph to have width
        0, unless the SPACE character maps to the same glyph.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20635 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago2007-03-31 Mitz Pettel <mitz@webkit.org>
ap [Sat, 31 Mar 2007 07:49:19 +0000 (07:49 +0000)]
2007-03-31  Mitz Pettel  <mitz@webkit.org>

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=8016
          REPRO: Safari hangs loading 18 MB page (deeply nested DOM tree)

WebCore:
        * html/HTMLTokenizer.cpp:
        (WebCore::HTMLTokenizer::parseTag): Made "<" terminate tag and attribute
        names. This matches Firefox.

LayoutTests:
        * fast/overflow/002-expected.checksum:
        * fast/overflow/002-expected.png:
        * fast/overflow/002-expected.txt:
        * fast/overflow/002.html: Added missing ">" to one of the BR tags.
        * fast/text/atsui-multiple-renderers-expected.checksum:
        * fast/text/atsui-multiple-renderers-expected.png:
        * fast/text/atsui-multiple-renderers-expected.txt:
        * fast/text/atsui-multiple-renderers.html: Added missing ">" to the TITLE tag.
        * fast/tokenizer/lessthan-terminates-tags-and-attrs-expected.txt: Added.
        * fast/tokenizer/lessthan-terminates-tags-and-attrs.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20634 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoReplaced bogus 'text/plain' svn property with svn:mime-type property set to 'text...
ddkilzer [Sat, 31 Mar 2007 02:25:21 +0000 (02:25 +0000)]
Replaced bogus 'text/plain' svn property with svn:mime-type property set to 'text/javascript'.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20631 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
andersca [Fri, 30 Mar 2007 21:41:00 +0000 (21:41 +0000)]
LayoutTests:

        Reviewed by Geoff.

        * plugins/destroy-stream-twice.html:
        Put the scripts before the embed element. Set the timeout in the onstreamload handler.

WebKitTools:

        Reviewed by Geoff.

        Add an "onstreamload" attribute to the plugin which is called when a stream starts loading.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
        * DumpRenderTree/TestNetscapePlugIn.subproj/main.c:
        (NPP_New):
        Look for the onstreamload attribute.

        (NPP_Destroy):
        Free the onstreamload attribute.

        (NPP_NewStream):
        Call the onstreamload handler.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20625 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Debug build fix.
ap [Fri, 30 Mar 2007 20:57:27 +0000 (20:57 +0000)]
    Debug build fix.

        * xml/XPathValue.cpp:
        (WebCore::XPath::): Define Value::adopt.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20622 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Removing an assertion I just added because it's crashing the Leopard
ggaren [Fri, 30 Mar 2007 20:54:25 +0000 (20:54 +0000)]
    Removing an assertion I just added because it's crashing the Leopard
        buildbot. The related layout test will still report a failure, so we
        don't really need this assertion.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c:
        (pluginInvoke):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20621 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Darin.
ap [Fri, 30 Mar 2007 20:31:26 +0000 (20:31 +0000)]
    Reviewed by Darin.

        http://bugs.webkit.org/show_bug.cgi?id=13021
        XPath can be very slow

        This patch finally makes us faster than Firefox 2.0 on this test case.

        * xml/XPathFunctions.cpp:
        (WebCore::XPath::FunConcat::evaluate): Use a Vector buffer for faster appending.
        (WebCore::XPath::FunId::evaluate): Move the result NodeSet into Value instead of copying it.
        (WebCore::XPath::FunLocalName::evaluate): Cosmetic changes.
        (WebCore::XPath::FunNamespaceURI::evaluate): Ditto.
        (WebCore::XPath::FunName::evaluate): Ditto.
        (WebCore::XPath::FunNormalizeSpace::evaluate): Ditto.

        * xml/XPathValue.cpp:
        (WebCore::XPath::Value::toNodeSet):
        (WebCore::XPath::Value::modifiableNodeSet):
        (WebCore::XPath::Value::toBoolean):
        (WebCore::XPath::Value::toNumber):
        (WebCore::XPath::Value::toString):
        * xml/XPathValue.h:
        (WebCore::XPath::ValueData::ValueData):
        (WebCore::XPath::Value::Value):
        Made Value copying faster by putting large data members into a separate refcounted ValueData class.
        Added provisions for moving a NodeSet in and out of Value without copying.
        Made construction safer by moving bool version to a template constructor.
        Removed an (almost) unused default constructor.

        * xml/XPathPredicate.h: Store a Value for literal string and number expressions to avoid
        constructing one each time.

        * xml/XPathPath.cpp:
        (WebCore::XPath::Filter::evaluate):
        (WebCore::XPath::LocationPath::evaluate):
        (WebCore::XPath::Path::evaluate):
        * xml/XPathPath.h:
        * xml/XPathPredicate.cpp:
        (WebCore::XPath::NumericOp::evaluate):
        (WebCore::XPath::Union::evaluate):
        * xml/XPathResult.cpp:
        (WebCore::XPathResult::snapshotItem):
        * xml/XPathStep.cpp:
        (WebCore::XPath::Step::evaluate):
        (WebCore::XPath::Step::nodesInAxis):
        * xml/XPathStep.h:
        Got rid of unnecessary NodeSet copying in many cases.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20620 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Geoff.
andersca [Fri, 30 Mar 2007 20:13:58 +0000 (20:13 +0000)]
    Reviewed by Geoff.

        * DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.c:
        (testAllocate):
        (testEnumerate):
        Add casts.

        * DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.h:
        Don't use #import, use #include.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20619 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Alexey.
andersca [Fri, 30 Mar 2007 19:59:24 +0000 (19:59 +0000)]
    Reviewed by Alexey.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c:
        (pluginInvoke):
        Free the identifier string.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20618 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
justing [Fri, 30 Mar 2007 19:50:28 +0000 (19:50 +0000)]
LayoutTests:

        Reviewed by john

        <rdar://problem/5076323> Only one To Do can be created by a selection in a message

        * editing/selection/5076323-1-expected.checksum: Added.
        * editing/selection/5076323-1-expected.txt: Added.
        * editing/selection/5076323-1.html: Added.
        * editing/selection/5076323-2-expected.checksum: Added.
        * editing/selection/5076323-2-expected.txt: Added.
        * editing/selection/5076323-2.html: Added.
        * editing/selection/5076323-3-expected.checksum: Added.
        * editing/selection/5076323-3-expected.txt: Added.
        * editing/selection/5076323-3.html: Added.

WebCore:

        Reviewed by john

        <rdar://problem/5076323> Only one To Do can be created by a selection in a message

        When Mail's ToDoify operation called moveToEndOfDocument:
        with a selection that started just before a ToDo, we'd
        move them to the end of the document, not to the end
        of the editable root, because we incorrectly checked
        for the editability of positions before ToDos.  Then Mail
        would refuse to create a ToDo because the selection was
        outside of the region where they allowed editing.

        * editing/SelectionController.cpp:
        (WebCore::SelectionController::modifyExtendingRightForward):
        Check the position for editability, not the position's node,
        since there are editable positions of the form [node, offset]
        where node is contenteditable=false.
        Re-use pos.  Nothing that happens after it's created can invalidate
        it or the position and affinity used to create it.
        (WebCore::SelectionController::modifyMovingRightForward): Ditto.
        (WebCore::SelectionController::modifyExtendingLeftBackward): Ditto.
        (WebCore::SelectionController::modifyMovingLeftBackward): Ditto.
        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::paintCaret): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20617 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Brian Dash
beidson [Fri, 30 Mar 2007 18:24:51 +0000 (18:24 +0000)]
    Reviewed by Brian Dash

        Fixes an intermittent failure in the port blocking layout test where
        some sort of indeterminacy would change the error code the networking
        layer reported.

        * security/block-test-expected.txt:
        * security/block-test.html:  Use 255.255.255.255 instead of a bogus domain name

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20616 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoWebCore:
adele [Fri, 30 Mar 2007 18:13:54 +0000 (18:13 +0000)]
WebCore:

        Reviewed by Darin.

        Added DeleteWordBackward and DeleteWordForward to execCommand.

        * editing/Editor.cpp:
        (WebCore::execDeleteWordBackward):
        (WebCore::execDeleteWordForward):
        (WebCore::CommandEntry::):

WebKit:

        Reviewed by Darin.

        Call execCommand for deleteWordForward and deleteWordBackward instead of calling
        deleteWithDirection directly.

        * WebView/WebHTMLView.mm:
        (-[WebHTMLView deleteWordForward:]):
        (-[WebHTMLView deleteWordBackward:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20615 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Build fix.
andersca [Fri, 30 Mar 2007 18:10:53 +0000 (18:10 +0000)]
    Build fix.

        * bindings/NP_jsobject.cpp:
        (_NPN_Enumerate):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20614 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoJavaScriptCore:
andersca [Fri, 30 Mar 2007 17:29:50 +0000 (17:29 +0000)]
JavaScriptCore:

        Reviewed by Geoff.

        Implement _NPN_Enumerate support.

        * JavaScriptCore.exp:
        * bindings/NP_jsobject.cpp:
        (_NPN_Enumerate):
        * bindings/c/c_instance.cpp:
        (KJS::Bindings::CInstance::getPropertyNames):
        * bindings/c/c_instance.h:
        * bindings/npapi.h:
        * bindings/npruntime.h:
        * bindings/npruntime_impl.h:
        * bindings/runtime.h:
        (KJS::Bindings::Instance::getPropertyNames):
        * bindings/runtime_object.cpp:
        (RuntimeObjectImp::getPropertyNames):
        * bindings/runtime_object.h:
        (KJS::RuntimeObjectImp::getInternalInstance):

LayoutTests:

        Reviewed by Geoff.

        Add enumeration test.

        * plugins/netscape-enumerate-expected.txt: Added.
        * plugins/netscape-enumerate.html: Added.

WebKit:

        Reviewed by Geoff.

        * Plugins/WebNetscapePluginPackage.m:
        (-[WebNetscapePluginPackage load]):
        Initialize pushpopupsenabledstate, poppopupsenabledstate and enumerate.

        * Plugins/npapi.m:
        (NPN_PushPopupsEnabledState):
        (NPN_PopPopupsEnabledState):
        Add stubs for these functions.

        * Plugins/npfunctions.h:
        Add new methods to NPNetscapeFuncs.

WebKitTools:

        Reviewed by Geoff.

        * DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
        Add TestObject.c and TestObject.h

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c:
        (pluginGetProperty):
        Implement the testObject property.

        (pluginInvoke):
        Implement testEnumerate which takes an object and an array and enumerates
        the properties of the object and adds them to the array.

        (pluginAllocate):
        Allocate the test object.

        (pluginDeallocate):
        Free the test object.

        * DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.c: Added.
        * DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.h: Added.
        Add a test object with two enumerable properties.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20613 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoFix Qt release builds
hausmann [Fri, 30 Mar 2007 15:33:11 +0000 (15:33 +0000)]
Fix Qt release builds

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20612 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by bdash.
antti [Fri, 30 Mar 2007 12:55:51 +0000 (12:55 +0000)]
    Reviewed by bdash.

        Spotted an int that should be float in previous patch.

        * platform/Font.cpp:
        (WebCore::WidthIterator::advance):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20611 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Hyatt.
antti [Fri, 30 Mar 2007 09:59:57 +0000 (09:59 +0000)]
    Reviewed by Hyatt.

        Render tree memory savings, part 1

        - Eliminate caching of monospace char width from RenderText. This optimization is not needed
          anymore since Font already caches space width (RenderText -4 bytes)
        - Eliminate caching of tab width from RenderBlock. Similary this optimization has become
          obsolete (RenderBlock -4 bytes)

        * platform/Font.cpp:
        (WebCore::WidthIterator::advance):
        (WebCore::Font::spaceWidth):
        * platform/Font.h:
        (WebCore::Font::tabWidth):
        * platform/TextStyle.h:
        (WebCore::TextStyle::TextStyle):
        (WebCore::TextStyle::allowTabs):
        * rendering/InlineTextBox.cpp:
        (WebCore::InlineTextBox::selectionRect):
        (WebCore::InlineTextBox::paint):
        (WebCore::InlineTextBox::paintSelection):
        (WebCore::InlineTextBox::paintMarkedTextBackground):
        (WebCore::InlineTextBox::paintSpellingOrGrammarMarker):
        (WebCore::InlineTextBox::paintTextMatchMarker):
        (WebCore::InlineTextBox::offsetForPosition):
        (WebCore::InlineTextBox::positionForOffset):
        * rendering/RenderBlock.cpp:
        (WebCore:::RenderFlow):
        (WebCore::RenderBlock::setStyle):
        * rendering/RenderBlock.h:
        * rendering/RenderObject.cpp:
        * rendering/RenderObject.h:
        * rendering/RenderText.cpp:
        (WebCore::RenderText::RenderText):
        (WebCore::RenderText::setStyle):
        (WebCore::RenderText::widthFromCache):
        (WebCore::RenderText::trimmedMinMaxWidth):
        (WebCore::RenderText::calcMinMaxWidthInternal):
        (WebCore::RenderText::setTextInternal):
        (WebCore::RenderText::width):
        * rendering/RenderText.h:
        (WebCore::RenderText::allowTabs):
        * rendering/bidi.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20610 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Fix (take 2) for bug 4334, flickering when floating elements gain/lose transp...
hyatt [Fri, 30 Mar 2007 09:51:24 +0000 (09:51 +0000)]
    Fix (take 2) for bug 4334, flickering when floating elements gain/lose transparency.

        Reviewed by mitz

        * rendering/RenderBlock.cpp:
        * rendering/RenderBlock.h:
        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::setStyle):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20609 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Layout test for <rdar://problem/5061807>. (This one's for you, Boyscout!)
ggaren [Fri, 30 Mar 2007 03:17:50 +0000 (03:17 +0000)]
    Layout test for <rdar://problem/5061807>. (This one's for you, Boyscout!)

        * fast/frames/remove-frame-with-scrollbars-crash-expected.txt: Added.
        * fast/frames/remove-frame-with-scrollbars-crash.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20608 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
weinig [Fri, 30 Mar 2007 02:36:16 +0000 (02:36 +0000)]
LayoutTests:

        Reviewed by Darin.

        - test for http://bugs.webkit.org/show_bug.cgi?id=13166
          REGRESSION (2005-10-16 - 2005-10-19): Ampersands showing up in the last column of the Gmail contacts table

        * fast/tokenizer/write-partial-entity-expected.txt: Added.
        * fast/tokenizer/write-partial-entity.html: Added.

WebCore:

        Reviewed by Darin.

        - fix http://bugs.webkit.org/show_bug.cgi?id=13166
          REGRESSION (2005-10-16 - 2005-10-19): Ampersands showing up in the last column of the Gmail contacts table

        Test: fast/tokenizer/write-partial-entity.html

        * html/HTMLTokenizer.cpp:
        (WebCore::HTMLTokenizer::parseTag): Reordered to avoid resetting the buffer position
        after script execution which can write to the buffer.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20607 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
justing [Fri, 30 Mar 2007 00:11:24 +0000 (00:11 +0000)]
LayoutTests:

        Reviewed by john

        <rdar://problem/5099303> Deleting after a ToDo puts the caret in the wrong position

        Tests for the bug:
        * editing/deleting/5099303-expected.checksum: Added.
        * editing/deleting/5099303-expected.png: Added.
        * editing/deleting/5099303-expected.txt: Added.
        * editing/deleting/5099303.html: Added.
        Tests that extending an editable selection skips non-editable
        content:
        * editing/selection/5099303-expected.checksum: Added.
        * editing/selection/5099303-expected.png: Added.
        * editing/selection/5099303-expected.txt: Added.
        * editing/selection/5099303.html: Added.

WebCore:

        Reviewed by john

        <rdar://problem/5099303> Deleting after a ToDo puts the caret in the wrong position

        We were failing to create the correct selection
        for DeleteSelectionCommand because the selection extension
        operations don't skip over non-editable content for editable
        selections.

        * editing/SelectionController.cpp:
        (WebCore::SelectionController::modifyExtendingRightForward):
        When extending an editable selection, don't extend into
        non-editable content.
        (WebCore::SelectionController::modifyExtendingLeftBackward):
        Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20606 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
ggaren [Thu, 29 Mar 2007 23:26:30 +0000 (23:26 +0000)]
LayoutTests:

        Reviewed by Beth Dakin.

        Layout test for <rdar://problem/5091330> REGRESSION: Repro crash in
        -[WebBaseNetscapePluginView(WebNPPCallbacks) destroyStream:reason:]
        navigating away from page with DivX movie plug-in (13203)

        * plugins/destroy-stream-twice-expected.txt: Added.
        * plugins/destroy-stream-twice.html: Added.

WebKit:

        Reviewed by Beth Dakin, reviewed by Maciej Stachowiak.

        Layout test for <rdar://problem/5091330> REGRESSION: Repro crash in
        -[WebBaseNetscapePluginView(WebNPPCallbacks) destroyStream:reason:]
        navigating away from page with DivX movie plug-in (13203)

        Changed LOG_ERROR to LOG so the layout test doesn't produce console spew
        every time you run it.

        * Plugins/WebBaseNetscapePluginView.mm:
        (-[WebBaseNetscapePluginView destroyStream:reason:]):

WebKitTools:

        Reviewed by Beth Dakin.

        Layout test for <rdar://problem/5091330> REGRESSION: Repro crash in
        -[WebBaseNetscapePluginView(WebNPPCallbacks) destroyStream:reason:]
        navigating away from page with DivX movie plug-in (13203)

        Added hasStream property and destroyStream function, used by layout test.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.c:
        (pluginGetProperty):
        (pluginInvoke):
        (pluginAllocate):
        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
        * DumpRenderTree/TestNetscapePlugIn.subproj/main.c:
        (NPP_NewStream):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20597 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Rubber-stamped by Adam.
bdakin [Thu, 29 Mar 2007 23:24:41 +0000 (23:24 +0000)]
    Rubber-stamped by Adam.

        Keep OpenSource building.

        * WebKitSystemInterface.h:
        * libWebKitSystemInterface.a:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20596 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago <rdar://problem/4918397> Exception raise inspecting note contents in the background
pyeh [Thu, 29 Mar 2007 23:24:05 +0000 (23:24 +0000)]
 <rdar://problem/4918397> Exception raise inspecting note contents in the background

        * bridge/mac/WebCoreAXObject.mm:
        (-[WebCoreAXObject subrole]):
        (-[WebCoreAXObject title]):
Check that attachment supports the attribute before requesting it.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20595 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoI forgot add this expected result in my previous checkin.
justing [Thu, 29 Mar 2007 23:16:44 +0000 (23:16 +0000)]
I forgot add this expected result in my previous checkin.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20594 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years ago Reviewed by Brady.
bdakin [Thu, 29 Mar 2007 23:13:03 +0000 (23:13 +0000)]
    Reviewed by Brady.

        Fix for <rdar://problem/4674537> REGRESSION: Adobe Acrobat 8 - Text
        blinks when mouse is moved, and is invisible otherwise

        -and-

        <rdar://problem/4992521> Please adjust WebKit's Acrobat-workaround
        methodology

        The fix for the first bug is to compare against the bundle
        identifiers for Adobe Reader and the non-Pro Adobe Acrobat in
        addition to Adobe Acrobat Pro. The fix for the second bug is to
        check the version number of Acrobat/Reader through
        WebKitSystemInterface instead of checking which version of WebKit
        it has been linked against.

        * English.lproj/StringsNotToBeLocalized.txt: Two new bundle
        identifiers.
        * Misc/WebKitVersionChecks.h: Remove Acrobat quirk constant.
        * WebView/WebView.mm:
        (-[WebView _updateWebCoreSettingsFromPreferences:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20593 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoLayoutTests:
justing [Thu, 29 Mar 2007 22:34:28 +0000 (22:34 +0000)]
LayoutTests:

        Reviewed by john

        <rdar://problem/5032066> <Delete> should work between To Dos

        Demonstrates the fix:
        * editing/deleting/5032066-expected.checksum: Added.
        * editing/deleting/5032066-expected.png: Added.
        * editing/deleting/5032066-expected.txt: Added.
        * editing/deleting/5032066.html: Added.

        Fixed:
        * editing/deleting/delete-block-table-expected.checksum:
        * editing/deleting/delete-block-table-expected.png:
        * editing/deleting/delete-block-table-expected.txt:

        shouldInsertNode:... now being called during moves:
        * editing/deleting/delete-3857753-fix-expected.txt:
        * editing/deleting/delete-at-paragraph-boundaries-003-expected.txt:
        * editing/deleting/delete-at-paragraph-boundaries-007-expected.txt:
        * editing/deleting/delete-at-paragraph-boundaries-009-expected.txt:
        * editing/deleting/delete-block-merge-contents-001-expected.txt:
        * editing/deleting/delete-block-merge-contents-002-expected.txt:
        * editing/deleting/delete-block-merge-contents-003-expected.txt:
        * editing/deleting/delete-block-merge-contents-004-expected.txt:
        * editing/deleting/delete-block-merge-contents-005-expected.txt:
        * editing/deleting/delete-block-merge-contents-006-expected.txt:
        * editing/deleting/delete-block-merge-contents-007-expected.txt:
        * editing/deleting/delete-block-merge-contents-008-expected.txt:
        * editing/deleting/delete-block-merge-contents-009-expected.txt:
        * editing/deleting/delete-block-merge-contents-010-expected.txt:
        * editing/deleting/delete-block-merge-contents-012-expected.txt:
        * editing/deleting/delete-block-merge-contents-013-expected.txt:
        * editing/deleting/delete-block-merge-contents-014-expected.txt:
        * editing/deleting/delete-block-merge-contents-015-expected.txt:
        * editing/deleting/delete-block-merge-contents-016-expected.txt:
        * editing/deleting/delete-block-merge-contents-017-expected.txt:
        * editing/deleting/delete-block-merge-contents-018-expected.txt:
        * editing/deleting/delete-block-merge-contents-019-expected.txt:
        * editing/deleting/delete-block-merge-contents-020-expected.txt:
        * editing/deleting/delete-block-merge-contents-021-expected.txt:
        * editing/deleting/delete-block-merge-contents-022-expected.txt:
        * editing/deleting/delete-br-010-expected.txt:
        * editing/deleting/delete-line-002-expected.txt:
        * editing/deleting/delete-line-004-expected.txt:
        * editing/deleting/delete-line-006-expected.txt:
        * editing/deleting/delete-line-007-expected.txt:
        * editing/deleting/delete-line-008-expected.txt:
        * editing/deleting/delete-line-011-expected.txt:
        * editing/deleting/delete-line-012-expected.txt:
        * editing/deleting/delete-listitem-001-expected.txt:
        * editing/deleting/delete-to-end-of-paragraph-expected.txt:
        * editing/deleting/merge-different-styles-expected.txt:
        * editing/deleting/merge-no-br-expected.txt:
        * editing/deleting/pruning-after-merge-1-expected.txt:
        * editing/deleting/pruning-after-merge-2-expected.txt:
        * editing/execCommand/create-list-from-range-selection-expected.txt:
        * editing/execCommand/create-list-with-hr-expected.txt:
        * editing/execCommand/format-block-expected.txt:
        * editing/execCommand/format-block-from-range-selection-expected.txt:
        * editing/execCommand/format-block-with-braces-expected.txt:
        * editing/execCommand/format-block-with-trailing-br-expected.txt:
        * editing/execCommand/indent-list-item-expected.txt:
        * editing/execCommand/indent-selection-expected.txt:
        * editing/execCommand/insert-list-and-stitch-expected.txt:
        * editing/execCommand/insert-list-with-id-expected.txt:
        * editing/execCommand/nsresponder-indent-expected.txt:
        * editing/execCommand/outdent-selection-expected.txt:
        * editing/execCommand/queryCommandState-01-expected.txt:
        * editing/execCommand/remove-list-1-expected.txt:
        * editing/execCommand/remove-list-from-range-selection-expected.txt:
        * editing/execCommand/remove-list-item-1-expected.txt:
        * editing/execCommand/remove-list-items-expected.txt:
        * editing/execCommand/switch-list-type-expected.txt:
        * editing/inserting/insert-3907422-fix-expected.txt:
        * editing/inserting/insert-div-026-expected.txt:
        * editing/inserting/multiple-lines-selected-expected.txt:
        * editing/inserting/return-key-with-selection-001-expected.txt:
        * editing/inserting/return-key-with-selection-002-expected.txt:
        * editing/pasteboard/4076267-2-expected.txt:
        * editing/pasteboard/4700297-expected.txt:
        * editing/pasteboard/8145-3-expected.txt:
        * editing/pasteboard/display-block-on-spans-expected.txt:
        * editing/pasteboard/emacs-ctrl-k-y-001-expected.txt:
        * editing/pasteboard/interchange-newline-1-expected.txt:
        * editing/pasteboard/interchange-newline-2-expected.txt:
        * editing/pasteboard/merge-after-delete-1-expected.txt:
        * editing/pasteboard/merge-after-delete-2-expected.txt:
        * editing/pasteboard/merge-after-delete-expected.txt:
        * editing/pasteboard/merge-end-1-expected.txt:
        * editing/pasteboard/merge-end-2-expected.txt:
        * editing/pasteboard/merge-end-3-expected.txt:
        * editing/pasteboard/merge-end-4-expected.txt:
        * editing/pasteboard/merge-end-5-expected.txt:
        * editing/pasteboard/merge-end-blockquote-expected.txt:
        * editing/pasteboard/merge-end-borders-expected.txt:
        * editing/pasteboard/merge-end-list-expected.txt:
        * editing/pasteboard/merge-end-table-expected.txt:
        * editing/pasteboard/paste-line-endings-007-expected.txt:
        * editing/pasteboard/paste-line-endings-008-expected.txt:
        * editing/pasteboard/paste-line-endings-009-expected.txt:
        * editing/pasteboard/paste-match-style-001-expected.txt:
        * editing/pasteboard/paste-text-002-expected.txt:
        * editing/pasteboard/paste-text-003-expected.txt:
        * editing/pasteboard/paste-text-004-expected.txt:
        * editing/pasteboard/paste-text-005-expected.txt:
        * editing/pasteboard/paste-text-006-expected.txt:
        * editing/pasteboard/paste-text-008-expected.txt:
        * editing/pasteboard/paste-text-009-expected.txt:
        * editing/pasteboard/paste-text-010-expected.txt:
        * editing/pasteboard/smart-paste-001-expected.txt:
        * editing/pasteboard/smart-paste-002-expected.txt:
        * editing/pasteboard/smart-paste-003-expected.txt:
        * editing/pasteboard/smart-paste-004-expected.txt:
        * editing/pasteboard/smart-paste-005-expected.txt:
        * editing/pasteboard/smart-paste-006-expected.txt:
        * editing/pasteboard/smart-paste-007-expected.txt:
        * editing/pasteboard/smart-paste-008-expected.txt:
        * editing/style/create-block-for-style-001-expected.txt:
        * editing/style/create-block-for-style-002-expected.txt:
        * editing/style/create-block-for-style-003-expected.txt:
        * editing/style/create-block-for-style-004-expected.txt:
        * editing/style/create-block-for-style-007-expected.txt:
        * editing/style/create-block-for-style-009-expected.txt:
        * editing/style/create-block-for-style-010-expected.txt:
        * editing/style/create-block-for-style-011-expected.txt:
        * editing/style/create-block-for-style-012-expected.txt:
        * editing/style/create-block-for-style-013-expected.txt:
        * editing/style/smoosh-styles-001-expected.txt:
        * editing/style/smoosh-styles-002-expected.txt:
        * editing/style/style-boundary-005-expected.txt:
        * editing/unsupported-content/list-delete-001-expected.txt:

WebCore:

        Reviewed by john

        <rdar://problem/5032066> Delete should work between ToDos

        Stop selecting the table when the user attempts to delete
        the line break just after it, instead, extend the selection
        into the last table cell.
        Let deletion merge content into table cells (as long as
        its not from another table cell).
        Start calling shouldInsertNode:replacingDOMRange:givenAction:
        when moving content with moveParagraphs, so that Mail can
        modify content before it is moved into a ToDo during a delete.

        * editing/CompositeEditCommand.cpp:
        (WebCore::CompositeEditCommand::moveParagraphs): Ask the
        delegate before performing the move.  This will let Mail
        prevent styles from entering a ToDo.
        Added a FIXME about adding a new WebViewInsertAction
        * editing/DeleteSelectionCommand.cpp:
        (WebCore::DeleteSelectionCommand::initializePositionData):
        Allow merging into table cells from content not in a table.
        * editing/SelectionController.cpp:
        (WebCore::SelectionController::modifyExtendingRightForward):
        Don't select the table when extending by character/word from
        just before it.  This code was here to make it easier to select
        tables in order to delete them, but the deletion UI makes it
        largely unnecessary.
        (WebCore::SelectionController::modifyExtendingLeftBackward):
        Ditto.
        * editing/TypingCommand.cpp:
        (WebCore::isFirstPositionAfterTable): Added.
        (WebCore::isLastPositionBeforeTable): Ditto.
        (WebCore::TypingCommand::deleteKeyPressed): When pressing
        the delete key with a caret at the beginning of a paragraph
        just after a table, move the paragraph into the last table
        cell.
        Do not do this if it would mean moving a block table.
        Removed some old irrelevant comments.
        * editing/htmlediting.cpp: Removed unused functions.
        * editing/htmlediting.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20591 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoBuild fix for Leopard.
thatcher [Thu, 29 Mar 2007 22:14:49 +0000 (22:14 +0000)]
Build fix for Leopard.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20590 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoAdd a comment since our current fix is not really acceptable.
staikos [Thu, 29 Mar 2007 19:00:27 +0000 (19:00 +0000)]
Add a comment since our current fix is not really acceptable.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20579 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 years agoRemove redundant line.
staikos [Thu, 29 Mar 2007 18:44:02 +0000 (18:44 +0000)]
Remove redundant line.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20578 268f45cc-cd09-0410-ab3c-d52691b4dbfc