Plumb NotifyAnimationAborted from the compositor to blink animation delegate.

Plumb NotifyAnimationAborted from the compositor to blink animation delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

BUG=576438
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/73f33b9bb4e534d75915757b7bf746349f817663
Cr-Commit-Position: refs/heads/master@{#369856}

[ymalik, 2016-01-15 16:00:35]

Description was changed from

==========
Plumb NotifyAnimationAborted from the compositor to blink animtion delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

BUG=576438
==========

to

==========
Plumb NotifyAnimationAborted from the compositor to blink animtion delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

[ymalik, 2016-01-15 16:01:50]

Description was changed from

==========
Plumb NotifyAnimationAborted from the compositor to blink animtion delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

to

==========
Plumb NotifyAnimationAborted from the compositor to blink animtion delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

[ymalik, 2016-01-15 16:01:50]

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

[ymalik, 2016-01-15 16:15:59]

Description was changed from

==========
Plumb NotifyAnimationAborted from the compositor to blink animtion delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

to

==========
Plumb NotifyAnimationAborted from the compositor to blink animation delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

[ajuma, 2016-01-15 16:49:54]

lgtm with nit

https://codereview.chromium.org/1587073011/diff/1/cc/animation/layer_animatio...
File cc/animation/layer_animation_controller_unittest.cc (right):

https://codereview.chromium.org/1587073011/diff/1/cc/animation/layer_animatio...
cc/animation/layer_animation_controller_unittest.cc:1033: : started_(false),
finished_(false), start_time_(base::TimeTicks()) {}
aborted_ needs to be initialized

[ymalik, 2016-01-15 17:30:49]

ymalik@chromium.org changed reviewers:
+ bokan@chromium.org, jbroman@chromium.org, rbyers@chromium.org

[ymalik, 2016-01-15 17:30:49]

+bokan for Source/web/
+rbyers for public/platform/
+jbroman for Source/platform/

[ymalik, 2016-01-15 17:38:30]

ymalik@chromium.org changed reviewers:
- bokan@chromium.org, jbroman@chromium.org

[ymalik, 2016-01-15 17:39:44]

On 2016/01/15 17:30:49, ymalik1 wrote:
> +bokan for Source/web/
> +rbyers for public/platform/
> +jbroman for Source/platform/

Removing others because rbyers is WebKit/OWNERS

:)

[Rick Byers, 2016-01-15 19:55:13]

WebKit LGTM

I see there's no test coverage of the WebKit side of this, but it's pretty
trivial plumbing so if you feel it's not worth the effort to add a test, I'm OK
with that.

[ymalik, 2016-01-15 20:35:47]

The CQ bit was checked by ymalik@chromium.org

[ymalik, 2016-01-15 20:35:47]

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/1587073011/#ps20001
(title: "initialize aborted_ in unittest")

[commit-bot: I haz the power, 2016-01-15 20:35:57]

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

[ymalik, 2016-01-15 20:36:57]

On 2016/01/15 19:55:13, Rick Byers wrote:
> WebKit LGTM
> 
> I see there's no test coverage of the WebKit side of this, but it's pretty
> trivial plumbing so if you feel it's not worth the effort to add a test, I'm
OK
> with that.

Agreed that it's pretty trivial plumbing and leaving tests out for now. Thanks!

[commit-bot: I haz the power, 2016-01-15 21:59:53]

Description was changed from

==========
Plumb NotifyAnimationAborted from the compositor to blink animation delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

to

==========
Plumb NotifyAnimationAborted from the compositor to blink animation delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

[commit-bot: I haz the power, 2016-01-15 21:59:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-15 22:00:55]

Description was changed from

==========
Plumb NotifyAnimationAborted from the compositor to blink animation delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

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

to

==========
Plumb NotifyAnimationAborted from the compositor to blink animation delegate.

This is needed to clear the blink scroll animator state when an animation
started on the MT is cancelled by the compositor.

TESTED=forced MT animation to abort from the compositor and verified that
the delegate override on the blink side is called.

BUG=576438
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/73f33b9bb4e534d75915757b7bf746349f817663
Cr-Commit-Position: refs/heads/master@{#369856}
==========

[commit-bot: I haz the power, 2016-01-15 22:00:56]

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