Detect/report animations that are cancelled after they start.

Detect/report animations that are cancelled after they start.

Previously, we were early-exiting from |animationPlayStateChange| if an
animation was contained in |m_idToAnimation|. However, this means that
we miss cancel events for animations that have already been started,
since the 'start' event causes them to be placed in |m_idToAnimation|.

BUG=686091
Review-Url: https://codereview.chromium.org/2656273003
Cr-Commit-Position: refs/heads/master@{#447074}
Committed: https://chromium.googlesource.com/chromium/src/+/e04864cab7755bdc01abf0431cfd825d488b309c

[smcgruer, 2017-01-27 15:14:25]

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

[commit-bot: I haz the power, 2017-01-27 15:14:34]

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

[smcgruer, 2017-01-27 15:15:25]

smcgruer@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org, suzyh@chromium.org

[commit-bot: I haz the power, 2017-01-27 17:11:27]

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

[commit-bot: I haz the power, 2017-01-27 17:11:27]

Dry run: This issue passed the CQ dry run.

[kozy, 2017-01-30 17:22:31]

kozyatinskiy@chromium.org changed reviewers:
+ dgozman@chromium.org

[kozy, 2017-01-30 17:22:31]

lgtm, thanks.

Added Dmitry since I'm not an owner of this file.

[dgozman, 2017-01-30 17:31:54]

lgtm

[smcgruer, 2017-01-30 17:32:20]

The CQ bit was checked by smcgruer@chromium.org

[commit-bot: I haz the power, 2017-01-30 17:32:34]

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

[suzyh_UTC10 (ex-contributor), 2017-01-30 19:58:32]

lgtm

[commit-bot: I haz the power, 2017-01-30 21:38:46]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485797540331220,
"parent_rev": "80341e502bc01bdab13c7e69c2d713150a9f833b", "commit_rev":
"e04864cab7755bdc01abf0431cfd825d488b309c"}

[commit-bot: I haz the power, 2017-01-30 21:39:22]

Description was changed from

==========
Detect/report animations that are cancelled after they start.

Previously, we were early-exiting from |animationPlayStateChange| if an
animation was contained in |m_idToAnimation|. However, this means that
we miss cancel events for animations that have already been started,
since the 'start' event causes them to be placed in |m_idToAnimation|.

BUG=686091
==========

to

==========
Detect/report animations that are cancelled after they start.

Previously, we were early-exiting from |animationPlayStateChange| if an
animation was contained in |m_idToAnimation|. However, this means that
we miss cancel events for animations that have already been started,
since the 'start' event causes them to be placed in |m_idToAnimation|.

BUG=686091

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

[commit-bot: I haz the power, 2017-01-30 21:39:40]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/e04864cab7755bdc01abf0431cfd...