Animations: Register LinkHighlight content layer for animations

Animations: Register LinkHighlight content layer for animations

We want to add animations to content layer. So let's register it for animations.

Next episode (chromium part):
https://codereview.chromium.org/1101823002/

This is a prerequisite to implement compositor animation timelines.
https://codereview.chromium.org/947033002/

BUG=394777
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194471

[loyso (OOO), 2015-04-24 03:21:59]

loyso@chromium.org changed reviewers:
+ aelias@chromium.org, esprehn@chromium.org, kbr@chromium.org

[loyso (OOO), 2015-04-24 03:22:33]

On 2015/04/24 03:22:00, loyso wrote:

This is a one-liner. PTAL, anyone?

[loyso (OOO), 2015-04-24 03:26:54]

https://codereview.chromium.org/1089043003/diff/1/Source/web/LinkHighlight.cpp
File Source/web/LinkHighlight.cpp (right):

https://codereview.chromium.org/1089043003/diff/1/Source/web/LinkHighlight.cp...
Source/web/LinkHighlight.cpp:84:
m_contentLayer->layer()->setAnimationDelegate(this);
Note: The problem is that LayerAnimationController is not yet created at this
point. Alternatively we could move the setAnimationDelegate call later, right
after updateGeometry(). updateGeometry adds LinkHighlight's content layer to
parent GraphicsLayer calling cc::Layer::AddChild, cc::Layer::SetParent,
cc::Layer::SetLayerThreeHost, cc::Layer::RegisterForAnimation.

[loyso (OOO), 2015-04-24 03:35:59]

loyso@chromium.org changed reviewers:
+ jbroman@chromium.org

[aelias_OOO_until_Jul13, 2015-04-24 19:25:09]

lgtm

[loyso (OOO), 2015-04-26 23:58:39]

The CQ bit was checked by loyso@chromium.org

[commit-bot: I haz the power, 2015-04-26 23:58:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1089043003/1

[commit-bot: I haz the power, 2015-04-27 01:33:43]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194471