Ink Drop cleanup.

Ink Drop cleanup.

Some stuff got missed in the original CL here:
https://codereview.chromium.org/1937353002.

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

[bruthig, 2016-05-10 18:42:31]

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

[bruthig, 2016-05-10 18:42:31]

tdanderson@, can you provide a second set of eyes on this?

[tdanderson, 2016-05-10 18:51:47]

lgtm with one comment

https://codereview.chromium.org/1963193002/diff/1/ui/views/animation/test/ink...
File ui/views/animation/test/ink_drop_impl_test_api.h (right):

https://codereview.chromium.org/1963193002/diff/1/ui/views/animation/test/ink...
ui/views/animation/test/ink_drop_impl_test_api.h:31: explicit
InkDropImplTestApi(InkDropImpl* ink_drop_);
No _ at end.

[bruthig, 2016-05-10 18:55:26]

https://codereview.chromium.org/1963193002/diff/1/ui/views/animation/test/ink...
File ui/views/animation/test/ink_drop_impl_test_api.h (right):

https://codereview.chromium.org/1963193002/diff/1/ui/views/animation/test/ink...
ui/views/animation/test/ink_drop_impl_test_api.h:31: explicit
InkDropImplTestApi(InkDropImpl* ink_drop_);
On 2016/05/10 18:51:46, tdanderson wrote:
> No _ at end.

Done.

[bruthig, 2016-05-10 18:55:30]

The CQ bit was checked by bruthig@chromium.org

[bruthig, 2016-05-10 18:55:31]

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/1963193002/#ps20001
(title: "Fixed tdanderson@ nit from patch set 1.")

[commit-bot: I haz the power, 2016-05-10 18:56:29]

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

[commit-bot: I haz the power, 2016-05-10 19:06:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-10 19:06:20]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[bruthig, 2016-05-10 19:14:59]

The CQ bit was checked by bruthig@chromium.org

[commit-bot: I haz the power, 2016-05-10 19:15:25]

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

[commit-bot: I haz the power, 2016-05-10 19:23:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-10 19:24:00]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[bruthig, 2016-05-10 20:35:25]

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

[bruthig, 2016-05-10 20:35:25]

sky@, can you PTAL?

I'm not sure why chrome_presubmit bot is failing when I am an OWNER of the files
mentioned in the error.  Does per file ownership not apply to deleted files?

[sky, 2016-05-12 15:20:59]

Not sure, LGTM

[bruthig, 2016-05-12 16:18:23]

The CQ bit was checked by bruthig@chromium.org

[commit-bot: I haz the power, 2016-05-12 16:18:39]

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

[commit-bot: I haz the power, 2016-05-12 17:05:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-12 17:07:27]

Description was changed from

==========
Ink Drop cleanup.

Some stuff got missed in the original CL here:
https://codereview.chromium.org/1937353002.

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

to

==========
Ink Drop cleanup.

Some stuff got missed in the original CL here:
https://codereview.chromium.org/1937353002.

BUG=608423

Committed: https://crrev.com/225a87231b3c0e81f17faebf241f5b548b2b1c1c
Cr-Commit-Position: refs/heads/master@{#393277}
==========

[commit-bot: I haz the power, 2016-05-12 17:07:28]

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