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

chrome/browser/ui/views/bar_control_button.cc

 // 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.
 
 #include "chrome/browser/ui/views/bar_control_button.h"
 
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/vector_icons_public.h"
 #include "ui/views/animation/button_ink_drop_delegate.h"
@@ skipping 37 matching lines @@
                     views::ImageButton::ALIGN_MIDDLE);
 }
 
 void BarControlButton::OnThemeChanged() {
   SkColor icon_color =
       color_utils::DeriveDefaultIconColor(get_text_color_callback_.Run());
   gfx::ImageSkia image = gfx::CreateVectorIcon(id_, 16, icon_color);
   SetImage(views::CustomButton::STATE_NORMAL, &image);
   image = gfx::CreateVectorIcon(id_, 16, SkColorSetA(icon_color, 0xff / 2));
   SetImage(views::CustomButton::STATE_DISABLED, &image);
+  set_ink_drop_base_color(icon_color);

[bruthig, 2016/02/09 21:01:06]

Shouldn't some of the other Views use a similar pattern in order to handle theme
changes?

[Evan Stade, 2016/02/09 22:54:28]

like which one?

[bruthig, 2016/02/10 15:09:54]

Nevermind, I follow now.

 }
 
 void BarControlButton::OnNativeThemeChanged(const ui::NativeTheme* theme) {
   OnThemeChanged();
 }
 
 bool BarControlButton::OnMousePressed(const ui::MouseEvent& event) {
   if (IsTriggerableEvent(event))
     ink_drop_delegate()->OnAction(views::InkDropState::ACTION_PENDING);
 
   return ImageButton::OnMousePressed(event);
 }