Add monotonic time and target property to cc::AnimationDelegate notifications

Add monotonic time and target property to cc::AnimationDelegate notifications

This adds monotonic time and target property as arguments to
cc::AnimationDelegate::NotifyAnimation{Started, Finished}.

The existing wall clock time argument can be removed once Blink's legacy
implementation of CSS animations and transitions is removed.

The target property argument is needed so that Blink can distinguish
between notifications for CSS animations/transitions and notifications for
the scroll animations being added for the CSSOM View smooth scroll API.

BUG=299945, 243871
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=239554

[ajuma, 2013-12-03 00:04:51]

The Blink side of this is http://crrev.com/99333005 (the chromium side needs to
land before the Blink side).

[Ian Vollick, 2013-12-05 21:43:49]

On 2013/12/03 00:04:51, ajuma wrote:
> The Blink side of this is http://crrev.com/99333005 (the chromium side needs
to
> land before the Blink side).

lgtm!

[ajuma, 2013-12-05 21:51:59]

+jamesr for compositor_bindings/

[jamesr, 2013-12-05 22:01:39]

https://codereview.chromium.org/99733003/diff/1/cc/animation/animation_delega...
File cc/animation/animation_delegate.h (right):

https://codereview.chromium.org/99733003/diff/1/cc/animation/animation_delega...
cc/animation/animation_delegate.h:18: double monotonic_time,
please use base::TimeTicks for monotonic time values. we can convert to double
at the blink API boundary since that's what blink needs, but all monotonic time
base logic in chromium should be using base::TimeTicks

[ajuma, 2013-12-06 00:41:34]

PTAL.

https://codereview.chromium.org/99733003/diff/1/cc/animation/animation_delega...
File cc/animation/animation_delegate.h (right):

https://codereview.chromium.org/99733003/diff/1/cc/animation/animation_delega...
cc/animation/animation_delegate.h:18: double monotonic_time,
On 2013/12/05 22:01:39, jamesr wrote:
> please use base::TimeTicks for monotonic time values. we can convert to double
> at the blink API boundary since that's what blink needs, but all monotonic
time
> base logic in chromium should be using base::TimeTicks

Done.

[ajuma, 2013-12-09 18:22:52]

jamesr, PTAL.

[jamesr, 2013-12-09 18:29:49]

lgtm

[commit-bot: I haz the power, 2013-12-09 18:36:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ajuma@chromium.org/99733003/60001

[commit-bot: I haz the power, 2013-12-09 21:30:28]

Change committed as 239554