Bug 111386 – [chromium] Move check for threaded compositing from WebCompositorSupport to Platform

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[James Robinson]

[chromium] Move check for threaded compositing from WebCompositorSupport to Platform

[James Robinson]

Created attachment 191363 [details]
Patch

[James Robinson]

Comment on attachment 191363 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=191363&action=review

> Tools/DumpRenderTree/chromium/TestShell.cpp:148
> +#if WEBKIT_SUPPORT_HAS_SET_THREADING_COMPOSITING_ENABLED

Throwing this in so this patch compiles before the chromium side lands+rolls, but the tests won't do the right thing until that happens so I won't land with this guard.

[WebKit Review Bot]

Please wait for approval from [e-mail], [e-mail], [e-mail], [e-mail] or [e-mail] before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.

[WebKit Review Bot]

Attachment 191363 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/Platform/ChangeLog', u'Source/Platform/chromium/public/Platform.h', u'Source/Platform/chromium/public/WebCompositorSupport.h', u'Source/WebCore/ChangeLog', u'Source/WebCore/platform/graphics/chromium/Canvas2DLayerBridge.cpp', u'Source/WebCore/platform/graphics/chromium/Canvas2DLayerBridge.h', u'Source/WebCore/platform/graphics/chromium/DrawingBufferChromium.cpp', u'Source/WebCore/platform/graphics/skia/ImageBufferSkia.cpp', u'Source/WebKit/chromium/ChangeLog', u'Source/WebKit/chromium/src/WebViewImpl.cpp', u'Source/WebKit/chromium/tests/Canvas2DLayerBridgeTest.cpp', u'Source/WebKit/chromium/tests/Canvas2DLayerManagerTest.cpp', u'Tools/ChangeLog', u'Tools/DumpRenderTree/chromium/TestShell.cpp']" exit_code: 1
Source/WebCore/platform/graphics/chromium/DrawingBufferChromium.cpp:99:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 1 in 13 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Adrienne Walker]

Comment on attachment 191363 [details]
Patch

R=me.

[James Robinson]

Created attachment 191513 [details]
Patch

[WebKit Review Bot]

Comment on attachment 191513 [details]
Patch

Rejecting attachment 191513 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', '--bot-id=gce-cq-04', 'apply-attachment', '--no-update', '--non-interactive', 191513, '--port=chromium-xvfb']" exit_code: 2 cwd: /mnt/git/webkit-commit-queue

Last 500 characters of output:
8.
2 out of 4 hunks FAILED -- saving rejects to file Source/WebKit/chromium/src/WebViewImpl.cpp.rej
patching file Source/WebKit/chromium/tests/Canvas2DLayerBridgeTest.cpp
patching file Source/WebKit/chromium/tests/Canvas2DLayerManagerTest.cpp
patching file Tools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Tools/DumpRenderTree/chromium/TestShell.cpp

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue

Full output: http://webkit-commit-queue.appspot.com/results/16994104

[James Robinson]

Committed r144827: <http://trac.webkit.org/changeset/144827>