Adds MD ink ripple animations to buttons within location bar

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

Index: chrome/browser/ui/views/toolbar/toolbar_view.cc
diff --git a/chrome/browser/ui/views/toolbar/toolbar_view.cc b/chrome/browser/ui/views/toolbar/toolbar_view.cc
index d1624fcd5b4158d8c7e07c5873429a433d16dd82..2ec85da4f119779bfbadd2dee768010ffcf6c14b 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_view.cc
+++ b/chrome/browser/ui/views/toolbar/toolbar_view.cc
@@ -329,6 +329,17 @@ views::View* ToolbarView::GetTranslateBubbleAnchor() {
              : 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) {