Bug 159209 – [Coordinated Graphics] Move CompositingCoordinator from platform to WebKit2 layer

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Carlos Garcia Campos]

It's a layering violation, that code belongs to the WebKit2 layer.

[Carlos Garcia Campos]

Created attachment 282238 [details]
Patch

To make the review easier I haven't changed any code, just moved it, updated the header includes and added some WebCore::. I plan to do further cleanup in a follow up patch.

[Carlos Garcia Campos]

Created attachment 282341 [details]
Rebased patch

This should apply on trunk now

[Build Bot]

Comment on attachment 282341 [details]
Rebased patch

Attachment 282341 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/1594193

New failing tests:
animations/multiple-backgrounds.html

[Build Bot]

Created attachment 282343 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.4

[Carlos Garcia Campos]

(In reply to comment #3)
> Comment on attachment 282341 [details]
> Rebased patch
> 
> Attachment 282341 [details] did not pass ios-sim-ews (ios-simulator-wk2):
> Output: http://webkit-queues.webkit.org/results/1594193
> 
> New failing tests:
> animations/multiple-backgrounds.html

This is just code moved and shouldn't affect iOS at all.

[Carlos Garcia Campos]

Committed r202675: <http://trac.webkit.org/changeset/202675>