cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

This makes RenderSurfaceImpl::ClipTreeIndex use the surface's
effect node rather than its owning layer to find its clip id.
The clip id on an effect node is actually for the parent of the
owning layer's clip node, but the parent clip node is in fact
the one that call sites care about since it represents the clip
applied to the surface's output rather than to its input.

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

Review-Url: https://codereview.chromium.org/2636953002
Cr-Commit-Position: refs/heads/master@{#443935}
Committed: https://chromium.googlesource.com/chromium/src/+/ed86f4eb6b5a10332b8d4477afc19834faa6079a

[ajuma, 2017-01-16 18:11:34]

Description was changed from

==========
cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

This makes RenderSurfaceImpl::ClipTreeIndex use the surface's
effect node rather than its owning layer to find its clip id.
The clip id on an effect node is actually for the parent of the
owning layer's clip node, but the parent clip node is in fact
the one that callsites care about since it represents the clip
applied to the surface's output rather than to its input.

BUG=679837
==========

to

==========
cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

This makes RenderSurfaceImpl::ClipTreeIndex use the surface's
effect node rather than its owning layer to find its clip id.
The clip id on an effect node is actually for the parent of the
owning layer's clip node, but the parent clip node is in fact
the one that callsites care about since it represents the clip
applied to the surface's output rather than to its input.

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

[ajuma, 2017-01-16 18:13:37]

The CQ bit was checked by ajuma@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-16 18:13:57]

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

[ajuma, 2017-01-16 19:08:03]

Description was changed from

==========
cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

This makes RenderSurfaceImpl::ClipTreeIndex use the surface's
effect node rather than its owning layer to find its clip id.
The clip id on an effect node is actually for the parent of the
owning layer's clip node, but the parent clip node is in fact
the one that callsites care about since it represents the clip
applied to the surface's output rather than to its input.

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

to

==========
cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

This makes RenderSurfaceImpl::ClipTreeIndex use the surface's
effect node rather than its owning layer to find its clip id.
The clip id on an effect node is actually for the parent of the
owning layer's clip node, but the parent clip node is in fact
the one that call sites care about since it represents the clip
applied to the surface's output rather than to its input.

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

[ajuma, 2017-01-16 19:08:03]

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

[commit-bot: I haz the power, 2017-01-16 19:11:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-16 19:11:39]

Dry run: This issue passed the CQ dry run.

[weiliangc, 2017-01-16 19:49:29]

LGTM

[ajuma, 2017-01-16 19:50:35]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2017-01-16 19:50:43]

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-01-16 19:54:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484596235288860, "parent_rev":
"386b2a829c8ca86b04fad4ae527a35991b872bd0", "commit_rev":
"ed86f4eb6b5a10332b8d4477afc19834faa6079a"}

[commit-bot: I haz the power, 2017-01-16 19:55:03]

Description was changed from

==========
cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

This makes RenderSurfaceImpl::ClipTreeIndex use the surface's
effect node rather than its owning layer to find its clip id.
The clip id on an effect node is actually for the parent of the
owning layer's clip node, but the parent clip node is in fact
the one that call sites care about since it represents the clip
applied to the surface's output rather than to its input.

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

to

==========
cc: Compute RenderSurfaceImpl's ClipTreeIndex using its effect node

This makes RenderSurfaceImpl::ClipTreeIndex use the surface's
effect node rather than its owning layer to find its clip id.
The clip id on an effect node is actually for the parent of the
owning layer's clip node, but the parent clip node is in fact
the one that call sites care about since it represents the clip
applied to the surface's output rather than to its input.

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

Review-Url: https://codereview.chromium.org/2636953002
Cr-Commit-Position: refs/heads/master@{#443935}
Committed:
https://chromium.googlesource.com/chromium/src/+/ed86f4eb6b5a10332b8d4477afc1...
==========

[commit-bot: I haz the power, 2017-01-16 19:55:03]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ed86f4eb6b5a10332b8d4477afc1...