Fix GraphicsLayer traversal in SimCompositor

Fix GraphicsLayer traversal in SimCompositor

Match what FrameView::synchronizedPaint does, rather than only
traversing the mainGraphicsLayer.

BUG=490942
R=chrishtr@chromium.org,skyostil@chromium.org

Committed: https://crrev.com/dd7e49692b1288ee1a3774bb84062df1525de62e
Cr-Commit-Position: refs/heads/master@{#416998}

[Sami, 2016-09-07 14:48:43]

lgtm % two nits. Thanks for the fix!

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/tests/FrameThrottlingTest.cpp (right):

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/tests/FrameThrottlingTest.cpp:360: 
Unintended whitespace?

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/tests/sim/SimCompositor.cpp (right):

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/tests/sim/SimCompositor.cpp:23: static void
paintLayers(PaintLayer& layer, SimDisplayItemList& displayList)
Probably meant to delete this completely?

[chrishtr, 2016-09-07 16:40:49]

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/tests/sim/SimCompositor.cpp (right):

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/tests/sim/SimCompositor.cpp:47: if
(!RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
I don't think you need this conditional.

[szager1, 2016-09-07 16:52:41]

Description was changed from

==========
Fix FrameThrottlingTest for root layer scrolls.

Make the layer-traversing code in SimCompositor match what
FrameView::synchronizedPaint does, rather than only traversing
the mainGraphicsLayer.

BUG=490942
R=chrishtr@chromium.org,skyostil@chromium.org
==========

to

==========
Fix GraphicsLayer traversal in SimCompositor

Match what FrameView::synchronizedPaint does, rather than only
traversing the mainGraphicsLayer.

BUG=490942
R=chrishtr@chromium.org,skyostil@chromium.org
==========

[szager1, 2016-09-07 16:53:01]

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/tests/FrameThrottlingTest.cpp (right):

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/tests/FrameThrottlingTest.cpp:360: 
On 2016/09/07 14:48:43, Sami wrote:
> Unintended whitespace?

Fixed.

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
File third_party/WebKit/Source/web/tests/sim/SimCompositor.cpp (right):

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/tests/sim/SimCompositor.cpp:23: static void
paintLayers(PaintLayer& layer, SimDisplayItemList& displayList)
On 2016/09/07 14:48:43, Sami wrote:
> Probably meant to delete this completely?

Yep, done.

https://codereview.chromium.org/2315083002/diff/1/third_party/WebKit/Source/w...
third_party/WebKit/Source/web/tests/sim/SimCompositor.cpp:47: if
(!RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
On 2016/09/07 16:40:49, chrishtr wrote:
> I don't think you need this conditional.

Done.

[chrishtr, 2016-09-07 16:57:07]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2016-09-07 16:57:08]

lgtm

[chrishtr, 2016-09-07 16:57:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/2315083002/#ps40001
(title: "Drop unneeded #include")

[commit-bot: I haz the power, 2016-09-07 16:57:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 18:19:38]

Description was changed from

==========
Fix GraphicsLayer traversal in SimCompositor

Match what FrameView::synchronizedPaint does, rather than only
traversing the mainGraphicsLayer.

BUG=490942
R=chrishtr@chromium.org,skyostil@chromium.org
==========

to

==========
Fix GraphicsLayer traversal in SimCompositor

Match what FrameView::synchronizedPaint does, rather than only
traversing the mainGraphicsLayer.

BUG=490942
R=chrishtr@chromium.org,skyostil@chromium.org
==========

[commit-bot: I haz the power, 2016-09-07 18:19:39]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-07 18:22:02]

Description was changed from

==========
Fix GraphicsLayer traversal in SimCompositor

Match what FrameView::synchronizedPaint does, rather than only
traversing the mainGraphicsLayer.

BUG=490942
R=chrishtr@chromium.org,skyostil@chromium.org
==========

to

==========
Fix GraphicsLayer traversal in SimCompositor

Match what FrameView::synchronizedPaint does, rather than only
traversing the mainGraphicsLayer.

BUG=490942
R=chrishtr@chromium.org,skyostil@chromium.org

Committed: https://crrev.com/dd7e49692b1288ee1a3774bb84062df1525de62e
Cr-Commit-Position: refs/heads/master@{#416998}
==========

[commit-bot: I haz the power, 2016-09-07 18:22:03]

Patchset 3 (id:??) landed as
https://crrev.com/dd7e49692b1288ee1a3774bb84062df1525de62e
Cr-Commit-Position: refs/heads/master@{#416998}