Refactor accessible focus ring code so the layers are animation observers.

Refactor accessible focus ring code so the layers are animation observers.

I'd like to make more of the focus ring layers animate, so it will be
simpler if the layers are animation observers and keep track of their own
compositor, rather than storing them all in AccessibilityFocusRingController.

I'm assuming it's not too expensive to have OnAnimationStep() called for a
layer that doesn't need to animate all the time. If that's not true, this
refactoring would make it easier to add StartAnimating and StopAnimating
methods to FocusRingLayer for better efficiency.

BUG=314889
Committed: https://crrev.com/82a45a85f0b2a9292d9524e90cc82ee3da68d876
Cr-Commit-Position: refs/heads/master@{#397825}

[dmazzoni, 2016-06-03 19:32:51]

dmazzoni@chromium.org changed reviewers:
+ xiyuan@chromium.org

[xiyuan, 2016-06-03 19:45:52]

lgtm

[dmazzoni, 2016-06-03 20:52:41]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2016-06-03 20:53:30]

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

[commit-bot: I haz the power, 2016-06-03 22:33:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-03 22:35:54]

Description was changed from

==========
Refactor accessible focus ring code so the layers are animation observers.

I'd like to make more of the focus ring layers animate, so it will be
simpler if the layers are animation observers and keep track of their own
compositor, rather than storing them all in AccessibilityFocusRingController.

I'm assuming it's not too expensive to have OnAnimationStep() called for a
layer that doesn't need to animate all the time. If that's not true, this
refactoring would make it easier to add StartAnimating and StopAnimating
methods to FocusRingLayer for better efficiency.

BUG=314889
==========

to

==========
Refactor accessible focus ring code so the layers are animation observers.

I'd like to make more of the focus ring layers animate, so it will be
simpler if the layers are animation observers and keep track of their own
compositor, rather than storing them all in AccessibilityFocusRingController.

I'm assuming it's not too expensive to have OnAnimationStep() called for a
layer that doesn't need to animate all the time. If that's not true, this
refactoring would make it easier to add StartAnimating and StopAnimating
methods to FocusRingLayer for better efficiency.

BUG=314889

Committed: https://crrev.com/82a45a85f0b2a9292d9524e90cc82ee3da68d876
Cr-Commit-Position: refs/heads/master@{#397825}
==========

[commit-bot: I haz the power, 2016-06-03 22:35:55]

Patchset 1 (id:??) landed as
https://crrev.com/82a45a85f0b2a9292d9524e90cc82ee3da68d876
Cr-Commit-Position: refs/heads/master@{#397825}