Enhance the material design ripple API so the ripple's state can be controlled by it's owning View.

chrome/browser/ui/views/toolbar/back_button.cc

 // Copyright 2013 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/toolbar/back_button.h"
 
 #include "ui/gfx/geometry/insets.h"
+#include "ui/views/animation/ink_drop_animation_controller.h"
 #include "ui/views/controls/button/label_button_border.h"
 #include "ui/views/painter.h"
 
 BackButton::BackButton(views::ButtonListener* listener,
                        ui::MenuModel* model)
     : ToolbarButton(listener, model),
       margin_leading_(0) {}
 
 BackButton::~BackButton() {}
 
 void BackButton::SetLeadingMargin(int margin) {
   margin_leading_ = margin;
 
   UpdateThemedBorder();
 
   // Similarly fiddle the focus border. Value consistent with LabelButton.
   // TODO(gbillock): Refactor this magic number somewhere global to views,
   // probably a FocusBorder constant.
   const int kFocusRectInset = 3;
   SetFocusPainter(views::Painter::CreateDashedFocusPainterWithInsets(
                       gfx::Insets(kFocusRectInset, kFocusRectInset + margin,
                                   kFocusRectInset, kFocusRectInset)));
 
   InvalidateLayout();
 }
 
+void BackButton::LayoutInkDrop() {
+  ink_drop_animation_controller()->SetBounds(
+      gfx::Rect(margin_leading_, 0, width() - margin_leading_, height()));

[jonross, 2015/08/07 17:14:31]

Is this something that could be gained by applying the view's insets to the ink
drop?

Otherwise we'll be having similar code littered throughout.

[bruthig, 2015/08/07 22:00:45]

Discussed offline, the inset's track the space between the border and the
contents and not the view's edge and the border.

+}
+
 const char* BackButton::GetClassName() const {
   return "BackButton";
 }
 
 scoped_ptr<views::LabelButtonBorder> BackButton::CreateDefaultBorder() const {
   scoped_ptr<views::LabelButtonBorder> border =
       ToolbarButton::CreateDefaultBorder();
 
   // Adjust border insets to follow the margin change,
   // which will be reflected in where the border is painted
   // through |GetThemePaintRect|.
   const gfx::Insets insets(border->GetInsets());
   border->set_insets(gfx::Insets(insets.top(), insets.left() + margin_leading_,
                                  insets.bottom(), insets.right()));
 
   return border.Pass();
 }
 
 gfx::Rect BackButton::GetThemePaintRect() const  {
   gfx::Rect rect(LabelButton::GetThemePaintRect());
   rect.Inset(margin_leading_, 0, 0, 0);
   return rect;
 }