Discard SampledEffects on elements if they are redundant

Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change [0] and have their effects occluded by other
SampledEffects that will never change.

[0] https://docs.google.com/document/d/1NomOWRrGQHlynQGO64CgdqRPAAEHhi3fSa8sf0Ip6xE

BUG=360473
Committed: https://crrev.com/4dd3920845f5f628f606909cdb447e6576fd4292
Cr-Commit-Position: refs/heads/master@{#375877}

[alancutter (OOO until 2018), 2016-02-16 06:43:34]

alancutter@chromium.org changed reviewers:
+ timloh@chromium.org

[alancutter (OOO until 2018), 2016-02-16 06:43:57]

Description was changed from

==========
Discard Animation and AnimationEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change (their Animation has been GC'd) and have their
effects occluded by other SampledEffects that will never change.

BUG=360473
==========

to

==========
Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change (their Animation has been GC'd) and have their
effects occluded by other SampledEffects that will never change.

BUG=360473
==========

[alancutter (OOO until 2018), 2016-02-17 01:15:04]

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

[commit-bot: I haz the power, 2016-02-17 01:17:38]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1698093005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1698093005/20001

[commit-bot: I haz the power, 2016-02-17 02:36:27]

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

[commit-bot: I haz the power, 2016-02-17 02:36:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[alancutter (OOO until 2018), 2016-02-17 05:28:22]

Description was changed from

==========
Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change (their Animation has been GC'd) and have their
effects occluded by other SampledEffects that will never change.

BUG=360473
==========

to

==========
Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change [0] and have their effects occluded by other
SampledEffects that will never change.

[0]
https://docs.google.com/document/d/1NomOWRrGQHlynQGO64CgdqRPAAEHhi3fSa8sf0Ip6xE
BUG=360473
==========

[alancutter (OOO until 2018), 2016-02-17 05:38:30]

Description was changed from

==========
Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change [0] and have their effects occluded by other
SampledEffects that will never change.

[0]
https://docs.google.com/document/d/1NomOWRrGQHlynQGO64CgdqRPAAEHhi3fSa8sf0Ip6xE
BUG=360473
==========

to

==========
Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change [0] and have their effects occluded by other
SampledEffects that will never change.

[0]
https://docs.google.com/document/d/1NomOWRrGQHlynQGO64CgdqRPAAEHhi3fSa8sf0Ip6xE

BUG=360473
==========

[alancutter (OOO until 2018), 2016-02-17 05:43:55]

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

[commit-bot: I haz the power, 2016-02-17 05:44:27]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1698093005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1698093005/40001

[Timothy Loh, 2016-02-17 06:27:35]

lgmt

[Timothy Loh, 2016-02-17 06:29:51]

On 2016/02/17 06:27:35, Timothy Loh wrote:
> lgmt

lgtm

[commit-bot: I haz the power, 2016-02-17 06:52:41]

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

[commit-bot: I haz the power, 2016-02-17 06:52:42]

Dry run: This issue passed the CQ dry run.

[alancutter (OOO until 2018), 2016-02-17 14:09:47]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-02-17 14:10:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1698093005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1698093005/40001

[commit-bot: I haz the power, 2016-02-17 14:17:04]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-17 14:18:33]

Description was changed from

==========
Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change [0] and have their effects occluded by other
SampledEffects that will never change.

[0]
https://docs.google.com/document/d/1NomOWRrGQHlynQGO64CgdqRPAAEHhi3fSa8sf0Ip6xE

BUG=360473
==========

to

==========
Discard SampledEffects on elements if they are redundant

This change is a memory performance optimisation to discard SampledEffects
that will never change [0] and have their effects occluded by other
SampledEffects that will never change.

[0]
https://docs.google.com/document/d/1NomOWRrGQHlynQGO64CgdqRPAAEHhi3fSa8sf0Ip6xE

BUG=360473

Committed: https://crrev.com/4dd3920845f5f628f606909cdb447e6576fd4292
Cr-Commit-Position: refs/heads/master@{#375877}
==========

[commit-bot: I haz the power, 2016-02-17 14:18:34]

Patchset 3 (id:??) landed as
https://crrev.com/4dd3920845f5f628f606909cdb447e6576fd4292
Cr-Commit-Position: refs/heads/master@{#375877}