No need to suppress ZoomBubble if !window_->IsActive().

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 2f7f206b449567917db080f7bae7ea9d88bac48e..f591baf06a78f5c5953b0779a7ea71d3fad62c5f 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -1939,8 +1939,7 @@ void Browser::OnZoomChanged(
     const ui_zoom::ZoomController::ZoomChangedEventData& data) {
   if (data.web_contents == tab_strip_model_->GetActiveWebContents()) {
     // Only show the zoom bubble for zoom changes in the active window.

[rohitrao (ping after 24h), 2015/02/25 18:28:01]

Is this comment now incorrect?

[wjmaclean, 2015/02/25 18:40:12]

Ooops, yes ... I'll remove it.

-    window_->ZoomChangedForActiveTab(data.can_show_bubble &&
-                                     window_->IsActive());
+    window_->ZoomChangedForActiveTab(data.can_show_bubble);

[rohitrao (ping after 24h), 2015/02/25 18:28:01]

Is it still correct for this method to have "ForActiveTab" in its name?

[wjmaclean, 2015/02/25 18:40:12]

I think so, though it is an unfortunate coincidence in the naming ... a tab can
be active without its window being active (by the logic that otherwise
GetActiveWebContents() would return null). It seems to be referring to the
"currently visible tab".

     // Change the zoom commands state based on the zoom state
     command_controller_->ZoomStateChanged();
   }