WebKit-https.git
10 years ago2011-05-31 Abhishek Arya <inferno@chromium.org>
inferno@chromium.org [Tue, 31 May 2011 16:51:09 +0000 (16:51 +0000)]
2011-05-31  Abhishek Arya  <inferno@chromium.org>

        Reviewed by Dimitri Glazkov.

        Convert raw ptr to RefPtr for documentElement.
        https://bugs.webkit.org/show_bug.cgi?id=61688

        Test: fast/dom/xml-parser-error-message-crash.svg

        * dom/XMLDocumentParser.cpp:
        (WebCore::XMLDocumentParser::insertErrorMessageBlock):
2011-05-31  Abhishek Arya  <inferno@chromium.org>

        Reviewed by Dimitri Glazkov.

        Test that we do not crash when a DOMNodeInsertedIntoDocument event
        is fired while generating a error message block in xml parsing.
        https://bugs.webkit.org/show_bug.cgi?id=61688

        * fast/dom/xml-parser-error-message-crash-expected.txt: Added.
        * fast/dom/xml-parser-error-message-crash.svg: Added.

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:28:33 +0000 (16:28 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for setShadow()
        https://bugs.webkit.org/show_bug.cgi?id=61786

        Move CanvasRenderingContext2D.setShadow() to auto-generated JSC bindings.

        There is a subtle difference to the previous behavior: invalid numbers of
        arguments now raise TypeError instead of SyntaxError. This is in accordance
        with Web IDL, and matches the existing V8 bindings.

        Test: fast/canvas/canvas-overloads-setShadow.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.idl:
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for setShadow()
        https://bugs.webkit.org/show_bug.cgi?id=61786

        Add a test to verify the behavior of setShadow() when called
        with different numbers of arguments.

        * fast/canvas/canvas-overloads-setShadow-expected.txt: Added.
        * fast/canvas/canvas-overloads-setShadow.html: Added.
        * fast/canvas/script-tests/canvas-overloads-setShadow.js: Added.

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:25:34 +0000 (16:25 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for drawImageFromRect()
        https://bugs.webkit.org/show_bug.cgi?id=61709

        Add a test to verify the behavior of drawImageFromRect() when called
        with different numbers of arguments.

        * fast/canvas/canvas-overloads-drawImageFromRect-expected.txt: Added.
        * fast/canvas/canvas-overloads-drawImageFromRect.html: Added.
        * fast/canvas/script-tests/canvas-overloads-drawImageFromRect.js: Added.
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for drawImageFromRect()
        https://bugs.webkit.org/show_bug.cgi?id=61709

        Move CanvasRenderingContext2D.drawImageFromRect() to auto-generated JSC bindings.
        Make it [RequiresAllArguments=Raise] to match the old behavior.
        Side effect: calling drawImageFromRect() with zero arguments will now raise
        SyntaxError instead of TypeError.

        Test: fast/canvas/canvas-overloads-drawImageFromRect.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.h:
        * html/canvas/CanvasRenderingContext2D.idl:

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:22:31 +0000 (16:22 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for drawImage()
        https://bugs.webkit.org/show_bug.cgi?id=61703

        Move CanvasRenderingContext2D.drawImage() to auto-generated JSC bindings.

        There is a subtle difference to the previous behavior: invalid numbers of
        arguments now raise TypeError instead of SyntaxError. This is in accordance
        with Web IDL, and matches the existing V8 bindings.

        Test: fast/canvas/canvas-overloads-drawImage.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.idl:
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for drawImage()
        https://bugs.webkit.org/show_bug.cgi?id=61703

        Add a test to verify the behavior of drawImage() when called with
        different numbers of arguments.

        * fast/canvas/canvas-overloads-drawImage-expected.txt: Added.
        * fast/canvas/canvas-overloads-drawImage.html: Added.
        * fast/canvas/script-tests/canvas-overloads-drawImage.js: Added.

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:20:32 +0000 (16:20 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for strokeRect()
        https://bugs.webkit.org/show_bug.cgi?id=61641

        Add a test to verify the behavior of strokeRect() when called with
        different numbers of arguments.

        * fast/canvas/canvas-overloads-strokeRect-expected.txt: Added.
        * fast/canvas/canvas-overloads-strokeRect.html: Added.
        * fast/canvas/script-tests/canvas-overloads-strokeRect.js: Added.
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for strokeRect()
        https://bugs.webkit.org/show_bug.cgi?id=61641

        Move CanvasRenderingContext2D.setStrokeColor() to auto-generated JSC bindings.

        No behavioral change.

        Test: fast/canvas/canvas-overloads-strokeRect.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.idl:

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:15:27 +0000 (16:15 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for setFillColor()
        https://bugs.webkit.org/show_bug.cgi?id=61635

        Move CanvasRenderingContext2D.setStrokeColor() to auto-generated JSC bindings.

        There is a subtle difference to the previous behavior: invalid numbers of
        arguments now raise TypeError instead of SyntaxError. This is in accordance
        with Web IDL, but doesn't matter much anyway since this method is WebKit-only.

        Test: fast/canvas/canvas-overloads-setFillColor.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.idl:
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for setFillColor()
        https://bugs.webkit.org/show_bug.cgi?id=61635

        Add a test to verify the behavior of setFillColor() when called with
        different numbers of arguments.

        * fast/canvas/canvas-overloads-setFillColor-expected.txt: Added.
        * fast/canvas/canvas-overloads-setFillColor.html: Added.
        * fast/canvas/script-tests/canvas-overloads-setFillColor.js: Added.

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:12:35 +0000 (16:12 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for setStrokeColor()
        https://bugs.webkit.org/show_bug.cgi?id=61629

        Add a test to verify the behavior of setStrokeColor() when called with
        different numbers of arguments.

        * fast/canvas/canvas-overloads-setStrokeColor-expected.txt: Added.
        * fast/canvas/canvas-overloads-setStrokeColor.html: Added.
        * fast/canvas/script-tests/canvas-overloads-setStrokeColor.js: Added.
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for setStrokeColor()
        https://bugs.webkit.org/show_bug.cgi?id=61629

        Move CanvasRenderingContext2D.setStrokeColor() to auto-generated JSC bindings.

        There is a subtle difference to the previous behavior: invalid numbers of
        arguments now raise TypeError instead of SyntaxError. This is in accordance
        with Web IDL, but doesn't matter much anyway since this method is WebKit-only.

        Test: fast/canvas/canvas-overloads-setStrokeColor.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.idl:

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:10:39 +0000 (16:10 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for strokeText()
        https://bugs.webkit.org/show_bug.cgi?id=61626

        Move CanvasRenderingContext2D.strokeText() to auto-generated JSC bindings.
        Make it [RequiresAllArguments=Raise] to match the old behavior.
        This has the side-effect of aligning the behaviors of JSC and V8.

        Test: fast/canvas/canvas-overloads-strokeText.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.idl:
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for strokeText()
        https://bugs.webkit.org/show_bug.cgi?id=61626

        Add a test to verify the behavior of strokeText() when called with different
        numbers of arguments. There are two differences to the previous behavior:

        - SyntaxError exceptions are now raised with the message "Not enough arguments."
        - Calling strokeText() with superfluous arguments now lets the call through
          instead of raising a SyntaxError. This matches both Gecko and Presto.

        * fast/canvas/canvas-overloads-strokeText-expected.txt: Added.
        * fast/canvas/canvas-overloads-strokeText.html: Added.
        * fast/canvas/script-tests/canvas-overloads-strokeText.js: Added.

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

10 years ago2011-05-31 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Tue, 31 May 2011 16:08:49 +0000 (16:08 +0000)]
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for fillText()
        https://bugs.webkit.org/show_bug.cgi?id=61623

        Add a test to verify the behavior of fillText() when called with different
        numbers of arguments. There are two differences to the previous behavior:

        - SyntaxError exceptions are now raised with the message "Not enough arguments."
        - Calling fillText() with superfluous arguments now lets the call through
          instead of raising a SyntaxError. This matches both Gecko and Presto.

        * fast/canvas/canvas-overloads-fillText-expected.txt: Added.
        * fast/canvas/canvas-overloads-fillText.html: Added.
        * fast/canvas/script-tests/canvas-overloads-fillText.js: Added.
2011-05-31  Andreas Kling  <kling@webkit.org>

        Reviewed by Antti Koivisto.

        Canvas/JSC: Auto-generate overloads for fillText()
        https://bugs.webkit.org/show_bug.cgi?id=61623

        Move CanvasRenderingContext2D.fillText() to auto-generated JSC bindings.
        Make it [RequiresAllArguments=Raise] to match the old behavior.
        This has the side-effect of aligning the behaviors of JSC and V8.

        Test: fast/canvas/canvas-overloads-fillText.html

        * bindings/js/JSCanvasRenderingContext2DCustom.cpp:
        * html/canvas/CanvasRenderingContext2D.idl:

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

10 years ago2011-05-31 Cary Clark <caryclark@google.com>
commit-queue@webkit.org [Tue, 31 May 2011 16:05:31 +0000 (16:05 +0000)]
2011-05-31  Cary Clark  <caryclark@google.com>

        Reviewed by Eric Seidel.

        Allow Chromium porting layer to use Skia on Mac.
        https://bugs.webkit.org/show_bug.cgi?id=61548

        * public/WebCommon.h:
        Don't allow WebKit to use CG if it is already using Skia.

        * src/PlatformBridge.cpp:
        (WebCore::PlatformBridge::paintScrollbarThumb):
        Allow scrollbar thumb draw to take WebCanvas.

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

10 years ago2011-05-31 ChangeLog <james.cooley@nokia.com>
commit-queue@webkit.org [Tue, 31 May 2011 15:38:37 +0000 (15:38 +0000)]
2011-05-31  ChangeLog  <james.cooley@nokia.com>

        Reviewed by Andreas Kling.

        [Qt][WK2] Qt port needs test content for QWKHistory
        https://bugs.webkit.org/show_bug.cgi?id=57850

        Created data-driven QTestLib tests for existing QWKHistory APIs.

        This walks through loading four simple pages, navigating
        backwards and forwards.

        We check the results at each action by creating expected forward/back
        lists and comparing those lists in count and membership to those kept by
        WebKit and retrieved via the QWKHistory APIs.

        * UIProcess/API/qt/tests/html/a.htm: Added.
        * UIProcess/API/qt/tests/html/b.htm: Added.
        * UIProcess/API/qt/tests/html/c.htm: Added.
        * UIProcess/API/qt/tests/html/d.htm: Added.
        Four added simple pages

        * UIProcess/API/qt/tests/qwkhistory: Added.
        * UIProcess/API/qt/tests/qwkhistory/qwkhistory.pro: Added.
        Created qwkhistory folder and pro

        * UIProcess/API/qt/tests/qwkhistory/tst_qwkhistory.cpp: Added.
        (TestHistoryItem::title):
        (TestHistoryItem::url):
        (TestHistoryItem::TestHistoryItem):
        Utility class to hold expected url,title for test

        (tst_QWKHistory::initTestCase):
        (tst_QWKHistory::historyFBTest_data):
        (tst_QWKHistory::historyFBTest):
        init, data, and data-driven test case

        * UIProcess/API/qt/tests/tests.pro:
        Included qwkhistory in tests.pro

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

10 years ago2011-05-31 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Tue, 31 May 2011 14:20:05 +0000 (14:20 +0000)]
2011-05-31  Simon Fraser  <simon.fraser@apple.com>

        Reviewed by Kenneth Rohde Christiansen.

        Frame flattening is broken with nested frames
        https://bugs.webkit.org/show_bug.cgi?id=61491

        * fast/frames/flattening/iframe-flattening-nested.html: Added.
        * fast/frames/flattening/resources/inner-frame.html: Added.
        * fast/frames/flattening/resources/intermediate-frame.html: Added.
        * platform/mac/fast/frames/flattening/iframe-flattening-nested-expected.png: Added.
        * platform/mac/fast/frames/flattening/iframe-flattening-nested-expected.txt: Added.
2011-05-31  Yael Aharon  <yael.aharon@nokia.com>

        Reviewed by Kenneth Rohde Christiansen.

        Frame flattening is broken with nested frames
        https://bugs.webkit.org/show_bug.cgi?id=61491

        After r77988, when frame flattening is enabled, performPostLayoutTasks() is called on a timer for iframes.
        This causes layout of nested iframes to sometimes happen asynchronously, but WebCore expects layout to always finish synchronously.
        Added a call to updateWidgetPosition() for cases that performPostLayoutTasks() is now happening asynchronously.

        Test: fast/frames/flattening/iframe-flattening-nested.html

        * page/FrameView.cpp:
        (WebCore::FrameView::layout):

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

10 years ago2011-05-31 Tommy Widenflycht <tommyw@google.com>
commit-queue@webkit.org [Tue, 31 May 2011 13:46:13 +0000 (13:46 +0000)]
2011-05-31  Tommy Widenflycht  <tommyw@google.com>

        Reviewed by Tony Gentilcore.

        Fix getUserMedia callback bindings for JSC.
        https://bugs.webkit.org/show_bug.cgi?id=60174

        This patch initializes the exception code variable to 0.

        Test: fast/dom/MediaStream/argument-types.html

        * bindings/js/JSNavigatorCustom.cpp:
        (WebCore::JSNavigator::webkitGetUserMedia):

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

10 years ago[Qt] Implement find feature for QtTestBrowser
ossy@webkit.org [Tue, 31 May 2011 13:32:46 +0000 (13:32 +0000)]
[Qt] Implement find feature for QtTestBrowser
https://bugs.webkit.org/show_bug.cgi?id=61425

Patch by Kristóf Kosztyó <Kosztyo.Kristof@stud.u-szeged.hu> on 2011-05-31
Reviewed by Csaba Osztrogonác.

Buildfix for --minimal and CONFIG+=qt_minimal build.

* QtTestBrowser/launcherwindow.cpp:
(LauncherWindow::LauncherWindow):
(LauncherWindow::createChrome):
* QtTestBrowser/launcherwindow.h:

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

10 years ago2011-05-31 Oleg Romashin <oleg.romashin@nokia.com>
commit-queue@webkit.org [Tue, 31 May 2011 12:43:11 +0000 (12:43 +0000)]
2011-05-31  Oleg Romashin  <oleg.romashin@nokia.com>

        Reviewed by Benjamin Poulain.

        Fix compilation with debug enabled, m_lightSource.type is not valid anymore
        https://bugs.webkit.org/show_bug.cgi?id=61719

        * platform/graphics/filters/arm/FELightingNEON.h:
        (WebCore::FELighting::platformApplyNeon):

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

10 years ago2011-05-31 Mikhail Naganov <mnaganov@chromium.org>
mnaganov@chromium.org [Tue, 31 May 2011 11:49:22 +0000 (11:49 +0000)]
2011-05-31  Mikhail Naganov  <mnaganov@chromium.org>

        [Chromium] Unreviewed tests results rebaseline after r87721.

        * platform/chromium-linux-x86/svg/custom/invalid-fill-expected.png: Added.
        * platform/chromium-linux-x86/svg/custom/invalid-fill-expected.txt: Copied from LayoutTests/platform/chromium-linux/svg/custom/invalid-fill-expected.txt.
        * platform/chromium-linux-x86/svg/custom/invalid-fill-hex-expected.png: Added.
        * platform/chromium-linux-x86/svg/custom/invalid-stroke-hex-expected.png: Added.
        * platform/chromium-linux-x86/svg/custom/invalid-uri-stroke-expected.png: Added.
        * platform/chromium-linux/svg/custom/invalid-fill-expected.png:
        * platform/chromium-linux/svg/custom/invalid-fill-expected.txt:
        * platform/chromium-linux/svg/custom/invalid-fill-hex-expected.png:
        * platform/chromium-linux/svg/custom/invalid-stroke-hex-expected.png: Added.
        * platform/chromium-linux/svg/custom/invalid-uri-stroke-expected.png: Added.
        * platform/chromium-win-vista/svg/custom/invalid-fill-expected.png: Added.
        * platform/chromium-win-vista/svg/custom/invalid-fill-expected.txt: Copied from LayoutTests/platform/chromium-linux/svg/custom/invalid-fill-expected.txt.
        * platform/chromium-win-vista/svg/custom/invalid-fill-hex-expected.png: Added.
        * platform/chromium-win-vista/svg/custom/invalid-fill-hex-expected.txt: Copied from LayoutTests/platform/chromium-linux/svg/custom/invalid-fill-expected.txt.
        * platform/chromium-win-vista/svg/custom/invalid-stroke-hex-expected.png: Added.
        * platform/chromium-win-vista/svg/custom/invalid-stroke-hex-expected.txt: Added.
        * platform/chromium-win-vista/svg/custom/invalid-uri-stroke-expected.png: Added.
        * platform/chromium-win-vista/svg/custom/invalid-uri-stroke-expected.txt: Added.

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

10 years ago2011-05-31 Oliver Varga <Varga.Oliver@stud.u-szeged.hu>
commit-queue@webkit.org [Tue, 31 May 2011 10:46:25 +0000 (10:46 +0000)]
2011-05-31  Oliver Varga  <Varga.Oliver@stud.u-szeged.hu>

        Reviewed by Nikolas Zimmermann.

        Invalid color handling is broken for SVG
        https://bugs.webkit.org/show_bug.cgi?id=58531

        Fix invalid color fallback handling. If the fill/stroke attributes computed
        value leads to a an invalid color, inherit the desired color from the parent
        style instead. Matches Opera/FF and SVG 1.1 Second Edition (http://www.w3.org/TR/SVG/).

        * platform/chromium-linux/svg/custom/invalid-fill-expected.txt:
        * platform/chromium-linux/svg/custom/invalid-fill-hex-expected.txt:
        * platform/chromium-linux/svg/custom/invalid-stroke-hex-expected.txt: Added.
        * platform/chromium-linux/svg/custom/invalid-uri-stroke-expected.txt: Added.
        * platform/mac/svg/custom/invalid-fill-expected.png:
        * platform/mac/svg/custom/invalid-fill-expected.txt:
        * platform/mac/svg/custom/invalid-fill-hex-expected.png:
        * platform/mac/svg/custom/invalid-fill-hex-expected.txt:
        * platform/mac/svg/custom/invalid-stroke-hex-expected.png: Added.
        * platform/mac/svg/custom/invalid-stroke-hex-expected.txt: Added.
        * platform/mac/svg/custom/invalid-uri-stroke-expected.png: Added.
        * platform/mac/svg/custom/invalid-uri-stroke-expected.txt: Added.
        * platform/qt/svg/custom/invalid-fill-expected.txt:
        * platform/qt/svg/custom/invalid-fill-hex-expected.txt:
        * platform/qt/svg/custom/invalid-stroke-hex-expected.txt: Added.
        * platform/qt/svg/custom/invalid-uri-stroke-expected.txt: Added.
        * svg/custom/invalid-fill-hex.svg:
        * svg/custom/invalid-fill.svg:
        * svg/custom/invalid-stroke-hex.svg: Added.
        * svg/custom/invalid-uri-stroke.svg: Added.
2011-05-31  Oliver Varga  <Varga.Oliver@stud.u-szeged.hu>

        Reviewed by Nikolas Zimmermann.

        Invalid color handling is broken for SVG
        https://bugs.webkit.org/show_bug.cgi?id=58531

        Fix invalid color fallback handling. If the fill/stroke attributes computed
        value leads to a an invalid color, inherit the desired color from the parent
        style instead. Matches Opera/FF and SVG 1.1 Second Edition (http://www.w3.org/TR/SVG/).

        Tests: svg/custom/invalid-stroke-hex.svg
               svg/custom/invalid-uri-stroke.svg

        * rendering/svg/RenderSVGResource.cpp:
        (WebCore::inheritColorFromParentStyleIfNeeded):
        (WebCore::requestPaintingResource):

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

10 years ago2011-05-31 Kristóf Kosztyó <Kosztyo.Kristof@stud.u-szeged.hu>
commit-queue@webkit.org [Tue, 31 May 2011 10:42:01 +0000 (10:42 +0000)]
2011-05-31  Kristóf Kosztyó  <Kosztyo.Kristof@stud.u-szeged.hu>

        Reviewed by Csaba Osztrogonác.

        [Qt] Implement find feature for QtTestBrowser
        https://bugs.webkit.org/show_bug.cgi?id=61425

        * QtTestBrowser/launcherwindow.cpp:
        (LauncherWindow::LauncherWindow):
        (LauncherWindow::createChrome):
        (LauncherWindow::showFindBar):
        (LauncherWindow::find):
        * QtTestBrowser/launcherwindow.h:

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

10 years ago2011-05-31 Yuta Kitamura <yutak@chromium.org>
yutak@chromium.org [Tue, 31 May 2011 10:20:05 +0000 (10:20 +0000)]
2011-05-31  Yuta Kitamura  <yutak@chromium.org>

        Reviewed by Kent Tamura.

        WebSocket: Make the output of client-close.html more verbose
        https://bugs.webkit.org/show_bug.cgi?id=61770

        * http/tests/websocket/tests/client-close-expected.txt:
        * http/tests/websocket/tests/client-close.html:
        * http/tests/websocket/tests/client-close_wsh.py:
        Echo back the first two bytes received from the client.

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

10 years ago2011-05-31 Vangelis Kokkevis <vangelis@chromium.org>
vangelis@chromium.org [Tue, 31 May 2011 09:58:31 +0000 (09:58 +0000)]
2011-05-31  Vangelis Kokkevis  <vangelis@chromium.org>

        Reviewed by Kent Tamura.

        [chromium] Avoid double deletion of DrawingBuffer's color buffer
        texture.
        https://bugs.webkit.org/show_bug.cgi?id=61761

        Test: Verified that the DCHECK in GraphicsContext3D implementation
        doesn't trigger.

        * platform/graphics/chromium/DrawingBufferChromium.cpp:
        (WebCore::DrawingBuffer::~DrawingBuffer):

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

10 years ago[Qt][WK2] Ossyize editing/pasteboard/copy-standalone-image-crash.html since eventSend...
zoltan@webkit.org [Tue, 31 May 2011 09:10:42 +0000 (09:10 +0000)]
[Qt][WK2] Ossyize editing/pasteboard/copy-standalone-image-crash.html since eventSender is not implemented.

* platform/qt-wk2/Skipped:

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

10 years ago2011-05-30 Hayato Ito <hayato@chromium.org>
hayato@chromium.org [Tue, 31 May 2011 07:39:32 +0000 (07:39 +0000)]
2011-05-30  Hayato Ito  <hayato@chromium.org>

        Reviewed by Ryosuke Niwa.

        Add a utility function for dumping a tree for the Node, including a document of a frame.
        https://bugs.webkit.org/show_bug.cgi?id=61727

        No new tests since added functions are only available in debug builds.

        * dom/Node.cpp:
        (WebCore::parentOrHostOrFrameOwner):
        (WebCore::traverseNextNodeAcrossFrame):
        (WebCore::Node::showTreeForThisAcrossFrame):
        * dom/Node.h:

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

10 years ago2011-05-31 Yuta Kitamura <yutak@chromium.org>
yutak@chromium.org [Tue, 31 May 2011 07:29:20 +0000 (07:29 +0000)]
2011-05-31  Yuta Kitamura  <yutak@chromium.org>

        Unreviewed. Update Chromium test expectation for http/tests/websocket/tests/client-close.html.

        * platform/chromium/test_expectations.txt:

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

10 years ago2011-05-30 Gabor Loki <loki@webkit.org>
loki@webkit.org [Tue, 31 May 2011 07:01:28 +0000 (07:01 +0000)]
2011-05-30 Gabor Loki <loki@webkit.org>

        Rubber-stamped by Csaba Osztrogonác.

        [Qt] Skip crashed test after r87691-r87693 on ARM
        https://bugs.webkit.org/show_bug.cgi?id=61764

        * platform/qt-arm/Skipped:

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

10 years ago2011-05-30 James Kozianski <koz@chromium.org>
koz@chromium.org [Tue, 31 May 2011 06:28:58 +0000 (06:28 +0000)]
2011-05-30  James Kozianski  <koz@chromium.org>

        Reviewed by Kent Tamura.

        [Chromium] Make isValidProtocol() accept protocols with '+'.
        https://bugs.webkit.org/show_bug.cgi?id=61759

        * platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt: Added.
        * platform/chromium/fast/dom/register-protocol-handler-expected.txt: Added.
2011-05-30  James Kozianski  <koz@chromium.org>

        Reviewed by Kent Tamura.

        [Chromium] Make isValidProtocol() accept protocols with '+'.
        https://bugs.webkit.org/show_bug.cgi?id=61759

        Also, detect syntax errors before security errors; some syntax errors
        will also trigger a security error, but it's more helpful to the
        programmer to know if they have a syntax error, which are well-defined
        in the spec, rather than a security error, which aren't.

        * page/Navigator.cpp:
        (WebCore::Navigator::registerProtocolHandler):
        Detect syntax errors before security errors.
        * platform/KURLGoogle.cpp:
        (WebCore::isSchemeChar):
        Include '+' in the list of valid characters.

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

10 years ago2011-05-30 Andrey Petrov <andrey.petrov@gmail.com>
commit-queue@webkit.org [Tue, 31 May 2011 05:00:51 +0000 (05:00 +0000)]
2011-05-30  Andrey Petrov  <andrey.petrov@gmail.com>

        Reviewed by Hajime Morita.

        Using jQuery to show/hide IMG elements crashes WebKit
        https://bugs.webkit.org/show_bug.cgi?id=31721

        * editing/pasteboard/copy-standalone-image-crash-expected.txt: Added.
        * editing/pasteboard/copy-standalone-image-crash.html: Added.
2011-05-30  Andrey Petrov  <andrey.petrov@gmail.com>

        Reviewed by Hajime Morita.

        Using jQuery to show/hide IMG elements crashes WebKit
        https://bugs.webkit.org/show_bug.cgi?id=31721

        For platform code, do not assert nodes always have renderer at
        the time context menu action is invoked.
        Renderer can actually be empty for a given node if it was hidden
        or removed from domtree by some non-user generated event (e.g timeout)
        after the popup menu had been created.

        Changing Chromium, Gtk, Mac, Qt, Win and WinCE.

        Test: editing/pasteboard/copy-standalone-image-crash.html

        * platform/chromium/PasteboardChromium.cpp:
        (WebCore::Pasteboard::writeImage):
        * platform/gtk/PasteboardGtk.cpp:
        (WebCore::Pasteboard::writeImage):
        * platform/mac/PasteboardMac.mm:
        (WebCore::Pasteboard::writeImage):
        * platform/qt/PasteboardQt.cpp:
        (WebCore::Pasteboard::writeImage):
        * platform/win/PasteboardWin.cpp:
        (WebCore::Pasteboard::writeImage):
        * platform/wince/PasteboardWinCE.cpp:
        (WebCore::Pasteboard::writeImage):

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

10 years ago2011-05-30 Daniel Bates <dbates@webkit.org>
dbates@webkit.org [Tue, 31 May 2011 04:03:56 +0000 (04:03 +0000)]
2011-05-30  Daniel Bates  <dbates@webkit.org>

        Reviewed by Adam Barth.

        Rename XSSFilter to XSSAuditor
        https://bugs.webkit.org/show_bug.cgi?id=61718

        Currently we use the names XSSFilter and XSSAuditor throughout the project.
        Instead, we should choose one name for consistency.

        No functionality was changed. So, no new tests.

        * XSSAuditor: Copied from PerformanceTests/XSSFilter.
        * XSSFilter: Removed.
        * XSSFilter/large-post-many-events.html: Removed.
        * XSSFilter/large-post-many-inline-scripts-and-events.html: Removed.
        * XSSFilter/resources: Removed.
        * XSSFilter/resources/target-for-large-post-many-inline-scripts-and-events.html: Removed.
2011-05-30  Daniel Bates  <dbates@webkit.org>

        Reviewed by Adam Barth.

        Rename XSSFilter to XSSAuditor
        https://bugs.webkit.org/show_bug.cgi?id=61718

        Currently we use the names XSSFilter and XSSAuditor throughout the project.
        Instead, we should choose one name for consistency.

        No functionality was changed. So, no new tests.

        * CMakeLists.txt:
        * GNUmakefile.list.am:
        * WebCore.gypi:
        * WebCore.pro:
        * WebCore.vcproj/WebCore.vcproj:
        * WebCore.xcodeproj/project.pbxproj:
        * html/parser/HTMLDocumentParser.cpp:
        (WebCore::HTMLDocumentParser::HTMLDocumentParser):
        (WebCore::HTMLDocumentParser::pumpTokenizer):
        * html/parser/HTMLDocumentParser.h:
        * html/parser/XSSAuditor.cpp: Copied from Source/WebCore/html/parser/XSSFilter.cpp.
        (WebCore::XSSAuditor::XSSAuditor):
        (WebCore::XSSAuditor::init):
        (WebCore::XSSAuditor::filterToken):
        (WebCore::XSSAuditor::filterTokenInitial):
        (WebCore::XSSAuditor::filterTokenAfterScriptStartTag):
        (WebCore::XSSAuditor::filterScriptToken):
        (WebCore::XSSAuditor::filterObjectToken):
        (WebCore::XSSAuditor::filterParamToken):
        (WebCore::XSSAuditor::filterEmbedToken):
        (WebCore::XSSAuditor::filterAppletToken):
        (WebCore::XSSAuditor::filterIframeToken):
        (WebCore::XSSAuditor::filterMetaToken):
        (WebCore::XSSAuditor::filterBaseToken):
        (WebCore::XSSAuditor::filterFormToken):
        (WebCore::XSSAuditor::eraseDangerousAttributesIfInjected):
        (WebCore::XSSAuditor::eraseAttributeIfInjected):
        (WebCore::XSSAuditor::snippetForRange):
        (WebCore::XSSAuditor::snippetForAttribute):
        (WebCore::XSSAuditor::isContainedInRequest):
        (WebCore::XSSAuditor::isSameOriginResource):
        * html/parser/XSSAuditor.h: Copied from Source/WebCore/html/parser/XSSFilter.h.
        * html/parser/XSSFilter.cpp: Removed.
        * html/parser/XSSFilter.h: Removed.

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

10 years ago2011-05-30 No'am Rosenthal <noam.rosenthal@nokia.com>
noam.rosenthal@nokia.com [Tue, 31 May 2011 02:21:49 +0000 (02:21 +0000)]
2011-05-30  No'am Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Simon Hausmann.

        WebKit2: Enable serializing of data types needed for cross-process accelerated compositing
        https://bugs.webkit.org/show_bug.cgi?id=61694

        Expose a public clearAll() function to reset WebCore::Animation.

        No new functionality, so no new tests.

        * platform/animation/Animation.h:
        (WebCore::Animation::clearAll):
2011-05-30  No'am Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Simon Hausmann.

        WebKit2: Enable serializing of data types needed for cross-process accelerated compositing
        https://bugs.webkit.org/show_bug.cgi?id=61694

        Create an ArgumentCoder for WebCore::Animation.

        * Scripts/webkit2/messages.py:
        * Shared/WebCoreArgumentCoders.h:

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

10 years ago2011-05-30 Raphael Kubo da Costa <kubo@profusion.mobi>
commit-queue@webkit.org [Tue, 31 May 2011 02:18:05 +0000 (02:18 +0000)]
2011-05-30  Raphael Kubo da Costa  <kubo@profusion.mobi>

        Reviewed by Kent Tamura.

        [efl] Fix _ewk_frame_children_iterator_next().
        https://bugs.webkit.org/show_bug.cgi?id=61749

        The iterator was returning always the first child, making tests like
        fast/history/history-back-initial-vs-final-url.html to timeout and
        eventually run out of memory.

        * ewk/ewk_frame.cpp:
        (_ewk_frame_children_iterator_next):
        (ewk_frame_children_iterator_new):

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

10 years ago2011-05-30 Kent Tamura <tkent@chromium.org>
tkent@chromium.org [Tue, 31 May 2011 02:04:53 +0000 (02:04 +0000)]
2011-05-30  Kent Tamura  <tkent@chromium.org>

        [Mac] Skip a flaky test: svg/zoom/page/zoom-svg-through-object-with-text.xhtml

        * platform/mac/Skipped:

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

10 years ago2011-05-30 Eric Carlson <eric.carlson@apple.com>
eric.carlson@apple.com [Tue, 31 May 2011 01:44:55 +0000 (01:44 +0000)]
2011-05-30  Eric Carlson  <eric.carlson@apple.com>

        Reviewed by Alexey Proskuryakov.

        Audio and video files saved to the Application Cache should preserve the original file extension
        https://bugs.webkit.org/show_bug.cgi?id=61750
        <rdar://9524922>

        No new tests, it isn't possible to check the name of the file in the cache from within
        DRT. Changes verified manually.

        * loader/appcache/ApplicationCacheStorage.cpp:
        (WebCore::ApplicationCacheStorage::store): Append the original file extension to the cache
            file name.
        (WebCore::ApplicationCacheStorage::writeDataToUniqueFileInDirectory): Add extension parameter.
        * loader/appcache/ApplicationCacheStorage.h:

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

10 years ago2011-05-30 Jer Noble <jer.noble@apple.com>
jer.noble@apple.com [Tue, 31 May 2011 00:05:02 +0000 (00:05 +0000)]
2011-05-30  Jer Noble  <jer.noble@apple.com>

        Reviewed by Dan Bernstein.

        Play/Pause button in wrong state when entering full screen on Vimeo
        https://bugs.webkit.org/show_bug.cgi?id=61754

        Update the play/pause button inside reset().  This ensures the button is in
        the correct state when the controls are created after playback has already
        started.

        * html/shadow/MediaControlRootElement.cpp:
        (WebCore::MediaControlRootElement::reset):

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

10 years ago2011-05-30 Hojong Han <hojong.han@samsung.com>
commit-queue@webkit.org [Mon, 30 May 2011 23:49:51 +0000 (23:49 +0000)]
2011-05-30  Hojong Han  <hojong.han@samsung.com>

        Reviewed by Geoffrey Garen.

        [JSC] malfunction during arithmetic condition check with negative number (-2147483648)
        https://bugs.webkit.org/show_bug.cgi?id=61416

        * assembler/MacroAssemblerARM.h:
        (JSC::MacroAssemblerARM::branch32):
        * tests/mozilla/ecma/Expressions/11.12-1.js:
        (getTestCases):

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

10 years ago2011-05-30 No'am Rosenthal <noam.rosenthal@nokia.com>
noam.rosenthal@nokia.com [Mon, 30 May 2011 23:43:58 +0000 (23:43 +0000)]
2011-05-30  No'am Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Simon Hausmann.

        WebKit2: Enable serializing of data types needed for cross-process accelerated compositing
        https://bugs.webkit.org/show_bug.cgi?id=61694

        Added a public TimingFunction::type() method.

        No new functionality, so no new tests.

        * platform/animation/TimingFunction.h:
        (WebCore::TimingFunction::type):
2011-05-30  No'am Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Simon Hausmann.

        WebKit2: Enable serializing of data types needed for cross-process accelerated compositing
        https://bugs.webkit.org/show_bug.cgi?id=61694

        Add an ArgumentCoder for WebCore::TimingFunction. This serializer can create the appropriate
        TimingFunction subclass based on the type of timing function.

        * Scripts/webkit2/messages.py:
        * Shared/WebCoreArgumentCoders.h:

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

10 years ago2011-05-30 Noam Rosenthal <noam.rosenthal@nokia.com>
noam.rosenthal@nokia.com [Mon, 30 May 2011 22:29:50 +0000 (22:29 +0000)]
2011-05-30  Noam Rosenthal  <noam.rosenthal@nokia.com>

        BUILD FIX for r87697 on Windows/Symbian

        For now enabling TextureMapper for Mac/Linux only.

        * WebCore.pri:

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

10 years ago2011-05-30 No'am Rosenthal <noam.rosenthal@nokia.com>
noam.rosenthal@nokia.com [Mon, 30 May 2011 22:02:57 +0000 (22:02 +0000)]
2011-05-30  No'am Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Simon Hausmann.

        WebKit2: Enable serializing of data types needed for cross-process accelerated compositing
        https://bugs.webkit.org/show_bug.cgi?id=61694

        Add WebCore::Length and WebCore::TransformationMatrix to the types that use SimpleArgumentCoder.

        * Scripts/webkit2/messages.py:
        * Shared/WebCoreArgumentCoders.h:

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

10 years agoBUILD FIX for r87692 with !ENABLE(FULLSCREEN_API)
ddkilzer@apple.com [Mon, 30 May 2011 22:00:57 +0000 (22:00 +0000)]
BUILD FIX for r87692 with !ENABLE(FULLSCREEN_API)

* dom/EventDispatcher.cpp:
(WebCore::EventDispatcher::determineDispatchBehavior): Mark
shadowRoot as an unused parameter with !ENABLE(FULLSCREEN_API).

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

10 years ago2011-05-30 No'am Rosenthal <noam.rosenthal@nokia.com>
noam.rosenthal@nokia.com [Mon, 30 May 2011 20:57:18 +0000 (20:57 +0000)]
2011-05-30  No'am Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Andreas Kling.

        [Texmap][Qt] Enable TextureMapper by default
        https://bugs.webkit.org/show_bug.cgi?id=61740

        Make CONFIG+=texmap an opt-out instead of an opt-in.

        No new functionality so no new tests.

        * WebCore.pri:

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

10 years ago2011-05-30 Martin Robinson <mrobinson@igalia.com>
commit-queue@webkit.org [Mon, 30 May 2011 20:08:51 +0000 (20:08 +0000)]
2011-05-30  Martin Robinson  <mrobinson@igalia.com>

        Reviewed by Xan Lopez.

        [GTK] fast/css/font-face-zero-hash-key.html is crashing on the bots
        https://bugs.webkit.org/show_bug.cgi?id=61693

        * platform/gtk/Skipped: Unskip the test which is no longer crashing.
2011-05-30  Martin Robinson  <mrobinson@igalia.com>

        Reviewed by Xan Lopez.

        [GTK] fast/css/font-face-zero-hash-key.html is crashing on the bots
        https://bugs.webkit.org/show_bug.cgi?id=61693

        Do not try to read the FT_Face from m_scaledFont if it m_scaledFont is
        null. A recent change fixing 0-pixel-size fonts means that m_scaledFont
        may now be null.

        No new tests. This change should cause fast/css/font-face-zero-hash-key.html
        to stop crashing.

        * platform/graphics/freetype/FontPlatformDataFreeType.cpp:
        (WebCore::FontPlatformData::FontPlatformData): Do a null check before accessing
        m_scaledFont.

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

10 years ago2011-05-30 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
caio.oliveira@openbossa.org [Mon, 30 May 2011 19:02:21 +0000 (19:02 +0000)]
2011-05-30  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>

        Reviewed by Andreas Kling.

        [Qt] Fix unnecessary wait in API autotest tst_QWebFrame::scrollbarsOff
        https://bugs.webkit.org/show_bug.cgi?id=61711

        The loadFinished() signal was emitted directly inside the call for setHtml, so
        the test was waiting the loadFinished() signal with a big timeout. Change this by
        a very small timeout and a verification with signal spy.

        In practice, setHtml() will either directly call loadFinished() or queue it to
        the next event loop run, and test will work for both situations.

        * tests/qwebframe/tst_qwebframe.cpp:
        (tst_QWebFrame::scrollbarsOff):

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

10 years ago2011-05-30 Noam Rosenthal <noam.rosenthal@nokia.com>
noam.rosenthal@nokia.com [Mon, 30 May 2011 18:28:08 +0000 (18:28 +0000)]
2011-05-30  Noam Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Kenneth Rohde Christiansen.

        [Texmap][Qt] Disable accelerated plugins/media until they're working.
        https://bugs.webkit.org/show_bug.cgi?id=61687

        Until this works properly, we should indicate that our media implementation
        in texture mapper doesn't support accelerated compositing.

        This makes LayoutTests/compositing/video/* work again.

        * platform/graphics/qt/MediaPlayerPrivateQt.h:
        (WebCore::MediaPlayerPrivateQt::supportsAcceleratedRendering):
        (WebCore::MediaPlayerPrivateQt::platformLayer):
2011-05-30  Noam Rosenthal  <noam.rosenthal@nokia.com>

        Reviewed by Kenneth Rohde Christiansen.

        [Texmap][Qt] Disable accelerated plugins/media in until they're working.
        https://bugs.webkit.org/show_bug.cgi?id=61687

        Turn off accelerated compositing for media and plugins when in texture-mapper.

        * Api/qwebsettings.cpp:
        (QWebSettingsPrivate::apply):

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

10 years ago2011-05-30 Mikhail Naganov <mnaganov@chromium.org>
abarth@webkit.org [Mon, 30 May 2011 17:21:14 +0000 (17:21 +0000)]
2011-05-30  Mikhail Naganov  <mnaganov@chromium.org>

        Reviewed by Adam Barth.

        [Chromium] Fix regression after r87628.
        https://bugs.webkit.org/show_bug.cgi?id=61733

        Having r87628 in place, Chrome reliability bot crashes in
        WebCore::HTMLLinkElement::onloadTimerFired.

        This is because the change makes WebCore::CachedResource::setRequest to
        call checkNotify on request reset.  HTMLLinkElement registers itself as
        CachedResource client via m_cachedSheet, which can happen even if
        m_cachedLinkResource wasn't set.  As a result,
        WebCore::HTMLLinkElement::notifyFinished is got called with unset
        m_cachedLinkResource, which causes a crash in
        HTMLLinkElement::onloadTimerFired

        * html/HTMLLinkElement.cpp:
        (WebCore::HTMLLinkElement::notifyFinished):

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

10 years ago2011-05-30 Jer Noble <jer.noble@apple.com>
darin@apple.com [Mon, 30 May 2011 17:09:48 +0000 (17:09 +0000)]
2011-05-30  Jer Noble  <jer.noble@apple.com>

        Reviewed by Darin Adler and Simon Fraser.

        REGRESSION (r87622): Scrubbing a Vimeo movie when in fullscreen stops playback; no way to make it start again
        https://bugs.webkit.org/show_bug.cgi?id=61717
        rdar://problem/9522272

        May be some good way to test this later. No immediate idea about the best way.

        When a media element is taken into full-screen mode, stop events from propagating
        outside the media element's shadow DOM, EventDispatcher::determineDispatchBehavior()
        has been changed to take a shadow root node. In our full screen media element check,
        we check to see if the passed shadow root is the shadow root of the full screen media
        element, and if so, specify events should StayInsideShadowDOM. The end result is that
        inside EventDispatcher::ensureEventAncestors, an ancestor chain is built up all the
        way from the SliderThumb to the video element's shadow root, but no further.

        * dom/EventDispatcher.cpp:
        (WebCore::EventDispatcher::determineDispatchBehavior): Restrict events to the
        shadow DOM when showing a full screen video.

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::HTMLMediaElement): Removed code to manage full screen controls.
        The events telling us about activity in the shadow DOM no longer bubble out so we need to
        handle this inside the shadow DOM on the root element.
        (WebCore::HTMLMediaElement::play): Ditto.
        (WebCore::HTMLMediaElement::playbackProgressTimerFired): Ditto.
        (WebCore::HTMLMediaElement::defaultEventHandler): Ditto.
        (WebCore::HTMLMediaElement::enterFullscreen): Ditto.
        (WebCore::HTMLMediaElement::exitFullscreen): Ditto.
        * html/HTMLMediaElement.h: Added isPlaying function, removed things moved to the root element.

        * html/shadow/MediaControlElements.cpp:
        (WebCore::MediaControlVolumeSliderContainerElement::defaultEventHandler): Rolled out
        changes that tried to make special rules for events using preDispatchEventHandler and such.
        This rolls out both r87622 and r87655.
        (WebCore::MediaControlMuteButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlPanelMuteButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlPlayButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlSeekButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlRewindButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlReturnToRealtimeButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlToggleClosedCaptionsButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlTimelineElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlVolumeSliderElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlFullscreenButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlFullscreenVolumeMinButtonElement::defaultEventHandler): Ditto.
        (WebCore::MediaControlFullscreenVolumeMaxButtonElement::defaultEventHandler): Ditto.
        * html/shadow/MediaControlElements.h: Ditto.

        * html/shadow/MediaControlRootElement.cpp:
        (WebCore::MediaControlRootElement::MediaControlRootElement): Initialize new data members.
        (WebCore::MediaControlRootElement::playbackStarted): Start the timer so we will consider
        hiding the controls later.
        (WebCore::MediaControlRootElement::playbackProgressed): Hide the controls if the mouse
        is no longer over the controls.
        (WebCore::MediaControlRootElement::playbackStopped): Stop the timer since we only hide
        automatically if we're playing.
        (WebCore::MediaControlRootElement::enteredFullscreen): Start the timer.
        (WebCore::MediaControlRootElement::exitedFullscreen): Stop the timer since we only care
        if we are full screen.
        (WebCore::MediaControlRootElement::containsRelatedTarget): Added. Helper for next function.
        (WebCore::MediaControlRootElement::defaultEventHandler): Do the hide/show and timer functions
        as the mouse is moved in and out.
        (WebCore::MediaControlRootElement::startHideFullscreenControlsTimer): Start the timer if
        needed.
        (WebCore::MediaControlRootElement::hideFullscreenControlsTimerFired): Hide if the conditions
        are met.
        (WebCore::MediaControlRootElement::stopHideFullscreenControlsTimer): Stop the timer.

        * html/shadow/MediaControlRootElement.h: Added new functions and data members.

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

10 years ago2011-05-30 Gavin Peters <gavinp@chromium.org>
abarth@webkit.org [Mon, 30 May 2011 17:05:37 +0000 (17:05 +0000)]
2011-05-30  Gavin Peters  <gavinp@chromium.org>

        Reviewed by Adam Barth.

        prevent HTMLLinkElement from watching multiple CachedResources
        https://bugs.webkit.org/show_bug.cgi?id=61686

        If we modify an existing link element, stop listening to the previous
        cached resource, to prevent double notifications (which crash).

        Test: fast/dom/HTMLLinkElement/prefetch-too-many-clients.html

        * html/HTMLLinkElement.cpp:
        (WebCore::HTMLLinkElement::process):
2011-05-30  Gavin Peters  <gavinp@chromium.org>

        Reviewed by Adam Barth.

        prevent HTMLLinkElement from watching multiple CachedResources
        https://bugs.webkit.org/show_bug.cgi?id=61686

        If we modify an existing link element, stop listening to the previous
        cached resource, to prevent double notifications (which crash).

        * fast/dom/HTMLLinkElement/prefetch-too-many-clients-expected.txt: Added.
        * fast/dom/HTMLLinkElement/prefetch-too-many-clients.html: Added.
        * platform/gtk/Skipped:
        * platform/mac/Skipped:
        * platform/qt/Skipped:
        * platform/win/Skipped:

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

10 years ago2011-05-30 Carlos Garcia Campos <cgarcia@igalia.com>
carlosgc@webkit.org [Mon, 30 May 2011 16:25:06 +0000 (16:25 +0000)]
2011-05-30  Carlos Garcia Campos  <cgarcia@igalia.com>

        Reviewed by Martin Robinson.

        [GTK] Implement JavaScript dialogs in MiniBrowser
        https://bugs.webkit.org/show_bug.cgi?id=61725

        * MiniBrowser/gtk/BrowserWindow.c:
        (WKStringGetCString): Helper function to get a UTF-8 c string from
        a WKStringRef.
        (WKURLGetCString): Helper function to get a UTF-8 c string from a
        WKURLRef.
        (browserWindowUpdateURL): Update to use WKURLGetCString.
        (didReceiveTitleForFrame): Update to use WKStringGetCString.
        (createMessageDialog): Create a GtkMessageDialog.
        (runJavaScriptAlert):
        (runJavaScriptConfirm):
        (runJavaScriptPrompt):
        (browserWindowUIClientInit):

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

10 years ago2011-05-30 Anders Carlsson <andersca@apple.com>
carlosgc@webkit.org [Mon, 30 May 2011 16:17:56 +0000 (16:17 +0000)]
2011-05-30  Anders Carlsson  <andersca@apple.com>

        Reviewed by Simon Fraser.

        Remove incorrect asserts from PluginProcessConnection
        https://bugs.webkit.org/show_bug.cgi?id=61731

        There might be pending incoming messages from a PluginProxy even though we've
        destroyed the plug-in and removed it from the m_plugins map, so remove the incorrect assertions.

        * WebProcess/Plugins/PluginProcessConnection.cpp:
        (WebKit::PluginProcessConnection::didReceiveMessage):
        (WebKit::PluginProcessConnection::didReceiveSyncMessage):

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

10 years ago2011-05-30 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Mon, 30 May 2011 16:14:29 +0000 (16:14 +0000)]
2011-05-30  Anders Carlsson  <andersca@apple.com>

        Reviewed by Simon Fraser.

        Remove incorrect asserts from PluginProcessConnection
        https://bugs.webkit.org/show_bug.cgi?id=61731

        There might be pending incoming messages from a PluginProxy even though we've
        destroyed the plug-in and removed it from the m_plugins map, so remove the incorrect assertions.

        * WebProcess/Plugins/PluginProcessConnection.cpp:
        (WebKit::PluginProcessConnection::didReceiveMessage):
        (WebKit::PluginProcessConnection::didReceiveSyncMessage):

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

10 years ago2011-05-30 Peter Varga <pvarga@webkit.org>
pvarga@webkit.org [Mon, 30 May 2011 15:44:02 +0000 (15:44 +0000)]
2011-05-30  Peter Varga  <pvarga@webkit.org>

        Unreviewed build fix.

        [Qt][V8] Fix debug build.

        No new tests needed.

        * css/CSSStyleDeclaration.cpp: Add missing header.
        * dom/Text.cpp: Ditto.
        * rendering/InlineTextBox.cpp: Ditto.

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

10 years ago2011-05-30 Sheriff Bot <webkit.review.bot@gmail.com>
commit-queue@webkit.org [Mon, 30 May 2011 14:50:13 +0000 (14:50 +0000)]
2011-05-30  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r87683.
        http://trac.webkit.org/changeset/87683
        https://bugs.webkit.org/show_bug.cgi?id=61729

        Breaks inspector tests on Chromium (Requested by apavlov on
        #webkit).

        * http/tests/inspector/inspect-iframe-from-different-domain.html:
        * inspector/elements/elements-delete-inline-style.html:
        * inspector/elements/elements-panel-styles.html:
        * inspector/styles/styles-iframe.html:
        * inspector/styles/styles-update-from-js-expected.txt:
        * inspector/styles/styles-update-from-js.html:
        * inspector/styles/styles-url-linkify.html:
2011-05-30  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r87683.
        http://trac.webkit.org/changeset/87683
        https://bugs.webkit.org/show_bug.cgi?id=61729

        Breaks inspector tests on Chromium (Requested by apavlov on
        #webkit).

        * inspector/front-end/StylesSidebarPane.js:
        (WebInspector.StylesSidebarPane.prototype.update):

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

10 years ago2011-05-30 Carlos Garcia Campos <cgarcia@igalia.com>
carlosgc@webkit.org [Mon, 30 May 2011 14:13:32 +0000 (14:13 +0000)]
2011-05-30  Carlos Garcia Campos  <cgarcia@igalia.com>

        Reviewed by Martin Robinson.

        [GTK] Remove some unneeded header includes
        https://bugs.webkit.org/show_bug.cgi?id=60536

        * Platform/CoreIPC/Connection.cpp: Remove WebProcess.h.
        * Platform/CoreIPC/unix/ConnectionUnix.cpp: Remove
        ProcessLauncher.h and WebProcessProxy.h. Add <wtf/OwnArrayPtr.h>.
        * WebProcess/Plugins/Netscape/NPJSObject.cpp: Remove PluginView.h.

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

10 years ago[Qt] Remove passing tests from ARM Skipped list
rgabor@webkit.org [Mon, 30 May 2011 14:06:58 +0000 (14:06 +0000)]
[Qt] Remove passing tests from ARM Skipped list

Rubber-stamped by Csaba Osztrogonác.

* platform/qt-arm/Skipped:
* platform/qt-arm/fast: Removed.
* platform/qt-arm/fast/block: Removed.
* platform/qt-arm/fast/block/float: Removed.
* platform/qt-arm/fast/block/float/float-not-removed-from-next-sibling-expected.txt: Removed.
* platform/qt-arm/fast/block/float/float-not-removed-from-next-sibling2-expected.txt: Removed.
* platform/qt-arm/fast/block/float/float-not-removed-from-next-sibling3-expected.txt: Removed.
* platform/qt-arm/fast/block/float/float-not-removed-from-next-sibling4-expected.txt: Removed.
* platform/qt-arm/fast/transforms: Removed.
* platform/qt-arm/fast/transforms/transform-positioned-ancestor-expected.txt: Removed.
* platform/qt-arm/transforms: Removed.
* platform/qt-arm/transforms/2d: Removed.
* platform/qt-arm/transforms/2d/compound-transforms-vs-containers-expected.txt: Removed.
* platform/qt-arm/transforms/2d/transform-borderbox-expected.txt: Removed.
* platform/qt-arm/transforms/2d/transform-origin-borderbox-expected.txt: Removed.
* platform/qt-arm/transforms/2d/zoom-menulist-expected.txt: Removed.
* platform/qt-arm/transforms/no_transform_hit_testing-expected.txt: Removed.
* platform/qt-arm/transforms/svg-vs-css-expected.txt: Removed.

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

10 years ago2011-05-27 Alexander Pavlov <apavlov@chromium.org>
apavlov@chromium.org [Mon, 30 May 2011 13:48:09 +0000 (13:48 +0000)]
2011-05-27  Alexander Pavlov  <apavlov@chromium.org>

        Reviewed by Yury Semikhatsky.

        Web Inspector: node selection is slow in the Elements panel.
        https://bugs.webkit.org/show_bug.cgi?id=60813

        * http/tests/inspector/inspect-iframe-from-different-domain.html:
        * inspector/elements/elements-delete-inline-style.html:
        * inspector/elements/elements-panel-styles.html:
        * inspector/styles/styles-iframe.html:
        * inspector/styles/styles-update-from-js.html:
        * inspector/styles/styles-update-from-js-expected.txt:
        * inspector/styles/styles-url-linkify.html:
2011-05-25  Alexander Pavlov  <apavlov@chromium.org>

        Reviewed by Yury Semikhatsky.

        Web Inspector: node selection is slow in the Elements panel.
        https://bugs.webkit.org/show_bug.cgi?id=60813

        Schedule full style updates rather than request all node styles from the backend up front.

        * inspector/front-end/StylesSidebarPane.js:
        (WebInspector.StylesSidebarPane.prototype.update.reloadAllStyles):
        (WebInspector.StylesSidebarPane.prototype.update):

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

10 years ago2011-05-30 Mikhail Naganov <mnaganov@chromium.org>
mnaganov@chromium.org [Mon, 30 May 2011 10:36:55 +0000 (10:36 +0000)]
2011-05-30  Mikhail Naganov  <mnaganov@chromium.org>

        Unreviewed.
        [Chromium] Rebaseline tests after r87680.

        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-in-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelMatrix-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelUnitLength-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-order-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-preserveAlpha-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetX-attr-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-bias-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-divisor-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-in-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelMatrix-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelUnitLength-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-order-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-preserveAlpha-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-targetX-prop-expected.png: Added.
        * platform/chromium-linux-x86/svg/filters/feConvolveFilter-y-bounds-expected.png: Added.
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-in-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelMatrix-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelUnitLength-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-order-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-preserveAlpha-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetX-attr-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-bias-prop-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-divisor-prop-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-in-prop-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelMatrix-prop-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelUnitLength-prop-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-order-prop-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-preserveAlpha-prop-expected.png:
        * platform/chromium-linux/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-targetX-prop-expected.png:
        * platform/chromium-linux/svg/filters/feConvolveFilter-y-bounds-expected.png: Added.
        * platform/chromium-mac-leopard/svg/W3C-SVG-1.1/filters-conv-01-f-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-in-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelMatrix-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelUnitLength-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-order-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetX-attr-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-bias-prop-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-divisor-prop-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-in-prop-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelMatrix-prop-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelUnitLength-prop-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-order-prop-expected.png:
        * platform/chromium-mac-leopard/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-targetX-prop-expected.png:
        * platform/chromium-mac/svg/W3C-SVG-1.1/filters-conv-01-f-expected.png: Added.
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-in-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelMatrix-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelUnitLength-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-order-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetX-attr-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-bias-prop-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-divisor-prop-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-in-prop-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelMatrix-prop-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelUnitLength-prop-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-order-prop-expected.png:
        * platform/chromium-mac/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-targetX-prop-expected.png:
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-in-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelMatrix-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-kernelUnitLength-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-order-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-preserveAlpha-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-targetX-attr-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-bias-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-divisor-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-edgeMode-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-in-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelMatrix-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-kernelUnitLength-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-order-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-preserveAlpha-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/dynamic-updates/SVGFEConvolveMatrixElement-svgdom-targetX-prop-expected.png: Added.
        * platform/chromium-win-vista/svg/filters/feConvolveFilter-y-bounds-expected.png: Added.

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

10 years ago011-05-30 Andrey Kosyakov <caseq@chromium.org>
caseq@chromium.org [Mon, 30 May 2011 09:18:16 +0000 (09:18 +0000)]
011-05-30  Andrey Kosyakov  <caseq@chromium.org>

        Reviewed by Pavel Feldman.

        Web Inspector: do not issue frontendReused for reload or navigation.
        https://bugs.webkit.org/show_bug.cgi?id=61192

       - only issue frontendReused when inspector is attached after main resource committed load
       - upon committing loading main resource, reset agents before sending FrameNavigated to page agent
       - Ensure main resource is set early so that abbreviations of URLs in resource tree are consistent after reload
       - do not enable agents explicitly in frontendReused (this is done by backend when resotring state from cookie)

        * inspector/InspectorAgent.cpp:
        (WebCore::InspectorAgent::InspectorAgent):
        (WebCore::InspectorAgent::restore):
        (WebCore::InspectorAgent::clearFrontend):
        (WebCore::InspectorAgent::didCommitLoad):
        * inspector/InspectorAgent.h:
        * inspector/InspectorInstrumentation.cpp:
        (WebCore::InspectorInstrumentation::didCommitLoadImpl):
        * inspector/front-end/NetworkManager.js:
        * inspector/front-end/ResourceTreeModel.js:
        (WebInspector.ResourceTreeModel.prototype._processCachedResources):
        (WebInspector.ResourceTreeModel.prototype._frameNavigated):
        (WebInspector.ResourceTreeModel.prototype._cleanupFramesAfterNavigation):
        (WebInspector.ResourceTreeModel.prototype._addFramesRecursively):
        * inspector/front-end/ResourcesPanel.js:
        (WebInspector.ResourcesPanel.prototype.reset):
        (WebInspector.ResourcesPanel.prototype._resetResourcesTree):
        (WebInspector.DatabaseTreeElement.prototype.onpopulate):
        (WebInspector.SearchResultsTreeElementsTraverser.prototype._lastTreeElement):
        * inspector/front-end/inspector.js:
        (WebInspector.reset):
        (WebInspector.frontendReused):

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

10 years ago2011-05-30 Ryan Sleevi <rsleevi@chromium.org>
commit-queue@webkit.org [Mon, 30 May 2011 08:30:17 +0000 (08:30 +0000)]
2011-05-30  Ryan Sleevi  <rsleevi@chromium.org>

        Reviewed by Darin Adler.

        FEConvolveMatrix::getPixelValue() fails to properly check if y is within bounds, causing it to fail to correctly apply the kernel and edge mode to the first targetY pixels
        https://bugs.webkit.org/show_bug.cgi?id=61603

        * platform/mac/svg/filters/feConvolveFilter-y-bounds-expected.png: Added.
        * platform/mac/svg/filters/feConvolveFilter-y-bounds-expected.txt: Added.
        * svg/filters/feConvolveFilter-y-bounds.svg: Added.
2011-05-30  Ryan Sleevi  <rsleevi@chromium.org>

        Reviewed by Darin Adler.

        FEConvolveMatrix::getPixelValue() fails to properly check if y is within bounds, causing it to fail to correctly apply the kernel and edge mode to the first targetY pixels
        https://bugs.webkit.org/show_bug.cgi?id=61603

        Test: svg/filters/feConvolveFilter-y-bounds.svg

        * platform/graphics/filters/FEConvolveMatrix.cpp:
        (WebCore::FEConvolveMatrix::getPixelValue):

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

10 years ago2011-05-30 Nate Chapin <japhet@chromium.org>
japhet@chromium.org [Mon, 30 May 2011 07:45:41 +0000 (07:45 +0000)]
2011-05-30  Nate Chapin  <japhet@chromium.org>

        Reviewed by Antti Koivisto.

        Refactor the CachedResourceLoader -> CachedResource -> CachedResourceRequest
        flow so it's a little less circular.
        https://bugs.webkit.org/show_bug.cgi?id=61666

        * loader/cache/CachedFont.cpp:
        (WebCore::CachedFont::beginLoadIfNeeded):
        * loader/cache/CachedResource.cpp:
        (WebCore::CachedResource::load):
        * loader/cache/CachedResourceLoader.cpp:
        (WebCore::CachedResourceLoader::requestImage):
        (WebCore::CachedResourceLoader::setAutoLoadImages):
        (WebCore::CachedResourceLoader::loadStarted):
        * loader/cache/CachedResourceLoader.h:
        * loader/cache/CachedResourceRequest.cpp:
        (WebCore::CachedResourceRequest::load):

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

10 years ago2011-05-30 Sheriff Bot <webkit.review.bot@gmail.com>
abarth@webkit.org [Mon, 30 May 2011 07:22:21 +0000 (07:22 +0000)]
2011-05-30  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r87618.
        http://trac.webkit.org/changeset/87618
        https://bugs.webkit.org/show_bug.cgi?id=61720

        Causes BrowserFocusTest.FocusOnReload to fail (Requested by
        abarth|gardener on #webkit).

        * fast/dom/HTMLLinkElement/prefetch-too-many-clients-expected.txt: Removed.
        * fast/dom/HTMLLinkElement/prefetch-too-many-clients.html: Removed.
        * platform/gtk/Skipped:
        * platform/mac/Skipped:
        * platform/qt/Skipped:
        * platform/win/Skipped:
2011-05-30  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r87618.
        http://trac.webkit.org/changeset/87618
        https://bugs.webkit.org/show_bug.cgi?id=61720

        Causes BrowserFocusTest.FocusOnReload to fail (Requested by
        abarth|gardener on #webkit).

        * html/HTMLLinkElement.cpp:
        (WebCore::HTMLLinkElement::process):

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

10 years ago2011-05-29 Yuta Kitamura <yutak@chromium.org>
yutak@chromium.org [Mon, 30 May 2011 06:22:06 +0000 (06:22 +0000)]
2011-05-29  Yuta Kitamura  <yutak@chromium.org>

        Reviewed by Kent Tamura.

        WebSocket closing handshake
        https://bugs.webkit.org/show_bug.cgi?id=35721

        * http/tests/websocket/tests/client-close-expected.txt: Added.
        * http/tests/websocket/tests/client-close.html: Added. Test client-initiated close.
        * http/tests/websocket/tests/client-close_wsh.py: Added.
        * http/tests/websocket/tests/close-before-open-expected.txt: Add a new console message.
        * http/tests/websocket/tests/close-event-expected.txt:
        * http/tests/websocket/tests/close-event.html: Test if closeEvent.wasClean is true.
        * http/tests/websocket/tests/close-unref-websocket-expected.txt: Add a new console message.
        * http/tests/websocket/tests/frame-length-longer-than-buffer_wsh.py:
        We need to stop pywebsocket from starting the closing handshake. Otherwise, pywebsocket
        waits for a close frame to arrive and this test will time out.
        * http/tests/websocket/tests/server-close-expected.txt: Added.
        * http/tests/websocket/tests/server-close.html: Added. Test server-initiated close.
        * http/tests/websocket/tests/server-close_wsh.py: Added.
        * http/tests/websocket/tests/websocket-event-target-expected.txt: Add a new console message.
2011-05-29  Yuta Kitamura  <yutak@chromium.org>

        Reviewed by Kent Tamura.

        WebSocket closing handshake
        https://bugs.webkit.org/show_bug.cgi?id=35721

        Implement WebSocket closing handshake based on Ian Hickson's
        WebSocket protocol draft 76.

        Tests: http/tests/websocket/tests/client-close.html
               http/tests/websocket/tests/server-close.html

        * platform/network/SocketStreamHandleBase.cpp:
        (WebCore::SocketStreamHandleBase::send):
        Do not send a message if we are in Closing state.
        (WebCore::SocketStreamHandleBase::close):
        Do not disconnect if we have pending data which have not been sent yet.
        In this case, the actual disconnection will happen in sendPendingData().
        (WebCore::SocketStreamHandleBase::disconnect):
        Renamed from close(). Disconnect the connection immediately.
        (WebCore::SocketStreamHandleBase::sendPendingData):
        * platform/network/SocketStreamHandleBase.h:
        * websockets/ThreadableWebSocketChannelClientWrapper.cpp:
        Add didStartClosingHandshake(). Add a function argument (ClosingHandshakeCompletionStatus)
        to didClose().
        (WebCore::ThreadableWebSocketChannelClientWrapper::didStartClosingHandshake):
        (WebCore::ThreadableWebSocketChannelClientWrapper::didClose):
        (WebCore::ThreadableWebSocketChannelClientWrapper::didStartClosingHandshakeCallback):
        (WebCore::ThreadableWebSocketChannelClientWrapper::didCloseCallback):
        * websockets/ThreadableWebSocketChannelClientWrapper.h:
        * websockets/WebSocket.cpp:
        (WebCore::WebSocket::send):
        (WebCore::WebSocket::close):
        Fail if close() is attempted before the connection is established.
        Otherwise, set the state to CLOSING and start the closing handshake.
        (WebCore::WebSocket::bufferedAmount):
        If the state is CLOSING, we need to consider buffered data in m_channel and sent after close().
        (WebCore::WebSocket::didConnect):
        (WebCore::WebSocket::didReceiveMessage):
        We need to invoke message event in CLOSING state as well as OPEN state.
        (WebCore::WebSocket::didReceiveMessageError):
        (WebCore::WebSocket::didStartClosingHandshake):
        (WebCore::WebSocket::didClose):
        * websockets/WebSocket.h:
        * websockets/WebSocketChannel.cpp:
        (WebCore::WebSocketChannel::WebSocketChannel):
        (WebCore::WebSocketChannel::close):
        Start the closing handshake.
        (WebCore::WebSocketChannel::disconnect):
        Disconnect the socket stream, instead of close.
        (WebCore::WebSocketChannel::didClose):
        (WebCore::WebSocketChannel::didReceiveData): Ditto.
        (WebCore::WebSocketChannel::didFail): Ditto.
        (WebCore::WebSocketChannel::processBuffer):
        Ditto.
        Handle 0xFF 0x00 byte sequence, and discard received data once the closing handshake has started.
        (WebCore::WebSocketChannel::startClosingHandshake):
        Send 0xFF 0x00 byte sequence.
        (WebCore::WebSocketChannel::closingTimerFired):
        Disconnect the socket stream if the closing handshake has timed out.
        * websockets/WebSocketChannel.h:
        m_closing is true if "the WebSocket closing handshake has started" (as stated in the protocol
        specification).
        * websockets/WebSocketChannelClient.h:
        (WebCore::WebSocketChannelClient::didStartClosingHandshake): Added.
        (WebCore::WebSocketChannelClient::didClose): Add closingHandshakeCompletion parameter.
        * websockets/WorkerThreadableWebSocketChannel.cpp:
        Add closingHandshakeCompletion parameter to didClose(), and add didStartClosingHandshake().
        (WebCore::WorkerThreadableWebSocketChannel::Peer::close):
        (WebCore::workerContextDidStartClosingHandshake):
        (WebCore::WorkerThreadableWebSocketChannel::Peer::didStartClosingHandshake):
        (WebCore::workerContextDidClose):
        (WebCore::WorkerThreadableWebSocketChannel::Peer::didClose):
        * websockets/WorkerThreadableWebSocketChannel.h:
2011-05-29  Yuta Kitamura  <yutak@chromium.org>

        Reviewed by Kent Tamura.

        WebSocket closing handshake
        https://bugs.webkit.org/show_bug.cgi?id=35721

        * Scripts/webkitpy/thirdparty/__init__.py:
        Pull in pywebsocket 0.6b1. We need to update pywebsocket
        to get the right behavior of closing handshake.

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

10 years ago2011-05-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Mon, 30 May 2011 06:17:28 +0000 (06:17 +0000)]
2011-05-29  Adam Barth  <abarth@webkit.org>

        More IMAGE results.

        * platform/chromium-mac-leopard/svg/wicd/test-scalable-background-image1-expected.png: Added.

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

10 years ago2011-05-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Mon, 30 May 2011 06:07:34 +0000 (06:07 +0000)]
2011-05-29  Adam Barth  <abarth@webkit.org>

        Add image baselines for all these beautiful platforms.

        * platform/chromium-linux-x86/fullscreen: Added.
        * platform/chromium-linux-x86/fullscreen/full-screen-iframe-zIndex-expected.png: Added.
        * platform/chromium-linux/fullscreen/full-screen-iframe-zIndex-expected.png: Added.
        * platform/chromium-mac-leopard/fullscreen: Added.
        * platform/chromium-mac-leopard/fullscreen/full-screen-iframe-zIndex-expected.png: Added.
        * platform/chromium-mac/fullscreen: Added.
        * platform/chromium-mac/fullscreen/full-screen-iframe-zIndex-expected.png: Added.
        * platform/chromium-win-vista/fullscreen: Added.
        * platform/chromium-win-vista/fullscreen/full-screen-iframe-zIndex-expected.png: Added.

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

10 years ago2011-05-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Mon, 30 May 2011 05:54:59 +0000 (05:54 +0000)]
2011-05-29  Adam Barth  <abarth@webkit.org>

        Attempt to fix build by implementing pure virtual function.

        * html/shadow/MediaControlRootElementChromium.cpp:
        (WebCore::MediaControlRootElementChromium::shouldHideControls):
        * html/shadow/MediaControlRootElementChromium.h:

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

10 years ago2011-05-29 Brian Weinstein <bweinstein@apple.com>
jer.noble@apple.com [Mon, 30 May 2011 03:37:14 +0000 (03:37 +0000)]
2011-05-29  Brian Weinstein  <bweinstein@apple.com>

        Reviewed by Darin Adler.

        Controls never hide in full screen after user stops moving mouse
        https://bugs.webkit.org/show_bug.cgi?id=61715
        <rdar://problem/9522182>

        When we get a mouse move event in HTMLMediaElement::defaultEventHandler, and we are in full screen,
        show the media controls, and then start a timer.

        The timer fires 3 seconds after the user's last mouse movement (timer is restarted on every mouse
        move), and hides the controls.

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize our new timer.
        (WebCore::HTMLMediaElement::play): If we are in full screen mode, start our timer to hide the full screen
            controls. We don't want the user to have to move the mouse to hide them when they use the spacebar
            to play.
        (WebCore::HTMLMediaElement::startHideFullscreenControlsTimer): Starts a oneshot timer 3 seconds in the future
            if we are in full screen.
        (WebCore::HTMLMediaElement::hideFullscreenControlsTimerFired): Make sure that we are currently playing, and
            we are in full screen, and hide the controls. We don't want to hide the controls if we are paused.
        (WebCore::HTMLMediaElement::stopHideFullscreenControlsTimer): Stops the timer.
        (WebCore::HTMLMediaElement::defaultEventHandler): If we get a mouse move event and are in full screen, show the
            controls and start a timer to hide them.
        (WebCore::HTMLMediaElement::enterFullscreen): Start a timer to hide the full screen controls. The user shouldn't
            have the move the mouse once they enter full screen to hide the controls.
        (WebCore::HTMLMediaElement::exitFullscreen): Stop the timer to hide the full screen controls.
        * html/HTMLMediaElement.h:
        * html/shadow/MediaControls.h: Added pure virtual shouldHideControls() method.
        * html/shadow/MediaControlRootElement.cpp:
        (WebCore::MediaControlRootElement::playbackStopped): Stop the timer to hide the full screen controls.
        (WebCore::MediaControlRootElement::shouldHideControls): Added, only report that
            the caller should hide the controls if the panel is not hovered.
        * html/shadow/MediaControlRootElement.h:

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

10 years ago2011-05-29 Jer Noble <jer.noble@apple.com>
jer.noble@apple.com [Mon, 30 May 2011 02:05:30 +0000 (02:05 +0000)]
2011-05-29  Jer Noble  <jer.noble@apple.com>

        Reviewed by Simon Fraser.

        Embedded Vimeo video, when taken fullscreen, is overlapped by content from the enclosing page
        https://bugs.webkit.org/show_bug.cgi?id=61712

        * fullscreen/full-screen-iframe-zIndex-expected.txt: Added.
        * fullscreen/full-screen-iframe-zIndex.html: Added.
        * platform/mac/fullscreen/full-screen-iframe-zIndex-expected.png: Added.
2011-05-29  Jer Noble  <jer.noble@apple.com>

        Reviewed by Simon Fraser.

        Embedded Vimeo video, when taken fullscreen, is overlapped by content from the enclosing page
        https://bugs.webkit.org/show_bug.cgi?id=61712

        Tests: fullscreen/full-screen-iframe-zIndex.html

        Walk up the ancestor chain for the full-screen element, marking them as full-screen
        ancestors. Then, we apply a pseudo-class to those elements, which disable all their
        stacking-context-creating styles.  Set the z-index of all full-screen elements (and
        the iframes which contain them) to a large value, ensuring they appear above other
        sibling elements.

        This new behavior replaces the previous "-webkit-full-screen-media-document" behavior
        and pesudo-class, so remove it and replace it with the new pseudo-class.

        * css/CSSSelector.cpp:
        (WebCore::nameToPseudoTypeMap): Add -webkit-full-screen-ancestor and remove
            -webkit-full-screen-media-document.
        (WebCore::CSSSelector::pseudoId): Add PseudoFullScreenAncestor and remove
            PseudoFullScreenMediaDocument
        (WebCore::CSSSelector::extractPseudoType): Ditto.
        * css/CSSSelector.h: Ditto.
        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): Remove the
            PseudoFullScreenMediaDocument implementation and replace it with
            PseudoFullScreenAncestor.
        * css/fullscreen.css:
        (:-webkit-full-screen): Add a z-index property.
        (:-webkit-full-screen-ancestor:not(iframe)): Added.  Reset the z-index to
            auto, and reset other stacking-context creating properties.
        * dom/Document.cpp:
        (WebCore::Document::setContainsFullScreenElementRecursively): Recurse up every element
            instead of just the iframe elements.
        (WebCore::Document::webkitWillEnterFullScreenForElement): Ditto.
        * dom/Element.cpp:
        (WebCore::Element::willRemove): Reset our parent's containsFullScreenElement property.
        (WebCore::Element::insertedIntoTree): Ditto.
        (WebCore::Element::containsFullScreenElement): Moved here from HTMLElementBase.
        (WebCore::Element::setContainsFullScreenElement): Ditto.
        * dom/Element.h:
        * dom/ElementRareData.h:
        (WebCore::ElementRareData::ElementRareData): Added ivar and initializer for
            m_containsFullScreenElement.
        * html/HTMLFrameElementBase.cpp:
        (WebCore::HTMLFrameElementBase::HTMLFrameElementBase): Removed ivar and initializer.
        (WebCore::HTMLFrameElementBase::setContainsFullScreenElement): Moved into Element.
        * html/HTMLFrameElementBase.h:
        * rendering/style/RenderStyleConstants.h:

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

10 years ago2011-05-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Mon, 30 May 2011 00:49:32 +0000 (00:49 +0000)]
2011-05-29  Adam Barth  <abarth@webkit.org>

        Add expected result.

        * platform/chromium-mac-leopard/svg/wicd/test-scalable-background-image2-expected.png: Added.

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

10 years ago2011-05-29 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Mon, 30 May 2011 00:42:47 +0000 (00:42 +0000)]
2011-05-29  Adam Barth  <abarth@webkit.org>

        Slightly adjust the type of the expected failure.

        * platform/chromium/test_expectations.txt:

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

10 years ago<rdar://problem/9515650> 30 second rewind button obscured in fullscreen controller...
mitz@apple.com [Mon, 30 May 2011 00:19:51 +0000 (00:19 +0000)]
<rdar://problem/9515650> 30 second rewind button obscured in fullscreen controller when video uses custom inline controls
https://bugs.webkit.org/show_bug.cgi?id=61714

Reviewed by Darin Adler.

* dom/Document.cpp:
(WebCore::Document::webkitDidEnterFullScreenForElement): Call didBecomeFullscreenElement() on the
new full screen element.
(WebCore::Document::webkitWillExitFullScreenForElement): Call willStopBeingFullscreenElement() on
the full screen element.
* dom/Element.h:
(WebCore::Element::didBecomeFullscreenElement): Added with a default implementation that does nothing.
(WebCore::Element::willStopBeingFullscreenElement): Ditto.
* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::didBecomeFullscreenElement): Added this override, which calls
MediaControls::enteredFullscreen().
(WebCore::HTMLMediaElement::willStopBeingFullscreenElement): Added this override, which calls
MediaControls::exitedFullscreen().
* html/HTMLMediaElement.h:
* html/shadow/MediaControlElements.cpp:
(WebCore::MediaControlFullscreenButtonElement::preDispatchEventHandler): Removed calls to
enteredFullscreen() and exitedFullscreen() from here, since they are now called in response to
changes to the fullscreen-ness of the element.

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

10 years ago2011-05-29 Darin Adler <darin@apple.com>
darin@apple.com [Mon, 30 May 2011 00:19:45 +0000 (00:19 +0000)]
2011-05-29  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        Race condition in full screen controller, which leads to problem when web process crashes
        https://bugs.webkit.org/show_bug.cgi?id=61707

        Second try at this. First try could lead to a WKView leak.

        * UIProcess/mac/WKFullScreenWindowController.h: Added _isExitingAcceleratedCompositingMode.
        Needed to track whether we have retained so we don't leak if the page goes away before
        we get the callback.

        * UIProcess/mac/WKFullScreenWindowController.mm:
        (-[WKFullScreenWindowController exitAcceleratedCompositingMode]): Added code to set
        the new variable to YES.
        (-[WKFullScreenWindowController exitCompositedModeRepaintCompleted]): Added code to
        deal with the new boolean and to release.
        (exitCompositedModeRepaintCompleted): Removed the release that was here.
        (-[WKFullScreenWindowController close]): Added a call to exitCompositedModeRepaintCompleted
        here. We're as complete as we'll ever be when we're closed; we can't get the callback
        after that point.

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

10 years ago2011-05-29 Darin Adler <darin@apple.com>
darin@apple.com [Mon, 30 May 2011 00:00:34 +0000 (00:00 +0000)]
2011-05-29  Darin Adler  <darin@apple.com>

        Reviewed by Kevin Decker.

        REGRESSION (r87622): In media documents, clicking the full screen button and the play button pauses the video
        https://bugs.webkit.org/show_bug.cgi?id=61713

        We need to come up with a way to regression-test these changes. Nothing at the moment.

        The bug is that we removed calls to preventDefault, but we still do need to prevent
        default handling of this event.

        * html/shadow/MediaControlElements.cpp:
        (WebCore::MediaControlElement::preDispatchEventHandler): Add back the preventDefault
        that was in here before r87622 as well as the stopPropagation that was added in r87622.
        (WebCore::MediaControlMuteButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlPlayButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlSeekButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlRewindButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlReturnToRealtimeButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlToggleClosedCaptionsButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlFullscreenButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlFullscreenVolumeMinButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlFullscreenVolumeMaxButtonElement::preDispatchEventHandler): Ditto.

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

10 years agoREGRESSION (r87637): nytimes.com freezes after going back to it
weinig@apple.com [Sun, 29 May 2011 22:14:49 +0000 (22:14 +0000)]
REGRESSION (r87637): nytimes.com freezes after going back to it
<rdar://problem/9521928>

Reviewed by Anders Carlsson.

* WebProcess/WebPage/DrawingAreaImpl.cpp:
(WebKit::DrawingAreaImpl::forceRepaint):
Add call to setShouldNotifyAfterNextScheduledLayerFlush(false) to prevent a
redundant call to layerHostDidFlushLayers which would cause the layer state
to get out of sync with the UIProcess.

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

10 years ago2011-05-29 Geoffrey Garen <ggaren@apple.com>
ggaren@apple.com [Sun, 29 May 2011 21:56:53 +0000 (21:56 +0000)]
2011-05-29  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Sam Weinig.

        Some heap refactoring
        https://bugs.webkit.org/show_bug.cgi?id=61704

        SunSpider says no change.

        * JavaScriptCore.exp: Export!

        * heap/Heap.cpp: COLLECT_ON_EVERY_ALLOCATION can actually do so now.

        (JSC::Heap::Heap): Changed Heap sub-objects to point to the heap.

        (JSC::Heap::allocate): Changed inline allocation code to only select the
        size class, since this can be optimized out at compile time -- everything
        else is now inlined into this out-of-line function.

        No need to duplicate ASSERTs made in our caller.

        * heap/Heap.h:
        (JSC::Heap::heap):
        (JSC::Heap::isMarked):
        (JSC::Heap::testAndSetMarked):
        (JSC::Heap::testAndClearMarked):
        (JSC::Heap::setMarked): Call directly into MarkedBlock instead of adding
        a layer of indirection through MarkedSpace.

        (JSC::Heap::allocate): See above.

        * heap/MarkedBlock.cpp:
        (JSC::MarkedBlock::create):
        (JSC::MarkedBlock::MarkedBlock):
        * heap/MarkedBlock.h: Changed Heap sub-objects to point to the heap.

        * heap/MarkedSpace.cpp:
        (JSC::MarkedSpace::MarkedSpace):
        (JSC::MarkedSpace::allocateBlock):
        * heap/MarkedSpace.h:
        (JSC::MarkedSpace::allocate): Updated to match changes above.

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

10 years ago2011-05-29 Jer Noble <jer.noble@apple.com>
jer.noble@apple.com [Sun, 29 May 2011 21:28:24 +0000 (21:28 +0000)]
2011-05-29  Jer Noble  <jer.noble@apple.com>

        Reviewed by Simon Fraser.

        REGRESSION (r86924): webkitRequestFullScreen does not work properly (black screen, can’t interact)
        https://bugs.webkit.org/show_bug.cgi?id=61710

        When taking the documentElement full screen, we don't create a RenderFullScreen
        object and thus never enter accelerated compositing mode for the animation. Tell
        the WKFullScreenWindowController to enter accelerated compositing mode anyway
        (with an empty layer context) so as to fulfill its expectations and cause the
        full screen animation to complete.

        * UIProcess/mac/WKFullScreenWindowController.mm:
        (-[WKFullScreenWindowController finishedEnterFullScreenAnimation:]): Added a
            comment explaining when we expect the animation to complete.
        (-[WKFullScreenWindowController enterAcceleratedCompositingMode:WebKit::]):
            Remove an ASSERT which is no longer valid.
        * WebProcess/FullScreen/mac/WebFullScreenManagerMac.mm:
        (WebKit::WebFullScreenManagerMac::beginEnterFullScreenAnimation): Tell the UIProcess
            to enter then exit full-screen mode.
        (WebKit::WebFullScreenManagerMac::beginExitFullScreenAnimation): Ditto.

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

10 years ago2011-05-29 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Sun, 29 May 2011 20:45:10 +0000 (20:45 +0000)]
2011-05-29  Andreas Kling  <kling@webkit.org>

        Reviewed by Kenneth Rohde Christiansen.

        Element: Micro-cleanup of scroll methods.
        https://bugs.webkit.org/show_bug.cgi?id=61705

        Do an early return without calculating element boundaries when asked
        to scroll an element without a renderer().

        * dom/Element.cpp:
        (WebCore::Element::scrollIntoView):
        (WebCore::Element::scrollIntoViewIfNeeded):
        (WebCore::Element::scrollByUnits):

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

10 years ago2011-05-29 Darin Adler <darin@apple.com>
darin@apple.com [Sun, 29 May 2011 20:12:27 +0000 (20:12 +0000)]
2011-05-29  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        Race condition in full screen controller, which leads to problem when web process crashes
        https://bugs.webkit.org/show_bug.cgi?id=61707

        Saw this while debugging a crash. We don't have a way to make regression tests for this
        kind of issue at this time.

        * UIProcess/mac/WKFullScreenWindowController.mm:
        (-[WKFullScreenWindowController exitAcceleratedCompositingMode]): Retain the controller
        here since we are keeping a pointer to it.
        (exitCompositedModeRepaintCompleted): Release the controller here.

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

10 years ago2011-05-29 Darin Adler <darin@apple.com>
darin@apple.com [Sun, 29 May 2011 20:12:01 +0000 (20:12 +0000)]
2011-05-29  Darin Adler  <darin@apple.com>

        Reviewed by Dan Bernstein.

        Fix assertion seen when entering full screen mode for standalone video
        https://bugs.webkit.org/show_bug.cgi?id=61708

        We don't currently have a way to make a regression test for this.

        * dom/Document.cpp:
        (WebCore::Document::setAnimatingFullScreen): Call scheduleForcedStyleRecalc,
        because scheduleStyleRecalc is intended to only be called when the style
        system itself detects recalc is needed.

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

10 years ago2011-05-29 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Sun, 29 May 2011 20:06:10 +0000 (20:06 +0000)]
2011-05-29  Anders Carlsson  <andersca@apple.com>

        Reviewed by Darin Adler.

        REGRESSION (87637): Entering WebKit Full Screen mode causes an assertion in DrawingAreaImpl::layerHostDidFlushLayers
        https://bugs.webkit.org/show_bug.cgi?id=61706
        <rdar://problem/9521970>

        If we've already let the UI process know that we're in compositing mode, we just need to force a repaint.

        * WebProcess/WebPage/DrawingAreaImpl.cpp:
        (WebKit::DrawingAreaImpl::forceRepaint):

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

10 years ago2011-05-29 Daniel Bates <dbates@rim.com>
dbates@webkit.org [Sun, 29 May 2011 19:38:46 +0000 (19:38 +0000)]
2011-05-29  Daniel Bates  <dbates@rim.com>

        Reviewed by David Kilzer.

        REGRESSION (r86515): svn-apply ignores diffs that omit line count in chunk range
        https://bugs.webkit.org/show_bug.cgi?id=61162

        Fixes an issue where svn-apply may ignore a diff that contains a chunk range line
        that omits a line count. In particular, the chunk range regular expression does
        not match a chunk range line that omits a line count. GNU diff(1) will omit the
        line count in the chunk range if the line count is exactly 1. For example, appending
        a new line to the end of an existing file F that contains exactly one line of text will
        be represented in a diff with a chunk range line that omits the line count for F.

        * Scripts/VCSUtils.pm:
          (parseChunkRange): Added.
        * Scripts/webkitperl/VCSUtils_unittest/parseChunkRange.pl: Added.
        * Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
          - Added unit test "Git: Append new line to the end of an existing file".

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

10 years ago2011-05-27 Andreas Kling <kling@webkit.org>
andreas.kling@nokia.com [Sun, 29 May 2011 19:13:46 +0000 (19:13 +0000)]
2011-05-27  Andreas Kling  <kling@webkit.org>

        Reviewed by James Robinson.

        [Qt] Add area check to ImageBufferData::getImageData()
        https://bugs.webkit.org/show_bug.cgi?id=61375

        No change in behavior, just a sanity check.

        * platform/graphics/qt/ImageBufferQt.cpp:
        (WebCore::getImageData):

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

10 years agohttps://bugs.webkit.org/show_bug.cgi?id=61700
darin@apple.com [Sun, 29 May 2011 19:00:11 +0000 (19:00 +0000)]
https://bugs.webkit.org/show_bug.cgi?id=61700
Need WebKit2 API to get the size of the render tree
-and corresponding-
<rdar://problem/9512733>

Patch by Beth Dakin <bdakin@apple.com> on 2011-05-29
Reviewed by Darin Adler.

New member variable on RenderArena keeps track of the current amount of memory
allocated in the arena. The new client function sends this piece of data to
WebKit.
* page/ChromeClient.h:
(WebCore::ChromeClient::setRenderTreeSize):
* page/FrameView.cpp:
(WebCore::FrameView::performPostLayoutTasks):
* rendering/RenderArena.cpp:
(WebCore::RenderArena::RenderArena):
(WebCore::RenderArena::allocate):
(WebCore::RenderArena::free):
* rendering/RenderArena.h:
(WebCore::RenderArena::totalRenderArenaSize):

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

10 years ago2011-05-29 Beth Dakin <bdakin@apple.com>
darin@apple.com [Sun, 29 May 2011 18:13:14 +0000 (18:13 +0000)]
2011-05-29  Beth Dakin  <bdakin@apple.com>

        Reviewed by Darin Adler.

        https://bugs.webkit.org/show_bug.cgi?id=61700
        Need WebKit2 API to get the size of the render tree
        -and corresponding-
        <rdar://problem/9512733>

        * UIProcess/API/C/WKPage.cpp:
        (WKPageGetRenderTreeSize):
        * UIProcess/API/C/WKPage.h:
        * UIProcess/WebPageProxy.h:
        (WebKit::WebPageProxy::renderTreeSize):
        (WebKit::WebPageProxy::setRenderTreeSize):
        * UIProcess/WebPageProxy.messages.in:
        * WebProcess/WebCoreSupport/WebChromeClient.cpp:
        (WebKit::WebChromeClient::setRenderTreeSize):
        * WebProcess/WebCoreSupport/WebChromeClient.h:

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

10 years ago<rdar://problem/9519453>
darin@apple.com [Sun, 29 May 2011 18:03:30 +0000 (18:03 +0000)]
<rdar://problem/9519453>
When calling ForceRepaint on a page with accelerated compositing, we can get
a race condition where the UI process paints the old content, resulting in showing
the previous page. That happens because forceRepaint's callback waits for the next
repaint to happen, but nothing notifies UI process that the page is in accelerated
compositing mode.

Reviewed by Anders Carlsson and Simon Fraser.

* WebProcess/WebPage/DrawingAreaImpl.cpp:
(WebKit::DrawingAreaImpl::forceRepaint): Call layerHostDidFlushLayers instead of
just calling forceRepaint on the layer tree host.

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

10 years ago2011-05-27 Philippe Normand <pnormand@igalia.com>
philn@webkit.org [Sun, 29 May 2011 09:49:03 +0000 (09:49 +0000)]
2011-05-27  Philippe Normand  <pnormand@igalia.com>

        Reviewed by Martin Robinson.

        [GTK] Data directory is webkitgtk-x.y not webkit-x.y
        https://bugs.webkit.org/show_bug.cgi?id=61642

        * GNUmakefile.am: updated data directory.

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

10 years ago2011-05-29 Sheriff Bot <webkit.review.bot@gmail.com>
commit-queue@webkit.org [Sun, 29 May 2011 08:30:35 +0000 (08:30 +0000)]
2011-05-29  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r87566.
        http://trac.webkit.org/changeset/87566
        https://bugs.webkit.org/show_bug.cgi?id=61702

        It made all tests assert on Qt in debug mode (Requested by
        Ossy_weekend on #webkit).

        * WebCore.exp.in:
        * bindings/ScriptControllerBase.cpp:
        (WebCore::ScriptController::executeIfJavaScriptURL):
        * dom/Document.cpp:
        (WebCore::Document::Document):
        (WebCore::Document::suggestedMIMEType):
        (WebCore::Document::lastModified):
        (WebCore::Document::initSecurityContext):
        (WebCore::Document::updateURLForPushOrReplaceState):
        * dom/Document.h:
        (WebCore::Document::setDocumentLoader):
        (WebCore::Document::loader):
        * html/MediaDocument.cpp:
        (WebCore::MediaDocument::replaceMediaElementTimerFired):
        * html/PluginDocument.cpp:
        (WebCore::PluginDocumentParser::createDocumentStructure):
        * platform/mac/HTMLConverter.mm:
        (fileWrapperForElement):

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

10 years ago2011-05-28 Steve Falkenburg <sfalken@apple.com>
sfalken@apple.com [Sun, 29 May 2011 07:04:13 +0000 (07:04 +0000)]
2011-05-28  Steve Falkenburg  <sfalken@apple.com>

        Reviewed by Dan Bernstein.

        Don't add sub-frames to global history when navigating via back/forward
        https://bugs.webkit.org/show_bug.cgi?id=61701
        <rdar://problem/9521222>

        Instead of just checking whether the load is the first in the frame, we also need to
        check to make sure we're loading in the main frame.

        Not testable due to lack of global history infrastructure in DRT.

        * loader/FrameLoader.cpp:
        (WebCore::FrameLoader::transitionToCommitted):

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

10 years ago Suggested by Simon Fraser.
ap@apple.com [Sun, 29 May 2011 04:42:18 +0000 (04:42 +0000)]
    Suggested by Simon Fraser.

        REGRESSION (r85375): Load event is sometimes lost when multiple image elements use the same URL
        https://bugs.webkit.org/show_bug.cgi?id=61692
        <rdar://problem/9488628>

        * loader/ImageLoader.cpp: (WebCore::ImageLoader::notifyFinished): There was no need to use
        ASSERT_UNUSED here.

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

10 years ago Reviewed by Geoff Garen.
ap@apple.com [Sun, 29 May 2011 01:37:59 +0000 (01:37 +0000)]
    Reviewed by Geoff Garen.

        REGRESSION (r85375): Load event is sometimes lost when multiple image elements use the same URL
        https://bugs.webkit.org/show_bug.cgi?id=61692
        <rdar://problem/9488628>

        Test: fast/dom/gc-image-element-2.html

        Manually verified that tests from bug 59604 and from bug 40926 still pass.

        The problem here was that HTMLImageElement::hasPendingActivity() could return false when
        a load (or error) event was still expected to fire.

        * loader/cache/CachedResource.cpp:
        (WebCore::CachedResource::setRequest):
        * loader/cache/CachedResource.h:
        (WebCore::CachedResource::wasCanceled):
        (WebCore::CachedResource::errorOccurred):
        Track whether the load was canceled. We want to always notify clients of load outcome,
        as that's the only way they could make intelligent decisions.

        * dom/ScriptElement.cpp: (WebCore::ScriptElement::execute): Cached resource clients now
        get a notifyFinished call on cancellation. Handle this case, where we don't need the
        execute the script, but also don't need to fire an error event.

        * html/HTMLImageElement.cpp: Moved hasPendingActivity() to header, since it's just a single
        function call now.

        * html/HTMLImageElement.h: (WebCore::HTMLImageElement::hasPendingActivity): There is a large
        window between when CachedResource::isLoading() becomes false and events are queued.
        ImageLoader::haveFiredLoadEvent() is a much better indication of whether we are expecting
        an event to fire.

        * html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::onloadTimerFired): Again, don't do
        anything on cancellation.

        * loader/ImageLoader.cpp:
        (WebCore::ImageEventSender::hasPendingEvents): Made it debug-only again, and fixed to
        give an accurate result while looping over the list of events to dispatch.
        (WebCore::ImageLoader::notifyFinished): Don't do anything when cancelled. We don't want to
        switch to a broken image icon, or to dispatch events.
        (WebCore::ImageEventSender::dispatchPendingEvents): Clear the current loader from dispatching
        list, as the event is no longer pending when it's being dispatched.

        * loader/ImageLoader.h: Removed unnecessary hasPendingLoadEvent(). We don't care whether one
        is already pending, we only care if one is expected at some time in the future, and
        !haveFiredLoadEvent() is our best idea of that.

        * dom/XMLDocumentParser.cpp: (WebCore::XMLDocumentParser::notifyFinished): Another place to
        handle cancellation.

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

10 years ago2011-05-28 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Sat, 28 May 2011 23:45:17 +0000 (23:45 +0000)]
2011-05-28  Simon Fraser  <simon.fraser@apple.com>

        Reviewed by Dan Bernstein, Maciej Stachowiak.

        Denying access to your keychain on login crashes WebKit2
        https://bugs.webkit.org/show_bug.cgi?id=61695
        <rdar://problem/9520570>

        Fix two sources of crashes if you hit the Deny button when WebKit2 is
        doing HTTP authentication.

        First, SecKeychainItemRequestData::attributeList() failed to initialize the
        length and data members of SecKeychainAttributes in the list if there was no data.
        This caused invalid memory reads later.

        Second, returning a non-zero error from the SecKeychainItemCopyContent shim method
        would cause a later crash in a system framework, which is not set up to handle
        errors. Instead, we always return noErr, and allow the authentication to fail.

        Finally, paranoically initialize the SecKeychainItemContext in two places
        to avoid uninitialized data members, and initialize length and outData
        to 0 in secKeychainItemCopyContent() in case SecKeychainItemCopyContent()
        fails to set them on error.

        * Shared/mac/SecKeychainItemRequestData.cpp:
        (WebKit::SecKeychainItemRequestData::attributeList):
        * UIProcess/mac/WebProcessProxyMac.mm:
        (WebKit::WebProcessProxy::secKeychainItemCopyContent):
        * WebProcess/mac/KeychainItemShimMethods.mm:
        (WebKit::webSecKeychainItemCopyContent):
        (WebKit::webSecKeychainItemCreateFromContent):

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

10 years ago2011-05-28 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Sat, 28 May 2011 22:26:15 +0000 (22:26 +0000)]
2011-05-28  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        EWS builds patches that fail to build twice, which seems useless and slows down the bots
        https://bugs.webkit.org/show_bug.cgi?id=55585

        This patch switches all the early warning system bots over to the new
        PatchAnalysisTask-based infrastructure.  This patch makes these bots
        more efficient (in the case where patches fail to build) and paves the
        way for running tests on these bots!

        * Scripts/webkitpy/tool/bot/commitqueuetask_unittest.py:
        * Scripts/webkitpy/tool/bot/earlywarningsystemtask.py:
        * Scripts/webkitpy/tool/bot/patchanalysistask.py:
        * Scripts/webkitpy/tool/commands/earlywarningsystem.py:
        * Scripts/webkitpy/tool/commands/earlywarningsystem_unittest.py:
        * Scripts/webkitpy/tool/commands/queues.py:

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

10 years ago2011-05-28 Martin Robinson <mrobinson@igalia.com>
mrobinson@webkit.org [Sat, 28 May 2011 21:45:43 +0000 (21:45 +0000)]
2011-05-28  Martin Robinson  <mrobinson@igalia.com>

        Skip a failing test and better classify some existing skipped tests.

        * platform/gtk/Skipped: Update the skipped list.

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

10 years ago2011-05-28 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Sat, 28 May 2011 20:39:23 +0000 (20:39 +0000)]
2011-05-28  Anders Carlsson  <andersca@apple.com>

        Reviewed by Dan Bernstein.

        REGRESSION (r86578): Flash content missing from mercurynews.com
        https://bugs.webkit.org/show_bug.cgi?id=61691
        <rdar://problem/9519893>

        If the plug-in doesn't have access to the toplevel URL, make sure to
        return *something* to the plug-in.

        * PluginProcess/PluginControllerProxy.cpp:
        (WebKit::PluginControllerProxy::tryToShortCircuitInvoke):

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

10 years ago2011-05-28 Adam Barth <abarth@webkit.org>
abarth@webkit.org [Sat, 28 May 2011 19:59:16 +0000 (19:59 +0000)]
2011-05-28  Adam Barth  <abarth@webkit.org>

        Reviewed by Alexey Proskuryakov.

        Audit all uses of KURL::prettyURL
        https://bugs.webkit.org/show_bug.cgi?id=61201

        Update callers of prettyURL to just call string.

        * WebCoreSupport/ChromeClientQt.cpp:
        (WebCore::ChromeClientQt::mouseDidMoveOverElement):
        * WebCoreSupport/FrameLoaderClientQt.cpp:
        (WebCore::FrameLoaderClientQt::updateGlobalHistory):
        (WebCore::FrameLoaderClientQt::cancelledError):
        (WebCore::FrameLoaderClientQt::blockedError):
        (WebCore::FrameLoaderClientQt::objectContentType):
        (WebCore::FrameLoaderClientQt::createPlugin):
2011-05-28  Adam Barth  <abarth@webkit.org>

        Reviewed by Alexey Proskuryakov.

        Audit all uses of KURL::prettyURL
        https://bugs.webkit.org/show_bug.cgi?id=61201

        Update callers of prettyURL to just call string.

        * WebCoreSupport/ChromeClientGtk.cpp:
        (WebKit::ChromeClient::mouseDidMoveOverElement):
        * WebCoreSupport/FrameLoaderClientGtk.cpp:
        (WebKit::FrameLoaderClient::dispatchDidChangeLocationWithinPage):
        (WebKit::FrameLoaderClient::dispatchDidCommitLoad):
        * webkit/webkitwebview.cpp:
        (webkit_web_view_get_icon_uri):
2011-05-28  Adam Barth  <abarth@webkit.org>

        Reviewed by Alexey Proskuryakov.

        Audit all uses of KURL::prettyURL
        https://bugs.webkit.org/show_bug.cgi?id=61201

        Update callers of prettyURL to just call string.

        * WebCoreSupport/ChromeClientEfl.cpp:
        (WebCore::ChromeClientEfl::mouseDidMoveOverElement):
        * WebCoreSupport/FrameLoaderClientEfl.cpp:
        (WebCore::FrameLoaderClientEfl::dispatchWillSendRequest):
        (WebCore::FrameLoaderClientEfl::assignIdentifierToInitialRequest):
        (WebCore::FrameLoaderClientEfl::dispatchDecidePolicyForNavigationAction):
        (WebCore::FrameLoaderClientEfl::download):
        (WebCore::FrameLoaderClientEfl::cancelledError):
        (WebCore::FrameLoaderClientEfl::blockedError):
        * ewk/ewk_frame.cpp:
        (ewk_frame_hit_test_new):
        (ewk_frame_uri_changed):
        * ewk/ewk_view.cpp:
        (_ewk_view_priv_new):
        (ewk_view_frame_create):
        (ewk_view_plugin_create):
2011-05-28  Adam Barth  <abarth@webkit.org>

        Reviewed by Alexey Proskuryakov.

        Audit all uses of KURL::prettyURL
        https://bugs.webkit.org/show_bug.cgi?id=61201

        As far as I can tell, all the callers of this API are confused.  There
        seems to be a cargo cult of using the "pretty" version of a URL, but in
        reality folks just want the URL itself.  The only case I'm unsure about
        is location.href, which could have some compatibility constraints.
        I've renamed prettyURL to deprecatedString to discourage folks from
        further cargo-culting.

        * WebCore.exp.in:
        * page/Location.cpp:
        (WebCore::Location::href):
        (WebCore::Location::toString):
        * platform/KURL.cpp:
        (WebCore::KURL::deprecatedString):
        * platform/KURL.h:
        * platform/KURLGoogle.cpp:
        (WebCore::KURL::deprecatedString):
        * platform/network/soup/ResourceHandleSoup.cpp:
        (WebCore::ResourceHandle::prepareForURL):
        * workers/WorkerLocation.cpp:
        (WebCore::WorkerLocation::href):
        (WebCore::WorkerLocation::toString):
2011-05-28  Adam Barth  <abarth@webkit.org>

        Reviewed by Alexey Proskuryakov.

        Audit all uses of KURL::prettyURL
        https://bugs.webkit.org/show_bug.cgi?id=61201

        Update callers of prettyURL to just call string.

        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
        (WebKit::WebFrameLoaderClient::createPlugin):
        * WebProcess/WebCoreSupport/qt/WebErrorsQt.cpp:
        (WebKit::cancelledError):
        (WebKit::blockedError):

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

10 years ago2011-05-28 Jer Noble <jer.noble@apple.com>
jer.noble@apple.com [Sat, 28 May 2011 19:41:14 +0000 (19:41 +0000)]
2011-05-28  Jer Noble  <jer.noble@apple.com>

        Reviewed by Maciej Stachowiak.

        Mouse clicks propagate outside full-screen media controls.
        https://bugs.webkit.org/show_bug.cgi?id=61689

        Mouse click events are propagating out of the media controls, so mouse click
        listeners registered on the video element are getting fired when the user
        clicks on media controller buttons.  By default, block propagation of click
        events from MediaControlElements by overriding preDispatchEventHandler, and
        convert all instances of defaultEventHandler -> preDispatchEventHandler.  Change
        all calls of event->setDefaultHandled() to event->stopPropagation().

        * html/shadow/MediaControlElements.cpp:
        (WebCore::MediaControlElement::preDispatchEventHandler): Added.  Block
            propagation of all mouse click events.
        (WebCore::MediaControlVolumeSliderContainerElement::preDispatchEventHandler):
            Renamed from setDefaultHandled.
        (WebCore::MediaControlMuteButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlPanelMuteButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlPlayButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlSeekButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlRewindButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlReturnToRealtimeButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlToggleClosedCaptionsButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlTimelineElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlVolumeSliderElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlFullscreenButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlFullscreenVolumeMinButtonElement::preDispatchEventHandler): Ditto.
        (WebCore::MediaControlFullscreenVolumeMaxButtonElement::preDispatchEventHandler): Ditto.
        * html/shadow/MediaControlElements.h:

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

10 years ago2011-05-28 Gavin Peters <gavinp@chromium.org>
commit-queue@webkit.org [Sat, 28 May 2011 18:18:49 +0000 (18:18 +0000)]
2011-05-28  Gavin Peters  <gavinp@chromium.org>

        Reviewed by Adam Barth.

        prevent HTMLLinkElement from watching multiple CachedResources
        https://bugs.webkit.org/show_bug.cgi?id=61686

        If we modify an existing link element, stop listening to the previous
        cached resource, to prevent double notifications (which crash).

        * fast/dom/HTMLLinkElement/prefetch-too-many-clients-expected.txt: Added.
        * fast/dom/HTMLLinkElement/prefetch-too-many-clients.html: Added.
        * platform/gtk/Skipped:
        * platform/mac/Skipped:
        * platform/qt/Skipped:
        * platform/win/Skipped:
2011-05-28  Gavin Peters  <gavinp@chromium.org>

        Reviewed by Adam Barth.

        prevent HTMLLinkElement from watching multiple CachedResources
        https://bugs.webkit.org/show_bug.cgi?id=61686

        If we modify an existing link element, stop listening to the previous
        cached resource, to prevent double notifications (which crash).

        Test: fast/dom/HTMLLinkElement/prefetch-too-many-clients.html

        * html/HTMLLinkElement.cpp:
        (WebCore::HTMLLinkElement::process):

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

10 years ago2011-05-28 Tonis Tiigi <tonistiigi@gmail.com>
commit-queue@webkit.org [Sat, 28 May 2011 17:55:16 +0000 (17:55 +0000)]
2011-05-28  Tonis Tiigi  <tonistiigi@gmail.com>

        Reviewed by Pavel Feldman.

        Web Inspector: there is a problem with computed style properties with wrong values
        https://bugs.webkit.org/show_bug.cgi?id=60535

        Adding cases that test invalid properties.

        * inspector/styles/styles-computed-trace-expected.txt:
        * inspector/styles/styles-computed-trace.html:
2011-05-28  Tonis Tiigi  <tonistiigi@gmail.com>

        Reviewed by Pavel Feldman.

        Web Inspector: there is a problem with computed style properties with wrong values
        https://bugs.webkit.org/show_bug.cgi?id=60535

        * inspector/front-end/StylesSidebarPane.js:
        (WebInspector.StylesSidebarPane.prototype._markUsedProperties):
        (WebInspector.ComputedStylePropertiesSection.prototype.rebuildComputedTrace):

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

10 years ago2011-05-26 Antonio Gomes <agomes@rim.com>
tonikitoo@webkit.org [Sat, 28 May 2011 17:02:18 +0000 (17:02 +0000)]
2011-05-26  Antonio Gomes  <agomes@rim.com>

        Reviewed by Andreas Kling.

        Logic error in WebCore/Page/SpatialNavigation.cpp::areRectsPartial      lyAligned
        https://bugs.webkit.org/show_bug.cgi?id=61606

        Removed duplicated statement within if condition.

        No new tests. Obvious clean up patch.

        * page/SpatialNavigation.cpp:
        (WebCore::areRectsPartiallyAligned):

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

11 years ago[Chromium] Unreviewed, clean up GPU tests expectations
mnaganov@chromium.org [Sat, 28 May 2011 16:16:12 +0000 (16:16 +0000)]
[Chromium] Unreviewed, clean up GPU tests expectations

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

11 years agoBUILD FIX when building only the interpreter
ddkilzer@apple.com [Sat, 28 May 2011 15:43:38 +0000 (15:43 +0000)]
BUILD FIX when building only the interpreter

Fixes the following compiler warning:

    JavaScriptCore/runtime/JSGlobalData.cpp:462:6: error: no previous prototype for function 'releaseExecutableMemory' [-Werror,-Wmissing-prototypes,3]
     void releaseExecutableMemory(JSGlobalData& globalData)
          ^

* jit/ExecutableAllocator.h: Moved declaration of
JSC::releaseExecutableMemory().

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

11 years ago[Chromium] Unreviewed. Update test_expectations to remove suppressions for passing...
mnaganov@chromium.org [Sat, 28 May 2011 15:27:31 +0000 (15:27 +0000)]
[Chromium] Unreviewed. Update test_expectations to remove suppressions for passing tests

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

11 years agoBUILD FIX after r87527 with ENABLE(BRANCH_COMPACTION)
ddkilzer@apple.com [Sat, 28 May 2011 15:24:46 +0000 (15:24 +0000)]
BUILD FIX after r87527 with ENABLE(BRANCH_COMPACTION)

* assembler/LinkBuffer.h:
(JSC::LinkBuffer::linkCode): Added missing argument.

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

11 years ago2011-05-28 Robert Hogan <robert@webkit.org>
robert@webkit.org [Sat, 28 May 2011 12:15:26 +0000 (12:15 +0000)]
2011-05-28  Robert Hogan  <robert@webkit.org>

        Reviewed by Antonio Gomes.

        [Qt] fast/events/backspace-nagivates-back fails on Qt bots (Mac and Linux)
        https://bugs.webkit.org/show_bug.cgi?id=60311

        If there is no editable text Qt interprets backspace as a navigation
        command, regardless of platform.

        * platform/qt/fast/events/backspace-navigates-back-expected.txt: Added.

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

11 years ago[Chromium] Unreviewed. Mark svg/wicd/test-scalable-background-image1.xhtml as crashin...
mnaganov@chromium.org [Sat, 28 May 2011 12:06:36 +0000 (12:06 +0000)]
[Chromium] Unreviewed. Mark svg/wicd/test-scalable-background-image1.xhtml as crashing on all platforms in debug

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