Close the save card bubble on tab switch.

chrome/browser/ui/views/autofill/save_card_icon_view.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/autofill/save_card_icon_view.h"
 
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/ui/autofill/save_card_bubble_controller_impl.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/view_ids.h"
 #include "chrome/browser/ui/views/autofill/save_card_bubble_views.h"
 #include "chrome/grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/vector_icons_public.h"
 
 namespace autofill {
 
 SaveCardIconView::SaveCardIconView(CommandUpdater* command_updater,
                                    Browser* browser)
     : BubbleIconView(command_updater, IDC_SAVE_CREDIT_CARD_FOR_PAGE),
       browser_(browser) {
   set_id(VIEW_ID_SAVE_CREDIT_CARD_BUTTON);
   SetTooltipText(l10n_util::GetStringUTF16(IDS_TOOLTIP_SAVE_CREDIT_CARD));
+  if (browser)
+    browser->tab_strip_model()->AddObserver(this);
 }
 
 SaveCardIconView::~SaveCardIconView() {}
 
 void SaveCardIconView::OnExecuting(
     BubbleIconView::ExecuteSource execute_source) {}
 
+bool SaveCardIconView::OnMousePressed(const ui::MouseEvent& event) {

[Evan Stade, 2016/01/11 18:05:24]

BubbleIconView handles more than just mouses, it also handles keyboard and
gestures.

I think what you want is to change this[1]

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

[Justin Donnelly, 2016/01/11 19:32:37]

Understood.
The BubbleIconView/BrowserCommandController code path handles showing the bubble
when it's not currently shown. It suppresses any action when it's already
showing which prevents side effects (e.g. logging). Closing the bubble only
works because any loss of focus closes the bubble (it doesn't matter whether you
click on the icon or not or whether it's a keyboard event that causes the loss
of focus).

This handler adds a mechanism (matching the one used by some of the other
location bar bubbles) to handle the specific case of closing the bubble when
it's not active. It might make sense to make the browser command path handle
opening *and* closing and being aware of whether the bubble is active or not but
that would requiring overhauling the way each of the bubbles work. If we want to
do that I'd be happy to discuss but I'd like to fix this bug before the broken
behavior lands on the branch.

+  bool result = BubbleIconView::OnMousePressed(event);
+
+  SaveCardBubbleControllerImpl* controller = GetController();
+  if (controller)
+    controller->HideBubble();
+
+  return result;
+}
+
 views::BubbleDelegateView* SaveCardIconView::GetBubble() const {
-  if (!browser_)
-    return nullptr;
-  content::WebContents* web_contents =
-      browser_->tab_strip_model()->GetActiveWebContents();
-  if (!web_contents)
-    return nullptr;
-  autofill::SaveCardBubbleControllerImpl* controller =
-      autofill::SaveCardBubbleControllerImpl::FromWebContents(web_contents);
+  SaveCardBubbleControllerImpl* controller = GetController();
   if (!controller)
     return nullptr;
 
   return static_cast<autofill::SaveCardBubbleViews*>(
       controller->save_card_bubble_view());
 }
 
 gfx::VectorIconId SaveCardIconView::GetVectorIcon() const {
   return gfx::VectorIconId::CREDIT_CARD;
 }
 
+void SaveCardIconView::TabDeactivated(content::WebContents* contents) {
+  SaveCardBubbleControllerImpl* controller = GetController();
+  if (controller)
+    controller->HideBubble();
+}
+
+SaveCardBubbleControllerImpl* SaveCardIconView::GetController() const {
+  if (!browser_)
+    return nullptr;
+  content::WebContents* web_contents =
+      browser_->tab_strip_model()->GetActiveWebContents();
+  if (!web_contents)
+    return nullptr;
+  return autofill::SaveCardBubbleControllerImpl::FromWebContents(web_contents);
+}
+
 }  // namespace autofill