CC Animation: Activate/deactivate ElementAnimations on Layer addition/removal.

CC Animation: Activate/deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

Blink side: We will need to fix the race in CompositedLayerMapping destruction vs. CompositorAnimationPlayer detachment separately.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/638ef70330224e93f144c2adbd864ac786bf1e70
Cr-Commit-Position: refs/heads/master@{#398209}

[loyso (OOO), 2016-06-06 06:04:28]

Description was changed from

==========
CC Animation: Activate/Deactivate ElementAnimations on Layer addition/removal.

BUG=615471
==========

to

==========
CC Animation: Activate/Deactivate ElementAnimations on Layer addition/removal.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[loyso (OOO), 2016-06-06 06:06:22]

Description was changed from

==========
CC Animation: Activate/Deactivate ElementAnimations on Layer addition/removal.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Activate/Deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

We will need to fix CompositedLayerMapping /CompositorAnimationPlayer
destruction race separately in Blink.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[loyso (OOO), 2016-06-06 06:06:22]

loyso@chromium.org changed reviewers:
+ ajuma@chromium.org

[loyso (OOO), 2016-06-06 06:07:05]

Description was changed from

==========
CC Animation: Activate/Deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

We will need to fix CompositedLayerMapping /CompositorAnimationPlayer
destruction race separately in Blink.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Activate/Deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

We will need to fix CompositedLayerMapping  vs. CompositorAnimationPlayer
destruction race separately in Blink.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[loyso (OOO), 2016-06-06 06:07:56]

Description was changed from

==========
CC Animation: Activate/Deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

We will need to fix CompositedLayerMapping  vs. CompositorAnimationPlayer
destruction race separately in Blink.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Activate/deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

Blink side: We will need to fix the race in CompositedLayerMapping destruction
vs. CompositorAnimationPlayer detachment separately.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[ajuma, 2016-06-06 13:17:02]

lgtm

https://codereview.chromium.org/2042713002/diff/1/cc/trees/layer_tree_host_im...
File cc/trees/layer_tree_host_impl_unittest.cc (right):

https://codereview.chromium.org/2042713002/diff/1/cc/trees/layer_tree_host_im...
cc/trees/layer_tree_host_impl_unittest.cc:1463:
root->test_properties()->force_render_surface = true;
Nit: the root always gets a render surface, so this line shouldn't be needed

[loyso (OOO), 2016-06-07 01:27:01]

The CQ bit was checked by loyso@chromium.org

[loyso (OOO), 2016-06-07 01:27:02]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2042713002/#ps20001
(title: "Fix the nit.")

[commit-bot: I haz the power, 2016-06-07 01:27:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2042713002/20001

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

Description was changed from

==========
CC Animation: Activate/deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

Blink side: We will need to fix the race in CompositedLayerMapping destruction
vs. CompositorAnimationPlayer detachment separately.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Activate/deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

Blink side: We will need to fix the race in CompositedLayerMapping destruction
vs. CompositorAnimationPlayer detachment separately.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-06-07 02:52:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-07 02:53:57]

Description was changed from

==========
CC Animation: Activate/deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

Blink side: We will need to fix the race in CompositedLayerMapping destruction
vs. CompositorAnimationPlayer detachment separately.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
CC Animation: Activate/deactivate ElementAnimations on Layer addition/removal.

This is a CC-side fix which can be useful for all CC clients.

Blink side: We will need to fix the race in CompositedLayerMapping destruction
vs. CompositorAnimationPlayer detachment separately.

BUG=615471
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/638ef70330224e93f144c2adbd864ac786bf1e70
Cr-Commit-Position: refs/heads/master@{#398209}
==========

[commit-bot: I haz the power, 2016-06-07 02:53:58]

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