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

chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc

Index: chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
diff --git a/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc b/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
index a8dc1224fda879c99215c43dc6f1668b9932346a..e6ab414efebafc5d4191020cba290c47245f39e1 100644
--- a/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
+++ b/chrome/browser/ui/views/location_bar/icon_label_bubble_view.cc
@@ -12,6 +12,7 @@
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/native_theme/native_theme.h"
+#include "ui/views/animation/ink_drop_hover.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/painter.h"
 
@@ -139,6 +140,28 @@ void IconLabelBubbleView::OnNativeThemeChanged(
   SetLabelBackgroundColor(background_color);
 }
 
+void IconLabelBubbleView::AddInkDropLayer(ui::Layer* ink_drop_layer) {
+  image()->SetPaintToLayer(true);
+  image()->SetFillsBoundsOpaquely(false);
+  InkDropHostView::AddInkDropLayer(ink_drop_layer);
+}
+
+void IconLabelBubbleView::RemoveInkDropLayer(ui::Layer* ink_drop_layer) {
+  InkDropHostView::RemoveInkDropLayer(ink_drop_layer);
+  image()->SetFillsBoundsOpaquely(true);

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

this isn't needed (because you destroy the layer on the next line).

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

good call. This is a copy-pasta error so I've updated several other affected
places as well.

I'm wondering if we need SetFillsBoundsOpaquely to take a parameter at all ---
do we ever want to set it to true? It seems not, given that's the default and
almost all views just pick one value and stick to it. I've started a secondary
CL to see if I can change that or at least clean up some more unnecessary calls.

[sky, 2016/02/25 02:26:55]

Seems like a weird limitation on view and layer. There are valid reasons to
toggle.

+  image()->SetPaintToLayer(false);
+}
+
+scoped_ptr<views::InkDropHover> IconLabelBubbleView::CreateInkDropHover()
+    const {
+  // Location bar views don't show hover effect.
+  return nullptr;
+}
+
+SkColor IconLabelBubbleView::GetInkDropBaseColor() const {
+  return color_utils::DeriveDefaultIconColor(GetTextColor());
+}
+
 SkColor IconLabelBubbleView::GetParentBackgroundColor() const {
   return ui::MaterialDesignController::IsModeMaterial()
              ? GetNativeTheme()->GetSystemColor(