cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
LayerImpl::is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2851193002
Cr-Commit-Position: refs/heads/master@{#468413}
Committed: https://chromium.googlesource.com/chromium/src/+/651848a51fcc175e9a42f588efc3d7d138ccaa36

[ajuma, 2017-05-01 14:56:56]

Description was changed from

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
==========

to

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-05-01 14:57:08]

Description was changed from

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
LayerImpl::is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-05-01 14:58:58]

Description was changed from

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
LayerImpl::is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
LayerImpl::is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-05-01 14:58:58]

[email protected] changed reviewers:
+ [email protected], [email protected]

[ajuma, 2017-05-01 15:36:40]

The CQ bit was checked by [email protected] to run a CQ dry run

[commit-bot: I haz the power, 2017-05-01 15:37:00]

Dry run: 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, 2017-05-01 16:38:58]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2017-05-01 16:38:59]

Dry run: This issue passed the CQ dry run.

[jaydasika, 2017-05-01 16:49:22]

lgtm

https://codereview.chromium.org/2851193002/diff/1/cc/trees/layer_tree_host_co...
File cc/trees/layer_tree_host_common_unittest.cc (right):

https://codereview.chromium.org/2851193002/diff/1/cc/trees/layer_tree_host_co...
cc/trees/layer_tree_host_common_unittest.cc:10308: // mask isn't a drawn RSLL
member. This means it has an empty visible rect,
Can you also change the comment, since there is no RSLL ?

[ajuma, 2017-05-01 17:23:59]

https://codereview.chromium.org/2851193002/diff/1/cc/trees/layer_tree_host_co...
File cc/trees/layer_tree_host_common_unittest.cc (right):

https://codereview.chromium.org/2851193002/diff/1/cc/trees/layer_tree_host_co...
cc/trees/layer_tree_host_common_unittest.cc:10308: // mask isn't a drawn RSLL
member. This means it has an empty visible rect,
On 2017/05/01 16:49:22, jaydasika wrote:
> Can you also change the comment, since there is no RSLL ?

Done.

[ajuma, 2017-05-01 17:24:03]

The CQ bit was checked by [email protected]

[ajuma, 2017-05-01 17:24:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
[email protected]
Link to the patchset: https://codereview.chromium.org/2851193002/#ps20001
(title: "Address review comment")

[commit-bot: I haz the power, 2017-05-01 17:24:47]

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, 2017-05-01 18:51:04]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2017-05-01 18:51:06]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[ajuma, 2017-05-01 19:36:13]

The CQ bit was checked by [email protected]

[ajuma, 2017-05-01 19:36:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
[email protected]
Link to the patchset: https://codereview.chromium.org/2851193002/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2017-05-01 19:36:46]

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, 2017-05-01 21:23:48]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493667373702470,
"parent_rev": "851c752c37f7400ce870f46bfcaaa0d65742f15f", "commit_rev":
"651848a51fcc175e9a42f588efc3d7d138ccaa36"}

[commit-bot: I haz the power, 2017-05-01 21:24:02]

Description was changed from

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
LayerImpl::is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Rename LayerImpl::is_drawn_render_surface_layer_list_member

Since there's no longer a render surface layer list, this renames
LayerImpl::is_drawn_render_surface_layer_list_member to
contributes_to_drawn_render_surface.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2851193002
Cr-Commit-Position: refs/heads/master@{#468413}
Committed:
https://chromium.googlesource.com/chromium/src/+/651848a51fcc175e9a42f588efc3...
==========

[commit-bot: I haz the power, 2017-05-01 21:24:03]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/651848a51fcc175e9a42f588efc3...