Color the ink drop ripple and hover effects based on theming.

ui/views/animation/ink_drop_host_view.h

Index: ui/views/animation/ink_drop_host_view.h
diff --git a/ui/views/animation/ink_drop_host_view.h b/ui/views/animation/ink_drop_host_view.h
index c77a39f065bb6501ca9466c02420d57744a36fcf..87832b7dd0ae7e3e5758e2e37dafa9dbf468bcfc 100644
--- a/ui/views/animation/ink_drop_host_view.h
+++ b/ui/views/animation/ink_drop_host_view.h
@@ -6,6 +6,7 @@
 #define UI_VIEWS_ANIMATION_INK_DROP_HOST_VIEW_H_
 
 #include "base/memory/scoped_ptr.h"
+#include "third_party/skia/include/core/SkColor.h"
 #include "ui/views/animation/ink_drop_host.h"
 #include "ui/views/view.h"
 
@@ -30,6 +31,7 @@ class VIEWS_EXPORT InkDropHostView : public views::View, public InkDropHost {
   // Overrideable methods to allow views to provide minor tweaks to the default
   // ink drop.
   virtual gfx::Point GetInkDropCenter() const;
+  virtual SkColor GetInkDropBaseColor() const;

[sky, 2016/02/25 00:38:32]

As your implementation has a NOTREACHED, is there a reason you didn't want to
make this pure virtual?

[Evan Stade, 2016/02/25 01:34:28]

yes, originally it was pure virtual, but there was a test button class that
inherited from Button which complained. I figured it would be valid to create a
button that didn't have a ripple, and if no ripple is added this should never be
called. If this is never called, it's annoying to have to provide an override.

 
  private:
   DISALLOW_COPY_AND_ASSIGN(InkDropHostView);