Moved the *PaintedLayerDelegates from ink_drop_animation.cc to ink_drop_painted_layer_delegates.h.

Moved the *PaintedLayerDelegates from ink_drop_animation.cc to ink_drop_painted_layer_delegates.h.

This CL promotes the *PaintedLayerDelegates from the ink_drop_animation.cc
file to the ink_drop_painted_layer_delegates.(h|cc) in order to be
reusable.

Pre-requisite CL for https://codereview.chromium.org/1430153003/

BUG=537238
Committed: https://crrev.com/a07c1b3adfa6905b340240a75070f7436318b5c6
Cr-Commit-Position: refs/heads/master@{#358385}

[bruthig, 2015-11-06 16:22:01]

Description was changed from

==========
Moved the *PaintedLayerDelegates from ink_drop_animation.cc to
ink_drop_painted_layer_delegates.h.

This CL promotes the *PaintedLayerDelegates from the ink_drop_animation.cc
file to the ink_drop_painted_layer_delegates.(h|cc) in order to be
reusable.

BUG=537238
==========

to

==========
Moved the *PaintedLayerDelegates from ink_drop_animation.cc to
ink_drop_painted_layer_delegates.h.

This CL promotes the *PaintedLayerDelegates from the ink_drop_animation.cc
file to the ink_drop_painted_layer_delegates.(h|cc) in order to be
reusable.

Pre-requisite CL for https://codereview.chromium.org/1430153003/

BUG=537238
==========

[bruthig, 2015-11-06 16:22:32]

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

[bruthig, 2015-11-06 16:22:33]

sadrul@, can you PTAL?

[sadrul, 2015-11-06 17:39:15]

lgtm

https://codereview.chromium.org/1430153003/diff/20001/ui/views/animation/ink_...
File ui/views/animation/ink_drop_painted_layer_delegates.h (right):

https://codereview.chromium.org/1430153003/diff/20001/ui/views/animation/ink_...
ui/views/animation/ink_drop_painted_layer_delegates.h:4: #ifndef
UI_VIEWS_ANIMATION_INK_DROP_PAINTED_LAYER_DELEGATES_H_
newline before

https://codereview.chromium.org/1430153003/diff/20001/ui/views/animation/ink_...
ui/views/animation/ink_drop_painted_layer_delegates.h:26: virtual gfx::PointF
GetCenterPoint() const = 0;
This looks new that doesn't exist in current code. It's also not used in the
larger CL (https://codereview.chromium.org/1390113006/), so unclear if this is
going to be useful.

[bruthig, 2015-11-06 18:51:21]

https://codereview.chromium.org/1430153003/diff/20001/ui/views/animation/ink_...
File ui/views/animation/ink_drop_painted_layer_delegates.h (right):

https://codereview.chromium.org/1430153003/diff/20001/ui/views/animation/ink_...
ui/views/animation/ink_drop_painted_layer_delegates.h:4: #ifndef
UI_VIEWS_ANIMATION_INK_DROP_PAINTED_LAYER_DELEGATES_H_
On 2015/11/06 17:39:14, sadrul wrote:
> newline before

Done.

https://codereview.chromium.org/1430153003/diff/20001/ui/views/animation/ink_...
ui/views/animation/ink_drop_painted_layer_delegates.h:26: virtual gfx::PointF
GetCenterPoint() const = 0;
On 2015/11/06 17:39:14, sadrul wrote:
> This looks new that doesn't exist in current code. It's also not used in the
> larger CL (https://codereview.chromium.org/1390113006/), so unclear if this is
> going to be useful.

I've added the use of it.

[bruthig, 2015-11-06 18:51:26]

The CQ bit was checked by bruthig@chromium.org

[bruthig, 2015-11-06 18:51:26]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/1430153003/#ps40001
(title: "Added use of BasePaintedLayerDelegate::GetCenterPoint().")

[bruthig, 2015-11-06 18:51:35]

The CQ bit was unchecked by bruthig@chromium.org

[bruthig, 2015-11-06 18:52:01]

The CQ bit was checked by bruthig@chromium.org

[commit-bot: I haz the power, 2015-11-06 18:52:41]

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

[commit-bot: I haz the power, 2015-11-06 20:54:48]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-06 20:55:28]

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