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

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

 // Copyright (c) 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/location_bar/location_bar_view.h"
 
 #include <algorithm>
 #include <map>
 
 #include "base/stl_util.h"
@@ skipping 380 matching lines @@
     StartFadeAnimation();
   } else {
     // Cancel any pending animations; switch to the final state immediately.
     StopFadeAnimation();
   }
 #endif
 }
 
 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();
 
   bool star_enabled = star_view_ && !model_->input_in_progress() &&
                       edit_bookmarks_enabled_.GetValue();
 
   command_updater_->UpdateCommandEnabled(IDC_BOOKMARK_PAGE, star_enabled);
   if (star_view_)
     star_view_->SetVisible(star_enabled);
 
   bool enabled = chrome_to_mobile_view_ && !model_->input_in_progress() &&
@@ skipping 1087 matching lines @@
 }
 
 void LocationBarView::CleanupFadeAnimation() {
   // Since we're no longer animating we don't need our layer.
   SetPaintToLayer(false);
   // Bubble labels don't need a transparent background anymore.
   ev_bubble_view_->SetLabelBackgroundColor(SK_ColorWHITE);
   selected_keyword_view_->SetLabelBackgroundColor(SK_ColorWHITE);
 }
 #endif  // USE_AURA