Fixes ink drop being "stuck" on when losing focus.

ui/views/animation/test/test_ink_drop_delegate.cc

Index: ui/views/animation/test/test_ink_drop_delegate.cc
diff --git a/ui/views/animation/test/test_ink_drop_delegate.cc b/ui/views/animation/test/test_ink_drop_delegate.cc
index 9d20cf0fffc09e5f7b6a6cacb83efcdc4e1b4330..ce37242f5995da695a70b07c7cec26463bf1fd6c 100644
--- a/ui/views/animation/test/test_ink_drop_delegate.cc
+++ b/ui/views/animation/test/test_ink_drop_delegate.cc
@@ -2,6 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "ui/views/animation/test/test_ink_drop.h"
 #include "ui/views/animation/test/test_ink_drop_delegate.h"
 
 namespace views {
@@ -29,7 +30,9 @@ InkDropState TestInkDropDelegate::GetTargetInkDropState() const {
 }
 
 InkDrop* TestInkDropDelegate::GetInkDrop() {
-  return nullptr;
+  if (!ink_drop_.get())

[bruthig, 2016/06/02 15:21:35]

Was this change necessary or are you just back filling the simple 'return
nullptr'?

As it stands now, I don't like how TestInkDropDelegate::GetTargetInkDropState()
and TestInkDropDelegate::GetInkDrop()::GetTargetState() can return different
values.  This is inconsistent with the TestInkDropDelegate being somewhat of a
test fake.

FTR I am in the middle of removing the bulk of the (Button)InkDropDelegates, if
not the entire class.  So, it might make more sense to not add this, assuming
it's not currently required...?

+    ink_drop_.reset(new TestInkDrop);
+  return ink_drop_.get();
 }
 
 }  // namespace test