Adding callback for finished property animations.

Adding callback for finished property animations.

This CL extends the ChromeAnimation class to notify
Animatable interfaces that a particular property
animation has finished.

BUG=

Committed: https://crrev.com/4107f387c09d36eb24f304e2bc33ec82410bdd9a
Cr-Commit-Position: refs/heads/master@{#381854}

[pedro (no code reviews), 2016-03-18 00:08:13]

Description was changed from

==========
Adding callback for finished property animations.

This CL extends the ChromeAnimation class to notify
Animatable interfaces that a particular property
animation has finished.

BUG=
==========

to

==========
Adding callback for finished property animations.

This CL extends the ChromeAnimation class to notify
Animatable interfaces that a particular property
animation has finished.

BUG=
==========

[pedro (no code reviews), 2016-03-18 00:08:14]

pedrosimonetti@chromium.org changed reviewers:
+ dtrainor@chromium.org

[pedro (no code reviews), 2016-03-18 00:10:04]

Hey David, please take a look at this change.

Based on our offline discussion, I changed the code to only fire when an
individual property animation is finished, and not just when all of the
concurrent animations finish.

As you requested, I also included a unit test to make sure that the callback
will fire for each individual property when multiple animations are running
concurrently.

[David Trainor- moved to gerrit, 2016-03-18 00:17:28]

like the test thanks! :)

lgtm!

[pedro (no code reviews), 2016-03-18 00:22:06]

The CQ bit was checked by pedrosimonetti@chromium.org

[commit-bot: I haz the power, 2016-03-18 00:22:48]

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

[commit-bot: I haz the power, 2016-03-18 00:50:51]

Description was changed from

==========
Adding callback for finished property animations.

This CL extends the ChromeAnimation class to notify
Animatable interfaces that a particular property
animation has finished.

BUG=
==========

to

==========
Adding callback for finished property animations.

This CL extends the ChromeAnimation class to notify
Animatable interfaces that a particular property
animation has finished.

BUG=
==========

[commit-bot: I haz the power, 2016-03-18 00:50:52]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-18 00:53:34]

Description was changed from

==========
Adding callback for finished property animations.

This CL extends the ChromeAnimation class to notify
Animatable interfaces that a particular property
animation has finished.

BUG=
==========

to

==========
Adding callback for finished property animations.

This CL extends the ChromeAnimation class to notify
Animatable interfaces that a particular property
animation has finished.

BUG=

Committed: https://crrev.com/4107f387c09d36eb24f304e2bc33ec82410bdd9a
Cr-Commit-Position: refs/heads/master@{#381854}
==========

[commit-bot: I haz the power, 2016-03-18 00:53:34]

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