Fix Animations, Remove Compostior Animations.

Fix Animations, Remove Compostior Animations.

This patch (re)hooks up animations so they start and removes the infrastructure required by blink to synchronize the compositor and main threads.

R=ojan@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/8c0e929f2e547092bdb8040bcc68611dfc05fcae

[rafaelw, 2014-12-01 21:34:14]

rafaelw@chromium.org changed reviewers:
+ ojan@chromium.org

[ojan, 2014-12-01 21:38:39]

ojan@chromium.org changed reviewers:
+ esprehn@chromium.org

[ojan, 2014-12-01 21:38:39]

esprehn: What do you think is the best way to rip out the compositor animations
infrastructure?

[ojan, 2014-12-01 21:53:11]

lgtm given that this is dead code.

[esprehn, 2014-12-02 07:02:02]

You might want to try forcing all animations to the main thread by returning
true/false from the method that checks and then delete all now unreachable code.
Any code that used to "cancel on compositor" is dead, you don't need any code
there now.

https://codereview.chromium.org/772673002/diff/60001/sky/engine/core/dom/Elem...
File sky/engine/core/dom/Element.cpp (right):

https://codereview.chromium.org/772673002/diff/60001/sky/engine/core/dom/Elem...
sky/engine/core/dom/Element.cpp:846: //
activeAnimations->cancelAnimationOnCompositor();
I don't think we need to do anything, the animation is already on the main
thread in sky.

[ojan, 2014-12-02 22:21:29]

lgtm

[rafaelw, 2014-12-02 22:38:12]

Committed patchset #9 (id:150001) manually as
8c0e929f2e547092bdb8040bcc68611dfc05fcae (presubmit successful).