zoom bubble: Close if anchor is clicked while bubble is showing.

chrome/browser/ui/views/location_bar/zoom_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_LOCATION_BAR_ZOOM_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_ZOOM_VIEW_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
@@ skipping 21 matching lines @@
   virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;
   virtual bool GetTooltipText(const gfx::Point& p,
                               base::string16* tooltip) const OVERRIDE;
   virtual bool OnMousePressed(const ui::MouseEvent& event) OVERRIDE;
   virtual void OnMouseReleased(const ui::MouseEvent& event) OVERRIDE;
   virtual bool OnKeyPressed(const ui::KeyEvent& event) OVERRIDE;
 
   // ui::EventHandler:
   virtual void OnGestureEvent(ui::GestureEvent* event) OVERRIDE;
 
-  // Helper method to show and focus the zoom bubble associated with this
-  // widget.
-  void ActivateBubble();
+  // Hide the zoom bubble if it's showing. Show the zoom bubble if it's hidden.

[msw, 2014/07/24 22:07:13]

It seems like ZoomView should be a BubbleIconView ("Represents an icon on the
omnibox that shows a bubble when clicked."). Then it'd get the toggle behavior
more cheaply via |suppress_mouse_released_action_|, and be more consistent with
StarView, TranslateIconView, and ManagePasswordsIconView. WDYT?

[Dan Beam, 2014/10/23 23:34:03]

Done.

+  void ToggleBubble();
 
   // The delegate used to get the currently visible WebContents.
   LocationBarView::Delegate* location_bar_delegate_;
 
+  // Was the zoom bubble showing when the mouse went down inside this view.
+  bool was_bubble_showing_;
+
   DISALLOW_COPY_AND_ASSIGN(ZoomView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_ZOOM_VIEW_H_