Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex

Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex (patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
>
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
>
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
>
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org

BUG=670074, 670679
Committed: https://crrev.com/9d01b1b4fb103fb74319a072cc8591dc3e196685
Cr-Commit-Position: refs/heads/master@{#435951}

[ajuma, 2016-12-02 15:41:54]

Created Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex

[ajuma, 2016-12-02 15:42:25]

Description was changed from

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
> 
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
> 
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=670074
==========

to

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
> 
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
> 
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.

BUG=670074,670679
==========

[ajuma, 2016-12-02 15:42:32]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2016-12-02 15:43:09]

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

[ajuma, 2016-12-02 15:43:16]

Description was changed from

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
> 
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
> 
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.

BUG=670074,670679
==========

to

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
> 
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
> 
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org

BUG=670074,670679
==========

[commit-bot: I haz the power, 2016-12-02 16:39:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480693352906280, "parent_rev":
"df73ebebfe72586771be1d07bfeb6fdb029aa5e1", "commit_rev":
"39874352e6123895d2dd3f0d6b3a9d2c22040b48"}

[commit-bot: I haz the power, 2016-12-02 16:40:19]

Description was changed from

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
> 
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
> 
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org

BUG=670074,670679
==========

to

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
> 
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
> 
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org

BUG=670074,670679
==========

[commit-bot: I haz the power, 2016-12-02 16:40:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-02 16:43:18]

Description was changed from

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
> 
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
> 
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org

BUG=670074,670679
==========

to

==========
Revert of cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
(patchset #1 id:1 of https://codereview.chromium.org/2540333002/ )

Reason for revert:
This is causing crashes: http://crbug.com/670679

Original issue's description:
> cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex
>
> The map lookup was added by http://crrev.com/434349, but caused a
> 5-10% regression in CalcDrawPropsTests cc_perftests. This CL sets
> the effect tree index on the surface during the surface update step
> (when EffectNode::render_surface gets updated).
>
> BUG=670074
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
>
> Committed: https://crrev.com/2d1272d61ee51ba2e2171c77c3cc4545cd288d99
> Cr-Commit-Position: refs/heads/master@{#435672}

TBR=weiliangc@chromium.org,jaydasika@chromium.org

BUG=670074,670679

Committed: https://crrev.com/9d01b1b4fb103fb74319a072cc8591dc3e196685
Cr-Commit-Position: refs/heads/master@{#435951}
==========

[commit-bot: I haz the power, 2016-12-02 16:43:19]

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