cc : Replace LayerImpl::children calls with LayerById

cc : Replace LayerImpl::children calls with LayerById

Replacing because these tests expect layer tree hierarchy to be pushed
during commit and we won't be doing this after switching to layer lists.

BUG=604467
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[jaydasika, 2016-04-20 21:00:02]

Description was changed from

==========
cc : Replace LayerImpl::children calls with LayerById

Replacing because these tests expect layer tree hierarchy to be pushed
during commit and we won't be doing this after switching to layer lists.

BUG=604467
==========

to

==========
cc : Replace LayerImpl::children calls with LayerById

Replacing because these tests expect layer tree hierarchy to be pushed
during commit and we won't be doing this after switching to layer lists.

BUG=604467
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[jaydasika, 2016-04-20 21:02:19]

jaydasika@chromium.org changed reviewers:
+ ajuma@chromium.org, sunxd@chromium.org

[jaydasika, 2016-04-20 21:02:19]

PTAL

[sunxd, 2016-04-21 17:51:18]

On 2016/04/20 21:02:19, jaydasika wrote:
> PTAL

lgtm % ajuma

[ajuma, 2016-04-21 18:33:19]

lgtm

https://codereview.chromium.org/1895873008/diff/1/cc/layers/scrollbar_layer_u...
File cc/layers/scrollbar_layer_unittest.cc (right):

https://codereview.chromium.org/1895873008/diff/1/cc/layers/scrollbar_layer_u...
cc/layers/scrollbar_layer_unittest.cc:99: ScrollbarLayerTest() :
fake_client_(FakeLayerTreeHostClient::DIRECT_3D) {
initialize scrollbar_layer_id_ here too

[jaydasika, 2016-04-21 18:47:00]

https://codereview.chromium.org/1895873008/diff/1/cc/layers/scrollbar_layer_u...
File cc/layers/scrollbar_layer_unittest.cc (right):

https://codereview.chromium.org/1895873008/diff/1/cc/layers/scrollbar_layer_u...
cc/layers/scrollbar_layer_unittest.cc:99: ScrollbarLayerTest() :
fake_client_(FakeLayerTreeHostClient::DIRECT_3D) {
On 2016/04/21 18:33:19, ajuma wrote:
> initialize scrollbar_layer_id_ here too

Done.

[jaydasika, 2016-04-21 18:47:03]

The CQ bit was checked by jaydasika@chromium.org

[jaydasika, 2016-04-21 18:47:04]

The patchset sent to the CQ was uploaded after l-g-t-m from sunxd@chromium.org,
ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/1895873008/#ps20001
(title: " ")

[commit-bot: I haz the power, 2016-04-21 18:47:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1895873008/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1895873008/20001

[commit-bot: I haz the power, 2016-04-21 19:45:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 19:38:01]

Description was changed from

==========
cc : Replace LayerImpl::children calls with LayerById

Replacing because these tests expect layer tree hierarchy to be pushed
during commit and we won't be doing this after switching to layer lists.

BUG=604467
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc : Replace LayerImpl::children calls with LayerById

Replacing because these tests expect layer tree hierarchy to be pushed
during commit and we won't be doing this after switching to layer lists.

BUG=604467
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-04-22 19:38:03]

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