Don't show the zoom bubble when the settings menu is showing on views or GTK.

chrome/browser/ui/views/toolbar_view.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_TOOLBAR_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_TOOLBAR_VIEW_H_
 
 #include <set>
 #include <string>
 
@@ skipping 141 matching lines @@
   virtual bool GetDropFormats(
       int* formats,
       std::set<OSExchangeData::CustomFormat>* custom_formats) OVERRIDE;
   virtual bool CanDrop(const ui::OSExchangeData& data) OVERRIDE;
   virtual int OnDragUpdated(const ui::DropTargetEvent& event) OVERRIDE;
   virtual int OnPerformDrop(const ui::DropTargetEvent& event) OVERRIDE;
   virtual void OnThemeChanged() OVERRIDE;
   virtual std::string GetClassName() const OVERRIDE;
   virtual bool AcceleratorPressed(const ui::Accelerator& acc) OVERRIDE;
 
+  // Whether the wrench/hotdogs menu is currently showing.
+  bool IsSettingsMenuShowing() const;

[sky, 2012/08/22 19:36:29]

Since we use wrench menu every where, this should be IsWrenchMenuShowing().

[Dan Beam, 2012/08/22 23:10:38]

OK, updated.  Here's hoping I'm not the one that has to s/Wrench/HotDogs/g
eventually.

+
   // The apparent horizontal space between most items, and the vertical padding
   // above and below them.
   static const int kStandardSpacing;
   // The top of the toolbar has an edge we have to skip over in addition to the
   // standard spacing.
   static const int kVertSpacing;
 
  protected:
   // Overridden from AccessiblePaneView
   virtual bool SetPaneFocusAndFocusDefault() OVERRIDE;
@@ skipping 71 matching lines @@
 
   // A list of listeners to call when the menu opens.
   ObserverList<views::MenuListener> menu_listeners_;
 
   content::NotificationRegistrar registrar_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(ToolbarView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TOOLBAR_VIEW_H_