cc : Check copy request count using effect tree

cc : Check copy request count using effect tree

BUG=598748
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[jaydasika, 2016-03-29 19:11:35]

Description was changed from

==========
cc : Check copy request count using effect tree

BUG=598748
==========

to

==========
cc : Check copy request count using effect tree

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

[jaydasika, 2016-03-29 19:12:59]

jaydasika@chromium.org changed reviewers:
+ ajuma@chromium.org, vollick@chromium.org, weiliangc@chromium.org

[jaydasika, 2016-03-29 19:13:00]

PTAL

[ajuma, 2016-03-29 19:30:43]

lgtm

https://codereview.chromium.org/1842883002/diff/1/cc/trees/layer_tree_impl.cc
File cc/trees/layer_tree_impl.cc (right):

https://codereview.chromium.org/1842883002/diff/1/cc/trees/layer_tree_impl.cc...
cc/trees/layer_tree_impl.cc:768: void SanityCheckCopyRequestCounts(LayerImpl*
root_layer) {
This is only using the root_layer in order to get the LayerTreeImpl, so it
should probably just take the LayerTreeImpl as an argument instead.

[jaydasika, 2016-03-29 19:35:15]

https://codereview.chromium.org/1842883002/diff/1/cc/trees/layer_tree_impl.cc
File cc/trees/layer_tree_impl.cc (right):

https://codereview.chromium.org/1842883002/diff/1/cc/trees/layer_tree_impl.cc...
cc/trees/layer_tree_impl.cc:768: void SanityCheckCopyRequestCounts(LayerImpl*
root_layer) {
On 2016/03/29 19:30:43, ajuma wrote:
> This is only using the root_layer in order to get the LayerTreeImpl, so it
> should probably just take the LayerTreeImpl as an argument instead.

Done.

[jaydasika, 2016-03-29 19:35:20]

The CQ bit was checked by jaydasika@chromium.org

[jaydasika, 2016-03-29 19:35:21]

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

[commit-bot: I haz the power, 2016-03-29 19:35:33]

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

[commit-bot: I haz the power, 2016-03-29 20:49:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-29 20:50:22]

Description was changed from

==========
cc : Check copy request count using effect tree

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

to

==========
cc : Check copy request count using effect tree

BUG=598748
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-03-29 20:50:23]

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