Adds MD ink ripple animations to buttons within location bar

chrome/browser/ui/views/toolbar/toolbar_view.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/toolbar_view.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/i18n/number_formatting.h"
@@ skipping 311 matching lines @@
              : app_menu_button_;
 }
 
 views::View* ToolbarView::GetTranslateBubbleAnchor() {
   views::View* translate_icon_view = location_bar()->translate_icon_view();
   return (translate_icon_view && translate_icon_view->visible())
              ? translate_icon_view
              : app_menu_button_;
 }
 
+void ToolbarView::OnBubbleCreatedForAnchor(views::View* anchor_view,
+                                           views::Widget* bubble_widget) {
+  if (bubble_widget &&
+      (anchor_view == location_bar()->star_view() ||
+       anchor_view == location_bar()->save_credit_card_icon_view() ||
+       anchor_view == location_bar()->translate_icon_view())) {
+    DCHECK(anchor_view);
+    bubble_widget->AddObserver(static_cast<BubbleIconView*>(anchor_view));

[msw, 2016/01/28 23:27:42]

q: Did you want the app list view to actually show a ripple?
Doesn't seem like this will handle that case.

[varkha, 2016/01/29 00:00:40]

Do you mean |app_menu_button_|? If so then no, we don't want a ripple update in
this case, only if the anchor is a BubbleIconView (which is where the ripple
would be shown on click and so needs to be hidden when the bubble widget is
hidden).

[msw, 2016/01/29 00:02:01]

Sorry, yeah that's what I meant; acknowledged.

+  }
+}
+
 void ToolbarView::ExecuteExtensionCommand(
     const extensions::Extension* extension,
     const extensions::Command& command) {
   browser_actions_->ExecuteExtensionCommand(extension, command);
 }
 
 int ToolbarView::GetMaxBrowserActionsWidth() const {
   // The browser actions container is allowed to grow, but only up until the
   // omnibox reaches its minimum size. So its maximum allowed width is its
   // current size, plus any that the omnibox could give up.
@@ skipping 469 matching lines @@
 void ToolbarView::OnShowHomeButtonChanged() {
   Layout();
   SchedulePaint();
 }
 
 int ToolbarView::content_shadow_height() const {
   return GetLayoutConstant(
       (browser_->host_desktop_type() == chrome::HOST_DESKTOP_TYPE_ASH) ?
           TOOLBAR_CONTENT_SHADOW_HEIGHT_ASH : TOOLBAR_CONTENT_SHADOW_HEIGHT);
 }