Enable accelerated animations for orphaned layers

Enable accelerated animations for orphan layers

This defines RenderWidgetCompositor::registerForAnimations,
which associates the LayerAnimationController of
newly-created cc::Layers with the AnimationRegistrar of the
RenderWidgetCompositor's LayerTreeHost. This also makes
cc::Layer::addAnimation accept animations whenever the
layer's LayerAnimationController has an AnimationRegistrar,
and makes LayerTreeHost only create an AnimationRegistrar
when accelerated animations are enabled.

BUG=129683
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=184488

[ajuma, 2013-02-22 22:33:07]

This depends on https://bugs.webkit.org/show_bug.cgi?id=106594

[Ian Vollick, 2013-02-22 22:55:55]

On 2013/02/22 22:33:07, ajuma wrote:
> This depends on https://bugs.webkit.org/show_bug.cgi?id=106594

lgtm.

[ajuma, 2013-02-22 23:21:29]

+kbr for content/renderer/gpu OWNERS.

[Ken Russell (switch to Gerrit), 2013-02-23 00:22:15]

lgtm

[commit-bot: I haz the power, 2013-02-24 21:03:15]

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

[commit-bot: I haz the power, 2013-02-24 21:21:14]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win7_aura.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[ajuma, 2013-02-24 21:29:36]

On 2013/02/24 21:21:14, I haz the power (commit-bot) wrote:
> Sorry for I got bad news for ya.
> Compile failed with a clobber build on win7_aura.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...
> Your code is likely broken or HEAD is junk. Please ensure your
> code is not broken then alert the build sheriffs.
> Look at the try server FAQ for more details.

HEAD was junk. Trying again.

[commit-bot: I haz the power, 2013-02-24 21:30:14]

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

[commit-bot: I haz the power, 2013-02-24 21:49:25]

Sorry for I got bad news for ya.
Compile failed with a clobber build on mac_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-02-24 21:58:06]

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

[commit-bot: I haz the power, 2013-02-25 14:12:07]

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

[ajuma, 2013-02-25 18:18:47]

Patch updated to fix
LayerLayerTreeHostTest.shouldNotAddAnimationWithoutLayerTreeHost. 

vollick@, PTAL.

[Ian Vollick, 2013-02-25 18:27:29]

https://codereview.chromium.org/12334041/diff/4008/cc/layer_unittest.cc
File cc/layer_unittest.cc (right):

https://codereview.chromium.org/12334041/diff/4008/cc/layer_unittest.cc#newco...
cc/layer_unittest.cc:865: // Case 3: with a LayerTreeHost, the animation should
be accepted.
Maybe we could add a LTH with no registrar and check that we can no longer add
the animation?

[ajuma, 2013-02-25 18:43:16]

On 2013/02/25 18:27:29, vollick wrote:
> https://codereview.chromium.org/12334041/diff/4008/cc/layer_unittest.cc
> File cc/layer_unittest.cc (right):
> 
>
https://codereview.chromium.org/12334041/diff/4008/cc/layer_unittest.cc#newco...
> cc/layer_unittest.cc:865: // Case 3: with a LayerTreeHost, the animation
should
> be accepted.
> Maybe we could add a LTH with no registrar and check that we can no longer add
> the animation?

Done.

[Ian Vollick, 2013-02-25 18:47:42]

On 2013/02/25 18:43:16, ajuma wrote:
> On 2013/02/25 18:27:29, vollick wrote:
> > https://codereview.chromium.org/12334041/diff/4008/cc/layer_unittest.cc
> > File cc/layer_unittest.cc (right):
> > 
> >
>
https://codereview.chromium.org/12334041/diff/4008/cc/layer_unittest.cc#newco...
> > cc/layer_unittest.cc:865: // Case 3: with a LayerTreeHost, the animation
> should
> > be accepted.
> > Maybe we could add a LTH with no registrar and check that we can no longer
add
> > the animation?
> 
> Done.
lgtm

[commit-bot: I haz the power, 2013-02-25 18:49:36]

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

[commit-bot: I haz the power, 2013-02-25 21:48:43]

Change committed as 184488