Converted all Views to use an InkDropDelegate instead of a InkDropAnimationController.

ui/views/animation/test/test_ink_drop_host.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_VIEWS_ANIMATION_TEST_TEST_INK_DROP_HOST_H_
 #define UI_VIEWS_ANIMATION_TEST_TEST_INK_DROP_HOST_H_
 
 #include "base/macros.h"
 #include "ui/views/animation/ink_drop_host.h"
 
 namespace views {
 
-// A non-functional, dummy implementation of an InkDropHost that can be used as
-// a placeholder during tests.
+// A non-functional, implementation of an InkDropHost that can be used as during

[Peter Kasting, 2015/12/18 18:44:02]

Nit: While here: Remove comma; remove "as"

[bruthig, 2015/12/21 16:54:34]

Done.

+// tests.  Tracks the current number of ink drop layers being hosted.

[Peter Kasting, 2015/12/18 18:44:02]

Nit: "Tracks the number of hosted ink drop layers"?

[bruthig, 2015/12/21 16:54:34]

Done.

 class TestInkDropHost : public InkDropHost {
  public:
   TestInkDropHost();
   ~TestInkDropHost() override;
 
+  int current_number_of_ink_drop_layers() const {

[Peter Kasting, 2015/12/18 18:44:02]

Nit: I would name all these like "num_ink_drop_layers()".

[bruthig, 2015/12/21 16:54:34]

Done.

+    return current_number_of_ink_drop_layers_;
+  }
+
   // TestInkDropHost:
   void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
   void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
   gfx::Point CalculateInkDropCenter() const override;
 
  private:
+  // The number of ink drop layers currently hosted by this.

[Peter Kasting, 2015/12/18 18:44:02]

Nit: Comment adds nothing to the code, remove.

[bruthig, 2015/12/21 16:54:34]

Done.

+  int current_number_of_ink_drop_layers_;
+
   DISALLOW_COPY_AND_ASSIGN(TestInkDropHost);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_ANIMATION_TEST_TEST_INK_DROP_HOST_H_