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

chrome/browser/ui/views/location_bar/zoom_view.h

Index: chrome/browser/ui/views/location_bar/zoom_view.h
diff --git a/chrome/browser/ui/views/location_bar/zoom_view.h b/chrome/browser/ui/views/location_bar/zoom_view.h
index 064d29edb95ed78511d8ed1711bbaf4fa754fbce..9a871fa7be468d19c29c514911abed9ca155f9db 100644
--- a/chrome/browser/ui/views/location_bar/zoom_view.h
+++ b/chrome/browser/ui/views/location_bar/zoom_view.h
@@ -39,13 +39,15 @@ class ZoomView : public views::ImageView {
   // 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);
 };