Closes zoom bubble when tab is changed within a browser window.

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 e18b1183ad24c99b1cd562e194c0d77a7bfe3484..657e03a1fe723894a66153e4c38a3a954136dfac 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.cc
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.cc
@@ -398,6 +398,10 @@ void LocationBarView::ModeChanged(const chrome::search::Mode& old_mode,
 
 void LocationBarView::Update(const WebContents* tab_for_state_restoring) {
   RefreshContentSettingViews();
+  // If |tab_for_state_restoring| is non-NULL, a tab change has occurred, so the
+  // zoom bubble should be hidden since it represents the zoom on another tab.

[Peter Kasting, 2012/08/15 19:40:28]

So, if we change URLs within the tab, and the new URL and old URL have different
zoom levels, will the bubble stay open but change its displayed value?  If so
that seems a little weird.

[Kyle Horimoto, 2012/08/15 20:30:19]

Actually, you're right. Probably better to get rid of the check. I tested it
without the check and it correctly gits rid of the bubble.

+  if (tab_for_state_restoring != NULL)
+    ZoomBubbleView::CloseBubble();
   zoom_view_->Update();

[Peter Kasting, 2012/08/15 19:40:28]

Having this call after CloseBubble() doesn't risk re-opening the bubble, does
it?

[Kyle Horimoto, 2012/08/15 20:30:19]

Nope!

   RefreshPageActionViews();