Adds MD ink ripple animations to buttons within location bar

chrome/browser/ui/views/frame/browser_view.cc

 // Copyright 2012 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/frame/browser_view.h"
 
 #include <stdint.h>
 #include <algorithm>
 #include <utility>
 
@@ skipping 1270 matching lines @@
 }
 
 void BrowserView::ShowUpdateChromeDialog() {
   UpdateRecommendedMessageBox::Show(GetWidget()->GetNativeWindow());
 }
 
 void BrowserView::ShowBookmarkBubble(const GURL& url, bool already_bookmarked) {
   scoped_ptr<BubbleSyncPromoDelegate> delegate;
   delegate.reset(new BookmarkBubbleSignInDelegate(browser_.get()));
 
-  BookmarkBubbleView::ShowBubble(GetToolbarView()->GetBookmarkBubbleAnchor(),
+  views::WidgetObserverView* anchor_view =
+      GetToolbarView()->GetBookmarkBubbleAnchor();
+  BookmarkBubbleView::ShowBubble(anchor_view,
                                  gfx::Rect(), nullptr, bookmark_bar_view_.get(),
                                  std::move(delegate), browser_->profile(), url,
                                  already_bookmarked);
+  anchor_view->ObserveWidget(
+      BookmarkBubbleView::bookmark_bubble()->GetWidget());

[sadrul, 2016/01/28 17:45:06]

I would do this differently:

. BubbleIconView becomes a WidgetObserver, and
BubbleIconView::ObserveBubble(Widget*) is added to notify it when there's a
bubble anchored to it.

. BrowserView notifies the ToolbarView when a bubble is created, e.g.:

    views::View* anchor = GetToolbarView()->GetBookmarkBubbleAnchor();
    BookmarkBubbleView::ShowBubble(anchor, ....);
    // Can ShowBubble() return the Widget instead?
    views::Widget* bubble = BookmarkBubbleView::bookmark_bubble()->GetWidget();
    GetToolbarView()->OnBubbleCreatedForAnchor(anchor, widget);


. ToolbarView::OnBubbleCreatedForAnchor(views::View* anchor, views::Widget*
bubble) looks like this:

  bool anchor_is_bubble_icon_view =
      anchor == location_bar()->star_view() ||
      anchor == location_bar()->translate_icon_view() ||
      anchor == location_bar()->save_credit_card_icon_view();
  if (anchor_is_bubble_icon_view)
    static_cast<BubbleIconView*>(anchor)->ObserveBubble(bubble);

This should take care of the issues, right?

[varkha, 2016/01/28 20:09:19]

Yes, this is precisely what I was looking for. Thanks for the idea! Done.

 }
 
 void BrowserView::ShowBookmarkAppBubble(
     const WebApplicationInfo& web_app_info,
     const ShowBookmarkAppBubbleCallback& callback) {
   BookmarkAppBubbleView::ShowBubble(GetToolbarView(), web_app_info, callback);
 }
 
 autofill::SaveCardBubbleView* BrowserView::ShowSaveCreditCardBubble(
     content::WebContents* web_contents,
     autofill::SaveCardBubbleController* controller,
     bool is_user_gesture) {
+  views::WidgetObserverView* anchor_view =
+      GetToolbarView()->GetSaveCreditCardBubbleAnchor();
   autofill::SaveCardBubbleViews* view = new autofill::SaveCardBubbleViews(
-      GetToolbarView()->GetSaveCreditCardBubbleAnchor(), web_contents,
+      anchor_view, web_contents,
       controller);
+  anchor_view->ObserveWidget(view->GetWidget());
   view->Show(is_user_gesture ? autofill::SaveCardBubbleViews::USER_GESTURE
                              : autofill::SaveCardBubbleViews::AUTOMATIC);
   return view;
 }
 
 void BrowserView::ShowTranslateBubble(
     content::WebContents* web_contents,
     translate::TranslateStep step,
     translate::TranslateErrors::Type error_type,
     bool is_user_gesture) {
   if (contents_web_view_->HasFocus() &&
       !GetLocationBarView()->IsMouseHovered()) {
     content::RenderViewHost* rvh = web_contents->GetRenderViewHost();
     if (rvh->IsFocusedElementEditable())
       return;
   }
 
   ChromeTranslateClient* chrome_translate_client =
       ChromeTranslateClient::FromWebContents(web_contents);
   translate::LanguageState& language_state =
       chrome_translate_client->GetLanguageState();
   language_state.SetTranslateEnabled(true);
 
   if (IsMinimized())
     return;
 
+  views::WidgetObserverView* anchor_view =
+      GetToolbarView()->GetTranslateBubbleAnchor();
   TranslateBubbleView::ShowBubble(
-      GetToolbarView()->GetTranslateBubbleAnchor(), web_contents, step,
+      anchor_view, web_contents, step,
       error_type, is_user_gesture ? TranslateBubbleView::USER_GESTURE
                                   : TranslateBubbleView::AUTOMATIC);
+  anchor_view->ObserveWidget(
+      TranslateBubbleView::GetCurrentBubble()->GetWidget());
 }
 
 #if defined(ENABLE_ONE_CLICK_SIGNIN)
 void BrowserView::ShowOneClickSigninBubble(
     OneClickSigninBubbleType type,
     const base::string16& email,
     const base::string16& error_message,
     const StartSyncCallback& start_sync_callback) {
   scoped_ptr<OneClickSigninBubbleDelegate> delegate;
   delegate.reset(new OneClickSigninBubbleLinksDelegate(browser()));
@@ skipping 1337 matching lines @@
   return immersive_mode_controller()->IsEnabled();
 }
 
 views::Widget* BrowserView::GetBubbleAssociatedWidget() {
   return GetWidget();
 }
 
 gfx::Rect BrowserView::GetTopContainerBoundsInScreen() {
   return top_container_->GetBoundsInScreen();
 }