⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Feb 7, 2011:

11:44 PM Changeset in webkit [77914] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2011-02-07 Cris Neckar <cdn@chromium.org>

Reviewed by Jian Li.

Test for null deref in DOMURL
https://bugs.webkit.org/show_bug.cgi?id=53811

  • fast/files/domurl-script-execution-context-crash-expected.txt: Added.
  • fast/files/domurl-script-execution-context-crash.html: Added.

2011-02-07 Cris Neckar <cdn@chromium.org>

Reviewed by Jian Li.

Fix null deref in DOMURL
https://bugs.webkit.org/show_bug.cgi?id=53811

Test: fast/files/domurl-script-execution-context-crash.html

  • html/DOMURL.cpp: (WebCore::DOMURL::DOMURL):
11:30 PM Changeset in webkit [77913] by tony@chromium.org
  • 19 edits in trunk/Tools

2011-02-07 Tony Chang <tony@chromium.org>

Reviewed by Adam Barth.

clean up python imports
https://bugs.webkit.org/show_bug.cgi?id=53966

Convert "import A.B.C.D as D" to "from A.B.C import D" and
make some imports absolute as required by PEP-8

  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/net/irc/ircbot.py:
  • Scripts/webkitpy/common/system/logutils.py:
  • Scripts/webkitpy/common/system/logutils_unittest.py:
  • Scripts/webkitpy/layout_tests/deduplicate_tests.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/mac.py: Remove some unused imports
  • Scripts/webkitpy/layout_tests/port/webkit.py:
  • Scripts/webkitpy/layout_tests/update_webgl_conformance_tests.py:
  • Scripts/webkitpy/tool/bot/irc_command.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/prettydiff.py:
  • Scripts/webkitpy/tool/commands/queries.py:
  • Scripts/webkitpy/tool/commands/roll.py:
  • Scripts/webkitpy/tool/commands/stepsequence.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/main.py:
11:29 PM Changeset in webkit [77912] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

2011-02-07 James Robinson <jamesr@chromium.org>

Reviewed by Adam Barth.

Teach do-webcore-rename about the WebKit move to Source/
https://bugs.webkit.org/show_bug.cgi?id=53967

  • Scripts/do-webcore-rename:
11:24 PM Changeset in webkit [77911] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed.

[Qt] Fix crashes in QMetaObject::metacall
https://bugs.webkit.org/show_bug.cgi?id=53912

  • platform/qt/Skipped: http/tests/websocket/tests/websocket-protocol-ignored.html added until fix.
11:13 PM Changeset in webkit [77910] by mrowe@apple.com
  • 2 edits in trunk/Source/WebCore

Fix the 32-bit build.

  • platform/mac/ScrollAnimatorMac.mm:

(WebCore::ScrollAnimatorMac::didAddVerticalScrollbar): Remove some extraneous characters.
(WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar): Ditto.

11:07 PM Changeset in webkit [77909] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-02-07 Kenji Imasaki <imasaki@chromium.org>

Reviewed by David Levin.

[Chromium] Remove non-Flaky media layout tests and update bugs that
are associated with media layout tests
https://bugs.webkit.org/show_bug.cgi?id=53948

  • platform/chromium/test_expectations.txt:
11:07 PM Changeset in webkit [77908] by mjs@apple.com
  • 16 edits
    2 adds in trunk

2011-02-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Dan Bernstein.

Add resource load client for injected bundle and move willSendRequest there
https://bugs.webkit.org/show_bug.cgi?id=53972

  • GNUmakefile.am:
  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp: (WKBundlePageSetPageLoaderClient): (WKBundlePageSetResourceLoadClient):
  • WebProcess/InjectedBundle/API/c/WKBundlePage.h:
  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h:
  • WebProcess/InjectedBundle/InjectedBundlePageResourceLoadClient.cpp: Added. (WebKit::InjectedBundlePageResourceLoadClient::didInitiateLoadForResource): (WebKit::InjectedBundlePageResourceLoadClient::willSendRequestForFrame): (WebKit::InjectedBundlePageResourceLoadClient::didReceiveResponseForResource): (WebKit::InjectedBundlePageResourceLoadClient::didReceiveContentLengthForResource): (WebKit::InjectedBundlePageResourceLoadClient::didFinishLoadForResource): (WebKit::InjectedBundlePageResourceLoadClient::didFailLoadForResource):
  • WebProcess/InjectedBundle/InjectedBundlePageResourceLoadClient.h: Added.
  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp: (WebKit::WebFrameLoaderClient::assignIdentifierToInitialRequest): (WebKit::WebFrameLoaderClient::dispatchWillSendRequest): (WebKit::WebFrameLoaderClient::dispatchDidReceiveResponse): (WebKit::WebFrameLoaderClient::dispatchDidReceiveContentLength): (WebKit::WebFrameLoaderClient::dispatchDidFinishLoading): (WebKit::WebFrameLoaderClient::dispatchDidFailLoading):
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::initializeInjectedBundleResourceLoadClient):
  • WebProcess/WebPage/WebPage.h: (WebKit::WebPage::injectedBundleResourceLoadClient):
  • win/WebKit2.vcproj:

2011-02-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Dan Bernstein.

Add resource load client for injected bundle and move willSendRequest there
https://bugs.webkit.org/show_bug.cgi?id=53972

  • MiniBrowser/mac/WebBundle/WebBundleMain.m: (didCreatePage):
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::InjectedBundlePage::InjectedBundlePage): (WTR::InjectedBundlePage::didInitiateLoadForResource): (WTR::InjectedBundlePage::didReceiveResponseForResource): (WTR::InjectedBundlePage::didReceiveContentLengthForResource): (WTR::InjectedBundlePage::didFinishLoadForResource): (WTR::InjectedBundlePage::didFailLoadForResource): (WTR::InjectedBundlePage::willSendRequestForFrame):
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:
11:07 PM Changeset in webkit [77907] by commit-queue@webkit.org
  • 2 edits
    1 add in trunk/Source/WebCore

2011-02-07 Xiyuan Xia <xiyuan@chromium.org>

Reviewed by Tony Chang.

[Chromium] Select popup box does not align with select button when appearance is not "menulist".
https://bugs.webkit.org/show_bug.cgi?id=53942

Added manual test select_webkit_appearance_off_popup_alignment.html.

  • manual-tests/select_webkit_appearance_off_popup_alignment.html: Added.
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::layout):
10:59 PM Changeset in webkit [77906] by commit-queue@webkit.org
  • 4 edits in trunk/Source

2011-02-07 Aparna Nandyal <aparna.nand@wipro.com>

Reviewed by Andreas Kling.

[Qt] - Context menu needs "Copy" as well when the selected text is a link.
https://bugs.webkit.org/show_bug.cgi?id=40983

Added copy option for Qt platform if selection is a link.
Added a test in tst_qwebpage.cpp.

  • page/ContextMenuController.cpp: (WebCore::ContextMenuController::populate):

2011-02-07 Aparna Nandyal <aparna.nand@wipro.com>

Reviewed by Andreas Kling.

[Qt] - Context menu needs "Copy" as well when the selected text is a link.
https://bugs.webkit.org/show_bug.cgi?id=40983

Added a test case to create a context sensitive menu.
A check is made if it has Copy menu item when user
selects a link and opens context menu.

  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::contextMenuCopy):
10:48 PM Changeset in webkit [77905] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-02-07 Mike Reed <reed@google.com>

Reviewed by Kenneth Russell.

Add compile-time flag ENABLE(SKIA_GPU) to use Skia's gpu backend
when drawing <canvas>. Disabled by default.

No new tests. existing canvas layouttests apply

  • platform/graphics/chromium/DrawingBufferChromium.cpp: (WebCore::DrawingBuffer::publishToPlatformLayer):
  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::GetGlobalGrContext): (WebCore::PlatformContextSkia::isPrinting): (WebCore::PlatformContextSkia::setSharedGraphicsContext3D): (WebCore::PlatformContextSkia::syncSoftwareCanvas):
10:45 PM Changeset in webkit [77904] by jcivelli@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-02-07 Jay Civelli <jcivelli@chromium.org>

Reviewed by Darin Fisher.

Removing unused code from PopupMenuChromium.
https://bugs.webkit.org/show_bug.cgi?id=53866

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::layout): (WebCore::PopupMenuChromium::show):
    • platform/chromium/PopupMenuChromium.h:
10:39 PM Changeset in webkit [77903] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-02-07 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

bring back diff context lines into the review tool
https://bugs.webkit.org/show_bug.cgi?id=53974

  • code-review.js:
10:30 PM Changeset in webkit [77902] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2011-02-07 Kenichi Ishibashi <bashi@google.com>

Reviewed by Andreas Kling.

HTMLOutputElement::childrenChanged() should call its base class childrenChanged()
https://bugs.webkit.org/show_bug.cgi?id=53791

Test that a live list is updated after the child of an HTMLOutputElement was removed.

  • fast/dom/HTMLOutputElement/htmloutputelement-children-removed-expected.txt: Added.
  • fast/dom/HTMLOutputElement/htmloutputelement-children-removed.html: Added.

2011-02-07 Kenichi Ishibashi <bashi@chromium.org>

Reviewed by Andreas Kling.

HTMLOutputElement::childrenChanged() should call its base class childrenChanged()
https://bugs.webkit.org/show_bug.cgi?id=53791

Calls HTMLFormControlElement::childrenChanged() in HTMLOutputElement::childrenChanged().

Test: fast/dom/HTMLOutputElement/htmloutputelement-children-removed.html

  • html/HTMLOutputElement.cpp: (WebCore::HTMLOutputElement::childrenChanged): Calls super class's childrenChanged().
10:27 PM Changeset in webkit [77901] by commit-queue@webkit.org
  • 6 edits in trunk/Source

2011-02-07 Benjamin Poulain <ikipou@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] cookiesEnabled could create the cookieJar on the wrong thread.
https://bugs.webkit.org/show_bug.cgi?id=52289

Use the QtNAMThreadSafeProxy to make sure the cookie jar is created
in the same thread as the QNetworkAccessManager.

  • platform/network/qt/QtNAMThreadSafeProxy.cpp: (WebCore::QtNAMThreadSafeProxy::QtNAMThreadSafeProxy): (WebCore::QtNAMThreadSafeProxy::hasCookieJar):
  • platform/network/qt/QtNAMThreadSafeProxy.h:
  • platform/qt/CookieJarQt.cpp: (WebCore::cookiesEnabled):

2011-02-07 Benjamin Poulain <ikipou@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] cookiesEnabled could create the cookieJar on the wrong thread.
https://bugs.webkit.org/show_bug.cgi?id=52289

Test if the cookie jar is created in the right thread when accessed from
the navigator object.

  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::navigatorCookieEnabledForNetworkAccessManagerOnDifferentThread):
10:25 PM Changeset in webkit [77900] by commit-queue@webkit.org
  • 9 edits in trunk/Source

2011-02-07 Alexey Marinichev <amarinichev@chromium.org>

Reviewed by Kenneth Russell.

Hook up WebGraphicsContext3D::setContextLostCallback.
https://bugs.webkit.org/show_bug.cgi?id=53722

Test: fast/canvas/webgl/context-lost.html, comment out the line with
extension.loseContext() and kill the GPU process "Lose context"
message shows up.

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContextLostCallback::WebGLRenderingContextLostCallback): (WebCore::WebGLRenderingContextLostCallback::onContextLost): (WebCore::WebGLRenderingContextLostCallback::~WebGLRenderingContextLostCallback): (WebCore::WebGLRenderingContext::initializeNewContext): (WebCore::WebGLRenderingContext::~WebGLRenderingContext):
  • platform/graphics/GraphicsContext3D.h: (WebCore::GraphicsContext3D::ContextLostCallback::~ContextLostCallback):
  • platform/graphics/mac/GraphicsContext3DMac.mm: (WebCore::GraphicsContext3D::setContextLostCallback): stub
  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3D::setContextLostCallback): stub

2011-02-07 Alexey Marinichev <amarinichev@chromium.org>

Reviewed by Kenneth Russell.

Hook up WebGraphicsContext3D::setContextLostCallback.
https://bugs.webkit.org/show_bug.cgi?id=53722

Test: fast/canvas/webgl/context-lost.html, comment out the line with
extension.loseContext() and kill the GPU process "Lose context"
message shows up.

  • public/WebGraphicsContext3D.h: (WebKit::WebGraphicsContext3D::WebGraphicsContextLostCallback::~WebGraphicsContextLostCallback):
  • src/GraphicsContext3DChromium.cpp: (WebCore::GraphicsContextLostCallbackAdapter::~GraphicsContextLostCallbackAdapter): (WebCore::GraphicsContextLostCallbackAdapter::GraphicsContextLostCallbackAdapter): (WebCore::GraphicsContextLostCallbackAdapter::onContextLost): (WebCore::GraphicsContextLostCallbackAdapter::create): (WebCore::GraphicsContext3DInternal::setContextLostCallback):
  • src/GraphicsContext3DInternal.h:
10:24 PM Changeset in webkit [77899] by mrowe@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix the 32-bit build.

  • UIProcess/cf/WebBackForwardListCF.cpp:

(WebKit::WebBackForwardList::restoreFromCFDictionaryRepresentation): Add some casts
to prevent warnings about mixing signed and unsigned comparisons.

10:11 PM Changeset in webkit [77898] by abarth@webkit.org
  • 3 edits in trunk/Tools

2011-02-07 Adam Barth <abarth@webkit.org>

Add Leopard Debug back to the core builders.

  • Scripts/webkitpy/common/net/buildbot/buildbot.py:
  • Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
10:08 PM Changeset in webkit [77897] by commit-queue@webkit.org
  • 4 edits in trunk/LayoutTests

2011-02-07 Ami Fischman <fischman@google.com>

Reviewed by Eric Seidel.

Re-baseline media/video-zoom.html for http://crbug.com/70252 aka
https://bugs.webkit.org/show_bug.cgi?id=53737 to account for (very)
slightly different scrollbar shading.
This test isn't flaky on LINUX or MAC according to
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&showAllRuns=true&master=ChromiumWebkit&tests=media%2Fvideo-zoom.html
so take it out of test_expectations.txt; it passes 100% on LINUX and
fails 100% on MAC, and should pass 100% on MAC with this new
baseline.

  • platform/chromium-mac/media/video-zoom-expected.checksum:
  • platform/chromium-mac/media/video-zoom-expected.png:
  • platform/chromium/test_expectations.txt:
10:05 PM Changeset in webkit [77896] by commit-queue@webkit.org
  • 5 edits in trunk

2011-02-07 Adam Klein <adamk@chromium.org>

Reviewed by Jian Li.

[chromium] Remove deprecated method WebNotification::dir
https://bugs.webkit.org/show_bug.cgi?id=53735

  • public/WebNotification.h:
  • src/WebNotification.cpp:

2011-02-07 Adam Klein <adamk@chromium.org>

Reviewed by Jian Li.

[chromium] Remove deprecated method WebNotification::dir
https://bugs.webkit.org/show_bug.cgi?id=53735

  • DumpRenderTree/chromium/NotificationPresenter.cpp: (NotificationPresenter::show):
10:02 PM Changeset in webkit [77895] by Patrick Gansterer
  • 2 edits in trunk/Source/WebCore

2011-02-07 Patrick Gansterer <Patrick Gansterer>

Reviewed by Eric Seidel.

Add missing PaintInfo include in RenderThemeWin.cpp
https://bugs.webkit.org/show_bug.cgi?id=53885

  • rendering/RenderThemeWin.cpp:
9:39 PM Changeset in webkit [77894] by jamesr@google.com
  • 1 edit
    3 moves in trunk/LayoutTests

2011-02-07 James Robinson <jamesr@chromium.org>

Move sputnik expectations to match test movement.

  • platform/gtk/fast/js/sputnik: Removed.
  • platform/gtk/fast/js/sputnik/Unicode: Removed.
  • platform/gtk/fast/js/sputnik/Unicode/Unicode_510: Removed.
  • platform/gtk/fast/js/sputnik/Unicode/Unicode_510/S7.6_A3.2-expected.txt: Removed.
  • platform/gtk/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.3_T1-expected.txt: Removed.
  • platform/gtk/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.3_T2-expected.txt: Removed.
  • platform/gtk/sputnik: Copied from LayoutTests/platform/gtk/fast/js/sputnik.
  • platform/mac-leopard/fast/js/sputnik: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_500: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_500/S7.6_A3.1-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_500/S7.6_A3.2-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_500/S7.6_A5.3_T1-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_500/S7.6_A5.3_T2-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S15.5.4.16_A1-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S15.5.4.18_A1-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A1.1_T1-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A1.1_T2-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A1.1_T4-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A2.2_T1-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A2.2_T2-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A2.3-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.2_T1-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.2_T2-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.2_T4-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.2_T7-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.2_T8-expected.txt: Removed.
  • platform/mac-leopard/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.2_T9-expected.txt: Removed.
  • platform/mac-leopard/sputnik: Copied from LayoutTests/platform/mac-leopard/fast/js/sputnik.
  • platform/win/fast/js/sputnik: Removed.
  • platform/win/fast/js/sputnik/Conformance: Removed.
  • platform/win/fast/js/sputnik/Conformance/11_Expressions: Removed.
  • platform/win/fast/js/sputnik/Conformance/11_Expressions/11.5_Multiplicative_Operators: Removed.
  • platform/win/fast/js/sputnik/Conformance/11_Expressions/11.5_Multiplicative_Operators/11.5.3_Percent: Removed.
  • platform/win/fast/js/sputnik/Conformance/11_Expressions/11.5_Multiplicative_Operators/11.5.3_Percent/S11.5.3_A4_T6-expected.txt: Removed.
  • platform/win/fast/js/sputnik/Unicode: Removed.
  • platform/win/fast/js/sputnik/Unicode/Unicode_510: Removed.
  • platform/win/fast/js/sputnik/Unicode/Unicode_510/S7.6_A3.1-expected.txt: Removed.
  • platform/win/fast/js/sputnik/Unicode/Unicode_510/S7.6_A3.2-expected.txt: Removed.
  • platform/win/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.3_T1-expected.txt: Removed.
  • platform/win/fast/js/sputnik/Unicode/Unicode_510/S7.6_A5.3_T2-expected.txt: Removed.
  • platform/win/sputnik: Copied from LayoutTests/platform/win/fast/js/sputnik.
9:37 PM Changeset in webkit [77893] by commit-queue@webkit.org
  • 37 edits
    38 deletes in trunk/LayoutTests

2011-02-07 Ami Fischman <fischman@google.com>

Reviewed by Eric Carlson.

Removed two of the three almost-duplicate test expectations
between {media,platform/chromium-{mac,win}} by excluding file
extensions from the emitted output.
Removed outdated test_expectation for deleted test.
http://crbug.com/28993

  • media/controls-right-click-on-timebar-expected.txt:
  • media/controls-right-click-on-timebar.html:
  • media/media-file.js: (stripExtension):
  • media/media-startTime-expected.txt:
  • media/media-startTime.html:
  • media/video-append-source-expected.txt:
  • media/video-append-source.html:
  • media/video-canvas-source-expected.txt:
  • media/video-canvas-source.html:
  • media/video-dom-src-expected.txt:
  • media/video-dom-src.html:
  • media/video-load-networkState-expected.txt:
  • media/video-load-networkState.html:
  • media/video-played-reset-expected.txt:
  • media/video-played-reset.html:
  • media/video-seek-past-end-paused-expected.txt:
  • media/video-seek-past-end-paused.html:
  • media/video-seek-past-end-playing-expected.txt:
  • media/video-seek-past-end-playing.html:
  • media/video-source-expected.txt:
  • media/video-source-type-expected.txt:
  • media/video-source-type-params-expected.txt:
  • media/video-source-type-params.html:
  • media/video-source-type.html:
  • media/video-source.html:
  • media/video-src-change-expected.txt:
  • media/video-src-change.html:
  • media/video-src-expected.txt:
  • media/video-src-remove-expected.txt:
  • media/video-src-remove.html:
  • media/video-src-set-expected.txt:
  • media/video-src-set.html:
  • media/video-src-source-expected.txt:
  • media/video-src-source.html:
  • media/video-src.html:
  • media/video-test.js: (enableFullTestDetailsPrinting): (test): (testExpected): (waitForEvent._eventCallback): (waitForEvent): (waitForEventAndTest._eventCallback): (waitForEventAndTest): (testException): (endTest):
  • platform/chromium-mac/media/controls-right-click-on-timebar-expected.txt: Removed.
  • platform/chromium-mac/media/media-startTime-expected.txt: Removed.
  • platform/chromium-mac/media/video-append-source-expected.txt: Removed.
  • platform/chromium-mac/media/video-canvas-source-expected.txt: Removed.
  • platform/chromium-mac/media/video-dom-src-expected.txt: Removed.
  • platform/chromium-mac/media/video-load-networkState-expected.txt: Removed.
  • platform/chromium-mac/media/video-played-reset-expected.txt: Removed.
  • platform/chromium-mac/media/video-seek-past-end-paused-expected.txt: Removed.
  • platform/chromium-mac/media/video-seek-past-end-playing-expected.txt: Removed.
  • platform/chromium-mac/media/video-source-add-src-expected.txt: Removed.
  • platform/chromium-mac/media/video-source-error-expected.txt: Removed.
  • platform/chromium-mac/media/video-source-expected.txt: Removed.
  • platform/chromium-mac/media/video-source-type-expected.txt: Removed.
  • platform/chromium-mac/media/video-source-type-params-expected.txt: Removed.
  • platform/chromium-mac/media/video-src-change-expected.txt: Removed.
  • platform/chromium-mac/media/video-src-expected.txt: Removed.
  • platform/chromium-mac/media/video-src-remove-expected.txt: Removed.
  • platform/chromium-mac/media/video-src-set-expected.txt: Removed.
  • platform/chromium-mac/media/video-src-source-expected.txt: Removed.
  • platform/chromium-win/media/controls-right-click-on-timebar-expected.txt: Removed.
  • platform/chromium-win/media/media-startTime-expected.txt: Removed.
  • platform/chromium-win/media/video-append-source-expected.txt: Removed.
  • platform/chromium-win/media/video-canvas-source-expected.txt: Removed.
  • platform/chromium-win/media/video-dom-src-expected.txt: Removed.
  • platform/chromium-win/media/video-load-networkState-expected.txt: Removed.
  • platform/chromium-win/media/video-played-reset-expected.txt: Removed.
  • platform/chromium-win/media/video-seek-past-end-paused-expected.txt: Removed.
  • platform/chromium-win/media/video-seek-past-end-playing-expected.txt: Removed.
  • platform/chromium-win/media/video-source-add-src-expected.txt: Removed.
  • platform/chromium-win/media/video-source-error-expected.txt: Removed.
  • platform/chromium-win/media/video-source-expected.txt: Removed.
  • platform/chromium-win/media/video-source-type-expected.txt: Removed.
  • platform/chromium-win/media/video-source-type-params-expected.txt: Removed.
  • platform/chromium-win/media/video-src-change-expected.txt: Removed.
  • platform/chromium-win/media/video-src-expected.txt: Removed.
  • platform/chromium-win/media/video-src-remove-expected.txt: Removed.
  • platform/chromium-win/media/video-src-set-expected.txt: Removed.
  • platform/chromium-win/media/video-src-source-expected.txt: Removed.
  • platform/chromium/test_expectations.txt:
9:37 PM Changeset in webkit [77892] by commit-queue@webkit.org
  • 14 edits in trunk/Source

2011-02-07 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Andreas Kling.

[Qt] enum QWebPage::WebAction doesn't have value QWebPage::CopyImageUrl
https://bugs.webkit.org/show_bug.cgi?id=52974

Add ContextMenuItemTagCopyImageUrlToClipboard into standard action but Qt platform specific.

  • page/ContextMenuController.cpp: (WebCore::ContextMenuController::contextMenuItemSelected): (WebCore::ContextMenuController::populate): (WebCore::ContextMenuController::checkOrEnableIfNeeded):
  • platform/ContextMenuItem.h:
  • platform/LocalizationStrategy.h:
  • platform/LocalizedStrings.cpp: (WebCore::contextMenuItemTagCopyImageUrlToClipboard):
  • platform/LocalizedStrings.h:

2011-02-07 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Andreas Kling.

[Qt] enum QWebPage::WebAction doesn't have value QWebPage::CopyImageUrl
https://bugs.webkit.org/show_bug.cgi?id=52974

Add QWebPage::CopyImageUrlToClipboard into QWebPage::WebAction.

  • Api/qwebpage.cpp: (webActionForContextMenuAction): (QWebPage::triggerAction): (QWebPage::action):
  • Api/qwebpage.h:
  • WebCoreSupport/WebPlatformStrategies.cpp: (WebPlatformStrategies::contextMenuItemTagCopyImageUrlToClipboard):
  • WebCoreSupport/WebPlatformStrategies.h:

2011-02-07 Qi Zhang <qi.2.zhang@nokia.com>

Reviewed by Andreas Kling.

[Qt] enum QWebPage::WebAction doesn't have value QWebPage::CopyImageUrl
https://bugs.webkit.org/show_bug.cgi?id=52974

Add ContextMenuItemTagCopyImageUrlToClipboard into standard action but Qt platform specific.

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp: (WebKit::WebPlatformStrategies::contextMenuItemTagCopyImageUrlToClipboard):
  • WebProcess/WebCoreSupport/WebPlatformStrategies.h:
9:28 PM Changeset in webkit [77891] by tkent@chromium.org
  • 4 edits
    2 adds in trunk

2011-02-07 Kenichi Ishibashi <bashi@chromium.org>

Reviewed by Kent Tamura.

HTMLFormElement::formElementIndex() returns a bad index into a vector of form associated elements
https://bugs.webkit.org/show_bug.cgi?id=53970

Adds HTMLObjectElement::virtualForm() to make HTMLObjectElement::form()
return the right value.

Test: fast/dom/HTMLObjectElement/form/nested-form-element.html

  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::virtualForm): Added.
  • html/HTMLObjectElement.h: Added virtualForm().

2011-02-07 Kenichi Ishibashi <bashi@chromium.org>

Reviewed by Kent Tamura.

HTMLFormElement::formElementIndex() returns a bad index into a vector of form associated elements
https://bugs.webkit.org/show_bug.cgi?id=53970

Adds a test that ensures WebKit parses successfully a page that
contains object elements which are in nested and defective form
elements.

  • fast/dom/HTMLObjectElement/form/nested-form-element-expected.txt: Added.
  • fast/dom/HTMLObjectElement/form/nested-form-element.html: Added.
8:39 PM Changeset in webkit [77890] by abarth@webkit.org
  • 3 edits in trunk/Tools

2011-02-07 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Remove Leopard Debug as a core builder
https://bugs.webkit.org/show_bug.cgi?id=53971

The Leopard Debug builder has been failing to compile for several days.
My understanding is that folks are working on fixing the build, but for
the time being we should probably remove it from the list of core
builders so that the core waterfall can be green. We can certainly add
it back once the build is fixed.

Also, I've removed the Tiger builders because those no longer appear to
be attached to the buildbot master.

  • Scripts/webkitpy/common/net/buildbot/buildbot.py:
  • Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
8:29 PM Changeset in webkit [77889] by ojan@chromium.org
  • 5 edits in trunk

2011-02-01 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Roben.

include svn revisions in git diffs for the code review tool to use
https://bugs.webkit.org/show_bug.cgi?id=53569

  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/checkout/scm_unittest.py:

2011-02-01 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Roben.

include svn revisions in git diffs for the code review tool to use
https://bugs.webkit.org/show_bug.cgi?id=53569

  • PrettyPatch/PrettyPatch.rb:
8:27 PM Changeset in webkit [77888] by rniwa@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-02-07 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed Leopard build fix.

  • WebCore.xcodeproj/project.pbxproj: let XCode have developmentRegion.
  • platform/mac/DragImageMac.mm: (WebCore::webkit_CGCeiling): Cast the return value of ceil to CGFloat.
8:21 PM Changeset in webkit [77887] by yuzo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Yuzo Fujishima <yuzo@google.com>

Unreviewed attempt to fix a flaky test.
https://bugs.webkit.org/show_bug.cgi?id=51757

  • fast/css/font-face-download-error.html:
8:07 PM Changeset in webkit [77886] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
7:55 PM Changeset in webkit [77885] by jamesr@google.com
  • 8 edits
    1 move in trunk/LayoutTests

2011-02-07 James Robinson <jamesr@chromium.org>

Reviewed by Dan Bernstein.

Move the sputnik test suite out of fast/js/
https://bugs.webkit.org/show_bug.cgi?id=53954

The sputnik suite shouldn't go in fast/js/ because it is not fast and
it is an imported test suite. This moves it to LayoutTests/sputnik
and updates references in skipped lists / expectations.

  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac-tiger/Skipped:
  • platform/mac-wk2/Skipped:
  • platform/qt-wk2/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
7:37 PM Changeset in webkit [77884] by rniwa@webkit.org
  • 1 edit in trunk/Source/JavaScriptCore/ChangeLog

Fixing the change log entry.

7:34 PM Changeset in webkit [77883] by rniwa@webkit.org
  • 5 edits in trunk/Source/JavaScriptCore

2011-02-07 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Removed some dead code from Heap
https://bugs.webkit.org/show_bug.cgi?id=53969


SunSpider reports no change.

  • runtime/MarkedSpace.cpp: (JSC::MarkedSpace::shrink): (JSC::MarkedSpace::sweep):
  • runtime/MarkedSpace.h: Removed resizeBlocks and growBlocks, and renamed shrinkBlocks to shrink, making it unconditionally shrink as much as possible.
7:13 PM Changeset in webkit [77882] by enrica@apple.com
  • 4 edits in trunk/Source

Source/WebCore: Fixing a build break on Mac caused by http://trac.webkit.org/changeset/77874

Reviewed by Sam Weinig.

When moving code from WebKit to WebCore we needed to move some additional
functions previously in NSStringExtras.

  • WebCore.xcodeproj/project.pbxproj:
  • platform/mac/DragImageMac.mm:

(WebCore::fontFromNSFont):
(WebCore::canUseFastRenderer):
(WebCore::widthWithFont):
(WebCore::webkit_CGCeiling):
(WebCore::drawAtPoint):
(WebCore::drawDoubledAtPoint):
(WebCore::createDragImageForLink):

Source/WebKit2: Fixing a build break on Mac caused by http://trac.webkit.org/changeset/77874

Reviewed by Sam Weinig.

Removed unused static function.

  • WebProcess/WebCoreSupport/mac/WebDragClientMac.mm:
7:08 PM Changeset in webkit [77881] by ggaren@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

2011-02-07 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Removed some dead code from Heap
https://bugs.webkit.org/show_bug.cgi?id=53969


SunSpider reports no change.

  • runtime/MarkedSpace.cpp: (JSC::MarkedSpace::shrink): (JSC::MarkedSpace::sweep):
  • runtime/MarkedSpace.h: Removed resizeBlocks and growBlocks, and renamed shrinkBlocks to shrink, making it unconditionally shrink as much as possible.
6:21 PM Changeset in webkit [77880] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/8969888>
The WebProcess should set LSFileQuarantineEnabled to true

Reviewed by Sam Weinig.

  • WebProcess/Info.plist:
6:13 PM Changeset in webkit [77879] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
6:04 PM Changeset in webkit [77878] by Lucas Forschler
  • 3 edits in trunk/Source/WebCore

2011-02-07 Lucas Forschler <Lucas Forschler>

Reviewed by Geoff Garen.

Tr #2, Fix the Leopard 32bit debug build.


No new tests required.

  • WebCore.xcodeproj/project.pbxproj:
  • accessibility/AccessibilityAllInOne.cpp:
5:58 PM Changeset in webkit [77877] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

2011-02-07 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Simplified the marked space's mark invariant
https://bugs.webkit.org/show_bug.cgi?id=53968


SunSpider reports no change.


  • runtime/MarkedSpace.cpp: (JSC::MarkedSpace::allocate): Mark objects when allocating them. This means that, at all times other than the mark phase, an object is live if and only if it is marked.

(JSC::MarkedSpace::containsSlowCase): Use the new mark invariant to
simplify testing whether an object is live.

5:53 PM Changeset in webkit [77876] by Beth Dakin
  • 3 edits in trunk/Source/WebCore

Fix for <rdar://problem/8946961> Fix repaint problems
with WKScrollbarPainter scrollers while resizing the window

Reviewed by Sam Weinig.

Remove setPainterForPainterController() and
removePainterFromPainterController() since they aren't
used anymore.

  • platform/mac/ScrollAnimatorMac.h:


Return early if they alpha hasn't changed.

  • platform/mac/ScrollAnimatorMac.mm:

(-[ScrollbarPainterDelegate scrollerImp:animateKnobAlphaTo:duration:]):
(-[ScrollbarPainterDelegate scrollerImp:animateTrackAlphaTo:duration:]):

If we're in a live resize, restore alpha to 1.
(WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):
(WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):

5:47 PM Changeset in webkit [77875] by morrita@google.com
  • 15 edits
    4 adds in trunk

2011-02-03 MORITA Hajime <morrita@google.com>

Reviewed by Darin Fisher.

[Chromium] Should implement EditorClientImpl::requestCheckingOfString()
https://bugs.webkit.org/show_bug.cgi?id=51013

Unskipped passed test.

  • platform/chromium/test_expectations.txt:

2011-02-03 MORITA Hajime <morrita@google.com>

Reviewed by Darin Fisher.

[Chromium] Should implement EditorClientImpl::requestCheckingOfString()
https://bugs.webkit.org/show_bug.cgi?id=51013

  • Added WebTextCheckingCompletion to receive the checked result asynchronously.
  • Added WebViewClient::requestCheckingOfText() and related classes.
  • Implemented EditorClientImpl::requestCheckingOfString().
  • WebKit.gyp:
  • public/WebSettings.h:
  • public/WebTextCheckingCompletion.h: Added. (WebKit::WebTextCheckingCompletion::~WebTextCheckingCompletion):
  • public/WebTextCheckingResult.h: Added. (WebKit::WebTextCheckingResult::error): (WebKit::WebTextCheckingResult::position): (WebKit::WebTextCheckingResult::length): (WebKit::WebTextCheckingResult::WebTextCheckingResult):
  • public/WebViewClient.h: (WebKit::WebViewClient::requestCheckingOfText): Gave an implementation
  • src/AssertMatchingEnums.cpp:
  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::requestCheckingOfString):
  • src/EditorClientImpl.h:
  • src/WebSettingsImpl.cpp: (WebKit::WebSettingsImpl::setAsynchronousSpellCheckingEnabled):
  • src/WebSettingsImpl.h:
  • src/WebTextCheckingCompletionImpl.cpp: Added. (WebKit::toCoreResults): (WebKit::WebTextCheckingCompletionImpl::didFinishCheckingText):
  • src/WebTextCheckingCompletionImpl.h: Added. (WebKit::WebTextCheckingCompletionImpl::WebTextCheckingCompletionImpl):

2011-02-03 MORITA Hajime <morrita@google.com>

Reviewed by Darin Fisher.

[Chromium] Should implement EditorClientImpl::requestCheckingOfString()
https://bugs.webkit.org/show_bug.cgi?id=51013

Gave DRT implementation for requestTextCheck().

  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::setAsynchronousSpellCheckingEnabled): Implemented.
  • DumpRenderTree/chromium/WebViewHost.cpp: (invokeFinishLastTextCheck): Added. (WebViewHost::requestTextCheck): Added. (WebViewHost::finishLastTextCheck): Added.
  • DumpRenderTree/chromium/WebViewHost.h:
5:44 PM Changeset in webkit [77874] by andersca@apple.com
  • 8 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Crash in web process after the connection had been closed
https://bugs.webkit.org/show_bug.cgi?id=51115
<rdar://problem/8964255>

Remove didCloseOnConnectionWorkQueue from Connection::Client and add
Connection::setDidCloseOnConnectionWorkQueueCallback which takes a static
function, eliminating the race condition in connectionDidClose where m_client could be
nulled out in the client thread right before we try to dereference it.

  • Platform/CoreIPC/Connection.cpp: (CoreIPC::Connection::Connection): Initialize m_didCloseOnConnectionWorkQueueCallback.

(CoreIPC::Connection::setDidCloseOnConnectionWorkQueueCallback):
Update m_didCloseOnConnectionWorkQueueCallback.

(CoreIPC::Connection::connectionDidClose):
Call m_didCloseOnConnectionWorkQueueCallback.

  • Platform/WorkQueue.h: Make WorkQueue noncopyable.
  • PluginProcess/PluginProcess.cpp: (WebKit::PluginProcess::initializeConnection): Call setDidCloseOnConnectionWorkQueueCallback.
  • Shared/ChildProcess.cpp: (WebKit::ChildProcess::didCloseOnConnectionWorkQueue):
  • Shared/ChildProcess.h: didCloseOnConnectionWorkQueue is now a static member function.
  • WebProcess/WebProcess.cpp: (WebKit::WebProcess::initialize): Call setDidCloseOnConnectionWorkQueueCallback.
5:39 PM Changeset in webkit [77873] by Martin Robinson
  • 2 edits
    59 adds in trunk/LayoutTests

2011-02-07 Martin Robinson <mrobinson@igalia.com>

Classify tests failing in the fast directory on GTK+. Add results for
tests that just need platform expectations.

  • platform/gtk/Skipped: Update the skipped list.
  • platform/gtk/fast/backgrounds/background-clip-text-expected.checksum: Added.
  • platform/gtk/fast/backgrounds/background-clip-text-expected.png: Added.
  • platform/gtk/fast/backgrounds/background-clip-text-expected.txt: Added.
  • platform/gtk/fast/canvas/canvas-zoom-expected.txt: Added.
  • platform/gtk/fast/css/css1_forward_compatible_parsing-expected.checksum: Added.
  • platform/gtk/fast/css/css1_forward_compatible_parsing-expected.png: Added.
  • platform/gtk/fast/css/css1_forward_compatible_parsing-expected.txt: Added.
  • platform/gtk/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt: Added.
  • platform/gtk/fast/css/invalid-percentage-property-expected.checksum: Added.
  • platform/gtk/fast/css/invalid-percentage-property-expected.png: Added.
  • platform/gtk/fast/css/invalid-percentage-property-expected.txt: Added.
  • platform/gtk/fast/css/text-align-expected.checksum: Added.
  • platform/gtk/fast/css/text-align-expected.png: Added.
  • platform/gtk/fast/css/text-align-expected.txt: Added.
  • platform/gtk/fast/dom/Element/getBoundingClientRect-expected.txt: Added.
  • platform/gtk/fast/dom/Range/getBoundingClientRect-expected.txt: Added.
  • platform/gtk/fast/dom/Range/getClientRects-expected.txt: Added.
  • platform/gtk/fast/dom/Window/webkitConvertPoint-expected.txt: Added.
  • platform/gtk/fast/dom/null-document-location-put-crash-expected.txt: Added.
  • platform/gtk/fast/dom/tabindex-clamp-expected.txt: Added.
  • platform/gtk/fast/events/js-keyboard-event-creation-expected.txt: Added.
  • platform/gtk/fast/events/offsetX-offsetY-expected.txt: Added.
  • platform/gtk/fast/events/pointer-events-2-expected.checksum: Added.
  • platform/gtk/fast/events/pointer-events-2-expected.png: Added.
  • platform/gtk/fast/events/pointer-events-2-expected.txt: Added.
  • platform/gtk/fast/fast-mobile-scrolling/fixed-position-element-expected.checksum: Added.
  • platform/gtk/fast/fast-mobile-scrolling/fixed-position-element-expected.png: Added.
  • platform/gtk/fast/fast-mobile-scrolling/fixed-position-element-expected.txt: Added.
  • platform/gtk/fast/fast-mobile-scrolling/no-fixed-position-elements-expected.checksum: Added.
  • platform/gtk/fast/fast-mobile-scrolling/no-fixed-position-elements-expected.png: Added.
  • platform/gtk/fast/fast-mobile-scrolling/no-fixed-position-elements-expected.txt: Added.
  • platform/gtk/fast/forms/listbox-hit-test-zoomed-expected.checksum: Added.
  • platform/gtk/fast/forms/listbox-hit-test-zoomed-expected.png: Added.
  • platform/gtk/fast/forms/listbox-hit-test-zoomed-expected.txt: Added.
  • platform/gtk/fast/html/tab-order-expected.txt: Added.
  • platform/gtk/fast/replaced/table-percent-height-expected.txt: Added.
  • platform/gtk/fast/selectors/lang-inheritance-expected.checksum: Added.
  • platform/gtk/fast/selectors/lang-inheritance-expected.png: Added.
  • platform/gtk/fast/selectors/lang-inheritance-expected.txt: Added.
  • platform/gtk/fast/selectors/lang-inheritance2-expected.checksum: Added.
  • platform/gtk/fast/selectors/lang-inheritance2-expected.png: Added.
  • platform/gtk/fast/selectors/lang-inheritance2-expected.txt: Added.
  • platform/gtk/fast/selectors/lang-vs-xml-lang-expected.checksum: Added.
  • platform/gtk/fast/selectors/lang-vs-xml-lang-expected.png: Added.
  • platform/gtk/fast/selectors/lang-vs-xml-lang-expected.txt: Added.
  • platform/gtk/fast/selectors/lang-vs-xml-lang-xhtml-expected.checksum: Added.
  • platform/gtk/fast/selectors/lang-vs-xml-lang-xhtml-expected.png: Added.
  • platform/gtk/fast/selectors/lang-vs-xml-lang-xhtml-expected.txt: Added.
  • platform/gtk/fast/text/international/danda-space-expected.checksum: Added.
  • platform/gtk/fast/text/international/danda-space-expected.png: Added.
  • platform/gtk/fast/text/international/danda-space-expected.txt: Added.
  • platform/gtk/fast/text/international/thai-baht-space-expected.checksum: Added.
  • platform/gtk/fast/text/international/thai-baht-space-expected.png: Added.
  • platform/gtk/fast/text/international/thai-baht-space-expected.txt: Added.
5:38 PM Changeset in webkit [77872] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
5:35 PM Changeset in webkit [77871] by Beth Dakin
  • 4 edits in trunk/Source

Fix for https://bugs.webkit.org/show_bug.cgi?id=53950
USE_WK_SCROLLBAR_PAINTER in ScrollbarThemeMac should be in
Platform.h instead

Reviewed by Eric Seidel.

../JavaScriptCore:

  • wtf/Platform.h:

../WebCore:

Use hot new USE(WK_SCROLLBAR_PAINTER) from Platform.h rather
than the silly, old in-file #define.

  • platform/mac/ScrollbarThemeMac.mm:

(+[ScrollbarPrefsObserver appearancePrefsChanged:]):
(WebCore::updateArrowPlacement):
(WebCore::ScrollbarThemeMac::registerScrollbar):
(WebCore::ScrollbarThemeMac::scrollbarThickness):
(WebCore::ScrollbarThemeMac::usesOverlayScrollbars):
(WebCore::ScrollbarThemeMac::hasThumb):
(WebCore::ScrollbarThemeMac::minimumThumbLength):
(WebCore::ScrollbarThemeMac::paint):

5:34 PM Changeset in webkit [77870] by enrica@apple.com
  • 47 edits
    3 adds in trunk/Source

Reviewed Adam Roben and Darin Adler.

Source/WebCore:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775
<rdar://problem/8514409>

  • loader/EmptyClients.h: Removed createDragImageFromLink.
  • page/DragClient.h: Ditto.
  • page/DragController.cpp:

(WebCore::DragController::startDrag): changed call to createDragImageFromLink.

  • platform/DragImage.h:
  • platform/mac/DragImageMac.mm:

(WebCore::createDragImageForLink): Moved from DragClient.

  • platform/win/ClipboardUtilitiesWin.cpp: Added a series of functions

to de-serialize the drag data from the map into an IDataObject.
(WebCore::getWebLocData):
(WebCore::texthtmlFormat):
(WebCore::createGlobalData):
(WebCore::getFullCFHTML):
(WebCore::urlWFormat):
(WebCore::urlFormat):
(WebCore::filenameWFormat):
(WebCore::filenameFormat):
(WebCore::htmlFormat):
(WebCore::smartPasteFormat):
(WebCore::getURL):
(WebCore::getPlainText):
(WebCore::getTextHTML):
(WebCore::ClipboardDataItem::ClipboardDataItem):
(WebCore::getStringData):
(WebCore::getUtf8Data):
(WebCore::getCFData):
(WebCore::setUCharData):
(WebCore::setCharData):
(WebCore::setCFData):
(WebCore::getClipboardMap):
(WebCore::getClipboardData):
(WebCore::setClipboardData):

  • platform/win/ClipboardUtilitiesWin.h:
  • platform/win/DragImageWin.cpp: Moved createDragImageForLink from DragClient

to a platform specific file in WebCore. It makes a lot more sense here, since
the drag image is always created in WebCore for all the other content types
except for links.
(WebCore::dragLabelFont):
(WebCore::createDragImageForLink):

  • platform/win/WCDataObject.cpp: Added new way to

construct a WCDataObject from a DragDataMap.
(WebCore::WCDataObject::createInstance):
(WebCore::WCDataObject::clearData):

  • platform/win/WCDataObject.h:

Source/WebKit/chromium:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • src/DragClientImpl.cpp:
  • src/DragClientImpl.h:

Source/WebKit/efl:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebCoreSupport/DragClientEfl.cpp:
  • WebCoreSupport/DragClientEfl.h:

Source/WebKit/gtk:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebCoreSupport/DragClientGtk.cpp:
  • WebCoreSupport/DragClientGtk.h:

Source/WebKit/haiku:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebCoreSupport/DragClientHaiku.cpp:
  • WebCoreSupport/DragClientHaiku.h:

Source/WebKit/mac:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebCoreSupport/WebDragClient.h:
  • WebCoreSupport/WebDragClient.mm:

(WebDragClient::startDrag):

Source/WebKit/qt:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebCoreSupport/DragClientQt.cpp:
  • WebCoreSupport/DragClientQt.h:

Source/WebKit/win:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebCoreSupport/WebDragClient.cpp:

(WebDragClient::startDrag):

  • WebCoreSupport/WebDragClient.h:

Source/WebKit/wince:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebCoreSupport/DragClientWinCE.cpp:
  • WebCoreSupport/DragClientWinCE.h:

Source/WebKit/wx:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775

Removed createDragImageForLink from DragClient.

  • WebKitSupport/DragClientWx.cpp:
  • WebKitSupport/DragClientWx.h:

Source/WebKit2:

WebKit2: drag and drop support on Windows.
https://bugs.webkit.org/show_bug.cgi?id=52775
<rdar://problem/8514409>

This patch contains the remaining work to support
a WebKit window as drag source on Windows.
I've added the code to generate the image for dragging a link as
well as the steps necessary to package the HBITMAP into a SharedMemory
object and send it over to the UI process where DoDragDrop is performed.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::setDragImage):
(WebKit::WebPageProxy::startDragDrop):

  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPageProxy.messages.in:
  • WebProcess/WebCoreSupport/WebDragClient.cpp:

(WebKit::WebDragClient::startDrag): Made startDrag for
platforms other than Windows and Mac.

  • WebProcess/WebCoreSupport/WebDragClient.h:
  • WebProcess/WebCoreSupport/mac/WebDragClientMac.mm:

(WebKit::WebDragClient::startDrag):

  • WebProcess/WebCoreSupport/win/WebDragClientWin.cpp: Added.

(WebKit::draggingSourceOperationMaskToDragCursors):
(WebKit::WebDragClient::startDrag):

  • WebProcess/WebCoreSupport/win/WebDragSource.cpp: Added.

This object implements the IDropSource interface.
(WebDragSource::createInstance):
(WebDragSource::WebDragSource):
(WebDragSource::QueryInterface):
(WebDragSource::AddRef):
(WebDragSource::Release):
(WebDragSource::QueryContinueDrag):
(WebDragSource::GiveFeedback):

  • WebProcess/WebCoreSupport/win/WebDragSource.h: Added.
  • win/WebKit2.vcproj:
5:32 PM Changeset in webkit [77869] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
5:23 PM Changeset in webkit [77868] by joone.hur@collabora.co.uk
  • 8 edits
    1 add in trunk

2011-02-07 Joone Hur <joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[Gtk] Implement layoutTestController.findString
https://bugs.webkit.org/show_bug.cgi?id=50237

Unskip editing/text-iterator/findString.html.

  • platform/gtk/Skipped: Unskip findString test.

2011-02-07 Joone Hur <joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[Gtk] Implement layoutTestController.findString
https://bugs.webkit.org/show_bug.cgi?id=50237

  • GNUmakefile.am: Add AssertMatchingEnums.cpp.
  • WebCoreSupport/AssertMatchingEnums.cpp: Add this file to assert that various WebKit API enum values continue matching WebCore defined enum values.
  • WebCoreSupport/DumpRenderTreeSupportGtk.cpp: (DumpRenderTreeSupportGtk::findString):
  • WebCoreSupport/DumpRenderTreeSupportGtk.h:

2011-02-07 Joone Hur <joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[Gtk] Implement layoutTestController.findString
https://bugs.webkit.org/show_bug.cgi?id=50237

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::findString):
5:02 PM Changeset in webkit [77867] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-02-07 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r77848.
http://trac.webkit.org/changeset/77848
https://bugs.webkit.org/show_bug.cgi?id=53963

caused 6 test failures (Requested by othermaciej on #webkit).

  • rendering/RenderBlockLineLayout.cpp: (WebCore::RenderBlock::computeInlineDirectionPositionsForLine):
5:00 PM Changeset in webkit [77866] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

UI process crashes if web process crashes while web inspector is focused
https://bugs.webkit.org/show_bug.cgi?id=53965
<rdar://problem/8961633>

Make sure to close the web inspector web page proxy before calling platformClose, since
platformClose can cause the web page proxy to try to send messages (such as SetActive)
to the web process while it's in a bad state.

  • UIProcess/WebInspectorProxy.cpp: (WebKit::WebInspectorProxy::didClose):
4:57 PM Changeset in webkit [77865] by kbr@google.com
  • 5 edits
    4 adds in trunk/Source/WebCore

2011-02-04 Kenneth Russell <kbr@google.com>

Reviewed by James Robinson.

Add shaders for GPU accelerated path rendering
https://bugs.webkit.org/show_bug.cgi?id=45520

Refactored SolidFillShader and TexShader to call new
generateVertex() and generateFragment() methods on base Shader
class, and added support for the Loop/Blinn shaders. Refactored
Loop/Blinn shader classes to derive from Shader.

Tested these changes with several of the IE9 Testdrive demos with
accelerated 2D canvas enabled, as well as a full layout test run
with --platform chromium-gpu.

  • platform/graphics/gpu/LoopBlinnShader.cpp: Added. (WebCore::LoopBlinnShader::LoopBlinnShader): (WebCore::LoopBlinnShader::use):
  • platform/graphics/gpu/LoopBlinnShader.h: Added.
  • platform/graphics/gpu/LoopBlinnSolidFillShader.cpp: Added. (WebCore::LoopBlinnSolidFillShader::create): (WebCore::LoopBlinnSolidFillShader::LoopBlinnSolidFillShader): (WebCore::LoopBlinnSolidFillShader::use):
  • platform/graphics/gpu/LoopBlinnSolidFillShader.h: Added.
  • platform/graphics/gpu/Shader.cpp: (WebCore::Shader::affineTo4x4): (WebCore::Shader::loadShader): (WebCore::Shader::loadProgram): (WebCore::Shader::generateVertex): (WebCore::Shader::generateFragment):
  • platform/graphics/gpu/Shader.h:
  • platform/graphics/gpu/SolidFillShader.cpp: (WebCore::SolidFillShader::create):
  • platform/graphics/gpu/TexShader.cpp: (WebCore::TexShader::TexShader): (WebCore::TexShader::create):
4:51 PM Changeset in webkit [77864] by mjs@apple.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Sam Weinig.

Skip a newly added test that uses unimplemented DRT functionality
https://bugs.webkit.org/show_bug.cgi?id=53938

  • platform/mac-wk2/Skipped:
4:30 PM Changeset in webkit [77863] by zmo@google.com
  • 5 edits
    1 add in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, rebaseline.

  • platform/chromium-linux/fast/html/keygen-expected.checksum:
  • platform/chromium-linux/fast/html/keygen-expected.png:
  • platform/chromium-linux/fast/html/keygen-expected.txt: Added.
  • platform/chromium-win/fast/html/keygen-expected.txt:
  • platform/chromium/test_expectations.txt:
4:29 PM Changeset in webkit [77862] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

Find bouncy is missing content on pages with framesets
https://bugs.webkit.org/show_bug.cgi?id=53958
<rdar://problem/8904447>

Fix paint rect computation; FrameView::paint expects the paint rect to be in the
parent widget's coordinate system.

  • WebProcess/WebPage/FindController.cpp: (WebKit::FindController::updateFindIndicator):
4:22 PM Changeset in webkit [77861] by beidson@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/8960434> and https://bugs.webkit.org/show_bug.cgi?id=53957
Crash after incorrectly restoring bogus session state.

Reviewed by Anders Carlsson.

In some cases we're writing an invalid session state for a back/forward list where the current entry is 0
but the number of entries is also 0.
In such cases the current entry should be "NoCurrentEntryIndex."

When we later read this state in, we set ourselves up to crash later.

Amusingly an ASSERT caught this, but we should've rejected it before the ASSERT fired.

  • UIProcess/cf/WebBackForwardListCF.cpp:

(WebKit::WebBackForwardList::restoreFromCFDictionaryRepresentation): Fail the restore if the "current index past the end

of the list" case occurs, and speculatively bail out of the case where we have no current index but do have a list.
Also remove the unhelpful ASSERT.

4:22 PM Changeset in webkit [77860] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
4:11 PM Changeset in webkit [77859] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
4:07 PM Changeset in webkit [77858] by mitz@apple.com
  • 3 edits in trunk/Source/WebCore

<rdar://problem/8946023> REGRESSION (r76674): SVG + BiDi text broken (text-intro-05-t.svg)
https://bugs.webkit.org/show_bug.cgi?id=53480

Reviewed by Oliver Hunt.

  • WebCore.xcodeproj/project.pbxproj:
  • platform/graphics/mac/ComplexTextController.cpp:

(WebCore::ComplexTextController::ComplexTextRun::ComplexTextRun): m_indexEnd should be an
offset into m_characters, not into the run’s characters.

  • platform/graphics/mac/ComplexTextControllerATSUI.cpp:

(WebCore::ComplexTextController::ComplexTextRun::ComplexTextRun): Ditto.

4:01 PM Changeset in webkit [77857] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
3:46 PM Changeset in webkit [77856] by weinig@apple.com
  • 7 edits in trunk/Source

Add bundle API to get the background color of a frame
<rdar://problem/8962670>
https://bugs.webkit.org/show_bug.cgi?id=53952

Reviewed by Anders Carlsson.

../WebCore:

  • WebCore.exp.in: Add export.

../WebKit2:

  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp:

(WKBundleFrameGetBodyBackgroundColor):

  • WebProcess/InjectedBundle/API/c/WKBundleFrame.h:

Add API function.

  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::getDocumentBackgroundColor):

  • WebProcess/WebPage/WebFrame.h:

Call down into the document to get either the body's, or if there is no body,
the root element's, background color.

3:45 PM Changeset in webkit [77855] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed.

[Qt] Fix crashes in QMetaObject::metacall
https://bugs.webkit.org/show_bug.cgi?id=53912

  • platform/qt/Skipped: http/tests/websocket/tests/websocket-pending-activity.html added until fix.
3:41 PM Changeset in webkit [77854] by zmo@google.com
  • 3 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, rebaseline.

  • platform/chromium-mac/fast/url/path-expected.txt:
  • platform/chromium-win/fast/url/path-expected.txt:
3:37 PM Changeset in webkit [77853] by ggaren@apple.com
  • 4 edits in trunk/Source/WebCore

2011-02-07 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.

Assertion failure in ~JSGlobalObject when loading apple.com HTML5
transitions demo page
https://bugs.webkit.org/show_bug.cgi?id=53853

This was a long-standing bookkeeping bug uncovered by some heap refactoring.

  • html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::createImageBuffer):
  • html/HTMLImageLoader.cpp: (WebCore::HTMLImageLoader::notifyFinished):
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::dropProtection): Take the JSLock before reporting extra cost to the JavaScriptCore heap, since all use of JavaScriptCore is supposed to be guarded by JSLock.
3:28 PM Changeset in webkit [77852] by jianli@chromium.org
  • 8 edits in trunk

Intermittent crash in fast/files/read-blob-async.html on the GTK+ debug
bots
https://bugs.webkit.org/show_bug.cgi?id=53104

Reviewed by David Levin.

Source/WebCore:

Covered by the existing tests.

  • fileapi/FileStreamProxy.cpp:

(WebCore::FileStreamProxy::startOnFileThread):

  • platform/network/BlobRegistryImpl.cpp:

(WebCore::BlobRegistryImpl::createResourceHandle):

  • platform/network/BlobResourceHandle.cpp:

(WebCore::BlobResourceHandle::BlobResourceHandle):
(WebCore::BlobResourceHandle::cancel):
(WebCore::delayedStartBlobResourceHandle):
(WebCore::BlobResourceHandle::start): Keep BlobResourceHandle alive
till the delay function is called.
(WebCore::BlobResourceHandle::doStart):
(WebCore::doNotifyFinish):
(WebCore::BlobResourceHandle::notifyFinish): Notify the client via the
standalone function to prevent the handle from being disposed immediately
by the client which will make the calls in the stack that're still bound
to the handle suffer.

  • platform/network/BlobResourceHandle.h:
  • platform/network/ResourceHandle.h: Change cancel() to virtual so that

BlobResourceHandle::cancel will be called when we abort a FileReader.

LayoutTests:

  • platform/gtk/Skipped: Enable the flakey test that has been fixed.
3:27 PM Changeset in webkit [77851] by abarth@webkit.org
  • 1 edit
    1 delete in trunk/LayoutTests

2011-02-07 Adam Barth <abarth@webkit.org>

Remove wrong platform-specific expectation.

  • platform/win/plugins/plugin-document-back-forward-expected.txt: Removed.
3:09 PM Changeset in webkit [77850] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-02-07 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r77845.
http://trac.webkit.org/changeset/77845
https://bugs.webkit.org/show_bug.cgi?id=53946

build failure (Requested by lforschler_ on #webkit).

  • WebCore.xcodeproj/project.pbxproj:
  • accessibility/AccessibilityAllInOne.cpp:
3:05 PM Changeset in webkit [77849] by Darin Adler
  • 2 edits in trunk/Source/WebCore

https://bugs.webkit.org/show_bug.cgi?id=53939
fast/css/text-overflow-ellipsis.html failure after r77818/77819

  • platform/text/TextCodecICU.cpp:

(WebCore::TextCodecICU::registerExtendedEncodingNames):
Add back the "utf8" synonym for "UTF-8".

2:58 PM Changeset in webkit [77848] by hyatt@apple.com
  • 2 edits in trunk/Source/WebCore

Remove an unnecessary extra computeLogicalWidth() from line layout.

Reviewed by Dan Bernstein.

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::computeInlineDirectionPositionsForLine):

2:34 PM Changeset in webkit [77847] by weinig@apple.com
  • 3 edits in trunk/Source/WebKit2

Fix typo. vectical -> vertical.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didChangeScrollbarsForMainFrame):

2:32 PM Changeset in webkit [77846] by weinig@apple.com
  • 8 edits in trunk/Source/WebKit2

Inform the UIProcess when main frame scrollbars change
<rdar://problem/8967496>
https://bugs.webkit.org/show_bug.cgi?id=53941

Reviewed by Anders Carlsson.

  • UIProcess/API/C/WKPage.cpp:

(WKPageHasHorizontalScrollbar):
(WKPageHasVerticalScrollbar):

  • UIProcess/API/C/WKPage.h:

Add API to query whether the main frame has scrollbars.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::close):
(WebKit::WebPageProxy::processDidCrash):
(WebKit::WebPageProxy::didChangeScrollbarsForMainFrame):

  • UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::hasHorizontalScrollbar):
(WebKit::WebPageProxy::hasVerticalScrollbar):
Store/update main frame scrollbar state.

  • UIProcess/WebPageProxy.messages.in:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::contentsSizeChanged):

  • WebProcess/WebCoreSupport/WebChromeClient.h:

(WebKit::WebChromeClient::WebChromeClient):
Post message when main frame scrollbar state changes.

2:28 PM Changeset in webkit [77845] by Lucas Forschler
  • 3 edits in trunk/Source/WebCore

2011-02-07 Lucas Forschler <Lucas Forschler>

Reviewed by Geoffrey Garen.

Fix Leopard Intel Debug Build.
<rdar://problem/8953606> REGRESSION: Can't link WebCore on 32-bit Leopard (53681)
This could potentially break windows, since I had to change #include <filename> to #include "filename"


  • WebCore.xcodeproj/project.pbxproj:
  • accessibility/AccessibilityAllInOne.cpp:
1:54 PM Changeset in webkit [77844] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
1:48 PM Changeset in webkit [77843] by weinig@apple.com
  • 2 edits in trunk/Source/WebCore

Fix build.

  • platform/mac/ScrollAnimatorMac.mm:

(WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):
(WebCore::ScrollAnimatorMac::willRemoveVerticalScrollbar):
(WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):
(WebCore::ScrollAnimatorMac::willRemoveHorizontalScrollbar):

1:42 PM Changeset in webkit [77842] by andersca@apple.com
  • 8 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Some content is not covered by Find-on-page overlay
https://bugs.webkit.org/show_bug.cgi?id=53937
<rdar://problem/8711353>

  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::didInstallPageOverlay): Call LayerTreeHost::didInstallPageOverlay.

(WebKit::DrawingAreaImpl::didUninstallPageOverlay):
Call LayerTreeHost::didUninstallPageOverlay.

(WebKit::DrawingAreaImpl::setPageOverlayNeedsDisplay):
Call LayerTreeHost::setPageOverlayNeedsDisplay.

  • WebProcess/WebPage/FindController.cpp: (WebKit::FindController::drawRect): Remove a bogus assertion.
  • WebProcess/WebPage/LayerTreeHost.h: Add new pure virtual member functions.
  • WebProcess/WebPage/PageOverlay.cpp: (WebKit::PageOverlay::setNeedsDisplay): Call DrawingArea::setPageOverlayNeedsDisplay.
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::installPageOverlay): Make sure to call PageOverlay::setNeedsDisplay after the page overlay has been installed.


  • WebProcess/WebPage/mac/LayerTreeHostMac.mm: (WebKit::LayerTreeHostMac::LayerTreeHostMac): Create the page overlay layer if needed.

(WebKit::LayerTreeHostMac::setNonCompositedContentsNeedDisplay):
Invalidate the page overlay layer as well.

(WebKit::LayerTreeHostMac::sizeDidChange):
Resize the page overlay layer.

(WebKit::LayerTreeHostMac::didInstallPageOverlay):
Create the page overlay layer.

(WebKit::LayerTreeHostMac::didUninstallPageOverlay):
Destroy the page overlay layer.

(WebKit::LayerTreeHostMac::setPageOverlayNeedsDisplay):
Call setNeedsDisplay on the page overlay layer.

(WebKit::LayerTreeHostMac::paintContents):
Handle painting the page overlay layer.

(WebKit::LayerTreeHostMac::flushPendingLayerChanges):
Flush the page overlay layer changes as well.

(WebKit::LayerTreeHostMac::createPageOverlayLayer):
Create a layer and add it as a sublayer of the root layer.

(WebKit::LayerTreeHostMac::destroyPageOverlayLayer):
Destroy the page overlay layer.

1:29 PM Changeset in webkit [77841] by weinig@apple.com
  • 9 edits in trunk/Source/WebCore

Add mechanism to inform the ScrollAnimator when Scrollbars are added
and removed.

Reviewed by Beth Dakin.

  • platform/ScrollAnimator.h:

(WebCore::ScrollAnimator::didAddVerticalScrollbar):
(WebCore::ScrollAnimator::willRemoveVerticalScrollbar):
(WebCore::ScrollAnimator::didAddHorizontalScrollbar):
(WebCore::ScrollAnimator::willRemoveHorizontalScrollbar):
Make virtual to allow platform specific overrides.

  • platform/ScrollView.cpp:

(WebCore::ScrollView::setHasHorizontalScrollbar):
(WebCore::ScrollView::setHasVerticalScrollbar):
Call new ScrollableArea functions.

  • platform/ScrollableArea.cpp:

(WebCore::ScrollableArea::didAddVerticalScrollbar):
(WebCore::ScrollableArea::willRemoveVerticalScrollbar):
(WebCore::ScrollableArea::didAddHorizontalScrollbar):
(WebCore::ScrollableArea::willRemoveHorizontalScrollbar):

  • platform/ScrollableArea.h:

Pipe to ScrollAnimator.

  • platform/mac/ScrollAnimatorMac.h:
  • platform/mac/ScrollAnimatorMac.mm:

(WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):
(WebCore::ScrollAnimatorMac::willRemoveVerticalScrollbar):
(WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):
(WebCore::ScrollAnimatorMac::willRemoveHorizontalScrollbar):

  • platform/mac/ScrollbarThemeMac.h:
  • platform/mac/ScrollbarThemeMac.mm:

(WebCore::ScrollbarThemeMac::registerScrollbar):
(WebCore::ScrollbarThemeMac::unregisterScrollbar):
(WebCore::ScrollbarThemeMac::painterForScrollbar):
Move painter delegate registration to the new didAdd/willRemove functions.

1:25 PM Changeset in webkit [77840] by zmo@google.com
  • 2 edits in trunk/LayoutTests

2011-02-07 Zhenyao Mo <zmo@google.com>

Unreviewed, test expectations update.

  • platform/chromium/test_expectations.txt:
1:19 PM Changeset in webkit [77839] by Adam Roben
  • 4 edits in trunk/Source/WebKit2

Make it possible to use DrawingAreaProxyImpl on Windows

It can be optionally turned on by setting the WEBKIT2_USE_NEW_DRAWING_AREA environment
variable. Turning it on by default is covered by <http://webkit.org/b/53805>.

Fixes <http://webkit.org/b/53879>.

Reviewed by Anders Carlsson.

  • UIProcess/BackingStore.h: Add an m_bitmap member on Windows.
  • UIProcess/win/BackingStoreWin.cpp:

(WebKit::BitmapDC::operator HDC):
(WebKit::BitmapDC::BitmapDC):
(WebKit::BitmapDC::~BitmapDC):
Added this helper class, which uses RAII to select a bitmap into a new memory DC.
(WebKit::BackingStore::paint): Wrap our bitmap in a BitmapDC and use ::BitBlt to copy the
specified rect into the destination DC.
(WebKit::createBitmap): New helper function, just creates a DIB section with the specified
size.
(WebKit::BackingStore::incorporateUpdate): Ported from BackingStoreMac.mm. It seems like
this code could be shared at some point.
(WebKit::BackingStore::scroll): Wrap our bitmap in a BitmapDC and use ::ScrollDC to scroll
the specified rect.

  • UIProcess/win/WebView.cpp:

(WebKit::useNewDrawingArea): Added. Returns true if the WEBKIT2_USE_NEW_DRAWING_AREA
environment variable is set.
(WebKit::drawPageBackground): New helper function. Code came from onPaintEvent.
(WebKit::WebView::onPaintEvent): Ported from -[WKView drawRect:].
(WebKit::WebView::createDrawingAreaProxy): Create a DrawingAreaProxyImpl if we're using the
new drawing area.

(WebKit::WebView::enterAcceleratedCompositingMode):
(WebKit::WebView::exitAcceleratedCompositingMode):
(WebKit::WebView::pageDidEnterAcceleratedCompositing):
(WebKit::WebView::pageDidLeaveAcceleratedCompositing):
Added some assertions to make sure these are only called when expected. We don't expect
enter/exitAcceleratedCompositingMode to be called at all right now, because accelerated
compositing is turned off in WebKit2 on Windows.

(WebKit::WebView::switchToDrawingAreaTypeIfNecessary): Ditto, and added a case for the
DrawingAreaInfo::Impl constant.

1:19 PM Changeset in webkit [77838] by Adam Roben
  • 13 edits
    2 copies in trunk/Source/WebKit2

Start compiling DrawingArea[Proxy]Impl and dependencies on Windows

Fixes <http://webkit.org/b/53875> WebKit2 on Windows should build DrawingArea[Proxy]Impl

Reviewed by Sam Weinig and Darin Adler.

  • Platform/Region.cpp:

(WebKit::Region::Shape::shapeOperation): Initialize variable to avoid a bogus "potentially
uninitialized variable" warning.

  • UIProcess/BackingStore.cpp: Removed #error for non-Mac platforms.
  • UIProcess/BackingStore.h: Added a PlatformGraphicsContext typedef for Windows.
  • UIProcess/DrawingAreaProxy.h:
  • UIProcess/DrawingAreaProxyImpl.cpp:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didReceiveMessage):
Allow Windows to compile the new drawing area code.

  • WebProcess/WebPage/DrawingArea.cpp:

(WebKit::DrawingArea::create): Allow creating a DrawingAreaImpl on Windows, too.

  • WebProcess/WebPage/DrawingArea.h:
  • WebProcess/WebPage/DrawingAreaImpl.cpp:
  • WebProcess/WebPage/LayerTreeHost.cpp:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::didReceiveMessage):
Allow Windows to compile the new drawing area code.

  • Shared/win/LayerTreeContextWin.cpp: Added.
  • UIProcess/win/BackingStoreWin.cpp: Added.

Stubbed out these files.

  • win/WebKit2.vcproj: Added new files.
12:57 PM Changeset in webkit [77837] by andersca@apple.com
  • 7 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Rename some drawing area related member functions
https://bugs.webkit.org/show_bug.cgi?id=53935

  • WebProcess/WebPage/DrawingArea.h: (WebKit::DrawingArea::setPageOverlayNeedsDisplay):
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::setNeedsDisplay): (WebKit::DrawingAreaImpl::setPageOverlayNeedsDisplay):
  • WebProcess/WebPage/DrawingAreaImpl.h:
  • WebProcess/WebPage/LayerTreeHost.h:
  • WebProcess/WebPage/mac/LayerTreeHostMac.h:
  • WebProcess/WebPage/mac/LayerTreeHostMac.mm: (WebKit::LayerTreeHostMac::setNonCompositedContentsNeedDisplay): (WebKit::LayerTreeHostMac::scrollNonCompositedContents):
12:49 PM Changeset in webkit [77836] by chang.shu@nokia.com
  • 2 edits in trunk/Tools

2011-02-07 Chang Shu <chang.shu@nokia.com>

Reviewed by Darin Adler.

We should disable spatial navigation explicitly during the initialization
of DumpRenderTree.
https://bugs.webkit.org/show_bug.cgi?id=53928

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
12:43 PM Changeset in webkit [77835] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Missing text match markers with accelerated content
https://bugs.webkit.org/show_bug.cgi?id=53934

When doing the fake paint that recomputes the text match rects, we need to
flatten compositing layers, otherwise the rects won't be updated until the next paint call.

(This worked by accident in WebKit1 because Safari calls -[NSWindow displayIfNeeded] on the containing
window before asking for the text match rects).

  • editing/Editor.cpp: (WebCore::Editor::countMatchesForText):
12:08 PM Changeset in webkit [77834] by Antti Koivisto
  • 4 edits in trunk/Source/WebCore

REGRESSION(r77740): CSSStyleSelector accessing deleted memory for svg/dom/use-transform.svg
https://bugs.webkit.org/show_bug.cgi?id=53900

Reviewed by Andreas Kling.

Ignore link elements in shadow trees.

  • dom/Element.cpp:

(WebCore::StyleSelectorParentPusher::StyleSelectorParentPusher):
(WebCore::StyleSelectorParentPusher::~StyleSelectorParentPusher):

Some asserts to catch cases like this.

(WebCore::Element::attach):
(WebCore::Element::recalcStyle):

  • html/HTMLLinkElement.cpp:

(WebCore::HTMLLinkElement::HTMLLinkElement):
(WebCore::HTMLLinkElement::process):
(WebCore::HTMLLinkElement::insertedIntoDocument):
(WebCore::HTMLLinkElement::removedFromDocument):

  • html/HTMLLinkElement.h:
11:39 AM BuildingQtOnSymbian edited by siddharth.mathur@nokia.com
add .slimdebug (diff)
11:18 AM Changeset in webkit [77833] by Dimitri Glazkov
  • 2 edits in trunk/LayoutTests

2011-02-07 Dimitri Glazkov <Dimitri Glazkov>

Mark test as flaky per dashboard.
https://bugs.webkit.org/show_bug.cgi?id=53824

  • platform/chromium/test_expectations.txt: Marked.
11:10 AM Changeset in webkit [77832] by jam@chromium.org
  • 7 edits
    2 copies in branches/chromium/648

Merge 77706 - 2011-02-04 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

PluginDocuments don't create widgets for plugins on back/forward
https://bugs.webkit.org/show_bug.cgi?id=53474

Test that plugin loads when in a plugin document on back-forward.

  • plugins/plugin-document-back-forward-expected.txt: Added.
  • plugins/plugin-document-back-forward.html: Added.

2011-02-04 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

PluginDocuments don't create widgets for plugins on back/forward
https://bugs.webkit.org/show_bug.cgi?id=53474

Long ago, PluginDocument always caused the HTMLEmbedElement to create
its widget synchronously during a post-layout task. Recently, however,
some changes to the HistroyController caused layout on back/forward to
become slightly more complicated (and added an extra level of recursion
to layout). This extra level of recursion triggered the "I've recursed
too many times" condition in the post-layout task queue, causing the
FrameView to run the remainder of the tasks asynchronously.
Unfortunately, that broke PluginDocument because it needs its the
HTMLEmbedElement's updateWidget task to run synchronously.

This patch adds a mechanism for "kicking off" the pending post-layout
tasks synchronously (instead of waiting for the timer to fire).
PluginDocument then uses that facility to ensure that the
HTMLEmbedElement's updateWidget task happens.

Test: plugins/plugin-document-back-forward.html

  • html/PluginDocument.cpp: (WebCore::PluginDocumentParser::appendBytes):
  • page/FrameView.cpp: (WebCore::FrameView::flushAnyPendingPostLayoutTasks):
  • page/FrameView.h:

2011-02-04 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

PluginDocuments don't create widgets for plugins on back/forward
https://bugs.webkit.org/show_bug.cgi?id=53474

Teach the test plugin how to call alert on load.

  • DumpRenderTree/TestNetscapePlugIn/main.cpp: (NPP_New):

TBR=abarth@webkit.org
Review URL: http://codereview.chromium.org/6286151

10:47 AM BuildingQtOnSymbian edited by siddharth.mathur@nokia.com
add -b syntax for compiling both Source and Tools project files (diff)
10:46 AM Changeset in webkit [77831] by Darin Adler
  • 9 edits in trunk/Source/WebCore

Turn UTF-8 decoder off for now so we can look into failing tests
without leaving the tree red.

  • platform/text/TextCodecICU.cpp:

(WebCore::TextCodecICU::registerBaseEncodingNames): Reverting
earlier change so this can handle UTF-8.
(WebCore::TextCodecICU::registerBaseCodecs): Ditto.
(WebCore::TextCodecICU::registerExtendedEncodingNames): Ditto.
(WebCore::TextCodecICU::registerExtendedCodecs): Ditto.

  • platform/text/TextCodecICU.h: Ditto.
  • platform/text/TextEncodingRegistry.cpp:

(WebCore::buildBaseTextCodecMaps): Ditto.
(WebCore::extendTextCodecMaps): Ditto.

  • platform/text/brew/TextCodecBrew.cpp:

(WebCore::TextCodecBrew::registerBaseEncodingNames): Ditto.
(WebCore::TextCodecBrew::registerBaseCodecs): Ditto.
(WebCore::TextCodecBrew::registerExtendedEncodingNames): Ditto.
(WebCore::TextCodecBrew::registerExtendedCodecs): Ditto.

  • platform/text/brew/TextCodecBrew.h: Ditto.
  • platform/text/gtk/TextCodecGtk.cpp:

(WebCore::TextCodecGtk::registerBaseEncodingNames): Ditto.
(WebCore::TextCodecGtk::registerBaseCodecs): Ditto.

  • platform/text/wince/TextCodecWinCE.cpp:

(WebCore::TextCodecWinCE::registerBaseEncodingNames): Ditto.
(WebCore::TextCodecWinCE::registerBaseCodecs): Ditto.
(WebCore::TextCodecWinCE::registerExtendedEncodingNames): Ditto.
(WebCore::TextCodecWinCE::registerExtendedCodecs): Ditto.

  • platform/text/wince/TextCodecWinCE.h: Ditto.
10:45 AM Changeset in webkit [77830] by Adam Roben
  • 2 edits in trunk/Source/WebKit2

Use the right calling convention when calling WebKitMain

Fixes <http://webkit.org/b/53923> REGRESSION (r77678): Crash on quit in web process in
wWinMain

Reviewed by Steve Falkenburg.

  • win/MainWin.cpp:

(wWinMain): Use cdecl instead of WINAPI (stdcall) when calling WebKitMain, since that is
how that function is declared.

10:39 AM Changeset in webkit [77829] by andersca@apple.com
  • 6 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

The drawing area should be in charge of drawing the page overlay
https://bugs.webkit.org/show_bug.cgi?id=53927

  • WebProcess/WebPage/DrawingArea.h: (WebKit::DrawingArea::didInstallPageOverlay): (WebKit::DrawingArea::didUninstallPageOverlay): (WebKit::DrawingArea::setOverlayNeedsDisplay): Add empty stubs.
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::didInstallPageOverlay): Add empty function. This will eventually ask the layer tree host to create a GraphicsLayer for the page overlay.

(WebKit::DrawingAreaImpl::didUninstallPageOverlay):
Tell the page that it needs to be repainted.

(WebKit::DrawingAreaImpl::setOverlayNeedsDisplay):
Just call setNeedsDisplay for now.

(WebKit::DrawingAreaImpl::display):
Paint the page overlay too.

  • WebProcess/WebPage/DrawingAreaImpl.h:
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::drawRect): Only call drawPageOverlay if we have an old drawing area.

(WebKit::WebPage::drawPageOverlay):
Draw the page overlay. Code moved here from WebPage::drawRect.

(WebKit::WebPage::installPageOverlay):
Call DrawingArea::installPageOverlay.

(WebKit::WebPage::uninstallPageOverlay):
Call DrawingArea::uninstallPageOverlay.

  • WebProcess/WebPage/WebPage.h: (WebKit::WebPage::hasPageOverlay): Add getter.
10:31 AM Changeset in webkit [77828] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Don't exclude overlay scrollers from the page overlay bounds
https://bugs.webkit.org/show_bug.cgi?id=53924

Reviewed by Maciej Stachowiak.

Export ScrollbarTheme::nativeTheme.

  • WebCore.exp.in:
10:31 AM Changeset in webkit [77827] by antonm@chromium.org
  • 3 edits in branches/chromium/648/Source/WebCore

Merge 77597 - 2011-02-03 Anton Muhin <antonm@chromium.org>

Reviewed by Adam Barth.

[v8] Bail out if to string conversion returned empty handle
https://bugs.webkit.org/show_bug.cgi?id=53687

This a temporary measure: actually one probably should never get empty handle
if there was no exception. The root cause is under investigation.
The bailout though allows Chromium not to crash---attempt to convert an empty
v8 hande into WebCore string crashes with invalid memory access.

See http://code.google.com/p/chromium/issues/detail?id=71544

There is no known reduction expressible as a layout test so far. The crash found with automated testing tools.

  • bindings/v8/V8Binding.cpp: (WebCore::v8NonStringValueToWebCoreString): Bail out on empty handle
  • bindings/v8/V8Binding.h: (WebCore::V8ParameterBase::prepareBase): Ditto

TBR=antonm@chromium.org
Review URL: http://codereview.chromium.org/6368135

10:26 AM Changeset in webkit [77826] by andersca@apple.com
  • 4 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Add WebPage::bounds and use it
https://bugs.webkit.org/show_bug.cgi?id=53926

  • WebProcess/WebPage/ChunkedUpdateDrawingArea.cpp: (WebKit::ChunkedUpdateDrawingArea::resumePainting):
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::resumePainting): (WebKit::DrawingAreaImpl::exitAcceleratedCompositingMode):
  • WebProcess/WebPage/WebPage.h: (WebKit::WebPage::bounds):
10:21 AM Changeset in webkit [77825] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-02-07 Anders Carlsson <andersca@apple.com>

Reviewed by Maciej Stachowiak.

Don't exclude overlay scrollers from the page overlay bounds
https://bugs.webkit.org/show_bug.cgi?id=53924

Don't subtract the scrollbars from the page overlay bounds if
overlay scrollers are enabled.

  • WebProcess/WebPage/PageOverlay.cpp: (WebKit::PageOverlay::bounds):
10:18 AM Changeset in webkit [77824] by Martin Robinson
  • 2 edits
    1 add in trunk/LayoutTests

2011-02-07 Martin Robinson <mrobinson@igalia.com>

Unskip a passing accessibility test and organize the rest of the tests missing
results as failing.

  • platform/gtk/Skipped: Unskip passing test and organize remaining failures.
  • platform/gtk/accessibility/aria-describedby-on-input-expected.txt: Added.
10:02 AM Changeset in webkit [77823] by Darin Adler
  • 2 edits in trunk/Source/JavaScriptCore

Add built-in decoder for UTF-8 for improved performance
https://bugs.webkit.org/show_bug.cgi?id=53898

Reviewed by Antti Koivisto.

  • wtf/unicode/UnicodeMacrosFromICU.h: Added U8_MAX_LENGTH and

U8_APPEND_UNSAFE. Also fixed header.

9:57 AM Changeset in webkit [77822] by mjs@apple.com
  • 2 edits in trunk/Tools

Not reviewed.

Fix WebKitTestRunner build on the SL bot.

  • WebKitTestRunner/Configurations/Base.xcconfig: Look for the JSC copy

of ICU headers.

9:29 AM Changeset in webkit [77821] by zmo@google.com
  • 4 edits
    2 adds in trunk

2011-02-03 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Crash in glDrawArrays with NaCl crystal model
https://bugs.webkit.org/show_bug.cgi?id=52831

  • html/canvas/WebGLRenderingContext.cpp: Allocate large enough buffer for atttrib0 if it's enabled and not used. (WebCore::WebGLRenderingContext::initVertexAttrib0): (WebCore::WebGLRenderingContext::simulateVertexAttrib0):
  • html/canvas/WebGLRenderingContext.h:

2011-02-03 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Crash in glDrawArrays with NaCl crystal model
https://bugs.webkit.org/show_bug.cgi?id=52831

  • fast/canvas/webgl/gl-vertex-attrib-zero-issues-expected.txt: Added.
  • fast/canvas/webgl/gl-vertex-attrib-zero-issues.html: Added.
9:24 AM Changeset in webkit [77820] by Chris Fleizach
  • 6 edits
    2 adds in trunk

Source/WebCore: WAI-ARIA @aria-sort not exposed on rowheader and columnheader roles.
https://bugs.webkit.org/show_bug.cgi?id=53842

Reviewed by Darin Adler.

Test: platform/mac/accessibility/aria-sort.html

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::sortDirection):

  • accessibility/AccessibilityObject.h:
  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper additionalAccessibilityAttributeNames]):
(-[AccessibilityObjectWrapper accessibilityAttributeValue:]):

  • html/HTMLAttributeNames.in:

LayoutTests: WAI-ARIA @aria-sort not exposed on rowheader and columnheader roles
https://bugs.webkit.org/show_bug.cgi?id=53842

Reviewed by Darin Adler.

  • platform/mac/accessibility/aria-sort-expected.txt: Added.
  • platform/mac/accessibility/aria-sort.html: Added.
8:35 AM Changeset in webkit [77819] by Darin Adler
  • 17 edits
    2 adds in trunk/Source/WebCore

2011-02-06 Darin Adler <Darin Adler>

Reviewed by Antti Koivisto.

Add built-in decoder for UTF-8 for improved performance
https://bugs.webkit.org/show_bug.cgi?id=53898

Correct behavior is covered by existing tests.

  • Android.mk: Added TextCodecUTF8.
  • CMakeLists.txt: Ditto.
  • GNUmakefile.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • platform/text/TextCodecICU.cpp: Removed registration of UTF-8 since that's now handled by the built-in UTF-8 codec. (WebCore::TextCodecICU::registerEncodingNames): Renamed to remove the word "extended" since all codecs from ICU are now extended. (WebCore::TextCodecICU::registerCodecs): Ditto.
  • platform/text/TextCodecICU.h: Ditto.
  • platform/text/TextCodecUTF16.cpp: (WebCore::TextCodecUTF16::decode): Added a FIXME about missing error handling.
  • platform/text/TextCodecUTF8.cpp: Added.
  • platform/text/TextCodecUTF8.h: Added.
  • platform/text/TextEncodingRegistry.cpp: (WebCore::buildBaseTextCodecMaps): Added TextCodecUTF8, and removed TextCodecICU, TextCodecBrew, and TextCodecWinCE. (WebCore::extendTextCodecMaps): Updated names for TextCodecICU and TextCodecWinCE.
  • platform/text/brew/TextCodecBrew.cpp: (WebCore::TextCodecBrew::registerEncodingNames): Renamed to remove the word "extended" since all codecs from Brew are now extended. (WebCore::TextCodecBrew::registerCodecs): Ditto.
  • platform/text/brew/TextCodecBrew.h: Ditto.
  • platform/text/gtk/TextCodecGtk.cpp: (WebCore::TextCodecGtk::registerBaseEncodingNames): Removed registration of UTF-8. It's questionable whether the other codecs here are really basic. If we think they qualify, then we should register them on other platforms, not just GTK. (WebCore::TextCodecGtk::registerBaseCodecs): Ditto.
  • platform/text/wince/TextCodecWinCE.cpp: (WebCore::getCodePage): Removed special case for UTF-8 since we no longer use this codec for UTF-8. (WebCore::TextCodecWinCE::registerEncodingNames): Renamed to remove the word "extended" since all codecs from WinCE are now extended. (WebCore::TextCodecWinCE::registerCodecs): Ditto.
  • platform/text/wince/TextCodecWinCE.h: Ditto.
8:35 AM Changeset in webkit [77818] by Martin Robinson
  • 4 edits in trunk

2011-02-07 Martin Robinson <mrobinson@igalia.com>

Reviewed by Daniel Bates.

[GTK] fast/history/history-subframe-with-name.html fails with GTK DRT
https://bugs.webkit.org/show_bug.cgi?id=44784

  • platform/gtk/Skipped: Unskip tests which are now passing.

2011-02-07 Martin Robinson <mrobinson@igalia.com>

Reviewed by Daniel Bates.

[GTK] fast/history/history-subframe-with-name.html fails with GTK DRT
https://bugs.webkit.org/show_bug.cgi?id=44784

Call FrameLoader::loadURLIntoChildFrame on the parent frame's loader, not the child's.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::createFrame): Fix the call to FrameLoader::loadURLIntoChildFrame.
8:23 AM Changeset in webkit [77817] by mario@webkit.org
  • 4 edits in trunk/Source

2011-02-07 Mario Sanchez Prada <msanchez@igalia.com>

Reviewed by Xan Lopez.

[Gtk] atk_text_get_caret_offset fails for list items
https://bugs.webkit.org/show_bug.cgi?id=53436

Consider list item markers when calculating the offset.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_text_get_caret_offset): Adjust the offset with the item marker's length for list items. (webkit_accessible_text_set_caret_offset): Replace usage of g_utf8_strlen() by calling to markerText.length().

2011-02-07 Mario Sanchez Prada <msanchez@igalia.com>

Reviewed by Xan Lopez.

[Gtk] atk_text_get_caret_offset fails for list items
https://bugs.webkit.org/show_bug.cgi?id=53436

Update unit test to check the fix for this bug.

  • tests/testatk.c: (testWebkitAtkCaretOffsets): Check that the caret offset returned match the value previously set.
8:03 AM Changeset in webkit [77816] by Adam Roben
  • 2 edits in trunk/Source/WebKit2

Windows build fix after r77794

  • win/WebKit2.def: Export WebCore symbols used by WebKitTestRunner.
8:02 AM Changeset in webkit [77815] by Adam Roben
  • 3 edits in trunk/Source/JavaScriptCore

Delete precompiled headers whenever any .vsprops file changes

Precompiled headers need to be rebuilt if, e.g., an ENABLE_* macro is changed in one of our
.vsprops files. Unfortunately, Visual Studio isn't smart enough to figure this out, so we
give it some assistance by deleting the precompiled headers whenever any .vsprops file
changes.

I also made some drive-by fixes while I was in the area.

Fixes <http://webkit.org/b/53826> react-to-vsprops-changes.py doesn't force precompiled
headers to be rebuilt, but should

Reviewed by David Kilzer.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCoreGenerated.make: Stop ignoring the

return code from react-to-vsprops-changes.py so we will notice when errors are introduced.
But skip the script entirely in production builds, where it is both unnecessary and can't
function correctly (due to not having the entire source tree available to it).

(main): Removed an extra '*' in the glob for finding manifest files. The extra '*' was
leftover from before we rearranged WebKitBuild in r75138. Moved code to delete an old file
to the new delete_if_older_than function. Added code to delete any precompiled headers older
than the newest .vsprops file.
(delete_if_older_than): Added. Code came from main.

7:57 AM Changeset in webkit [77814] by yi.4.shen@nokia.com
  • 4 edits in trunk

2011-02-07 Yi Shen <yi.4.shen@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Fix LayoutTests/media/restore-from-page-cache.html
https://bugs.webkit.org/show_bug.cgi?id=53621

Unskip media/restore-from-page-cache.html for Qt.

  • platform/qt/Skipped:

2011-02-07 Yi Shen <yi.4.shen@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Fix LayoutTests/media/restore-from-page-cache.html
https://bugs.webkit.org/show_bug.cgi?id=53621

Changed the networkState to MediaPlayer::Loaded when
MediaPlayerPrivateQt has enough data.

Test: media/restore-from-page-cache.html

  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivateQt::updateStates):
7:56 AM Changeset in webkit [77813] by Patrick Gansterer
  • 2 edits in trunk/Source/WebKit/haiku

2011-02-07 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

[Haiku] FrameLoaderClient calls loadURLInChildFrame on the child's frame loader
https://bugs.webkit.org/show_bug.cgi?id=53893

  • WebCoreSupport/FrameLoaderClientHaiku.cpp: (WebCore::FrameLoaderClientHaiku::createFrame): Use m_frame instead of childFrame.
7:31 AM Changeset in webkit [77812] by loislo@chromium.org
  • 2 edits in trunk/LayoutTests

2011-02-07 Ilya Tikhonovsky <loislo@chromium.org>

Unreviewed.

Chromium: three tests temporary disabled for rebaseline after r77781

  • platform/chromium/test_expectations.txt:
7:18 AM Changeset in webkit [77811] by Patrick Gansterer
  • 2 edits in trunk/Source/WebKit/efl

2011-02-07 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

[EFL] FrameLoaderClient calls loadURLInChildFrame on the child's frame loader
https://bugs.webkit.org/show_bug.cgi?id=53892

  • ewk/ewk_frame.cpp: (ewk_frame_child_add): Use the loader of the core frame instead.
7:15 AM Changeset in webkit [77810] by podivilov@chromium.org
  • 12 edits
    1 add in trunk

2011-02-04 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: implement beautification of scripts inlined in html documents.
https://bugs.webkit.org/show_bug.cgi?id=52706

  • http/tests/inspector/debugger-test2.js: (initialize_DebuggerTest.InspectorTest.setBreakpointInVisibleView):
  • inspector/debugger-breakpoints-not-activated-on-reload.html:
  • inspector/debugger-pause-on-breakpoint.html:

2011-01-31 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: implement beautification of scripts inlined in html documents.
https://bugs.webkit.org/show_bug.cgi?id=52706

To beautify scripts inlined in html, we need to cut individual scripts, beautify them using UglifyJS library, and
finally paste them back to html.

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/ScriptFormatter.js: (WebInspector.ScriptFormatter): Implement formatting of scripts inlined in html. (WebInspector.ScriptFormatter.prototype.formatContent.didFormatChunks): (WebInspector.ScriptFormatter.prototype.formatContent): (WebInspector.ScriptFormatter.prototype._splitContentIntoChunks): (WebInspector.ScriptFormatter.prototype._formatChunks.didFormat): (WebInspector.ScriptFormatter.prototype._formatChunks): (WebInspector.ScriptFormatter.prototype._buildContentFromChunks): (WebInspector.ScriptFormatter.prototype._formatScript.messageHandler): (WebInspector.ScriptFormatter.prototype._formatScript):
  • inspector/front-end/ScriptFormatterWorker.js: (onmessage): (buildMapping):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrameContent): Use SourceFrameContent and FormattedSourceFrameContent to perform operations with line numbers.
  • inspector/front-end/SourceFrameContent.js: Added. (WebInspector.SourceFrameContent): Helper class, holds resource text and scripts layout (WebInspector.SourceFrameContent.prototype.get scriptRanges): (WebInspector.SourceFrameContent.prototype.locationToPosition): (WebInspector.SourceFrameContent.prototype.positionToLocation): (WebInspector.SourceFrameContent.prototype.scriptLocationForLineNumber): (WebInspector.SourceFrameContent.prototype.scriptLocationForRange): (WebInspector.SourceFrameContent.prototype.lineNumberToRange): (WebInspector.SourceFrameContent.prototype._intersectingScriptRange): (WebInspector): (WebInspector.FormattedSourceFrameContent): Helper class, holds original and formatted versions of resource and mapping between them. (WebInspector.FormattedSourceFrameContent.prototype.get text): (WebInspector.FormattedSourceFrameContent.prototype.originalLocationToFormattedLocation): (WebInspector.FormattedSourceFrameContent.prototype.scriptLocationForFormattedLineNumber): (WebInspector.FormattedSourceFrameContent.prototype._convertPosition):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
6:55 AM Changeset in webkit [77809] by Patrick Gansterer
  • 2 edits in trunk/Source/WebKit/wx

2011-02-07 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

[wx] FrameLoaderClient calls loadURLInChildFrame on the child's frame loader
https://bugs.webkit.org/show_bug.cgi?id=53895

  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::createFrame): Use m_frame instead of childFrame.
6:49 AM Changeset in webkit [77808] by loislo@chromium.org
  • 1 edit
    41 adds in trunk/LayoutTests

2011-02-07 Ilya Tikhonovsky <loislo@chromium.org>

Unreviewed.

Chromium: add explicit "Passed" expectations for the tests which have "failed" expectations in platfrom/mac
The problem was introduced at r77719.
I have to do that because platform/mac is in the list of baseline folders.
See 'Tools/Scripts/webkitpy/layout_tests/port/chromium_linux.py'.

  • platform/chromium/canvas/philip/tests/2d.drawImage.broken-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.drawImage.image.incomplete.omitted-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-6-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.fillStyle.parse.rgb-eof-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.fillStyle.parse.rgba-eof-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.fillStyle.parse.system-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.gradient.radial.cone.front-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.gradient.radial.inside2-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.gradient.radial.inside3-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.gradient.radial.outside1-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.imageData.put.wrongtype-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.line.cap.closed-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.line.join.parallel-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.line.miter.lineedge-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.path.rect.winding-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.pattern.image.broken-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.pattern.image.incomplete.empty-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.pattern.image.incomplete.omitted-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.shadow.enable.x-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.shadow.enable.y-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.strokeRect.zero.4-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.baseline.bottom-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.baseline.hanging-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.baseline.middle-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.baseline.top-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.fill.maxWidth.fontface-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.space.collapse.end-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.space.collapse.nonspace-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.space.collapse.other-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.space.collapse.space-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.draw.space.collapse.start-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.font.parse.basic-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.font.parse.complex-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.font.parse.invalid-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.font.parse.size.percentage-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.font.parse.size.percentage.default-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.font.parse.system-expected.txt: Added.
  • platform/chromium/canvas/philip/tests/2d.text.measure.width.space-expected.txt: Added.
5:21 AM Changeset in webkit [77807] by Nikolas Zimmermann
  • 208 edits in trunk/LayoutTests

2011-02-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

small text which is scaled to be large renders pixelated
https://bugs.webkit.org/show_bug.cgi?id=12448

Update Snow Leopard SVG pixel test baseline after r77485.

  • platform/mac/svg/dom/SVGRectElement/rect-modify-rx-expected.png:
  • platform/mac/svg/dynamic-updates/SVGAElement-dom-href-attr-expected.png:

...

5:10 AM Changeset in webkit [77806] by Nikolas Zimmermann
  • 131 edits in trunk/LayoutTests

2011-02-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

small text which is scaled to be large renders pixelated
https://bugs.webkit.org/show_bug.cgi?id=12448

Update Snow Leopard SVG pixel test baseline after r77485.

  • platform/mac/svg/custom/js-update-polygon-removal-expected.png:
  • platform/mac/svg/custom/js-update-style-expected.png:

...

5:05 AM Changeset in webkit [77805] by Nikolas Zimmermann
  • 73 edits in trunk/LayoutTests

2011-02-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

small text which is scaled to be large renders pixelated
https://bugs.webkit.org/show_bug.cgi?id=12448

Update Snow Leopard SVG pixel test baseline after r77485.

  • platform/mac/svg/custom/class-baseValue-expected.png:
  • platform/mac/svg/custom/class-selector-expected.png:

...

4:59 AM Changeset in webkit [77804] by Nikolas Zimmermann
  • 91 edits in trunk/LayoutTests

2011-02-07 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

small text which is scaled to be large renders pixelated
https://bugs.webkit.org/show_bug.cgi?id=12448

Update Snow Leopard SVG pixel test baseline after r77485.

  • platform/mac/svg/batik/paints/gradientLimit-expected.png:
  • platform/mac/svg/batik/paints/patternRegionA-expected.png:

...

3:56 AM Changeset in webkit [77803] by alex
  • 2 edits
    3 adds in trunk/LayoutTests

2011-02-07 Alejandro G. Castro <alex@igalia.com>

Unreviewed, added new results for the slider-thumb-drag-release
test and removed use-infinite-recursion.svg from skipped, required
after r77778.

  • platform/gtk/Skipped:
  • platform/gtk/fast/repaint/slider-thumb-drag-release-expected.checksum: Added.
  • platform/gtk/fast/repaint/slider-thumb-drag-release-expected.png: Added.
  • platform/gtk/fast/repaint/slider-thumb-drag-release-expected.txt: Added.
3:46 AM Changeset in webkit [77802] by apavlov@chromium.org
  • 1 edit
    3 moves
    1 add in trunk/Websites/webkit.org

2011-02-07 Alexander Pavlov <apavlov@chromium.org>

Unreviewed. Add Web Inspector warning 10x10 PNG, move files from r77801 into the inspector/ directory.

  • blog-files/inspector/css_as_authored.png: Renamed from Websites/webkit.org/blog-files/inspector-css-as-authored.png.
  • blog-files/inspector/css_autocomplete.png: Renamed from Websites/webkit.org/blog-files/inspector-css-autocomplete.png.
  • blog-files/inspector/css_unknown_properties.png: Renamed from Websites/webkit.org/blog-files/inspector-unknown-css-properties.png.
  • blog-files/inspector/warning_10x10.png: Added.
3:27 AM Changeset in webkit [77801] by apavlov@chromium.org
  • 1 edit
    3 adds in trunk/Websites/webkit.org

2011-02-07 Alexander Pavlov <apavlov@chromium.org>

Unreviewed. Add Web Inspector style editing screenshots.

  • blog-files/inspector-css-as-authored.png: Added.
  • blog-files/inspector-css-autocomplete.png: Added.
  • blog-files/inspector-unknown-css-properties.png: Added.
3:26 AM Changeset in webkit [77800] by pfeldman@chromium.org
  • 7 edits in trunk/Source/WebCore

2011-02-07 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: "Reveal in elements panel" does not always work.
https://bugs.webkit.org/show_bug.cgi?id=53775

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView):
  • inspector/front-end/ContextMenu.js: (WebInspector.ContextMenu.prototype.show):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.switchToAndFocus):
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline.prototype._contextMenuEventFired.focusElement): (WebInspector.ElementsTreeOutline.prototype._contextMenuEventFired):
  • inspector/front-end/ObjectPropertiesSection.js: (WebInspector.ObjectPropertyTreeElement.prototype.update): (WebInspector.ObjectPropertyTreeElement.prototype._contextMenuEventFired):
  • inspector/front-end/inspector.js: (WebInspector.cancelSearch):
3:24 AM Changeset in webkit [77799] by abarth@webkit.org
  • 5 edits in trunk

2011-02-07 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Relative .. in URL paths shouldn't skip adjacent / characters
https://bugs.webkit.org/show_bug.cgi?id=53907

Add some more test cases.

  • fast/url/path-expected.txt:
  • fast/url/script-tests/path.js:

2011-02-07 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Relative .. in URL paths shouldn't skip adjacent / characters
https://bugs.webkit.org/show_bug.cgi?id=53907

Our new behavior matches Firefox, Chrome, and IE.

  • platform/KURL.cpp: (WebCore::copyPathRemovingDots):
3:16 AM Changeset in webkit [77798] by Csaba Osztrogonác
  • 2 edits in trunk/Tools

Unreviewed.

[Qt][WK2] Buildfix after r77794.

WebKitTestRunner does not block remote resources or complain about them
https://bugs.webkit.org/show_bug.cgi?id=42139
<rdar://problem/8183942>

  • WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro: Add missing includepaths.
3:01 AM Changeset in webkit [77797] by mjs@apple.com
  • 2 edits in trunk/Tools

Not reviewed.

More bot appeasement.

  • WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:
2:47 AM Changeset in webkit [77796] by mjs@apple.com
  • 2 edits in trunk/Tools

Not reviewed.

Remove accidental references to directories on my laptop.

  • WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:
1:59 AM Changeset in webkit [77795] by pfeldman@chromium.org
  • 2 edits in branches/chromium/648/Source/WebCore/inspector/front-end

Merge 76517 - 2011-01-24 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [REGRESSION] AppCache view on resources panel is broken.
https://bugs.webkit.org/show_bug.cgi?id=53002

  • inspector/front-end/ApplicationCacheItemsView.js: (WebInspector.ApplicationCacheItemsView.prototype._update):
  • inspector/front-end/DOMAgent.js: (WebInspector.ApplicationCacheDispatcher.getApplicationCachesAsync):

TBR=podivilov@chromium.org
Review URL: http://codereview.chromium.org/6368125

1:47 AM Changeset in webkit [77794] by mjs@apple.com
  • 8 edits in trunk

2011-02-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Antti Koivisto.

WebKitTestRunner does not block remote resources or complain about them
https://bugs.webkit.org/show_bug.cgi?id=42139
<rdar://problem/8183942>

  • Configurations/WebCore.xcconfig: Allow WebKitTestRunner to link WebCore.
  • WebCore.exp.in: Export a handful of KURL symbols.

2011-02-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Antti Koivisto.

WebKitTestRunner does not block remote resources or complain about them
https://bugs.webkit.org/show_bug.cgi?id=42139
<rdar://problem/8183942>

  • platform/mac-wk2/Skipped: Unskip formerly skipped tests.

2011-02-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Antti Koivisto.

WebKitTestRunner does not block remote resources or complain about them
https://bugs.webkit.org/show_bug.cgi?id=42139
<rdar://problem/8183942>

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::InjectedBundlePage::willSendRequestForFrame): Implement the required checks (using KURL, to avoid need to invent a whole URL API).
  • WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj: Link WebCore directly to get at KURL symbols.
1:31 AM Changeset in webkit [77793] by mjs@apple.com
  • 4 edits in trunk/Source/WebKit2

2011-02-07 Maciej Stachowiak <mjs@apple.com>

Reviewed by Antti Koivisto.

Add WKURLRequestCopyURL API (needed for tests and also generally useful)
https://bugs.webkit.org/show_bug.cgi?id=53909

  • Shared/API/c/WKURLRequest.cpp: (WKURLRequestCopyURL):
  • Shared/API/c/WKURLRequest.h:
  • Shared/WebURLRequest.h: (WebKit::WebURLRequest::url):
12:54 AM Changeset in webkit [77792] by Antti Koivisto
  • 2 edits in trunk/Source/JavaScriptCore

Not reviewed.

ASSERTS_DISABLED -> ASSERT_DISABLED

  • wtf/BloomFilter.h:

Feb 6, 2011:

11:20 PM Changeset in webkit [77791] by mjs@apple.com
  • 2 edits in trunk/Tools

2011-02-06 Maciej Stachowiak <mjs@apple.com>

Reviewed by Daniel Bates.

Add WebKit2 bot to list of expected builders in Python regression test results.
https://bugs.webkit.org/show_bug.cgi?id=53905

  • Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
10:49 PM Changeset in webkit [77790] by mjs@apple.com
  • 2 edits in trunk/Tools

2011-02-06 Maciej Stachowiak <mjs@apple.com>

Reviewed by Ryosuke Niwa.

Update test expectations for new core builder
https://bugs.webkit.org/show_bug.cgi?id=53904

  • Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
10:20 PM Changeset in webkit [77789] by mjs@apple.com
  • 2 edits in trunk/Tools

2011-02-06 Maciej Stachowiak <mjs@apple.com>

Reviewed by David Levin.

Add WebKit2 test bot to core builders
https://bugs.webkit.org/show_bug.cgi?id=53901

  • Scripts/webkitpy/common/net/buildbot/buildbot.py:
9:47 PM Changeset in webkit [77788] by rniwa@webkit.org
  • 2 edits in trunk/Tools

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Another unreviewed Chromium build fix.

  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::pathToLocalResource):
9:29 PM Changeset in webkit [77787] by rniwa@webkit.org
  • 4 edits in trunk

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed Chromium build fix.

  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::pathToLocalResource):

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed; speculative Qt build fix.

  • WebKit2.pro:
9:19 PM Changeset in webkit [77786] by rniwa@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed; speculative Qt build fix.

9:01 PM Changeset in webkit [77785] by rniwa@webkit.org
  • 44 edits
    3 deletes in trunk

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

OwnArraryPtr.h uses deleteOwnedPtr but doesn’t include OwnPtrCommon.h
https://bugs.webkit.org/show_bug.cgi?id=52867

  • DumpRenderTree/chromium/ImageDiff.cpp: (Image::craeteFromStdin): Call adoptArrayPtr.

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

OwnArraryPtr.h uses deleteOwnedPtr but doesn’t include OwnPtrCommon.h
https://bugs.webkit.org/show_bug.cgi?id=52867

Removed LOOSE_OWN_ARRAY_PTR and OwnArrayPtr<T>::set. Replaced all calls to OwnArrayPtr::set
and loose instantiation of OwnArrayPtr by calls to operator= and adoptArrayPtr. Also removed
OwnArrayPtrCommon.h since PassOwnArrayPtr.h needs to include OwnArrayPtr.h and there is
no point in putting deleteOwnedArrayPtr into a separate header.


Note: if this patch breaks build, the code is either instiantiating OwnArrayPtr
without calling adoptArrayPtr or calling set on ArrayOwnPtr instead of operator=.

No tests are added since this is a refactoring.

  • API/JSStringRefCF.cpp: (JSStringCreateWithCFString): Calls adoptArrayPtr.
  • GNUmakefile.am: Removed OwnArrayPtrCommon.h
  • JavaScriptCore.vcproj/WTF/WTF.vcproj: Ditto.
  • JavaScriptCore.xcodeproj/project.pbxproj: Ditto.
  • runtime/Arguments.cpp: (JSC::Arguments::deleteProperty): Calls adoptArrayPtr.
  • runtime/Arguments.h: (JSC::Arguments::copyRegisters): Ditto.
  • runtime/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::JSPropertyNameIterator): Ditto.
  • runtime/JSVariableObject.h: (JSC::JSVariableObject::setRegisters): Calls operator= instead of set.
  • runtime/StructureChain.cpp: (JSC::StructureChain::StructureChain): Ditto.
  • wtf/CMakeLists.txt:
  • wtf/DateMath.h: (JSC::GregorianDateTime::GregorianDateTime): No longer instnatiates OwnArrayPtr with a null pointer.
  • wtf/OwnArrayPtr.h:
  • wtf/OwnArrayPtrCommon.h: Removed.
  • wtf/PassOwnArrayPtr.h: No longer includes OwnArrayCommon.h (WTF::deleteOwnedArrayPtr): Moved from OwnArrayPtrCommon.h

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

OwnArraryPtr.h uses deleteOwnedPtr but doesn’t include OwnPtrCommon.h
https://bugs.webkit.org/show_bug.cgi?id=52867

  • ForwardingHeaders/wtf/OwnArrayPtrCommon.h: Removed.

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

OwnArraryPtr.h uses deleteOwnedPtr but doesn’t include OwnPtrCommon.h
https://bugs.webkit.org/show_bug.cgi?id=52867

  • WebHistory.cpp: (WebHistory::orderedLastVisitedDays): Calls adoptArrayPtr.
  • WebPreferences.cpp: (WebPreferences::copyWebKitPreferencesToCFPreferences): Ditto.

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

OwnArraryPtr.h uses deleteOwnedPtr but doesn’t include OwnPtrCommon.h
https://bugs.webkit.org/show_bug.cgi?id=52867

  • ForwardingHeaders/wtf/OwnArrayPtrCommon.h: Removed.
  • bindings/v8/NPV8Object.cpp: (WebCore::createValueListFromVariantArgs): Returns PassOwnArrayPtr<v8::Handle<v8::Value>> instead of v8::Handle<v8::Value>*. (_NPN_Invoke): Calls createValueListFromVariantArgs. (_NPN_InvokeDefault): createValueListFromVariantArgs. (_NPN_Construct): createValueListFromVariantArgs.
  • bindings/v8/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::call): Calls adoptArrayPtr. (WebCore::ScriptFunctionCall::construct): Ditto. (WebCore::ScriptCallback::call):
  • bindings/v8/V8DOMWindowShell.cpp: (WebCore::V8DOMWindowShell::createNewContext): Ditto.
  • bindings/v8/V8NPObject.cpp: (WebCore::npObjectInvokeImpl): Ditto
  • bindings/v8/V8NodeFilterCondition.cpp: (WebCore::V8NodeFilterCondition::acceptNode): Ditto,
  • html/HTMLAreaElement.cpp: (WebCore::HTMLAreaElement::parseMappedAttribute): Calls newCoordsArray.
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::parseMappedAttribute): Calls newLengthArray.
  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::getAttachedShaders): Calls adoptArraYptr. (WebCore::WebGLRenderingContext::simulateVertexAttrib0): Ditto. (WebCore::WebGLRenderingContext::LRUImageBufferCache::LRUImageBufferCache): Dito.
  • platform/Length.cpp: (WebCore::newCoordsArray): Returns PassOwnArrayPtr<Length> instead of Length*. (WebCore::newLengthArray): Returns PassOwnArrayPtr<Length> instead of Length*.
  • platform/Length.h: Prototype changes.
  • platform/graphics/ANGLEWebKitBridge.cpp: (WebCore::ANGLEWebKitBridge::validateShaderSource): Calls adoptArrayPtr.
  • platform/graphics/ca/win/CACFLayerTreeHost.cpp: (WebCore::getDirtyRects): Ditto.
  • platform/graphics/chromium/ComplexTextControllerLinux.cpp: (WebCore::ComplexTextController::getNormalizedTextRun): Ditto.
  • platform/graphics/chromium/HarfbuzzSkia.cpp: (WebCore::glyphsToAdvances): Ditto. (WebCore::canRender): Ditto.
  • platform/graphics/gpu/Texture.cpp: (WebCore::Texture::updateSubRect): Ditto.
  • platform/graphics/opengl/GraphicsContext3DOpenGL.cpp: (WebCore::GraphicsContext3D::paintRenderingResultsToCanvas): Ditto.
  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::readbackHardwareToSoftware): Ditto.
  • platform/graphics/wince/SharedBitmap.cpp: (WebCore::SharedBitmap::to16bit): Ditto.
  • platform/win/ContextMenuWin.cpp: (WebCore::ContextMenu::getContextMenuItems): Ditto.
  • platform/win/LoggingWin.cpp: (WebCore::initializeWithUserDefault): Ditto.
  • plugins/win/PluginPackageWin.cpp: (WebCore::PluginPackage::fetchInfo): Ditto.

2011-02-06 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

OwnArraryPtr.h uses deleteOwnedPtr but doesn’t include OwnPtrCommon.h
https://bugs.webkit.org/show_bug.cgi?id=52867

  • Shared/Plugins/Netscape/win/NetscapePluginModuleWin.cpp: (WebKit::NetscapePluginModule::getPluginInfo): Calls adoptArrayPtr.
8:54 PM Changeset in webkit [77784] by tkent@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-02-06 Kent Tamura <tkent@chromium.org>

Unreviewed.

  • WebCore.xcodeproj/project.pbxproj: Run sort-xcode-project-file.
8:08 PM Changeset in webkit [77783] by Patrick Gansterer
  • 2 edits in trunk/Source/WebKit/wince

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Martin Robinson.

[WINCE] FrameLoaderClient calls loadURLInChildFrame on the child's frame loader
https://bugs.webkit.org/show_bug.cgi?id=53896

  • WebView.cpp: (WebView::createFrame): Use coreFrame instead of childFrame.
6:20 PM Changeset in webkit [77782] by morrita@google.com
  • 2 edits in trunk/LayoutTests

2011-02-06 MORITA Hajime <morrita@google.com>

Unreviewed, unskip the html5lib test again which fails for another reason. (See Bug 53030)

  • platform/qt/Skipped:
5:40 PM Changeset in webkit [77781] by morrita@google.com
  • 13 edits in trunk

2011-02-06 Dominic Cooney <dominicc@google.com>

Reviewed by Dimitri Glazkov.

Convert <keygen> option elements to a shadow DOM
https://bugs.webkit.org/show_bug.cgi?id=51379

  • html5lib/runner-expected.txt: Two more tests pass.
  • platform/mac/fast/html/keygen-expected.txt: Shadow render tree.
  • platform/mac/fast/invalid/residual-style-expected.txt:

2011-02-03 Dominic Cooney <dominicc@google.com>

Reviewed by Dimitri Glazkov.

Convert <keygen> option elements to a shadow DOM
https://bugs.webkit.org/show_bug.cgi?id=51379

Covered by existing tests e.g. fast/html/keygen.html,
html5lib/runner.html, etc.

  • css/html.css: (form): Cleanup. (keygen, select): Border radii should match so focus ring looks good. (keygen::-webkit-keygen-select): No margin on the shadow element. (select): Hoisting border radius to keygen, select rule.
  • dom/SelectElement.cpp: (WebCore::toSelectElement): keygen is no longer a select element.
  • html/HTMLKeygenElement.cpp: Implements keygen shadow. (WebCore::KeygenSelectElement::create): (WebCore::KeygenSelectElement::shadowPseudoId): (WebCore::KeygenSelectElement::KeygenSelectElement): (WebCore::HTMLKeygenElement::HTMLKeygenElement): (WebCore::HTMLKeygenElement::parseMappedAttribute): (WebCore::HTMLKeygenElement::appendFormData): (WebCore::HTMLKeygenElement::formControlType): (WebCore::HTMLKeygenElement::reset): (WebCore::HTMLKeygenElement::selectShadow):
  • html/HTMLKeygenElement.h: keygen is no longer a select on C++ side. (WebCore::HTMLKeygenElement::canStartSelection): (WebCore::HTMLKeygenElement::isEnumeratable): (WebCore::HTMLKeygenElement::isResettable):
  • html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::ownerSelectElement): Only owned by selects.
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::HTMLSelectElement): Only 'select' tag name.
5:00 PM Changeset in webkit [77780] by commit-queue@webkit.org
  • 1 edit
    5 adds in trunk/Tools

2011-02-06 James Kozianski <koz@chromium.org>

Reviewed by Ojan Vafai.

Add classes for representing test outputs.
https://bugs.webkit.org/show_bug.cgi?id=52136

These classes will make it easier to write scripts that deal with test
outputs such as rebaselining and deduping scripts. The intent is that
eventually we will be have buildbot return TestOutputs for a
particular build which can be compared with TestOutputs derived from a
local LayoutTests directory.

  • Scripts/webkitpy/common/net/testoutput.py: Added.
  • Scripts/webkitpy/common/net/testoutput_unittest.py: Added.
  • Scripts/webkitpy/common/net/testoutputset.py: Added.
  • Scripts/webkitpy/common/net/testoutputset_unittest.py: Added.
  • Scripts/webkitpy/common/system/zip_mock.py: Added.
3:48 PM Changeset in webkit [77779] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-02-06 Benjamin Poulain <ikipou@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Fix coding style errors in ClipboardQt.h
https://bugs.webkit.org/show_bug.cgi?id=43520

Fix coding style: order of header and indentation.

  • platform/qt/ClipboardQt.h: (WebCore::ClipboardQt::create): (WebCore::ClipboardQt::clipboardData): (WebCore::ClipboardQt::invalidateWritableData):
2:16 PM Changeset in webkit [77778] by Patrick Gansterer
  • 1 edit
    6 deletes in trunk/LayoutTests

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Oliver Hunt.

Remove empty svg test
https://bugs.webkit.org/show_bug.cgi?id=53887

This files are empty. Empty tests are pretty useless.

  • platform/chromium-win/svg/custom/use-infinite-recursion-expected.png: Removed.
  • platform/mac/svg/custom/use-infinite-recursion-expected.checksum: Removed.
  • platform/mac/svg/custom/use-infinite-recursion-expected.png: Removed.
  • platform/mac/svg/custom/use-infinite-recursion-expected.txt: Removed.
  • platform/qt/svg/custom/use-infinite-recursion-expected.txt: Removed.
  • svg/custom/use-infinite-recursion.svg: Removed.
12:43 PM Changeset in webkit [77777] by Antti Koivisto
  • 8 edits
    2 adds in trunk/Source

Use bloom filter for descendant selector filtering
https://bugs.webkit.org/show_bug.cgi?id=53880

Reviewed by Maciej Stachowiak.

Source/JavaScriptCore:

Implement a bloom filter with k=2 and 8 bit counting.

(WTF::BloomFilter::maximumCount):
(WTF::BloomFilter::BloomFilter):
(WTF::BloomFilter::mayContain):
(WTF::BloomFilter::add):
(WTF::BloomFilter::remove):
(WTF::BloomFilter::firstSlot):
(WTF::BloomFilter::secondSlot):
(WTF::::add):
(WTF::::remove):
(WTF::::clear):
(WTF::::likelyEmpty):
(WTF::::isClear):

Source/WebCore:

Bloom filter is faster than a hash set in this kind of use.

Shark thinks this speeds up style matching by ~30% on sites
with lots of descendant selectors.

  • ForwardingHeaders/wtf/BloomFilter.h: Added.
  • css/CSSStyleSelector.cpp:

(WebCore::collectElementIdentifierHashes):
(WebCore::CSSStyleSelector::pushParent):
(WebCore::CSSStyleSelector::popParent):
(WebCore::CSSStyleSelector::fastRejectSelector):
(WebCore::RuleData::collectDescendantSelectorIdentifierHashes):

  • css/CSSStyleSelector.h:
12:05 PM Changeset in webkit [77776] by mjs@apple.com
  • 2 edits in trunk/Source/WebCore

2011-02-06 Maciej Stachowiak <mjs@apple.com>

Reviewed by Antti Koivisto.

media/video-document-types.html sometimes crashes the WebProcess
https://bugs.webkit.org/show_bug.cgi?id=53884

No new tests. Already covered by media/video-document-types.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::userCancelledLoad): Stop the load timer when cancelling the load.
10:52 AM Changeset in webkit [77775] by Patrick Gansterer
  • 15 edits in trunk/Source/WebCore

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Adam Roben.

[WIN] Use WCHAR instead of TCHAR
https://bugs.webkit.org/show_bug.cgi?id=53863

We always use the UNICODE versions of windows functions, so
the usage of TCHAR makes no sense and mixing them is bad style.

  • platform/graphics/win/IconWin.cpp: (WebCore::Icon::createIconForFiles):
  • platform/graphics/win/QTMovie.cpp:
  • platform/graphics/win/QTMovieGWorld.cpp: (QTMovieGWorld::fullscreenWndProc): (QTMovieGWorld::enterFullscreen):
  • platform/graphics/win/SimpleFontDataCGWin.cpp: (WebCore::SimpleFontData::platformInit):
  • platform/graphics/win/SimpleFontDataCairoWin.cpp:
  • platform/graphics/win/SimpleFontDataWin.cpp:
  • platform/graphics/wince/SimpleFontDataWinCE.cpp:
  • platform/win/BString.cpp:
  • platform/win/ClipboardWin.cpp: (WebCore::filesystemPathFromUrlOrTitle): (WebCore::createGlobalHDropContent): (WebCore::createGlobalImageFileDescriptor):
  • platform/win/ContextMenuWin.cpp:
  • platform/win/CursorWin.cpp: (WebCore::loadSharedCursor):
  • platform/win/FileChooserWin.cpp: (WebCore::FileChooser::basenameForWidth):
  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenuWin::popupClassName): (WebCore::PopupMenuWin::show):
  • platform/win/PopupMenuWin.h:
10:44 AM Changeset in webkit [77774] by Patrick Gansterer
  • 9 edits in trunk/Source/WebCore

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

Replace FLT_MAX with std::numeric_limits
https://bugs.webkit.org/show_bug.cgi?id=53861

Also move EFL's userIdleTime from TemporaryLinkStubs into SystemTimeEfl.

  • html/NumberInputType.cpp:
  • html/parser/HTMLParserIdioms.cpp: (WebCore::parseToDoubleForNumberType):
  • platform/brew/SystemTimeBrew.cpp: (WebCore::userIdleTime):
  • platform/efl/SystemTimeEfl.cpp: (WebCore::userIdleTime):
  • platform/efl/TemporaryLinkStubs.cpp:
  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::setupAnimation):
  • platform/win/SystemTimeWin.cpp: (WebCore::userIdleTime):
  • platform/wx/SystemTimeWx.cpp: (WebCore::userIdleTime):
10:39 AM Notes on Content Security Policies edited by abarth@webkit.org
(diff)
10:38 AM Notes on Content Security Policies created by abarth@webkit.org
10:31 AM Changeset in webkit [77773] by robert@webkit.org
  • 2 edits in trunk/Source/WebKit/qt

2011-02-06 Robert Hogan <robert@webkit.org>

Reviewed by Andreas Kling.

[Qt] Remove unnecessary QRect assignment
https://bugs.webkit.org/show_bug.cgi?id=53873

  • Api/qwebframe.cpp: (QWebFramePrivate::renderRelativeCoords):
10:18 AM Changeset in webkit [77772] by robert@webkit.org
  • 4 edits in trunk

2011-02-06 Robert Hogan <robert@webkit.org>

Reviewed by Andreas Kling.

[Qt] Clear page's groupName even when not in DRT
https://bugs.webkit.org/show_bug.cgi?id=53874

Matches behaviour of other ports.

  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::closeWindowSoon):

2011-02-06 Robert Hogan <robert@webkit.org>

Reviewed by Andreas Kling.

[Qt] Clear page's groupName even when not in DRT
https://bugs.webkit.org/show_bug.cgi?id=53874

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::windowCloseRequested):
9:47 AM Changeset in webkit [77771] by Patrick Gansterer
  • 6 edits in trunk

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

[CMake] Add dependencies for Visual Studio projects
https://bugs.webkit.org/show_bug.cgi?id=53773

Add a JavaScriptCore dependecy to WebCore, so CMake can
generate the correct build order for the solution.

  • CMakeLists.txt:

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

[CMake] Add dependencies for Visual Studio projects
https://bugs.webkit.org/show_bug.cgi?id=53773

Add a WebCore dependecy to WebKit, so CMake can
generate the correct build order for the solution.

Remove JavaScriptCore dependecy, since WebCore already depends on it.

  • CMakeLists.txt:

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

[CMake] Add dependencies for Visual Studio projects
https://bugs.webkit.org/show_bug.cgi?id=53773

Add a WebKit dependecy to WinCELauncher, so CMake can
generate the correct build order for the solution.

  • CMakeListsWinCE.txt:
9:14 AM Changeset in webkit [77770] by Patrick Gansterer
  • 2 edits in trunk/Source/WebCore

2011-02-06 Patrick Gansterer <Patrick Gansterer>

Reviewed by Andreas Kling.

[CMake] Unify ENABLE_SVG sections
https://bugs.webkit.org/show_bug.cgi?id=53778

  • CMakeLists.txt:
9:08 AM Changeset in webkit [77769] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2011-02-06 Martin Robinson <mrobinson@igalia.com>

Reclassify a failing test and unskip another which should be passing now.

  • platform/gtk/Skipped: Update the skipped list.
8:22 AM Changeset in webkit [77768] by andreas.kling@nokia.com
  • 5 edits in trunk/Source/WebCore

2011-02-06 Andreas Kling <kling@webkit.org>

Reviewed by Dirk Schulze.

Refactor Font::normalizeSpaces() to avoid an extra String copy.
https://bugs.webkit.org/show_bug.cgi?id=53871

Have normalizeSpaces() take UChar*,length arguments instead of a
String, since all call sites were creating temporaries from these
components anyway.

Font::normalizeSpaces() now always makes 1 String copy,
instead of 1 in the best case and 2 in the worst case.

  • platform/graphics/Font.cpp: (WebCore::Font::normalizeSpaces):
  • platform/graphics/Font.h:
  • platform/graphics/qt/FontQt.cpp: (WebCore::drawTextCommon): (WebCore::Font::floatWidthForSimpleText): (WebCore::Font::floatWidthForComplexText): (WebCore::Font::offsetForPositionForSimpleText): (WebCore::Font::offsetForPositionForComplexText): (WebCore::Font::selectionRectForSimpleText): (WebCore::Font::selectionRectForComplexText):
  • svg/SVGFont.cpp: (WebCore::SVGTextRunWalker::walk):
7:19 AM Changeset in webkit [77767] by andreas.kling@nokia.com
  • 4 edits in trunk/Tools

2011-02-06 Andreas Kling <kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt] MiniBrowser: Clean up handling of titleChanged() signal
https://bugs.webkit.org/show_bug.cgi?id=53869

  • MiniBrowser/qt/BrowserView.cpp: (BrowserView::BrowserView): Remove unnecessary titleChanged() handling.
  • MiniBrowser/qt/BrowserWindow.cpp: (BrowserWindow::BrowserWindow): Connect titleChanged() directly to setWindowTitle().
  • MiniBrowser/qt/BrowserWindow.h:
6:35 AM Changeset in webkit [77766] by andreas.kling@nokia.com
  • 2 edits in trunk/Source/WebCore

2011-02-06 Andreas Kling <kling@webkit.org>

Reviewed by Dirk Schulze.

Fix potential buffer overrun in SVGTextRunWalker::walk()
https://bugs.webkit.org/show_bug.cgi?id=53870

A new String was created from a UChar* with a 'length' argument
that could be greater than the number of UChars available.

  • svg/SVGFont.cpp: (WebCore::SVGTextRunWalker::walk):
5:23 AM Changeset in webkit [77765] by commit-queue@webkit.org
  • 11 edits in trunk/Source/WebKit2

2011-02-06 Benjamin Poulain <ikipou@gmail.com>

Reviewed by Andreas Kling.

[Qt] WebKit does not build on Mac with WebKit 2
https://bugs.webkit.org/show_bug.cgi?id=53380

  • Platform/CoreIPC/qt/ConnectionQt.cpp: Use fcntl() to set close on exec so we do not rely on recent kernel. (CoreIPC::Connection::readyReadHandler):
  • Platform/qt/SharedMemoryQt.cpp: ditto (WebKit::SharedMemory::create): (WebKit::SharedMemory::createHandle):
  • UIProcess/Launcher/qt/ProcessLauncherQt.cpp: Guard prctl() since it is a Linux specific function. (WebKit::QtWebProcess::setupChildProcess):
  • UIProcess/DrawingAreaProxy.h: Differentiate between building on Mac and the Mac port.
  • UIProcess/DrawingAreaProxyImpl.cpp: ditto
  • UIProcess/WebPageProxy.cpp: ditto (WebKit::WebPageProxy::didReceiveMessage):
  • WebProcess/WebPage/DrawingArea.cpp: ditto (WebKit::DrawingArea::create):
  • WebProcess/WebPage/DrawingArea.h:
  • WebProcess/WebPage/DrawingAreaImpl.cpp: ditto
  • WebProcess/WebPage/WebPage.cpp: ditto (WebKit::WebPage::didReceiveMessage):
2:46 AM Changeset in webkit [77764] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

2011-02-06 Robert Hogan <robert@webkit.org>

Unreviewed, skip failing test on Qt bot.

[Qt] fast/notifications/notifications-document-close-crash.html fails after r77738
https://bugs.webkit.org/show_bug.cgi?id=53868

  • platform/qt/Skipped: Skip fast/notifications/notifications-document-close-crash.html
2:35 AM Changeset in webkit [77763] by mjs@apple.com
  • 8 edits in trunk

2011-02-06 Maciej Stachowiak <mjs@apple.com>

Reviewed by Dan Bernstein.

WebKitTestRunner needs layoutTestController.setWillSendRequestReturnsNull
https://bugs.webkit.org/show_bug.cgi?id=42690
<rdar://problem/8213851>

  • platform/mac-wk2/Skipped: Unskip tests that now pass, except for the ones that mysteriously fail.

2011-02-06 Maciej Stachowiak <mjs@apple.com>

Reviewed by Dan Bernstein.

WebKitTestRunner needs layoutTestController.setWillSendRequestReturnsNull
https://bugs.webkit.org/show_bug.cgi?id=42690
<rdar://problem/8213851>

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::InjectedBundlePage::willSendRequestForFrame):
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::LayoutTestController):
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h: (WTR::LayoutTestController::willSendRequestReturnsNull): (WTR::LayoutTestController::setWillSendRequestReturnsNull):
Note: See TracTimeline for information about the timeline view.