Rename of InkDropAnimation classes to InkDropRipple.

Rename of InkDropAnimation classes to InkDropRipple.

The term 'Animation' is too overloaded with respect to the InkDrop.

BUG=608423
Committed: https://crrev.com/d4c4be6dcc782031c6b7e0e568ca1a187499f144
Cr-Commit-Position: refs/heads/master@{#391069}

[bruthig, 2016-05-02 18:22:40]

bruthig@chromium.org changed reviewers:
+ tdanderson@chromium.org

[bruthig, 2016-05-02 18:22:41]

tdanderson@, can you PTAL?

[bruthig, 2016-05-02 18:31:31]

Description was changed from

==========
Rename of InkDropAnimation classes to InkDropRipple.

The term 'Animation' is too overloaded with respect to the InkDrop.
==========

to

==========
Rename of InkDropAnimation classes to InkDropRipple.

The term 'Animation' is too overloaded with respect to the InkDrop.

BUG=608423
==========

[tdanderson, 2016-05-02 18:46:28]

On 2016/05/02 18:22:41, bruthig wrote:
> tdanderson@, can you PTAL?

LGTM if the trybots are happy. One nit below.

[tdanderson, 2016-05-02 18:49:33]

https://codereview.chromium.org/1937103003/diff/1/ui/views/animation/ink_drop...
File ui/views/animation/ink_drop_host.h (right):

https://codereview.chromium.org/1937103003/diff/1/ui/views/animation/ink_drop...
ui/views/animation/ink_drop_host.h:23: // Used by the InkDropAnimationController
to add and remove the ink drop layers
Should be InkDropRippleController

[bruthig, 2016-05-02 19:47:51]

bruthig@chromium.org changed reviewers:
+ sky@chromium.org

[bruthig, 2016-05-02 19:47:52]

sky@: Please review changes in:
- chrome/*
- ui/views/controls/*
- ui/views.gyp

https://codereview.chromium.org/1937103003/diff/1/ui/views/animation/ink_drop...
File ui/views/animation/ink_drop_host.h (right):

https://codereview.chromium.org/1937103003/diff/1/ui/views/animation/ink_drop...
ui/views/animation/ink_drop_host.h:23: // Used by the InkDropAnimationController
to add and remove the ink drop layers
On 2016/05/02 18:49:33, tdanderson wrote:
> Should be InkDropRippleController

Discussed offline.  This will be done in a follow up.

[sky, 2016-05-02 22:08:15]

LGTM

[bruthig, 2016-05-02 22:12:10]

The CQ bit was checked by bruthig@chromium.org

[bruthig, 2016-05-02 22:12:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdanderson@chromium.org
Link to the patchset: https://codereview.chromium.org/1937103003/#ps20001
(title: "Removed test_ink_drop_animation_observer.h from views.gyp and doc
update.")

[commit-bot: I haz the power, 2016-05-02 22:12:55]

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

[commit-bot: I haz the power, 2016-05-02 22:19:07]

Description was changed from

==========
Rename of InkDropAnimation classes to InkDropRipple.

The term 'Animation' is too overloaded with respect to the InkDrop.

BUG=608423
==========

to

==========
Rename of InkDropAnimation classes to InkDropRipple.

The term 'Animation' is too overloaded with respect to the InkDrop.

BUG=608423
==========

[commit-bot: I haz the power, 2016-05-02 22:19:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-02 22:21:40]

Description was changed from

==========
Rename of InkDropAnimation classes to InkDropRipple.

The term 'Animation' is too overloaded with respect to the InkDrop.

BUG=608423
==========

to

==========
Rename of InkDropAnimation classes to InkDropRipple.

The term 'Animation' is too overloaded with respect to the InkDrop.

BUG=608423

Committed: https://crrev.com/d4c4be6dcc782031c6b7e0e568ca1a187499f144
Cr-Commit-Position: refs/heads/master@{#391069}
==========

[commit-bot: I haz the power, 2016-05-02 22:21:41]

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