cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex

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}

[ajuma, 2016-11-30 23:08:45]

Description was changed from

==========
cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex

BUG=670074
==========

to

==========
cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex

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

[ajuma, 2016-11-30 23:09:22]

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

[commit-bot: I haz the power, 2016-11-30 23:09:56]

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

[ajuma, 2016-11-30 23:26:52]

Description was changed from

==========
cc: Remove map lookup from RenderSurfaceImpl::EffectTreeIndex

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

to

==========
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
==========

[commit-bot: I haz the power, 2016-11-30 23:55:05]

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

[commit-bot: I haz the power, 2016-11-30 23:55:06]

Dry run: This issue passed the CQ dry run.

[ajuma, 2016-11-30 23:59:20]

Description was changed from

==========
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
==========

to

==========
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
==========

[ajuma, 2016-11-30 23:59:20]

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

[jaydasika, 2016-12-01 02:15:12]

lgtm

[ajuma, 2016-12-01 19:13:08]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2016-12-01 19:13:42]

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-12-01 19:19:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480619588416830, "parent_rev":
"a26e40f406a83f0daf2256848591ed719ac7e495", "commit_rev":
"6d6d0f9d469ba6a05d516662664e16a51e8e3e87"}

[commit-bot: I haz the power, 2016-12-01 19:19:37]

Description was changed from

==========
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
==========

to

==========
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
==========

[commit-bot: I haz the power, 2016-12-01 19:19:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-01 19:23:48]

Description was changed from

==========
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
==========

to

==========
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}
==========

[commit-bot: I haz the power, 2016-12-01 19:23:49]

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

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

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

The reason for reverting is: This is causing crashes: http://crbug.com/670679.