Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl

Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl (patchset #2 id:20001 of https://codereview.chromium.org/2460413003/ )

Reason for revert:
Possibly caused a regression in draw properties computation time (see http://crbug.com/663068).

Original issue's description:
> cc: Remove more references to owning_layer_ from RenderSurfaceImpl
>
> This removes more dependencies of RenderSurfaceImpl on its owning layer.
> Specifically, this removes the use of owning_layer_ for accessing
> property trees and debug colors, and for gettng the surface's
> effect tree index. This also renames the OwningLayerId() method to id().
>
> BUG=611883
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
>
> Committed: https://crrev.com/740a49caf190bab13f8fdfd0979b29e09752da6b
> Cr-Commit-Position: refs/heads/master@{#428991}

TBR=jaydasika@chromium.org,weiliangc@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=611883, 663068
Committed: https://crrev.com/84667306ea7de0c234ad66737a3c519c21511ade
Cr-Commit-Position: refs/heads/master@{#430419}

[ajuma, 2016-11-07 21:46:48]

Created Revert of cc: Remove more references to owning_layer_ from
RenderSurfaceImpl

[ajuma, 2016-11-07 21:47:03]

Description was changed from

==========
Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl
(patchset #2 id:20001 of https://codereview.chromium.org/2460413003/ )

Reason for revert:
Possibly caused a regression in draw properties computation time (see
http://crbug.com/663068).

Original issue's description:
> cc: Remove more references to owning_layer_ from RenderSurfaceImpl
> 
> This removes more dependencies of RenderSurfaceImpl on its owning layer.
> Specifically, this removes the use of owning_layer_ for accessing
> property trees and debug colors, and for gettng the surface's
> effect tree index. This also renames the OwningLayerId() method to id().
> 
> BUG=611883
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/740a49caf190bab13f8fdfd0979b29e09752da6b
> Cr-Commit-Position: refs/heads/master@{#428991}

TBR=jaydasika@chromium.org,weiliangc@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=611883
==========

to

==========
Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl
(patchset #2 id:20001 of https://codereview.chromium.org/2460413003/ )

Reason for revert:
Possibly caused a regression in draw properties computation time (see
http://crbug.com/663068).

Original issue's description:
> cc: Remove more references to owning_layer_ from RenderSurfaceImpl
> 
> This removes more dependencies of RenderSurfaceImpl on its owning layer.
> Specifically, this removes the use of owning_layer_ for accessing
> property trees and debug colors, and for gettng the surface's
> effect tree index. This also renames the OwningLayerId() method to id().
> 
> BUG=611883
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/740a49caf190bab13f8fdfd0979b29e09752da6b
> Cr-Commit-Position: refs/heads/master@{#428991}

TBR=jaydasika@chromium.org,weiliangc@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=611883,663068
==========

[ajuma, 2016-11-07 21:47:40]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2016-11-07 21:48:19]

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-11-07 23:36:51]

Description was changed from

==========
Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl
(patchset #2 id:20001 of https://codereview.chromium.org/2460413003/ )

Reason for revert:
Possibly caused a regression in draw properties computation time (see
http://crbug.com/663068).

Original issue's description:
> cc: Remove more references to owning_layer_ from RenderSurfaceImpl
> 
> This removes more dependencies of RenderSurfaceImpl on its owning layer.
> Specifically, this removes the use of owning_layer_ for accessing
> property trees and debug colors, and for gettng the surface's
> effect tree index. This also renames the OwningLayerId() method to id().
> 
> BUG=611883
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/740a49caf190bab13f8fdfd0979b29e09752da6b
> Cr-Commit-Position: refs/heads/master@{#428991}

TBR=jaydasika@chromium.org,weiliangc@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=611883,663068
==========

to

==========
Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl
(patchset #2 id:20001 of https://codereview.chromium.org/2460413003/ )

Reason for revert:
Possibly caused a regression in draw properties computation time (see
http://crbug.com/663068).

Original issue's description:
> cc: Remove more references to owning_layer_ from RenderSurfaceImpl
> 
> This removes more dependencies of RenderSurfaceImpl on its owning layer.
> Specifically, this removes the use of owning_layer_ for accessing
> property trees and debug colors, and for gettng the surface's
> effect tree index. This also renames the OwningLayerId() method to id().
> 
> BUG=611883
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/740a49caf190bab13f8fdfd0979b29e09752da6b
> Cr-Commit-Position: refs/heads/master@{#428991}

TBR=jaydasika@chromium.org,weiliangc@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=611883,663068
==========

[commit-bot: I haz the power, 2016-11-07 23:37:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-07 23:43:41]

Description was changed from

==========
Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl
(patchset #2 id:20001 of https://codereview.chromium.org/2460413003/ )

Reason for revert:
Possibly caused a regression in draw properties computation time (see
http://crbug.com/663068).

Original issue's description:
> cc: Remove more references to owning_layer_ from RenderSurfaceImpl
> 
> This removes more dependencies of RenderSurfaceImpl on its owning layer.
> Specifically, this removes the use of owning_layer_ for accessing
> property trees and debug colors, and for gettng the surface's
> effect tree index. This also renames the OwningLayerId() method to id().
> 
> BUG=611883
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/740a49caf190bab13f8fdfd0979b29e09752da6b
> Cr-Commit-Position: refs/heads/master@{#428991}

TBR=jaydasika@chromium.org,weiliangc@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=611883,663068
==========

to

==========
Revert of cc: Remove more references to owning_layer_ from RenderSurfaceImpl
(patchset #2 id:20001 of https://codereview.chromium.org/2460413003/ )

Reason for revert:
Possibly caused a regression in draw properties computation time (see
http://crbug.com/663068).

Original issue's description:
> cc: Remove more references to owning_layer_ from RenderSurfaceImpl
>
> This removes more dependencies of RenderSurfaceImpl on its owning layer.
> Specifically, this removes the use of owning_layer_ for accessing
> property trees and debug colors, and for gettng the surface's
> effect tree index. This also renames the OwningLayerId() method to id().
>
> BUG=611883
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
>
> Committed: https://crrev.com/740a49caf190bab13f8fdfd0979b29e09752da6b
> Cr-Commit-Position: refs/heads/master@{#428991}

TBR=jaydasika@chromium.org,weiliangc@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=611883,663068

Committed: https://crrev.com/84667306ea7de0c234ad66737a3c519c21511ade
Cr-Commit-Position: refs/heads/master@{#430419}
==========

[commit-bot: I haz the power, 2016-11-07 23:43:42]

Patchset 1 (id:??) landed as
https://crrev.com/84667306ea7de0c234ad66737a3c519c21511ade
Cr-Commit-Position: refs/heads/master@{#430419}

[ajuma, 2016-11-21 18:35:52]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2517273002/ by ajuma@chromium.org.

The reason for reverting is: This turned out not to be the cause of the
regression in draw properties computation time..