Close the save card bubble on tab switch.

chrome/browser/ui/views/location_bar/location_bar_view.cc

Index: chrome/browser/ui/views/location_bar/location_bar_view.cc
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.cc b/chrome/browser/ui/views/location_bar/location_bar_view.cc
index a5195bdc14689d10b88cb522a16508c65b1a180d..56f9535988bf1d93dcb6d8ae7691c9be190370be 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.cc
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.cc
@@ -33,6 +33,7 @@
 #include "chrome/browser/ui/layout_constants.h"
 #include "chrome/browser/ui/passwords/manage_passwords_ui_controller.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/views/autofill/save_card_bubble_views.h"
 #include "chrome/browser/ui/views/autofill/save_card_icon_view.h"
 #include "chrome/browser/ui/views/browser_dialogs.h"
 #include "chrome/browser/ui/views/location_bar/background_with_1_px_border.h"
@@ -989,6 +990,8 @@ bool LocationBarView::RefreshSaveCreditCardIconView() {
   command_updater()->UpdateCommandEnabled(IDC_SAVE_CREDIT_CARD_FOR_PAGE,
                                           enabled);
   save_credit_card_icon_view_->SetVisible(enabled);
+  if (!enabled)
+    autofill::SaveCardBubbleViews::CloseBubble();

[Evan Stade, 2016/01/08 03:19:45]

save_credit_card_icon_view_->GetBubble()->Hide()?
save_credit_card_icon_view_->GetController()->HideBubble()?

[Justin Donnelly, 2016/01/08 18:14:05]

Yes, that's very much what I'd prefer to do. The problem is that this would get
the controller and view associated with the current webview. This code works by
running after we've switched tabs and are dealing with a new web view. The
static methods used by each of the location bar bubbles are a means of keeping a
reference to the currently visible bubble independent of which tab is showing.

 
   return was_visible != save_credit_card_icon_view_->visible();
 }