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

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

Index: chrome/browser/ui/views/location_bar/bubble_icon_view.h
diff --git a/chrome/browser/ui/views/location_bar/bubble_icon_view.h b/chrome/browser/ui/views/location_bar/bubble_icon_view.h
index 98d4a9297e0de6cc204eb6636ef6fa07297151e7..99ff92d16bbebb8d8ef8764809c7a729185bb41e 100644
--- a/chrome/browser/ui/views/location_bar/bubble_icon_view.h
+++ b/chrome/browser/ui/views/location_bar/bubble_icon_view.h
@@ -18,6 +18,7 @@ class BubbleIconView : public views::ImageView {
     EXECUTE_SOURCE_GESTURE,
   };
 
+  BubbleIconView();

[msw, 2014/10/24 00:00:55]

Remove this, ZoomBubbleView can just pass BubbleIconView(nullptr, 0).
(bonus points if you wanted to actually hook up IDC_ZOOM_MENU)

[Dan Beam, 2014/10/24 00:26:19]

Done.
i'm hesitant to do this as it's already used in the flapjacks menu

   explicit BubbleIconView(CommandUpdater* command_updater, int command_id);
   virtual ~BubbleIconView();

[msw, 2014/10/24 00:00:55]

optional nit: virtual -> override here and below.

[Dan Beam, 2014/10/24 00:26:19]

Done.

 
@@ -39,6 +40,10 @@ class BubbleIconView : public views::ImageView {
   virtual void OnGestureEvent(ui::GestureEvent* event) override;
 
  private:
+  // Executes the given |command_id_| with |command_updater_| if specified when

[msw, 2014/10/24 00:00:55]

optional nit: one-liner "// Calls OnExecuting and runs |command_id_| with a
valid |command_updater_|."

[Dan Beam, 2014/10/24 00:26:19]

Done.

+  // constructed.
+  void ExecuteCommand();
+
   // The CommandUpdater for the Browser object that owns the location bar.
   CommandUpdater* command_updater_;