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

Timeline



May 9, 2011:

10:41 PM Changeset in webkit [86131] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Try to fix the LLVM Compiler build.

  • page/EventHandler.h:
10:23 PM Changeset in webkit [86130] by weinig@apple.com
  • 15 edits in trunk

Unreviewed, rolling out r86108.
​http://trac.webkit.org/changeset/86108
​https://bugs.webkit.org/show_bug.cgi?id=60533

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-05-09
Broke makefile based builds and Cairo (Requested by weinig on
#webkit).

Source/WebKit/win:

  • WebKit.vcproj/WebKit.sln:

Tools:

  • Scripts/build-api-tests:
  • Scripts/run-api-tests:
  • TestWebKitAPI/Configurations/TestWebKitAPICommon.vsprops:
  • TestWebKitAPI/Test.h:

(TestWebKitAPI::Test::~Test):
(TestWebKitAPI::Test::name):
(TestWebKitAPI::Test::Register::Register):
(TestWebKitAPI::Test::Register::create):
(TestWebKitAPI::Test::Test):

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WTF/VectorBasic.cpp:

(TestWebKitAPI::TEST):

  • TestWebKitAPI/TestsController.cpp:

(TestWebKitAPI::TestsController::TestsController):
(TestWebKitAPI::TestsController::dumpTestNames):
(TestWebKitAPI::TestsController::runTestNamed):
(TestWebKitAPI::TestsController::testFailed):
(TestWebKitAPI::TestsController::registerCreateTestFunction):

  • TestWebKitAPI/TestsController.h:
  • TestWebKitAPI/mac/main.mm:

(main):

9:48 PM Changeset in webkit [86129] by dcheng@chromium.org
  • 1 edit
    1 add in trunk/Source/WebCore

2011-05-09 Daniel Cheng <​dcheng@chromium.org>

Fix the build.

Really add the file this time.

  • page/DragState.h: Added. (WebCore::DragState::DragState):
9:33 PM Changeset in webkit [86128] by dcheng@chromium.org
  • 12 edits in trunk/Source/WebCore

2011-05-09 Daniel Cheng <​dcheng@chromium.org>

Reviewed by Tony Chang.

Refactor RenderObject::draggableNode.
​https://bugs.webkit.org/show_bug.cgi?id=60503

This patch lays the ground for refactoring the drag start logic. It moves draggableNode to
DragController to avoid the awkward plumbing of calling DragController from RenderObject and
also moves EventHandlerDragState out from EventHandler so it can be shared between
EventHandler and DragController where appropriate.

No new tests since there should be no behavior change.

  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/DragController.cpp: (WebCore::DragController::draggableNode): (WebCore::DragController::mayStartDragAtEventLocation):
  • page/DragController.h:
  • page/DragState.h: Added. (WebCore::DragState::DragState):
  • page/EventHandler.cpp: (WebCore::EventHandler::dragState): (WebCore::EventHandler::eventMayStartDrag): (WebCore::EventHandler::handleDrag):
  • page/EventHandler.h:
  • rendering/RenderObject.cpp:
  • rendering/RenderObject.h:
8:58 PM Writing Reftests edited by hayato@chromium.org
(diff)
8:24 PM Changeset in webkit [86127] by cevans@google.com
  • 1 edit in branches/chromium/742/Source/WebCore/page/animation/CompositeAnimation.cpp

Merge 85693
BUG=82096
Review URL: ​http://codereview.chromium.org/6994006

7:40 PM Changeset in webkit [86126] by hayato@chromium.org
  • 5 edits in trunk/Tools

2011-05-09 Hayato Ito <​hayato@chromium.org>

Reviewed by Ojan Vafai.

Skip reftests if pixel tests are disabled.

​https://bugs.webkit.org/show_bug.cgi?id=59078

  • Scripts/webkitpy/layout_tests/layout_package/manager.py:
  • Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:
  • Scripts/webkitpy/layout_tests/layout_package/worker.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:
7:23 PM Changeset in webkit [86125] by Martin Robinson
  • 4 edits in trunk

2011-05-09 Martin Robinson <​mrobinson@igalia.com>

Update the results for a test after r86102. This test was susceptible to
scrollbar race conditions and r86102 changed timing slightly.

  • platform/gtk/scrollbars/custom-scrollbar-with-incomplete-style-expected.txt:

2011-05-09 Martin Robinson <​mrobinson@igalia.com>

Fix a test after r86102. Scrollbars update asynchronously, so we
must check the adjustments after a short delay.

  • tests/testwebview.c: (quit_after_short_delay_cb): Add a short delay before checking adjustments. (test_webkit_web_view_grab_focus): Ditto.
7:11 PM Changeset in webkit [86124] by commit-queue@webkit.org
  • 8 edits in trunk

2011-05-09 Chang Shu <​cshu@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] DumpRenderTreeQt needs an implementation of unmarkText
​https://bugs.webkit.org/show_bug.cgi?id=60499

Unskipped passed test.

  • platform/qt/Skipped:

2011-05-09 Chang Shu <​cshu@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] DumpRenderTreeQt needs an implementation of unmarkText
​https://bugs.webkit.org/show_bug.cgi?id=60499

The implementation is similar to gtk.

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::confirmComposition):
  • WebCoreSupport/DumpRenderTreeSupportQt.h:

2011-05-09 Chang Shu <​cshu@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] DumpRenderTreeQt needs an implementation of unmarkText
​https://bugs.webkit.org/show_bug.cgi?id=60499

The implementation is similar to gtk.

  • DumpRenderTree/qt/TextInputControllerQt.cpp: (TextInputController::unmarkText):
  • DumpRenderTree/qt/TextInputControllerQt.h:
7:05 PM Changeset in webkit [86123] by Lucas Forschler
  • 5 edits in tags/Safari-534.34.4/Source

Versioning to match updated WebBrowser.

7:01 PM Changeset in webkit [86122] by Lucas Forschler
  • 1 copy in tags/Safari-534.34.4

New tag.

6:55 PM Changeset in webkit [86121] by imasaki@chromium.org
  • 2 edits
    3 adds in trunk/LayoutTests

2011-05-09 Kenji Imasaki <​imasaki@chromium.org>

Reviewed by David Levin.

[Chromium] Add rebaseline images for video-controls-rendering.
​https://bugs.webkit.org/show_bug.cgi?id=60517

Chromium bug is ​http://crbug.com/82033. These rebaselines are necessary
due to unrelated bug (​http://crbug.com/74102). We are checking in bad
baselines for the time being so that we ensure no other problems occur.
This test will fail when above bug (​http://crbug.com/74102) is fixed.
At that time, we will checkin correct baselines.

  • platform/chromium-gpu-linux/media/video-controls-rendering -expected.png: Added.
  • platform/chromium-gpu-mac/media/video-controls-rendering-expected.png: Added.
  • platform/chromium-gpu-win/media/video-controls-rendering-expected.png: Added.
  • platform/chromium/test_expectations.txt:
6:52 PM Changeset in webkit [86120] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-05-09 Luke Macpherson <​macpherson@chromium.org>

Reviewed by Darin Adler.

Enable casting between CSSPrimitiveValue and FontWeight enum
​https://bugs.webkit.org/show_bug.cgi?id=60516

No new tests added as no functionality changed.

  • css/CSSPrimitiveValueMappings.h: (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): Implement cast from FontWeight to CSSPrimitiveValue. (WebCore::CSSPrimitiveValue::operator FontWeight): Implement cast from CSSPrimitiveValue to FontWeight
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Use new cast to reduce code size. Flatten switch inside if.
6:50 PM Changeset in webkit [86119] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

2011-05-09 James Robinson <​jamesr@chromium.org>

Reviewed by Eric Seidel.

[chromium] Teach new-run-webkit-tests to follow symlinks when trying to determine the arch of DumpRenderTree on linux
​https://bugs.webkit.org/show_bug.cgi?id=60520

  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
6:38 PM Changeset in webkit [86118] by xan@webkit.org
  • 2 edits in trunk/Source/WebKit/gtk

2011-05-09 Xan Lopez <​xlopez@igalia.com>

Unreviewed build fix.

Fix build, for real.

  • webkit/webkitwebview.cpp: (getHorizontalAdjustment):
6:30 PM Changeset in webkit [86117] by Martin Robinson
  • 2 edits in releases/WebKitGTK/webkit-1.4/Source/WebCore

Merging r85984

6:28 PM Changeset in webkit [86116] by xan@webkit.org
  • 2 edits in trunk/Source/WebKit/gtk

2011-05-09 Xan Lopez <​xlopez@igalia.com>

Unreviewed build fix..

Fix GTK+3.x build.

  • webkit/webkitwebview.cpp: (getHorizontalAdjustment): the adjustment is in adjustmentWatcher() now. (getVerticalAdjustment): ditto.
6:09 PM Changeset in webkit [86115] by Simon Fraser
  • 2 edits in trunk/Source/WebCore

2011-05-09 Simon Fraser <Simon Fraser>

Move IconURL.* into the dom grouping.

  • WebCore.xcodeproj/project.pbxproj:
6:01 PM Changeset in webkit [86114] by Martin Robinson
  • 4 edits
    1 add in releases/WebKitGTK/webkit-1.4/Source/WebCore

Merging r84979

5:59 PM Changeset in webkit [86113] by Martin Robinson
  • 2 edits in releases/WebKitGTK/webkit-1.4/Source/WebCore

Merging r84123

5:58 PM Changeset in webkit [86112] by Martin Robinson
  • 2 edits in releases/WebKitGTK/webkit-1.4/Source/WebKit/gtk

Merging r85221

5:57 PM Changeset in webkit [86111] by Martin Robinson
  • 2 edits in releases/WebKitGTK/webkit-1.4

Merging r85219

5:56 PM Changeset in webkit [86110] by Martin Robinson
  • 2 edits in releases/WebKitGTK/webkit-1.4/Source/WebKit/gtk

Merging r85390

5:25 PM Changeset in webkit [86109] by jer.noble@apple.com
  • 3 edits
    2 adds in trunk

2011-05-08 Jeremy Noble <​jer.noble@apple.com>

Reviewed by Dan Bernstein.

Full-screen video disappears behind black screen with scrollbar, followed by crash at jerryseinfeld.com
​https://bugs.webkit.org/show_bug.cgi?id=60449

  • fullscreen/full-screen-remove-sibling-expected.txt: Added.
  • fullscreen/full-screen-remove-sibling.html: Added.

2011-05-08 Jeremy Noble <​jer.noble@apple.com>

Reviewed by Dan Bernstein.

Full-screen video disappears behind black screen with scrollbar, followed by crash at jerryseinfeld.com
​https://bugs.webkit.org/show_bug.cgi?id=60449

Make RenderFullScreen a non-anonymous block, so that it does not get deleted when a parent attempts to
coalesce anonymous children.

Test: fullscreen/full-screen-remove-sibling.html

  • rendering/RenderObject.h: (WebCore::RenderObject::isAnonymousBlock):
5:17 PM Changeset in webkit [86108] by levin@chromium.org
  • 15 edits in trunk

Switch TestWebKitAPI to GTest
​https://bugs.webkit.org/show_bug.cgi?id=59561

Patch by Dmitry Lomov <​dslomov@google.com> on 2011-05-09
Reviewed by Sam Weinig.

.:

  • Source/ThirdParty/gtest/README.WebKit:
  • Source/ThirdParty/gtest/msvc/gtest-md.vcproj:

Source/WebKit/win:

  • WebKit.vcproj/WebKit.sln:

Tools:

  • Scripts/build-api-tests:
  • Scripts/run-api-tests:
  • TestWebKitAPI/Configurations/TestWebKitAPICommon.vsprops:
  • TestWebKitAPI/Test.h:
  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WTF/VectorBasic.cpp:

(TestWebKitAPI::TEST):

  • TestWebKitAPI/TestsController.cpp:

(TestWebKitAPI::TestsController::TestsController):
(TestWebKitAPI::TestsController::dumpTestNames):
(TestWebKitAPI::TestsController::runTestNamed):
(TestWebKitAPI::TestsController::runAllTests):

  • TestWebKitAPI/TestsController.h:
  • TestWebKitAPI/mac/main.mm:

(main):

4:50 PM Changeset in webkit [86107] by jer.noble@apple.com
  • 4 edits in trunk/Source

2011-05-09 Jeremy Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

Command-~/Exposé to another window while <video> is in full screen, breaks Safari, eventually leading to crash
​https://bugs.webkit.org/show_bug.cgi?id=60496

When animating into and out of full screen, order out and back in the original webView window,
disabling animations when necessary. The exitFullscreen method was targeting the wrong window,
as the webView is hosted in the full screen window when we ask for it's parent window.

  • WebView/WebFullScreenController.mm: (-[WebFullScreenController windowDidEnterFullscreen:]): (-[WebFullScreenController exitFullscreen]):

2011-05-09 Jeremy Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

Command-~/Exposé to another window while <video> is in full screen, breaks Safari, eventually leading to crash
​https://bugs.webkit.org/show_bug.cgi?id=60496

When animating into and out of full screen, order out and back in the original webView window,
disabling animations when necessary. The beganExitFullScreenAnimation method was targeting the wrong window,
as the webView is hosted in the full screen window when we ask for it's parent window.

  • UIProcess/mac/WKFullScreenWindowController.mm: (-[WKFullScreenWindowController finishedEnterFullScreenAnimation:]): (-[WKFullScreenWindowController beganExitFullScreenAnimation]):
4:45 PM Changeset in webkit [86106] by Darin Adler
  • 2 edits in trunk/Source/JavaScriptCore

2011-05-09 Darin Adler <Darin Adler>

Reviewed by Oliver Hunt.

​http://bugs.webkit.org/show_bug.cgi?id=60509
Wrong type used for return value from strlen

  • wtf/FastMalloc.cpp: (WTF::fastStrDup): Use size_t. Also don't bother checking for failure since fastMalloc won't return if it fails.
4:41 PM Changeset in webkit [86105] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebCore

2011-05-09 Jeremy Noble <​jer.noble@apple.com>

Reviewed by Simon Fraser.

REGRESSION (r84706): Crazy rendering of vimeo.com after exiting fullscreen video
​https://bugs.webkit.org/show_bug.cgi?id=60512

Initialize m_containsFullScreenElement.

  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::HTMLFrameElementBase):
4:37 PM Changeset in webkit [86104] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-09 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Darin Adler.

REGRESSION(r86039): DragCaretController doesn't draw caret
​https://bugs.webkit.org/show_bug.cgi?id=60505

Removed an early exit in setCaretPosition introduced by r86039.
Also replaced calls to containerNode by deprecatedNode since rendering code doesn't
support container node for rendering caret.

  • editing/FrameSelection.cpp: (WebCore::DragCaretController::setCaretPosition):
4:31 PM Changeset in webkit [86103] by mitz@apple.com
  • 3 edits in trunk/Source/WebKit2

A WKView with plug-ins paints white when first moved into a window
​https://bugs.webkit.org/show_bug.cgi?id=60511

Reviewed by Darin Adler.

The UI process’s wait for DidUpdateBackingStoreState would always time out because the Web process
was blocked on a synchronous message the UI process trying to get the connections for plug-ins in
the view. This is avoided by deferring the call to Page::setCanStartMedia().

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage): Initialize the new m_setCanStartMediaTimer.
(WebKit::WebPage::setCanStartMediaTimerFired): Added. Calls setCanStartMedia(true) when the timer
fires.
(WebKit::WebPage::setIsInWindow): When going out of a window, stop the timer. When going into a
window, start it.

  • WebProcess/WebPage/WebPage.h:
4:24 PM Changeset in webkit [86102] by Martin Robinson
  • 14 edits
    2 adds in trunk/Source

2011-04-29 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Untangle GtkAdjustments from WebCore
​https://bugs.webkit.org/show_bug.cgi?id=59821

Completely remove the ability for WebKit to set GtkAdjustments on ScrollViews
and, consequently, Scrollbars. Handling adjustment updates can now be managed
in WebKit.

No new tests. This is covered by existing Scrollbar tests.

  • platform/ScrollView.cpp: platformInit and platformDestroy can now be shared between ScrollView.cpp and ScrollViewGtk.cpp.
  • platform/ScrollView.h: Remove unused methods.
  • platform/gtk/MainFrameScrollbarGtk.cpp: (MainFrameScrollbarGtk::create): No longer need to pass in the adjustment. (MainFrameScrollbarGtk::MainFrameScrollbarGtk): Update the comments here.
  • platform/gtk/MainFrameScrollbarGtk.h: Lots of code removal.
  • platform/gtk/ScrollViewGtk.cpp: Ditto. (WebCore::ScrollView::createScrollbar): Ditto. (WebCore::ScrollView::setScrollbarModes): Ditto.
  • plugins/gtk/PluginViewGtk.cpp: This file needs the GRefPtrGtk.h include now that it doesn't receive it transitively.

2011-04-29 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Untangle GtkAdjustments from WebCore
​https://bugs.webkit.org/show_bug.cgi?id=59821

Have ChromeClient manage the GtkAdjustments of the parent widget instead of passing
them to WebCore to manage. This simplifies the code quite a bit and should be less
crash prone in the future.

  • GNUmakefile.am: Added new GtkAdjustmentWatcher class to the sources list.
  • WebCoreSupport/ChromeClientGtk.cpp: Added new GtkAdjustmentWatcher member. (WebKit::ChromeClient::ChromeClient) Initialize the new member. (WebKit::ChromeClient::scroll): Poke the adjustment watcher to update the adjustments. (WebKit::ChromeClient::contentsSizeChanged): Ditto. (WebKit::ChromeClient::scrollbarsModeDidChange): This may now be called at a time when there is no main frame, so do an early return in that case.
  • WebCoreSupport/ChromeClientGtk.h: Add the new member. (WebKit::ChromeClient::adjustmentWatcher): Add this getter for WebKitWebView.
  • WebCoreSupport/FrameLoaderClientGtk.cpp: Remove calls which pushed the adjustments into WebCore. (WebKit::FrameLoaderClient::savePlatformDataToCachedFrame): (WebKit::postCommitFrameViewSetup):
  • WebCoreSupport/GtkAdjustmentWatcher.cpp: Added. (WebKit::GtkAdjustmentWatcher::GtkAdjustmentWatcher): (WebKit::updateAdjustmentFromScrollbar): (WebKit::GtkAdjustmentWatcher::updateAdjustmentsFromScrollbars): (WebKit::updateAdjustmentCallback): (WebKit::GtkAdjustmentWatcher::updateAdjustmentsFromScrollbarsLater): (WebKit::adjustmentValueChangedCallback): (WebKit::setAdjustment): (WebKit::GtkAdjustmentWatcher::setHorizontalAdjustment): (WebKit::GtkAdjustmentWatcher::setVerticalAdjustment): (WebKit::GtkAdjustmentWatcher::adjustmentValueChanged):
  • WebCoreSupport/GtkAdjustmentWatcher.h: Added. (WebKit::GtkAdjustmentWatcher::horizontalAdjustment): (WebKit::GtkAdjustmentWatcher::verticalAdjustment):
  • webkit/webkitwebview.cpp: (setHorizontalAdjustment): (setVerticalAdjustment): Remove the adjustment member. Call into the adjustment watcher. (getHorizontalAdjustment): Ditto. (getVerticalAdjustment): Ditto. (webkit_web_view_size_allocate): Ditto. (webkit_web_view_set_scroll_adjustments): Ditto. (webkit_web_view_dispose): Ditto. (webkit_web_view_init): Ditto.
  • webkit/webkitwebviewprivate.h: Ditto.
4:08 PM Changeset in webkit [86101] by andersca@apple.com
  • 4 edits in trunk/Source/WebKit2

2011-05-09 Anders Carlsson <​andersca@apple.com>

Reviewed by Dan Bernstein.

Crash when sending a sync message to a crashed web process
​https://bugs.webkit.org/show_bug.cgi?id=60514
<rdar://problem/9281270>

Change WebProcessProxy::sendSync to return false if there's no connection.

Audit all WebProcessProxy::sendSync call sites to initialize reply parameters.

  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::firstRectForCharacterInSelectedRange): (WebKit::WebPageProxy::getSelectedText): (WebKit::WebPageProxy::gestureWillBegin):
  • UIProcess/WebProcessProxy.h: (WebKit::WebProcessProxy::sendSync):
  • UIProcess/mac/WebPageProxyMac.mm: (WebKit::WebPageProxy::insertText): (WebKit::WebPageProxy::getMarkedRange): (WebKit::WebPageProxy::getSelectedRange): (WebKit::WebPageProxy::characterIndexForPoint): (WebKit::WebPageProxy::executeKeypressCommands): (WebKit::WebPageProxy::writeSelectionToPasteboard): (WebKit::WebPageProxy::readSelectionFromPasteboard):
4:05 PM Changeset in webkit [86100] by abarth@webkit.org
  • 9 edits
    5 adds in trunk

2011-05-09 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

CSP should block Function constructor
​https://bugs.webkit.org/show_bug.cgi?id=60240

Test that the function constructor is properly blocked.

  • http/tests/security/contentSecurityPolicy/function-constructor-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/function-constructor-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/function-constructor-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/function-constructor-blocked.html: Added.
  • platform/chromium/test_expectations.txt:

2011-05-09 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

CSP should block Function constructor
​https://bugs.webkit.org/show_bug.cgi?id=60240

When eval is disabled, we need to block the use of the function
constructor. However, the WebCore JSC bindings call the function
constructor directly to create inline event listeners. To support that
use, this patch adds an entrypoint that bypasses the check for whether
eval is enabled.

  • JavaScriptCore.exp:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
  • runtime/FunctionConstructor.cpp: (JSC::constructFunction): (JSC::constructFunctionSkippingEvalEnabledCheck):
  • runtime/FunctionConstructor.h:

2011-05-09 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

CSP should block Function constructor
​https://bugs.webkit.org/show_bug.cgi?id=60240

Tests: http/tests/security/contentSecurityPolicy/function-constructor-allowed.html

http/tests/security/contentSecurityPolicy/function-constructor-blocked.html

  • bindings/js/JSLazyEventListener.cpp: (WebCore::JSLazyEventListener::initializeJSFunction):
    • Update call site to the new entrypoint.
  • bindings/v8/V8LazyEventListener.cpp: (WebCore::V8LazyEventListener::prepareListenerObject):
    • Add some comments about the rediculousness of this implementation.
4:02 PM Changeset in webkit [86099] by crogers@google.com
  • 4 edits
    1 add in trunk/Source/WebCore

2011-05-09 Chris Rogers <​crogers@google.com>

Reviewed by Kenneth Russell.

Add RAII-style DenormalDisabler class to disable denormals which hurt audio performance
​https://bugs.webkit.org/show_bug.cgi?id=60396

No new tests since audio API is not yet implemented.

  • WebCore.gypi:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/audio/DenormalDisabler.h: Added. (WebCore::DenormalDisabler::DenormalDisabler): (WebCore::DenormalDisabler::~DenormalDisabler): (WebCore::DenormalDisabler::getCSR): (WebCore::DenormalDisabler::setCSR):
  • webaudio/AudioDestinationNode.cpp: (WebCore::AudioDestinationNode::provideInput):
3:53 PM Changeset in webkit [86098] by tony@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-05-09 Tony Chang <​tony@chromium.org>

Reviewed by Eric Seidel.

Style cleanups in RenderFlexibleBox.cpp
​https://bugs.webkit.org/show_bug.cgi?id=60504

No new tests, just refactoring.

  • rendering/RenderFlexibleBox.cpp: (WebCore::FlexBoxIterator::FlexBoxIterator): Use m_ for member variables, and

use class initializer list.

(WebCore::FlexBoxIterator::reset):
(WebCore::FlexBoxIterator::next):
(WebCore::RenderFlexibleBox::RenderFlexibleBox):
(WebCore::marginWidthForChild): Pull out common code used in calc*PrefWidths
(WebCore::RenderFlexibleBox::calcHorizontalPrefWidths):
(WebCore::RenderFlexibleBox::calcVerticalPrefWidths):
(WebCore::RenderFlexibleBox::computePreferredLogicalWidths):
(WebCore::RenderFlexibleBox::layoutBlock):
(WebCore::gatherFlexChildrenInfo): Rewrite uses of FlexBoxIterator into for statements

rather than while loops.

(WebCore::RenderFlexibleBox::layoutHorizontalBox): ditto
(WebCore::RenderFlexibleBox::layoutVerticalBox): ditto
(WebCore::RenderFlexibleBox::applyLineClamp):
(WebCore::RenderFlexibleBox::allowedChildFlex): Rename variables w/h to width/height.

3:49 PM Changeset in webkit [86097] by andersca@apple.com
  • 6 edits in trunk/Source/WebKit2

2011-05-09 Anders Carlsson <​andersca@apple.com>

Reviewed by Sam Weinig.

Create a plug-in's MIME types preferences plist if it doesn't exist.
​https://bugs.webkit.org/show_bug.cgi?id=57204
<rdar://problem/9195048>

Spawn a plug-in process and let it create the plist. For now, we'll do this on the
main thread since it's a rare one-time computation and we have an API function,
WKFrameCanShowMIMEType, which would have to block anyway.

  • PluginProcess/mac/PluginProcessMainMac.mm: (WebKit::PluginProcessMain): If -createPluginMIMETypesPreferences is one of the command line argument keys, the plug-in path will be the argument value. Call NetscapePluginModule::createPluginMIMETypesPreferences to create the preferences file and then exit.
  • Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm: (WebKit::contentsOfPropertyListAtURL): Move code to read a property list out into a helper function.

(WebKit::getMIMETypesFromPluginBundle):
If there's no property list file, try to create one.

(WebKit::NetscapePluginModule::createPluginMIMETypesPreferences):
Load the plug-in, find the BP_CreatePluginMIMETypesPreferences and call it.

  • UIProcess/Plugins/mac/PluginProcessProxyMac.mm: (WebKit::PluginProcessProxy::createPropertyListFile): Spawn a plug-in process and pass the -createPluginMIMETypesPreferences flag to it and then wait for it to exit.
3:47 PM Changeset in webkit [86096] by robert@webkit.org
  • 3 edits in trunk/LayoutTests

2011-05-09 Robert Hogan <​robert@webkit.org>

Unreviewed, update test expectations.

Update platform-specific results after r86086

The unix test NPAPI plugin now supports the png mime type.

​https://bugs.webkit.org/show_bug.cgi?id=57137

  • platform/gtk/plugins/plugin-javascript-access-expected.txt:
  • platform/qt-linux/plugins/plugin-javascript-access-expected.txt:
3:36 PM Changeset in webkit [86095] by imasaki@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-09 Kenji Imasaki <​imasaki@chromium.org>

Reviewed by David Levin.

[Chromium] Remove unflaky media-related tests from test expectation.
​https://bugs.webkit.org/show_bug.cgi?id=60500

Chromium bug is ​http://crbug.com/81660.

  • platform/chromium/test_expectations.txt:
3:29 PM Changeset in webkit [86094] by commit-queue@webkit.org
  • 9 edits
    1 delete in trunk/Source/WebKit/chromium

2011-05-09 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r86091.
​http://trac.webkit.org/changeset/86091
​https://bugs.webkit.org/show_bug.cgi?id=60510

Broke Chromium win release build. (Requested by dave_levin on
#webkit).

  • WebKit.gyp:
  • features.gypi:
  • public/WebFrame.h:
  • public/WebFrameClient.h: (WebKit::WebFrameClient::didChangeIcons):
  • public/WebIconURL.h: Removed.
  • src/AssertMatchingEnums.cpp:
  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::dispatchDidChangeIcons):
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::favIconURL):
  • src/WebFrameImpl.h:
3:27 PM Changeset in webkit [86093] by eric@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-09 Eric Seidel <​eric@webkit.org>

Reviewed by Ryosuke Niwa.

Abstract line deletion code into a function
​https://bugs.webkit.org/show_bug.cgi?id=60501

No change in behavior, thus no testing.

  • rendering/RenderBlockLineLayout.cpp: (WebCore::deleteLineRange): (WebCore::RenderBlock::layoutRunsAndFloats): (WebCore::RenderBlock::determineStartPosition): (WebCore::RenderBlock::matchedEndLine):
3:14 PM Changeset in webkit [86092] by Darin Adler
  • 3 edits in trunk/Source/WebCore

Cleared some stray executable bits on source files.

  • bindings/js/WorkerScriptDebugServer.cpp: Removed property svn:executable.
  • bindings/js/WorkerScriptDebugServer.h: Removed property svn:executable.
3:04 PM Changeset in webkit [86091] by commit-queue@webkit.org
  • 9 edits
    1 add in trunk/Source/WebKit/chromium

2011-05-09 Tao Bai <​michaelbai@chromium.org>

Reviewed by David Levin.

Handle the touch icon.
​https://bugs.webkit.org/show_bug.cgi?id=60247

Added WebIconURL for the corresponding IconURL.
Added a parameter to favIconURL() to specify the type of icon need to
return.

  • WebKit.gyp:
  • features.gypi:
  • public/WebFrame.h:
  • public/WebFrameClient.h: (WebKit::WebFrameClient::didChangeIcons):
  • public/WebIconURL.h: Added. (WebKit::WebIconURL::WebIconURL):
  • src/AssertMatchingEnums.cpp:
  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::dispatchDidChangeIcons):
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::favIconURL):
  • src/WebFrameImpl.h:
2:56 PM Changeset in webkit [86090] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

2011-05-09 Igor Oliveira <​igor.oliveira@openbossa.org>

Reviewed by Andreas Kling.

[Qt] DRT eventSender.addTouchPoint method is not setting the start position
​https://bugs.webkit.org/show_bug.cgi?id=60487

When adding a new touch point, the start position of this point needs to be set.

  • DumpRenderTree/qt/EventSenderQt.cpp: (EventSender::addTouchPoint):
2:47 PM Changeset in webkit [86089] by andersca@apple.com
  • 3 edits
    2 adds in trunk/Source/WebKit2

2011-05-09 Anders Carlsson <​andersca@apple.com>

Reviewed by Sam Weinig.

Move EnvironmentVariables out into its own .cpp file
​https://bugs.webkit.org/show_bug.cgi?id=60507

  • UIProcess/Launcher/mac/EnvironmentVariables.cpp: Added. (WebKit::EnvironmentVariables::EnvironmentVariables): (WebKit::EnvironmentVariables::~EnvironmentVariables): (WebKit::EnvironmentVariables::set): (WebKit::EnvironmentVariables::get): (WebKit::EnvironmentVariables::appendValue): (WebKit::EnvironmentVariables::valueIfVariableHasName): (WebKit::EnvironmentVariables::createStringForVariable): (WebKit::EnvironmentVariables::copyEnvironmentVariables):
  • UIProcess/Launcher/mac/EnvironmentVariables.h: Added. (WebKit::EnvironmentVariables::environmentPointer):
  • UIProcess/Launcher/mac/ProcessLauncherMac.mm:
  • WebKit2.xcodeproj/project.pbxproj:
2:42 PM Changeset in webkit [86088] by arv@chromium.org
  • 3 edits
    2 adds in trunk

2011-05-09 Erik Arvidsson <​arv@chromium.org>

Reviewed by Darin Adler.

radiogroup: onchange not detected when triggered by keyboard
​https://bugs.webkit.org/show_bug.cgi?id=32013

  • fast/forms/radio-group-keyboard-change-event-expected.txt: Added.
  • fast/forms/radio-group-keyboard-change-event.html: Added.

2011-05-09 Erik Arvidsson <​arv@chromium.org>

Reviewed by Darin Adler.

radiogroup: onchange not detected when triggered by keyboard
​https://bugs.webkit.org/show_bug.cgi?id=32013

Test: fast/forms/radio-group-keyboard-change-event.html

  • html/RadioInputType.cpp: (WebCore::RadioInputType::handleKeydownEvent): Ensure that we do not check the radio input before we simulate the click

event. The simulated click event will check it for us but more importantly it will fire the "change" event as
expected.

2:20 PM Changeset in webkit [86087] by abarth@webkit.org
  • 2 edits
    2 adds in trunk

2011-05-09 Adam Barth <​abarth@webkit.org>

Reviewed by Daniel Bates.

XSSAuditor should be more selective about the <meta http-equivs> that it blocks
​https://bugs.webkit.org/show_bug.cgi?id=60489

We don't need to filter most http-equiv attributes. This patch
introduces a blacklist for two that we probably do want to filter.
It's possible a whitelist would be more appropriate, but I'm inclined
to start with a blacklist and see how it works.

This patch will hopefully fix a false positive that is causing errors
with copy-and-pasted text in Gmail in some configurations (due to using
the <meta> tag to request UTF-8 encoding both in the pasted text and in
the page itself).

  • html/parser/XSSFilter.cpp: (WebCore::isNonCanonicalCharacter): (WebCore::canonicalize): (WebCore::isRequiredForInjection): (WebCore::hasName): (WebCore::findAttributeWithName): (WebCore::isNameOfInlineEventHandler): (WebCore::isDangerousHTTPEquiv):
    • This function is new in the patch and includes a blacklist of dangerous http-equivs. Many of the other functions listed here are just being moved from an anonymous namespace to use static for internal linkage.

(WebCore::containsJavaScriptURL):
(WebCore::decodeURL):
(WebCore::XSSFilter::eraseAttributeIfInjected):

2:09 PM Changeset in webkit [86086] by robert@webkit.org
  • 5 edits in trunk

2011-05-09 Robert Hogan <​robert@webkit.org>

Reviewed by Adam Roben.

​https://bugs.webkit.org/show_bug.cgi?id=57137

Unskip plugins/embed-prefers-plugins-for-images.html

Add image/png mime-type to test netscape plugin on Unix.

  • platform/gtk/Skipped:
  • platform/qt/Skipped:

2011-05-09 Robert Hogan <​robert@webkit.org>

Reviewed by Adam Roben.

​https://bugs.webkit.org/show_bug.cgi?id=57137

Unskip plugins/embed-prefers-plugins-for-images.html

Add image/png mime-type to test netscape plugin on Unix.

  • DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp: (NP_GetMIMEDescription):
1:22 PM Changeset in webkit [86085] by mdelaney@apple.com
  • 2 edits in trunk/Source/WebCore

2011-05-05 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Simon Fraser.

Remove canvas usage of roundToDevicePixels
​https://bugs.webkit.org/show_bug.cgi?id=60331

No new tests. This could only affect canvases at non-integral scale factors. Our tests only run in 1.0x.

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::drawImage):
1:20 PM Changeset in webkit [86084] by xan@webkit.org
  • 2 edits in trunk

2011-05-09 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Fix logic in GCC version detection, it was reversed.

  • configure.ac:
12:57 PM Changeset in webkit [86083] by alexis.menard@openbossa.org
  • 4 edits in trunk

2011-05-09 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Andreas Kling.

[Qt] Fix the build on Embedded Linux.
​https://bugs.webkit.org/show_bug.cgi?id=60347

qpa is the name for lighthouse but the old name for Qt for Embedded Linux should
still be used in order to build properly.

  • Source/WebKit.pri:

2011-05-09 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Andreas Kling.

[Qt] Fix the build on Embedded Linux.
​https://bugs.webkit.org/show_bug.cgi?id=60347

qpa is the name for lighthouse but the old name for Qt for Embedded Linux should
still be used in order to build properly.

  • WebCore.pri:
12:49 PM Changeset in webkit [86082] by chang.shu@nokia.com
  • 2 edits
    1 move
    1 delete in trunk/LayoutTests

2011-05-09 Chang Shu <​cshu@webkit.org>

Unreviewed.

Expected result for editing/inserting/insert-bg-font.html can be shared
​https://bugs.webkit.org/show_bug.cgi?id=60498

  • editing/inserting/insert-bg-font-expected.txt: Copied from LayoutTests/platform/mac/editing/inserting/insert-bg-font-expected.txt.
  • platform/gtk/editing/inserting/insert-bg-font-expected.txt: Removed.
  • platform/mac/editing/inserting/insert-bg-font-expected.txt: Removed.
  • platform/qt/Skipped:
12:40 PM Changeset in webkit [86081] by andreas.kling@nokia.com
  • 2 edits in trunk/Source/WebCore

2011-05-09 Andreas Kling <​kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Simple small-caps text is displayed in uniform size.
​https://bugs.webkit.org/show_bug.cgi?id=60492

Covered by existing tests.

  • platform/graphics/qt/FontPlatformDataQt.cpp: (WebCore::FontPlatformData::FontPlatformData): In the FontPlatformData constructor used by SimpleFontData::scaledFontData(), pass the pixel size to both QFont and QRawFont to make small caps and emphasis marks have the correct size when rendered through the fast font path.
12:32 PM Changeset in webkit [86080] by evan@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-05-09 Evan Martin <​evan@chromium.org>

Reviewed by Tony Chang.

[chromium] make DumpRenderTree build in components for linux
​https://bugs.webkit.org/show_bug.cgi?id=60491

  • WebKit.gyp: move some code that was in a Windows-only block to be used on all platforms.
12:32 PM Changeset in webkit [86079] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitLibraries

2011-05-09 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r86075.
​http://trac.webkit.org/changeset/86075
​https://bugs.webkit.org/show_bug.cgi?id=60495

broke windows tests (Requested by jessieberlin on #webkit).

  • win/lib/WebKitSystemInterface.lib:
12:22 PM Changeset in webkit [86078] by andreas.kling@nokia.com
  • 2 edits in trunk/Source/WebCore

2011-05-09 Andreas Kling <​kling@webkit.org>

Reviewed by Tor Arne Vestbø.

[Qt] Remove unused FontPlatformData functions.
​https://bugs.webkit.org/show_bug.cgi?id=60485

  • platform/graphics/qt/FontPlatformData.h: (WebCore::FontPlatformData::bold): Removed. (WebCore::FontPlatformData::family): Removed. (WebCore::FontPlatformData::italic): Removed. (WebCore::FontPlatformData::smallCaps): Removed.
12:20 PM Changeset in webkit [86077] by Adam Roben
  • 2 edits in trunk/Source/WebKit2

Always set the font rendering mode based on preference values

WebPage::platformInitialize and WebPage::platformPreferencesDidChange were fighting over the
font rendering mode. The platformInitialize code predates preference-based font rendering
mode support, and is now redundant and incorrect.

Fixes <​http://webkit.org/b/60494> WebKit2 uses GDI text instead of CG text when
FontSmoothing preference is set to kWKFontSmoothingLevelNoSubpixelAntiAliasing in a .plist
on disk

Reviewed by Sam Weinig.

  • WebProcess/WebPage/win/WebPageWin.cpp:

(WebKit::WebPage::platformInitialize): Removed code to set the font rendering mode.
platformPreferencesDidChange will take care of this for us.

12:07 PM Changeset in webkit [86076] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-05-09 Anders Carlsson <​andersca@apple.com>

Reviewed by John Sullivan.

Clean up NetscapePluginModule::getPluginInfo
​https://bugs.webkit.org/show_bug.cgi?id=60486

Make all 'get' functions take a PluginInfoStore::Plugin reference, since getPluginInfoFromPropertyLists
is going to need it to know the path and plug-in architecture.

  • Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm: (WebKit::getPluginArchitecture): (WebKit::getPluginInfoFromPropertyLists): (WebKit::getPluginInfoFromCarbonResources): (WebKit::NetscapePluginModule::getPluginInfo):
12:04 PM Changeset in webkit [86075] by jberlin@webkit.org
  • 2 edits in trunk/WebKitLibraries

Take 2 on updating the WKSI libraries.

  • win/lib/WebKitSystemInterface.lib:
11:58 AM Changeset in webkit [86074] by mitz@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9005982> REGRESSION (r85780): Flash of white when a WKView with composited content moves on-screen
​https://bugs.webkit.org/show_bug.cgi?id=60488

Reviewed by Darin Adler.

  • WebProcess/WebPage/DrawingAreaImpl.cpp:

(WebKit::DrawingAreaImpl::sendDidUpdateBackingStoreState): Update m_compositingAccordingToProxyMessages,
since the DidUpdateBackingStoreState message also changes the compositing state in the proxy.

11:54 AM Changeset in webkit [86073] by inferno@chromium.org
  • 2 edits
    15 adds in trunk/LayoutTests

2011-05-09 Abhishek Arya <​inferno@chromium.org>

Unreviewed.

Rebaselines for r86065.

  • platform/chromium-linux/platform/gtk: Added.
  • platform/chromium-linux/platform/gtk/fonts: Added.
  • platform/chromium-linux/platform/gtk/fonts/fontconfig-synthetic-bold-expected.png: Added.
  • platform/chromium-mac-leopard/platform: Added.
  • platform/chromium-mac-leopard/platform/gtk: Added.
  • platform/chromium-mac-leopard/platform/gtk/fonts: Added.
  • platform/chromium-mac-leopard/platform/gtk/fonts/fontconfig-synthetic-bold-expected.png: Added.
  • platform/chromium-mac/platform/gtk: Added.
  • platform/chromium-mac/platform/gtk/fonts: Added.
  • platform/chromium-mac/platform/gtk/fonts/fontconfig-synthetic-bold-expected.png: Added.
  • platform/chromium-mac/platform/gtk/fonts/fontconfig-synthetic-bold-expected.txt: Added.
  • platform/chromium-win/platform/gtk: Added.
  • platform/chromium-win/platform/gtk/fonts: Added.
  • platform/chromium-win/platform/gtk/fonts/fontconfig-synthetic-bold-expected.png: Added.
  • platform/chromium-win/platform/gtk/fonts/fontconfig-synthetic-bold-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
11:26 AM Changeset in webkit [86072] by kevino@webkit.org
  • 2 edits in trunk/Source/WebCore

Reviewed by Kevin Ollivier.

[wx] Add invalid file handle check.

​https://bugs.webkit.org/show_bug.cgi?id=60483

11:23 AM Changeset in webkit [86071] by kevino@webkit.org
  • 2 edits in trunk/Source/WebCore

Reviewed by Kevin Ollivier.

[wx] Update the popup menu code to grab the native control and initialize the
event handler properly.

​https://bugs.webkit.org/show_bug.cgi?id=60482

11:20 AM Changeset in webkit [86070] by kevino@webkit.org
  • 2 edits in trunk/Source/WebCore

Reviewed by Kevin Ollivier.

[wx] Make sure other controls adjust the rect to handle transforms, and turn off the
adjustment in 2.9.2+ where it is handled internally by wx.

Also add a couple drawing and focus handling fixes for text fields and menulist controls.

​https://bugs.webkit.org/show_bug.cgi?id=60481

11:18 AM Changeset in webkit [86069] by tonyg@chromium.org
  • 4 edits in trunk/Source/WebCore

2011-05-09 Tony Gentilcore <​tonyg@chromium.org>

Reviewed by Darin Adler.

Style fix: Function names must begin with lowercase letter and should not use "get".
​https://bugs.webkit.org/show_bug.cgi?id=60475

  • dom/Document.cpp: (WebCore::Document::webkitVisibilityState):
  • page/PageVisibilityState.cpp: (WebCore::pageVisibilityStateString): Renamed.
  • page/PageVisibilityState.h:
11:14 AM Changeset in webkit [86068] by kevino@webkit.org
  • 3 edits in trunk/Source/WebKit/wx

Reviewed by Kevin Ollivier.

Create empty event handler to avoid new mouse capture assert added in wx 2.9 on wxMSW.

​https://bugs.webkit.org/show_bug.cgi?id=60480

11:13 AM Changeset in webkit [86067] by abarth@webkit.org
  • 40 edits in trunk/Source

2011-05-09 Adam Barth <​abarth@webkit.org>

Reviewed by Tony Gentilcore.

Make Chromium build with strict PassOwnPtr
​https://bugs.webkit.org/show_bug.cgi?id=60461

Fix many, many PassOwnPtr nits.

  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::precompileScript):
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::WindowSetTimeoutImpl):
  • bindings/v8/custom/V8MessageEventCustom.cpp: (WebCore::V8MessageEvent::initMessageEventCallback):
  • bindings/v8/custom/V8WorkerContextCustom.cpp: (WebCore::SetTimeoutOrInterval):
  • dom/Document.cpp: (WebCore::Document::removedLastRef): (WebCore::Document::detach):
  • fileapi/DOMFileSystem.h: (WebCore::DOMFileSystem::scheduleCallback):
  • fileapi/FileSystemCallbacks.cpp: (WebCore::FileSystemCallbacks::didOpenFileSystem): (WebCore::ResolveURICallbacks::didOpenFileSystem):
  • platform/graphics/chromium/GraphicsLayerChromium.cpp: (WebCore::GraphicsLayer::create):
  • platform/graphics/chromium/cc/CCLayerImpl.cpp: (WebCore::CCLayerImpl::createRenderSurface):
  • platform/graphics/gpu/BicubicShader.cpp: (WebCore::BicubicShader::create):
  • platform/graphics/gpu/ConvolutionShader.cpp: (WebCore::ConvolutionShader::create):
  • platform/graphics/gpu/LoopBlinnSolidFillShader.cpp: (WebCore::LoopBlinnSolidFillShader::create):
  • platform/graphics/gpu/SolidFillShader.cpp: (WebCore::SolidFillShader::create):
  • platform/graphics/gpu/TexShader.cpp: (WebCore::TexShader::create):
  • platform/graphics/gpu/Texture.cpp: (WebCore::Texture::create):
  • storage/chromium/DatabaseTrackerChromium.cpp: (WebCore::TrackerRemoveOpenDatabaseTask::create):
  • storage/chromium/SQLTransactionClientChromium.cpp: (WebCore::NotifyDatabaseChangedTask::create):

2011-05-09 Adam Barth <​abarth@webkit.org>

Reviewed by Tony Gentilcore.

Make Chromium build with strict PassOwnPtr
​https://bugs.webkit.org/show_bug.cgi?id=60461

Fixed many, many PassOwnPtr nits.

  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::autofill):
  • src/IDBDatabaseBackendProxy.cpp: (WebKit::IDBDatabaseBackendProxy::createObjectStore): (WebKit::IDBDatabaseBackendProxy::transaction):
  • src/IDBObjectStoreBackendProxy.cpp: (WebKit::IDBObjectStoreBackendProxy::createIndex): (WebKit::IDBObjectStoreBackendProxy::index):
  • src/IDBTransactionBackendProxy.cpp: (WebKit::IDBTransactionBackendProxy::objectStore):
  • src/PlatformMessagePortChannel.cpp: (WebCore::MessagePortChannel::create): (WebCore::PlatformMessagePortChannel::tryGetMessageFromRemote):
  • src/SharedWorkerRepository.cpp: (WebCore::SharedWorkerRepository::connect):
  • src/SocketStreamHandle.cpp: (WebCore::SocketStreamHandleInternal::create):
  • src/WebDevToolsAgentImpl.cpp: (WebKit::DebuggerTask::DebuggerTask): (WebKit::DebuggerTask::~DebuggerTask): (WebKit::DebuggerTask::run): (WebKit::WebDevToolsAgent::interruptAndDispatch):
  • src/WebDevToolsFrontendImpl.cpp: (WebKit::WebDevToolsFrontendImpl::WebDevToolsFrontendImpl):
  • src/WebIDBCallbacksImpl.cpp: (WebKit::WebIDBCallbacksImpl::onSuccess):
  • src/WebIDBCursorImpl.cpp: (WebKit::WebIDBCursorImpl::update): (WebKit::WebIDBCursorImpl::continueFunction): (WebKit::WebIDBCursorImpl::deleteFunction):
  • src/WebIDBDatabaseImpl.cpp: (WebKit::WebIDBDatabaseImpl::setVersion): (WebKit::WebIDBDatabaseImpl::open):
  • src/WebIDBFactoryImpl.cpp: (WebKit::WebIDBFactoryImpl::open):
  • src/WebIDBIndexImpl.cpp: (WebKit::WebIDBIndexImpl::openObjectCursor): (WebKit::WebIDBIndexImpl::openKeyCursor): (WebKit::WebIDBIndexImpl::getObject): (WebKit::WebIDBIndexImpl::getKey):
  • src/WebIDBObjectStoreImpl.cpp: (WebKit::WebIDBObjectStoreImpl::get): (WebKit::WebIDBObjectStoreImpl::put): (WebKit::WebIDBObjectStoreImpl::deleteFunction): (WebKit::WebIDBObjectStoreImpl::clear): (WebKit::WebIDBObjectStoreImpl::openCursor):
  • src/WebIDBTransactionImpl.cpp: (WebKit::WebIDBTransactionImpl::setCallbacks):
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::loadFrameRequest):
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::setDevToolsAgentClient):
  • src/WebWorkerClientImpl.cpp: (WebKit::WebWorkerClientImpl::postMessageToWorkerObject):
  • src/WebWorkerImpl.cpp: (WebKit::WebWorkerImpl::postMessageToWorkerContext):
  • src/WorkerFileWriterCallbacksBridge.cpp: (WebKit::WorkerFileWriterCallbacksBridge::initOnMainThread): (WebKit::WorkerFileWriterCallbacksBridge::didWrite): (WebKit::WorkerFileWriterCallbacksBridge::didFail): (WebKit::WorkerFileWriterCallbacksBridge::didTruncate):
11:04 AM Changeset in webkit [86066] by chang.shu@nokia.com
  • 2 edits
    1 delete in trunk/LayoutTests

2011-05-09 Chang Shu <​cshu@webkit.org>

Unreviewed.

[Qt] editing/inserting/5803706-1.html has the incorrect expected result
​https://bugs.webkit.org/show_bug.cgi?id=60479

  • platform/qt/editing/inserting/5803706-1-expected.txt: Removed.
10:52 AM WikiStart edited by Ademar Reis
(diff)
10:52 AM Changeset in webkit [86065] by Martin Robinson
  • 1 edit
    1 add in trunk/LayoutTests

2011-05-09 Martin Robinson <​mrobinson@igalia.com>

Add a pixel result for platform/gtk/fonts/fontconfig-synthetic-bold.html.

  • platform/gtk/fonts/fontconfig-synthetic-bold-expected.png: Added.
10:51 AM QtWebKitReleases edited by Ademar Reis
(diff)
10:47 AM Changeset in webkit [86064] by Martin Robinson
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Daniel Bates.

[Soup] Clean up error handling in ResourceHandleSoup
​https://bugs.webkit.org/show_bug.cgi?id=60436

Instead of repeating the ResourceError creation twice, abstract
it into a helper method.

No new tests. This is a code cleanup.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::soupErrorShouldCauseLoadFailure): Added. (WebCore::convertSoupErrorToResourceError): Added. (WebCore::sendRequestCallback): Use the new helpers. (WebCore::readCallback): Ditto.
10:32 AM Changeset in webkit [86063] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

2011-05-09 Alice Boxhall <​aboxhall@chromium.org>

Reviewed by Ojan Vafai.

Pull jsonresults_unittest test cases out into separate test methods.
​https://bugs.webkit.org/show_bug.cgi?id=60418

  • TestResultServer/model/jsonresults_unittest.py:
10:29 AM QtWebKitReleases edited by Ademar Reis
(diff)
10:19 AM Changeset in webkit [86062] by eric@webkit.org
  • 2 edits
    1 add in trunk/Websites/webkit.org

2011-05-09 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add guidelines for feature addition to webkit.org (per disccusion at contributor meeting)
​https://bugs.webkit.org/show_bug.cgi?id=59672

  • coding/adding-features.html: Added.
  • nav.inc:
10:13 AM QtWebKitRelease22 edited by Ademar Reis
(diff)
9:32 AM Changeset in webkit [86061] by inferno@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2011-05-08 Abhishek Arya <​inferno@chromium.org>

Reviewed by Dave Hyatt.

Add a crash test for r85964.
​https://bugs.webkit.org/show_bug.cgi?id=59473

  • fast/block/positioning/removing-inside-relpositioned-inline-crash-expected.txt: Added.
  • fast/block/positioning/removing-inside-relpositioned-inline-crash.html: Added.
9:29 AM Changeset in webkit [86060] by inferno@chromium.org
  • 3 edits
    2 adds in trunk

2011-05-09 Abhishek Arya <​inferno@chromium.org>

Reviewed by Dave Hyatt.

Regression(r85869): Restore dirty line logic in RenderInline::destroy.
​https://bugs.webkit.org/show_bug.cgi?id=60448

Test: fast/inline/inline-destroy-dirty-lines-crash.html

  • rendering/RenderInline.cpp: (WebCore::RenderInline::destroy):

2011-05-09 Abhishek Arya <​inferno@chromium.org>

Reviewed by Dave Hyatt.

Tests that we do not when failing to dirty lines while removing
a inline.
​https://bugs.webkit.org/show_bug.cgi?id=60448

  • fast/inline/inline-destroy-dirty-lines-crash-expected.txt: Added.
  • fast/inline/inline-destroy-dirty-lines-crash.html: Added.
9:15 AM Changeset in webkit [86059] by alex
  • 2 edits in trunk/LayoutTests

2011-05-09 Alejandro G. Castro <​alex@igalia.com>

Unreviewed, skip 2 more flaky svg tests failing by one pixel in
32-bits Debug, see bug 39022.

  • platform/gtk/Skipped:
8:38 AM Changeset in webkit [86058] by alex
  • 2 edits in trunk/Source/WebCore

2011-05-09 Alejandro G. Castro <​alex@igalia.com>

Reviewed by Martin Robinson.

[GTK] WidgetGtk setFocus can not access directly the widget in WK2
​https://bugs.webkit.org/show_bug.cgi?id=60016

Added code to check the situation where we do not have direct
access to the widget. We use the page to grab the focus which
would send a request to the UIProcess in WK2.

  • platform/gtk/WidgetGtk.cpp: (WebCore::Widget::setFocus):
8:27 AM BuildingQtOnOSX edited by andreas.kling@nokia.com
(diff)
8:23 AM Changeset in webkit [86057] by andreas.kling@nokia.com
  • 2 edits in trunk/LayoutTests

2011-05-09 Andreas Kling <​andreas.kling@nokia.com>

Unreviewed. Re-skip two failing inspector tests on Qt.

They still fail in the bot's environment for whatever reason.

  • platform/qt/Skipped:
8:00 AM Changeset in webkit [86056] by Adam Roben
  • 3 edits
    1 add
    1 delete in trunk/Source/JavaScriptCore

Automatically touch WebKit.idl whenever any other WebKit1 IDL file changes

Fixes <​http://webkit.org/b/60468> WebKit.idl needs to be manually touched whenever any other
WebKit1 IDL file changes to avoid build errors

Reviewed by Tim Hatcher.

Updated for script rename.

  • JavaScriptCore.vcproj/JavaScriptCore/react-to-vsprops-changes.py: Removed.
  • JavaScriptCore.vcproj/JavaScriptCore/work-around-vs-dependency-tracking-bugs.py: Renamed

from react-to-vsprops-changes.py.
(top level): Moved a constant here from main.
(main): Moved most code from here to react_to_vsprops_changes. Added a call to the new
react_to_webkit1_interface_changes function.
(react_to_vsprops_changes): Moved code here from main. Updated to use the
TOP_LEVEL_DIRECTORY global. Moved some code from here to mtime_of_newest_file_matching_globa
and touch_if_older_than.
(react_to_webkit1_interface_changes): Added. Touches WebKit.idl if any other WebKit1 IDL
file has changed.
(mtime_of_newest_file_matching_glob): Added. Code came from main.
(touch_if_older_than): Added. Code came from main.

7:58 AM Changeset in webkit [86055] by commit-queue@webkit.org
  • 10 edits in trunk

2011-05-09 Leandro Gracia Gil <​leandrogracia@chromium.org>

Reviewed by Tony Gentilcore.

Media Stream API: add exception raising to the getUserMedia method.
​https://bugs.webkit.org/show_bug.cgi?id=60387

Update the expected results for the argument types after the lastest updates
in the getUserMedia specification. The test has been disabled temporarily
and will be re-enabled by 60177 since some exceptions are risen after parsing
the provided options.

  • fast/dom/MediaStream/argument-types-expected.txt:
  • fast/dom/MediaStream/script-tests/argument-types.js: (ObjectThrowingException.prototype.toString):
  • platform/chromium/test_expectations.txt:

2011-05-09 Leandro Gracia Gil <​leandrogracia@chromium.org>

Reviewed by Tony Gentilcore.

Media Stream API: add exception raising to the getUserMedia method.
​https://bugs.webkit.org/show_bug.cgi?id=60387

Perform the required changes to support rising exceptions in the
navigator.getUserMedia method. This is required by the latest changes in the spec:
​http://www.whatwg.org/specs/web-apps/current-work/multipage/dnd.html#dom-navigator-getusermedia

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

  • bindings/js/JSNavigatorCustom.cpp: (WebCore::JSNavigator::webkitGetUserMedia):
  • bindings/v8/custom/V8NavigatorCustom.cpp: (WebCore::V8Navigator::webkitGetUserMediaCallback):
  • page/Navigator.cpp: (WebCore::Navigator::webkitGetUserMedia):
  • page/Navigator.h:
  • page/Navigator.idl:
7:13 AM Changeset in webkit [86054] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebCore

Unreviewed Qt 4.8 fix.

  • WebCore.pro: 4.8 bots are updated with the full QRawFont APIs, so we can set HAVE_QRAWFONT now.
7:06 AM Changeset in webkit [86053] by andreas.kling@nokia.com
  • 2 edits in trunk/LayoutTests

2011-05-09 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Benjamin Poulain.

[Qt] Minor skiplist cleanup.

Unskip two inspector tests that aren't failing.
Added explanation for why http/tests/misc/redirect-to-external-url.html is skipped.

  • platform/qt/Skipped:
7:06 AM Changeset in webkit [86052] by Adam Roben
  • 2 edits in trunk/Source/WebCore

Windows project file fix after r86047

  • WebCore.vcproj/WebCore.vcproj: Added missing closing tags.
6:51 AM Changeset in webkit [86051] by cmarrin@apple.com
  • 2 edits in trunk/Source/WebCore

2011-05-06 Chris Marrin <​cmarrin@apple.com>

Reviewed by Darin Adler.

Fix layer names to say "tiled" and show CALayer addresses, not PlatformCALayer addresses
​https://bugs.webkit.org/show_bug.cgi?id=60417

Layer name now has "Tiled" prepended if it is a TiledLayer. This is true on Mac where
the layer is a CATiledLayer and on Win where the layer is actually still a CACFLayer
but implements a tiled layer. I also now print the address of the CALayer or CACFLayer
instead of the PlatformCALayer everywhere.

  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::swapFromOrToTiledLayer):
6:41 AM Changeset in webkit [86050] by rwlbuis@webkit.org
  • 78 edits in trunk/Source/WebCore

2011-05-09 Rob Buis <​rbuis@rim.com>

Reviewed by Dirk Schulze.

Elements should assert that the QualifiedName given to their constructor corresponds to the Element constructed
​https://bugs.webkit.org/show_bug.cgi?id=22576

Add tag name ASSERTs in SVG Element constructors.

No new tests, since no change in behaviour.

4:22 AM Changeset in webkit [86049] by mahesh.kulkarni@nokia.com
  • 4 edits in trunk

2011-05-09 Mahesh Kulkarni <​mahesh.kulkarni@nokia.com>

Reviewed by Andreas Kling.

[Qt] fix fast/dom/Geolocation/window-close-crash.html
​https://bugs.webkit.org/show_bug.cgi?id=60458

Update all pages with Mock position/error/permission.

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setGeolocationPermission): (LayoutTestController::setMockGeolocationError): (LayoutTestController::setMockGeolocationPosition):

2011-05-09 Mahesh Kulkarni <​mahesh.kulkarni@nokia.com>

Reviewed by Andreas Kling.

[Qt] fix fast/dom/Geolocation/window-close-crash.html
​https://bugs.webkit.org/show_bug.cgi?id=60458

Unskipping fast/dom/Geolocation/window-close-crash.html.

  • platform/qt/Skipped:
4:17 AM Changeset in webkit [86048] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-09 Srinivasulu Chereddy <​srinivasulu.chereddy@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] one skipped 2d-transforms layout test passes on Qt Linux - skip list to be updated
​https://bugs.webkit.org/show_bug.cgi?id=60399

  • platform/qt/Skipped:
4:00 AM Changeset in webkit [86047] by commit-queue@webkit.org
  • 33 edits
    13 adds in trunk

2011-05-09 Shishir Agrawal <​shishir@chromium.org>

Reviewed by Tony Gentilcore.

Implement Page Visibility API.
​https://bugs.webkit.org/show_bug.cgi?id=54181

  • fast/events/page-visibility-iframe-delete-test-expected.txt: Added.
  • fast/events/page-visibility-iframe-delete-test.html: Added.
  • fast/events/page-visibility-iframe-move-test-expected.txt: Added.
  • fast/events/page-visibility-iframe-move-test.html: Added.
  • fast/events/page-visibility-iframe-propagation-test-expected.txt: Added.
  • fast/events/page-visibility-iframe-propagation-test.html: Added.
  • fast/events/page-visibility-transition-test-expected.txt: Added.
  • fast/events/page-visibility-transition-test.html: Added.
  • fast/events/resources/page-visibility-iframe-delete-test-frame.html: Added.
  • fast/events/resources/page-visibility-iframe-move-new-page.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

2011-05-09 Shishir Agrawal <​shishir@chromium.org>

Reviewed by Tony Gentilcore.

Implement Page Visibility API.
​https://bugs.webkit.org/show_bug.cgi?id=54181

Tests: fast/events/page-visibility-iframe-delete-test.html

fast/events/page-visibility-iframe-move-test.html
fast/events/page-visibility-iframe-propagation-test.html
fast/events/page-visibility-transition-test.html

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/Document.cpp: (WebCore::Document::visibilityState): (WebCore::Document::webkitVisibilityState): (WebCore::Document::webkitIsVisible): (WebCore::Document::dispatchVisibilityStateChangeEvent):
  • dom/Document.h:
  • dom/Document.idl:
  • dom/EventNames.h:
  • page/Frame.cpp: (WebCore::Frame::dispatchVisibilityStateChangeEvent):
  • page/Frame.h:
  • page/Page.cpp: (WebCore::Page::Page): (WebCore::Page::setVisibilityState): (WebCore::Page::visibilityState):
  • page/Page.h:
  • page/PageVisibilityState.cpp: Added. (WebCore::GetPageVisibilityStateString):
  • page/PageVisibilityState.h: Added.

2011-05-09 Shishir Agrawal <​shishir@chromium.org>

Reviewed by Tony Gentilcore.

Implement Page Visibility API.
​https://bugs.webkit.org/show_bug.cgi?id=54181

  • WebKit.gyp:
  • public/WebPageVisibilityState.h: Added.
  • public/WebView.h: (WebKit::WebView::setVisibilityState):
  • public/WebViewClient.h: (WebKit::WebViewClient::visibilityState):
  • src/AssertMatchingEnums.cpp:
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl): (WebKit::WebViewImpl::setVisibilityState):
  • src/WebViewImpl.h:

2011-05-09 Shishir Agrawal <​shishir@chromium.org>

Reviewed by Tony Gentilcore.

Implement Page Visibility API.
​https://bugs.webkit.org/show_bug.cgi?id=54181

  • DumpRenderTree/LayoutTestController.cpp: (setPageVisibilityCallback): (resetPageVisibilityCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h: (LayoutTestController::setPageVisibility): (LayoutTestController::resetPageVisibility):
  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::resetPageVisibility): (LayoutTestController::setPageVisibility):
  • DumpRenderTree/chromium/LayoutTestController.h:
3:55 AM Changeset in webkit [86046] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-09 Venkat Penukonda <​venkat.2.penukonda@nokia.com>

Reviewed by Csaba Osztrogonác.

[Qt]6 of the skipped canvas layout tests pass on Qt Linux
​https://bugs.webkit.org/show_bug.cgi?id=60382

  • platform/qt/Skipped: Removed passing canvas tests
3:48 AM Changeset in webkit [86045] by benjamin.poulain@nokia.com
  • 3 edits in trunk/Source/WebKit2

Build fix for WebKit 2 on Qt

Reviewed by Andreas Kling.

Qt treats warnings as errors so it does not build with unused variables. This patch removed two unused variable in WebKit 2.

  • UIProcess/TiledDrawingAreaProxy.cpp:

(WebKit::TiledDrawingAreaProxy::createTiles):

  • WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::~PluginView):

2:19 AM Changeset in webkit [86044] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-05-09 Luke Macpherson <​macpherson@chromium.org>

Reviewed by Eric Seidel.

Implement expanding CSS border properties in CSSStyleApplyProperty.
​https://bugs.webkit.org/show_bug.cgi?id=60456

No new tests added as no functionality changed.

  • css/CSSStyleApplyProperty.cpp: (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): Create CSS border expanding property handlers.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Remove property handlers that are no longer needed.

May 8, 2011:

9:37 PM Changeset in webkit [86043] by morrita@google.com
  • 2 edits in trunk/LayoutTests

2011-05-08 MORITA Hajime <​morrita@google.com>

Unreviewed, rolling out r86042.
​http://trac.webkit.org/changeset/86042

The change contains an incorrect specification.

  • platform/chromium/test_expectations.txt:
9:19 PM Changeset in webkit [86042] by morrita@google.com
  • 2 edits in trunk/LayoutTests

2011-05-08 MORITA Hajime <​morrita@google.com>

Unreviewed marking flaky test (insecure-plugin-in-iframe.html)

  • platform/chromium/test_expectations.txt:
8:29 PM Changeset in webkit [86041] by rniwa@webkit.org
  • 5 edits
    4 adds in trunk

2011-05-08 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Kent Tamura.

selectstart event does not fire when selection is made via select all
​https://bugs.webkit.org/show_bug.cgi?id=60376

Added tests to ensure selectstart event is fired when a user selects all contents
and script can prevent such selection change in selectstart's event handler.

  • fast/events/selectstart-on-selectall-expected.txt: Added.
  • fast/events/selectstart-on-selectall.html: Added.
  • fast/events/selectstart-prevent-selectall-expected.txt: Added.
  • fast/events/selectstart-prevent-selectall.html: Added.

2011-05-08 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Kent Tamura.

selectstart event does not fire when selection is made via select all
​https://bugs.webkit.org/show_bug.cgi?id=60376

Fire selectstart event when a user selects all contents (i.e. document.execCommand('SelectAll')) in
document, editable region, or text control.

Tests: editing/selection/selectstart-on-selectall.html

editing/selection/selectstart-prevent-selectall.html

  • dom/Node.h: Removed canSelectAll and selectAll as they are left over from WMLSelectElement.
  • editing/FrameSelection.cpp: (WebCore::FrameSelection::selectAll): Dispatch selectstart event on selectStartTarget, which is input element or textarea element when the current selection is inside a shadow DOM, and the root editable element if it's inside a non-shadow editable region, and the body element otherwise.
  • html/HTMLSelectElement.h: Made canSelectAll and selectAll public since they are no longer declared in Node.
7:16 PM Changeset in webkit [86040] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-05-08 Luke Macpherson <​macpherson@chromium.org>

Reviewed by Eric Seidel.

Make RenderStyle::setColumnBreakInside() reject unsupported enum values.
​https://bugs.webkit.org/show_bug.cgi?id=60249

No new tests added as no functionality changed.

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Remove special-case code and use appropriate macro.
  • rendering/style/RenderStyle.h: (WebCore::InheritedFlags::setColumnBreakInside): Assert that supported values are used.
5:40 PM Changeset in webkit [86039] by rniwa@webkit.org
  • 8 edits in trunk/Source/WebCore

2011-05-08 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Darin Adler.

Extract a DragCaretController from FrameSelection
​https://bugs.webkit.org/show_bug.cgi?id=60273

Extracted CaretBase and DragCaretController from FrameSelection. CaretBase is a base class for
DragCaretController and FrameSelection and provides functions to paint caret.

Since DragCaretController doesn't need to store selection, it only holds one VisiblePosition.

  • WebCore.exp.in:
  • editing/FrameSelection.cpp: (WebCore::CaretBase::CaretBase): Added. (WebCore::DragCaretController::DragCaretController): Added; Set m_caretVisible true. (WebCore::DragCaretController::isContentRichlyEditable): Added. (WebCore::FrameSelection::FrameSelection): Several member variables are move to CaretBase. (WebCore::DragCaretController::setCaretPosition): Extracted from FrameSelection::setSelection. (WebCore::FrameSelection::setSelection): Removed an early exit for DragCaretController. (WebCore::clearRenderViewSelection): Extracted from FrameSelection::respondToNodeModification. (WebCore::DragCaretController::nodeWillBeRemoved): Extracted from FrameSelection::nodeWillBeRemoved and respondToNodeModification. Note that the position DragCaretController holds is equivalent to FrameSelection's base or start so when removingNodeRemovesPosition returns true for m_position, we used to execute the first if clause and cleared render tree selection and DOM tree selection. This is exactly what new DragCaretController::nodeWillBeRemoved does. (WebCore::FrameSelection::nodeWillBeRemoved): Simplified early exist conditions. (WebCore::FrameSelection::respondToNodeModification): Calls clearRenderViewSelection. (WebCore::CaretBase::updateCaretRect): Takes document, caretPosition, selection type, and boolean isOrphaned. Note that we can't obtain the document from caretPosition because VisiblePosition can be null even if FrameSelection's start was not null. (WebCore::caretRendersInsideNode): Moved; Made static local. (WebCore::CaretBase::caretRenderer): Extracted from FrameSelection::caretRenderer. (WebCore::FrameSelection::caretRenderer): Calls CaretBase::caretRenderer. (WebCore::DragCaretController::caretRenderer): Ditto. (WebCore::FrameSelection::localCaretRect): Calls updateCaretRect with extra arguments. (WebCore::CaretBase::absoluteBoundsForLocalRect): Moved from FrameSelection; Takes Node*. (WebCore::CaretBase::caretRepaintRect): Ditto. (WebCore::FrameSelection::recomputeCaretRect): Calls absoluteBoundsForLocalRect, caretRepaintRect, and shouldRepaintCaret with extra arguments. (WebCore::CaretBase::shouldRepaintCaret): Takes a boolean isContentEditable. (WebCore::FrameSelection::invalidateCaretRect): Calls CaretBase::invalidateCaretRect. (WebCore::CaretBase::invalidateCaretRect): Extracted from FrameSelection::invalidateCaretRect. (WebCore::FrameSelection::paintCaret): Calls CaretBase::paintCaret. (WebCore::CaretBase::paintCaret): Extracted from FrameSelection::paintCaret. (WebCore::FrameSelection::updateAppearance): Removed an assertion that this function is never called for DragCaretController. (WebCore::DragCaretController::paintDragCaret): Moved from FrameSelection::paintDragCaret.
  • editing/FrameSelection.h: (WebCore::CaretBase::localCaretRectForPainting): Added. (WebCore::DragCaretController::isContentEditable): Added. (WebCore::DragCaretController::hasCaret): Added. (WebCore::DragCaretController::caretPosition): Added. (WebCore::DragCaretController::clear): Added.
  • page/DragController.cpp: (WebCore::DragController::tryDocumentDrag): Uses DragCaretController. (WebCore::DragController::dispatchTextInputEventFor): Ditto. (WebCore::DragController::concludeEditDrag): Ditto. (WebCore::DragController::placeDragCaret): Ditto.
  • page/Page.cpp: (WebCore::Page::Page): Instantiates DragCaretController instead of FrameSelection.
  • page/Page.h: (WebCore::Page::dragCaretController): Returns DragCaretController* instead of FrameSelection*.
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintCaret): Uses FrameSelection and DragCaretController.
2:49 PM Changeset in webkit [86038] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitLibraries

2011-05-08 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r86037.
​http://trac.webkit.org/changeset/86037
​https://bugs.webkit.org/show_bug.cgi?id=60453

broke Windows tests (Requested by jessieberlin on #webkit).

  • win/lib/WebKitSystemInterface.lib:
2:30 PM Changeset in webkit [86037] by jberlin@webkit.org
  • 2 edits in trunk/WebKitLibraries

Update the WKSI libraries.

Rubber-stamped by Alice Liu.

  • win/lib/WebKitSystemInterface.lib:
1:56 PM Changeset in webkit [86036] by jberlin@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Make JSRetainPtr work with JSGlobalContextRefs.
​https://bugs.webkit.org/show_bug.cgi?id=60452

Reviewed by Dan Bernstein.

Add specialized functions for JSRetain and JSRelease when dealing with JSGlobalContextRefs.

  • API/JSRetainPtr.h:

(JSRetain):
(JSRelease):

1:24 PM Changeset in webkit [86035] by inferno@chromium.org
  • 4 edits in trunk/LayoutTests

2011-05-08 Abhishek Arya <​inferno@chromium.org>

Unreviewed.

More rebaselines for r86028.

  • platform/chromium-linux/fast/text/basic/002-expected.png:
  • platform/chromium-win/fast/text/basic/002-expected.png:
  • platform/chromium-win/fast/text/basic/002-expected.txt:
1:13 PM Changeset in webkit [86034] by imasaki@chromium.org
  • 5 edits in trunk/LayoutTests

2011-05-08 Kenji Imasaki <​imasaki@chromium.org>

Reviewed by Eric Seidel.

[Chromium] add rebaseline for video-volume-slider for GPU.
​https://bugs.webkit.org/show_bug.cgi?id=60427

These tests are failing due to text.
​http://code.google.com/p/chromium/issues/detail?id=81482

  • platform/chromium-gpu-linux/media/video-volume-slider-expected.png:
  • platform/chromium-gpu-mac/media/video-volume-slider-expected.png:
  • platform/chromium-gpu-win/media/video-volume-slider-expected.png:
  • platform/chromium/test_expectations.txt:
12:47 PM Changeset in webkit [86033] by inferno@chromium.org
  • 15 edits
    6 adds
    2 deletes in trunk/LayoutTests

2011-05-08 Abhishek Arya <​inferno@chromium.org>

Unreviewed.

More chromium rebaseline for r86028.

  • platform/chromium-linux/css1/box_properties/float_on_text_elements-expected.png:
  • platform/chromium-linux/fast/text/justified-selection-expected.png:
  • platform/chromium-linux/fast/text/justify-ideograph-leading-expansion-expected.png: Added.
  • platform/chromium-linux/fast/text/justify-nbsp-expected.png: Removed.
  • platform/chromium-linux/fast/text/trailing-white-space-2-expected.png:
  • platform/chromium-linux/fast/text/trailing-white-space-expected.png:
  • platform/chromium-mac-leopard/fast/text/justify-nbsp-expected.png: Removed.
  • platform/chromium-mac/fast/text/justify-ideograph-leading-expansion-expected.png: Added.
  • platform/chromium-mac/fast/text/justify-ideograph-leading-expansion-expected.txt:
  • platform/chromium-win-xp/fast/text/justify-ideograph-leading-expansion-expected.png: Added.
  • platform/chromium-win/css1/box_properties/float_on_text_elements-expected.png:
  • platform/chromium-win/css1/box_properties/float_on_text_elements-expected.txt:
  • platform/chromium-win/fast/text/justified-selection-expected.png:
  • platform/chromium-win/fast/text/justified-selection-expected.txt:
  • platform/chromium-win/fast/text/justify-ideograph-leading-expansion-expected.png: Added.
  • platform/chromium-win/fast/text/justify-ideograph-leading-expansion-expected.txt: Added.
  • platform/chromium-win/fast/text/justify-nbsp-expected.png: Added.
  • platform/chromium-win/fast/text/trailing-white-space-2-expected.png:
  • platform/chromium-win/fast/text/trailing-white-space-2-expected.txt:
  • platform/chromium-win/fast/text/trailing-white-space-expected.png:
  • platform/chromium-win/fast/text/trailing-white-space-expected.txt:
  • platform/chromium/test_expectations.txt:
11:58 AM Changeset in webkit [86032] by inferno@chromium.org
  • 108 edits
    4 adds in trunk/LayoutTests

2011-05-08 Abhishek Arya <​inferno@chromium.org>

Unreviewed.

Chromium Rebaselines for r86028.

  • platform/chromium-linux/css1/box_properties/clear_float-expected.png:
  • platform/chromium-linux/css2.1/t0905-c5525-fltcont-00-d-g-expected.png:
  • platform/chromium-linux/css2.1/t0905-c5525-fltcont-00-d-g-expected.txt:
  • platform/chromium-linux/css2.1/t0905-c5525-fltwidth-00-c-g-expected.png:
  • platform/chromium-linux/css2.1/t0905-c5525-fltwidth-00-c-g-expected.txt:
  • platform/chromium-linux/css2.1/t1602-c546-txt-align-00-b-expected.png:
  • platform/chromium-linux/fast/multicol/float-avoidance-expected.png:
  • platform/chromium-linux/fast/multicol/vertical-lr/float-avoidance-expected.png:
  • platform/chromium-linux/fast/multicol/vertical-rl/float-avoidance-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-1-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-10-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-2-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-3-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-4-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-5-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-6-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-7-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-8-expected.png:
  • platform/chromium-linux/fast/repaint/line-flow-with-floats-9-expected.png:
  • platform/chromium-linux/fast/text/justified-selection-at-edge-expected.png:
  • platform/chromium-linux/svg/custom/svg-fonts-in-html-expected.png:
  • platform/chromium-linux/tables/mozilla/marvin/x_tbody_align_justify-expected.png:
  • platform/chromium-linux/tables/mozilla/marvin/x_td_align_justify-expected.png:
  • platform/chromium-linux/tables/mozilla/marvin/x_tfoot_align_justify-expected.png:
  • platform/chromium-linux/tables/mozilla/marvin/x_th_align_justify-expected.png:
  • platform/chromium-linux/tables/mozilla/marvin/x_thead_align_justify-expected.png:
  • platform/chromium-linux/tables/mozilla/marvin/x_tr_align_justify-expected.png:
  • platform/chromium-mac-leopard/css1/box_properties/clear_float-expected.png: Added.
  • platform/chromium-mac-leopard/css2.1/t0905-c5525-fltcont-00-d-g-expected.png: Added.
  • platform/chromium-mac-leopard/css2.1/t0905-c5525-fltwidth-00-c-g-expected.png: Added.
  • platform/chromium-mac-leopard/css2.1/t1602-c546-txt-align-00-b-expected.png:
  • platform/chromium-mac-leopard/fast/multicol/float-avoidance-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-1-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-10-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-2-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-3-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-4-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-5-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-6-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-7-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-8-expected.png:
  • platform/chromium-mac-leopard/fast/repaint/line-flow-with-floats-9-expected.png:
  • platform/chromium-mac-leopard/fast/text/justified-selection-at-edge-expected.png:
  • platform/chromium-mac-leopard/tables/mozilla/marvin/x_tbody_align_justify-expected.png:
  • platform/chromium-mac-leopard/tables/mozilla/marvin/x_td_align_justify-expected.png:
  • platform/chromium-mac-leopard/tables/mozilla/marvin/x_tfoot_align_justify-expected.png:
  • platform/chromium-mac-leopard/tables/mozilla/marvin/x_th_align_justify-expected.png:
  • platform/chromium-mac-leopard/tables/mozilla/marvin/x_thead_align_justify-expected.png: Added.
  • platform/chromium-mac-leopard/tables/mozilla/marvin/x_tr_align_justify-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-1-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-10-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-2-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-3-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-4-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-5-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-6-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-7-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-8-expected.png:
  • platform/chromium-mac/fast/repaint/line-flow-with-floats-9-expected.png:
  • platform/chromium-mac/svg/custom/svg-fonts-in-html-expected.png:
  • platform/chromium-win/css1/box_properties/clear_float-expected.png:
  • platform/chromium-win/css1/box_properties/clear_float-expected.txt:
  • platform/chromium-win/css2.1/t0905-c5525-fltcont-00-d-g-expected.png:
  • platform/chromium-win/css2.1/t0905-c5525-fltcont-00-d-g-expected.txt:
  • platform/chromium-win/css2.1/t0905-c5525-fltwidth-00-c-g-expected.png:
  • platform/chromium-win/css2.1/t0905-c5525-fltwidth-00-c-g-expected.txt:
  • platform/chromium-win/css2.1/t1602-c546-txt-align-00-b-expected.png:
  • platform/chromium-win/css2.1/t1602-c546-txt-align-00-b-expected.txt:
  • platform/chromium-win/fast/multicol/float-avoidance-expected.png:
  • platform/chromium-win/fast/multicol/float-avoidance-expected.txt:
  • platform/chromium-win/fast/multicol/vertical-lr/float-avoidance-expected.png:
  • platform/chromium-win/fast/multicol/vertical-lr/float-avoidance-expected.txt:
  • platform/chromium-win/fast/multicol/vertical-rl/float-avoidance-expected.png:
  • platform/chromium-win/fast/multicol/vertical-rl/float-avoidance-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-1-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-1-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-10-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-10-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-2-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-2-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-3-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-3-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-4-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-4-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-5-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-5-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-6-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-6-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-7-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-7-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-8-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-8-expected.txt:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-9-expected.png:
  • platform/chromium-win/fast/repaint/line-flow-with-floats-9-expected.txt:
  • platform/chromium-win/fast/text/justified-selection-at-edge-expected.png:
  • platform/chromium-win/fast/text/justified-selection-at-edge-expected.txt:
  • platform/chromium-win/svg/custom/svg-fonts-in-html-expected.png:
  • platform/chromium-win/svg/custom/svg-fonts-in-html-expected.txt:
  • platform/chromium-win/tables/mozilla/marvin/x_tbody_align_justify-expected.png:
  • platform/chromium-win/tables/mozilla/marvin/x_tbody_align_justify-expected.txt:
  • platform/chromium-win/tables/mozilla/marvin/x_td_align_justify-expected.png:
  • platform/chromium-win/tables/mozilla/marvin/x_td_align_justify-expected.txt:
  • platform/chromium-win/tables/mozilla/marvin/x_tfoot_align_justify-expected.png:
  • platform/chromium-win/tables/mozilla/marvin/x_tfoot_align_justify-expected.txt:
  • platform/chromium-win/tables/mozilla/marvin/x_th_align_justify-expected.png:
  • platform/chromium-win/tables/mozilla/marvin/x_th_align_justify-expected.txt:
  • platform/chromium-win/tables/mozilla/marvin/x_thead_align_justify-expected.png:
  • platform/chromium-win/tables/mozilla/marvin/x_thead_align_justify-expected.txt:
  • platform/chromium-win/tables/mozilla/marvin/x_tr_align_justify-expected.png:
  • platform/chromium-win/tables/mozilla/marvin/x_tr_align_justify-expected.txt:
  • platform/chromium/test_expectations.txt:
11:53 AM Changeset in webkit [86031] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/efl

2011-05-08 Kamil Blank <​k.blank@samsung.com>

Reviewed by Antonio Gomes.

Added new commands to be called by ewk_view_editor_command.
Due to this change ewk_view_select_* functions have been removed.
​https://bugs.webkit.org/show_bug.cgi?id=60435

  • ewk/ewk_view.cpp: (ewk_view_execute_editor_command):
  • ewk/ewk_view.h:
10:36 AM Changeset in webkit [86030] by mitz@apple.com
  • 2 edits in trunk/LayoutTests

Updated a platform-specific expected result again after r86028.

  • platform/qt/fast/multicol/vertical-lr/float-avoidance-expected.txt:
10:13 AM Changeset in webkit [86029] by mitz@apple.com
  • 48 edits in trunk/LayoutTests

Updated some platform-specific expected results after r86028.

  • platform/gtk/css1/box_properties/clear_float-expected.txt:
  • platform/gtk/css1/box_properties/float_on_text_elements-expected.txt:
  • platform/gtk/css2.1/t0905-c5525-fltcont-00-d-g-expected.txt:
  • platform/gtk/css2.1/t0905-c5525-fltwidth-00-c-g-expected.txt:
  • platform/gtk/css2.1/t1602-c546-txt-align-00-b-expected.txt:
  • platform/gtk/fast/multicol/float-avoidance-expected.txt:
  • platform/gtk/fast/multicol/vertical-lr/float-avoidance-expected.txt:
  • platform/gtk/fast/multicol/vertical-rl/float-avoidance-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-1-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-10-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-2-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-3-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-4-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-5-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-6-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-7-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-8-expected.txt:
  • platform/gtk/fast/repaint/line-flow-with-floats-9-expected.txt:
  • platform/gtk/fast/text/justified-selection-at-edge-expected.txt:
  • platform/gtk/fast/text/trailing-white-space-2-expected.txt:
  • platform/gtk/svg/custom/svg-fonts-in-html-expected.txt:
  • platform/gtk/tables/mozilla/marvin/x_tbody_align_justify-expected.txt:
  • platform/gtk/tables/mozilla/marvin/x_td_align_justify-expected.txt:
  • platform/gtk/tables/mozilla/marvin/x_tfoot_align_justify-expected.txt:
  • platform/gtk/tables/mozilla/marvin/x_th_align_justify-expected.txt:
  • platform/gtk/tables/mozilla/marvin/x_thead_align_justify-expected.txt:
  • platform/gtk/tables/mozilla/marvin/x_tr_align_justify-expected.txt:
  • platform/mac-leopard/fast/text/justify-ideograph-leading-expansion-expected.txt:
  • platform/qt/css1/box_properties/clear_float-expected.txt:
  • platform/qt/css1/box_properties/float_elements_in_series-expected.txt:
  • platform/qt/css1/box_properties/float_on_text_elements-expected.txt:
  • platform/qt/css2.1/t0905-c5525-fltcont-00-d-g-expected.txt:
  • platform/qt/css2.1/t0905-c5525-fltwidth-00-c-g-expected.txt:
  • platform/qt/css2.1/t1602-c546-txt-align-00-b-expected.txt:
  • platform/qt/fast/multicol/float-avoidance-expected.txt:
  • platform/qt/fast/multicol/vertical-lr/float-avoidance-expected.txt:
  • platform/qt/fast/multicol/vertical-rl/float-avoidance-expected.txt:
  • platform/qt/fast/text/justified-selection-at-edge-expected.txt:
  • platform/qt/fast/text/justify-nbsp-expected.txt:
  • platform/qt/svg/custom/svg-fonts-in-html-expected.txt:
  • platform/qt/tables/mozilla/marvin/x_tbody_align_justify-expected.txt:
  • platform/qt/tables/mozilla/marvin/x_td_align_justify-expected.txt:
  • platform/qt/tables/mozilla/marvin/x_tfoot_align_justify-expected.txt:
  • platform/qt/tables/mozilla/marvin/x_th_align_justify-expected.txt:
  • platform/qt/tables/mozilla/marvin/x_thead_align_justify-expected.txt:
  • platform/qt/tables/mozilla/marvin/x_tr_align_justify-expected.txt:
  • platform/win/css2.1/t0905-c5525-fltwidth-00-c-g-expected.txt:
9:27 AM Changeset in webkit [86028] by mitz@apple.com
  • 63 edits in trunk

<rdar://problem/9401853> REGRESSION (r78846): Insufficient expansion for justification when there are multiple inline boxes
​https://bugs.webkit.org/show_bug.cgi?id=60432

Reviewed by Darin Adler.

Source/WebCore:

  • rendering/InlineTextBox.h:

(WebCore::InlineTextBox::setExpansion): Changed back to take a int, since the m_expansion member
variable is a (truncated) int anyway.

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::computeExpansionForJustifiedText): Changed the expansion local variable into an int
so that the right amount is added to the total width. Also changed to use an early return.

LayoutTests:

  • fast/text/justify-nbsp-expected.png:
  • fast/text/justify-nbsp-expected.txt:
  • platform/mac-snowleopard/fast/text/justify-ideograph-leading-expansion-expected.txt:
  • platform/mac/css1/box_properties/clear_float-expected.png:
  • platform/mac/css1/box_properties/clear_float-expected.txt:
  • platform/mac/css1/box_properties/float_on_text_elements-expected.txt:
  • platform/mac/css2.1/t0905-c5525-fltcont-00-d-g-expected.png:
  • platform/mac/css2.1/t0905-c5525-fltcont-00-d-g-expected.txt:
  • platform/mac/css2.1/t0905-c5525-fltwidth-00-c-g-expected.png:
  • platform/mac/css2.1/t0905-c5525-fltwidth-00-c-g-expected.txt:
  • platform/mac/css2.1/t1602-c546-txt-align-00-b-expected.png:
  • platform/mac/css2.1/t1602-c546-txt-align-00-b-expected.txt:
  • platform/mac/fast/multicol/float-avoidance-expected.png:
  • platform/mac/fast/multicol/float-avoidance-expected.txt:
  • platform/mac/fast/multicol/vertical-lr/float-avoidance-expected.png:
  • platform/mac/fast/multicol/vertical-lr/float-avoidance-expected.txt:
  • platform/mac/fast/multicol/vertical-rl/float-avoidance-expected.png:
  • platform/mac/fast/multicol/vertical-rl/float-avoidance-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-1-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-1-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-10-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-10-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-2-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-2-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-3-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-3-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-4-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-4-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-5-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-5-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-6-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-6-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-7-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-7-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-8-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-8-expected.txt:
  • platform/mac/fast/repaint/line-flow-with-floats-9-expected.png:
  • platform/mac/fast/repaint/line-flow-with-floats-9-expected.txt:
  • platform/mac/fast/text/hyphenate-character-expected.png:
  • platform/mac/fast/text/hyphenate-character-expected.txt:
  • platform/mac/fast/text/hyphens-expected.png:
  • platform/mac/fast/text/hyphens-expected.txt:
  • platform/mac/fast/text/justified-selection-at-edge-expected.png:
  • platform/mac/fast/text/justified-selection-at-edge-expected.txt:
  • platform/mac/fast/text/justify-ideograph-leading-expansion-expected.png:
  • platform/mac/svg/custom/svg-fonts-in-html-expected.png:
  • platform/mac/svg/custom/svg-fonts-in-html-expected.txt:
  • platform/mac/tables/mozilla/marvin/x_tbody_align_justify-expected.png:
  • platform/mac/tables/mozilla/marvin/x_tbody_align_justify-expected.txt:
  • platform/mac/tables/mozilla/marvin/x_td_align_justify-expected.png:
  • platform/mac/tables/mozilla/marvin/x_td_align_justify-expected.txt:
  • platform/mac/tables/mozilla/marvin/x_tfoot_align_justify-expected.png:
  • platform/mac/tables/mozilla/marvin/x_tfoot_align_justify-expected.txt:
  • platform/mac/tables/mozilla/marvin/x_th_align_justify-expected.png:
  • platform/mac/tables/mozilla/marvin/x_th_align_justify-expected.txt:
  • platform/mac/tables/mozilla/marvin/x_thead_align_justify-expected.png:
  • platform/mac/tables/mozilla/marvin/x_thead_align_justify-expected.txt:
  • platform/mac/tables/mozilla/marvin/x_tr_align_justify-expected.png:
  • platform/mac/tables/mozilla/marvin/x_tr_align_justify-expected.txt:
9:09 AM Changeset in webkit [86027] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-05-08 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: ignore 0 responses in the inspector network instrumentation.

  • inspector/front-end/NetworkManager.js:
5:18 AM Changeset in webkit [86026] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-08 Jarkko Sakkinen <​jarkko.j.sakkinen@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] GraphicsContext3D::getImageData() does not retrieve image data correctly
​https://bugs.webkit.org/show_bug.cgi?id=58556

Tests: fast/canvas/webgl/gl-teximage.html

  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3D::getImageData):
Note: See TracTimeline for information about the timeline view.