Purge impl-only animations awaiting deletion in AnimationPlayer::UpdateState.

Purge impl-only animations awaiting deletion in AnimationPlayer::UpdateState.

Previously, animations in WAITING_FOR_DELETION state would only be purged via
AnimationPlayer::PushPropertiesTo, which synchronizes impl- and main-thread
players.  But the player in ScrollOffsetAnimationsImpl has no main-thread copy,
so its animations never got deleted after they finished.

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

Review-Url: https://codereview.chromium.org/2757453003
Cr-Commit-Position: refs/heads/master@{#457443}
Committed: https://chromium.googlesource.com/chromium/src/+/e26333eb717aa94b9f05ccf34ec91bfe31927f2a

[skobes, 2017-03-16 03:13:25]

Description was changed from

==========
AnimationPlayer::UpdateState purges impl-only animations marked for deletion.

Previously, animations in WAITING_FOR_DELETION state would only be purged via
AnimationPlayer::PushPropertiesTo, which synchronizes impl- and main-thread
players.  But the player in ScrollOffsetAnimationsImpl has no main-thread copy,
so its animations never got deleted after they finished.

BUG=701025
==========

to

==========
AnimationPlayer::UpdateState purges impl-only animations marked for deletion.

Previously, animations in WAITING_FOR_DELETION state would only be purged via
AnimationPlayer::PushPropertiesTo, which synchronizes impl- and main-thread
players.  But the player in ScrollOffsetAnimationsImpl has no main-thread copy,
so its animations never got deleted after they finished.

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

[skobes, 2017-03-16 03:21:34]

Description was changed from

==========
AnimationPlayer::UpdateState purges impl-only animations marked for deletion.

Previously, animations in WAITING_FOR_DELETION state would only be purged via
AnimationPlayer::PushPropertiesTo, which synchronizes impl- and main-thread
players.  But the player in ScrollOffsetAnimationsImpl has no main-thread copy,
so its animations never got deleted after they finished.

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

to

==========
Purge impl-only animations awaiting deletion in AnimationPlayer::UpdateState.

Previously, animations in WAITING_FOR_DELETION state would only be purged via
AnimationPlayer::PushPropertiesTo, which synchronizes impl- and main-thread
players.  But the player in ScrollOffsetAnimationsImpl has no main-thread copy,
so its animations never got deleted after they finished.

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

[skobes, 2017-03-16 03:22:30]

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

[commit-bot: I haz the power, 2017-03-16 03:22:45]

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

[skobes, 2017-03-16 03:32:35]

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

[commit-bot: I haz the power, 2017-03-16 04:24:19]

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

[commit-bot: I haz the power, 2017-03-16 04:24:19]

Dry run: This issue passed the CQ dry run.

[ajuma, 2017-03-16 14:28:33]

lgtm

[skobes, 2017-03-16 15:29:42]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2017-03-16 15:30:21]

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, 2017-03-16 15:36:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489678182490290, "parent_rev":
"3420248539cc1176ac975d7a87adb67937c7d8a8", "commit_rev":
"e26333eb717aa94b9f05ccf34ec91bfe31927f2a"}

[commit-bot: I haz the power, 2017-03-16 15:37:43]

Description was changed from

==========
Purge impl-only animations awaiting deletion in AnimationPlayer::UpdateState.

Previously, animations in WAITING_FOR_DELETION state would only be purged via
AnimationPlayer::PushPropertiesTo, which synchronizes impl- and main-thread
players.  But the player in ScrollOffsetAnimationsImpl has no main-thread copy,
so its animations never got deleted after they finished.

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

to

==========
Purge impl-only animations awaiting deletion in AnimationPlayer::UpdateState.

Previously, animations in WAITING_FOR_DELETION state would only be purged via
AnimationPlayer::PushPropertiesTo, which synchronizes impl- and main-thread
players.  But the player in ScrollOffsetAnimationsImpl has no main-thread copy,
so its animations never got deleted after they finished.

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

Review-Url: https://codereview.chromium.org/2757453003
Cr-Commit-Position: refs/heads/master@{#457443}
Committed:
https://chromium.googlesource.com/chromium/src/+/e26333eb717aa94b9f05ccf34ec9...
==========

[commit-bot: I haz the power, 2017-03-16 15:37:44]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e26333eb717aa94b9f05ccf34ec9...