Add VectorIconButton functionality to ImageButton.

ui/views/animation/ink_drop_host_view.cc

Index: ui/views/animation/ink_drop_host_view.cc
diff --git a/ui/views/animation/ink_drop_host_view.cc b/ui/views/animation/ink_drop_host_view.cc
index 85e62b88d93e8c24f9b4d3efb13a25dc2a5336bd..1fa48909c419a56c12d6b5861b17d9f17dbe7318 100644
--- a/ui/views/animation/ink_drop_host_view.cc
+++ b/ui/views/animation/ink_drop_host_view.cc
@@ -185,6 +185,9 @@ InkDropHostView::CreateDefaultInkDropHighlight(const gfx::PointF& center_point,
 }
 
 void InkDropHostView::SetInkDropMode(InkDropMode ink_drop_mode) {
+  if (ink_drop_mode == ink_drop_mode_)

[Evan Stade, 2017/03/17 13:54:44]

this doesn't seem like a bad thing but could it have unintended consequences?

Aside from perhaps tests, it seems like this is only called right after
construction. Can we make that a requirement?

Either way, I don't think it's necessary for this change any more, so maybe it
should be separated.

[Bret, 2017/03/17 23:41:41]

I didn't see any trouble with this when I was testing, but reverted for now
anyway.

+    return;
+
   ink_drop_mode_ = ink_drop_mode;
   ink_drop_ = nullptr;