Close avatar bubble when avatar button is clicked a second time

chrome/browser/ui/views/avatar_menu_bubble_view.h

Index: chrome/browser/ui/views/avatar_menu_bubble_view.h
diff --git a/chrome/browser/ui/views/avatar_menu_bubble_view.h b/chrome/browser/ui/views/avatar_menu_bubble_view.h
index 5490ac2e917b6a3c6c3c37a1fa52d3f3f7771a13..9dd02a1378a76f9bd90174273561947f265db98d 100644
--- a/chrome/browser/ui/views/avatar_menu_bubble_view.h
+++ b/chrome/browser/ui/views/avatar_menu_bubble_view.h
@@ -30,10 +30,14 @@ class AvatarMenuBubbleView : public views::BubbleDelegateView,
                              public views::LinkListener,
                              public AvatarMenuModelObserver {
  public:
-  AvatarMenuBubbleView(views::View* anchor_view,
-                       views::BubbleBorder::ArrowLocation arrow_location,
-                       const gfx::Rect& anchor_rect,
-                       Browser* browser);
+  static void ShowBubble(views::View* anchor_view,

[Peter Kasting, 2013/01/04 00:58:43]

Nit: Write a comment to note why we want this sort of function (i.e. that we
want to close when the button is clicked and the bubble is open; why this makes
that happen; and why without this we'd wind up not doing that).

(At least for me, the part that's least clear is why this causes the bubble to
close when the bubble is clicked a second time, as opposed to doing nothing.)

[sail, 2013/01/04 19:23:08]

Done.

+                         views::BubbleBorder::ArrowLocation arrow_location,
+                         views::BubbleBorder::BubbleAlignment border_alignment,
+                         const gfx::Rect& anchor_rect,
+                         Browser* browser);
+  static bool IsShowing();
+  static void Hide();
+
   virtual ~AvatarMenuBubbleView();
 
   // views::View implementation.
@@ -51,12 +55,18 @@ class AvatarMenuBubbleView : public views::BubbleDelegateView,
   // BubbleDelegate implementation.
   virtual gfx::Rect GetAnchorRect() OVERRIDE;
   virtual void Init() OVERRIDE;
+  virtual void WindowClosing() OVERRIDE;
 
   // AvatarMenuModelObserver implementation.
   virtual void OnAvatarMenuModelChanged(
       AvatarMenuModel* avatar_menu_model) OVERRIDE;
 
  private:
+  AvatarMenuBubbleView(views::View* anchor_view,
+                       views::BubbleBorder::ArrowLocation arrow_location,
+                       const gfx::Rect& anchor_rect,
+                       Browser* browser);
+
   views::Link* add_profile_link_;
   scoped_ptr<AvatarMenuModel> avatar_menu_model_;
   gfx::Rect anchor_rect_;
@@ -64,6 +74,8 @@ class AvatarMenuBubbleView : public views::BubbleDelegateView,
   std::vector<views::CustomButton*> item_views_;
   views::Separator* separator_;
 
+  static AvatarMenuBubbleView* avatar_bubble_;
+
   DISALLOW_COPY_AND_ASSIGN(AvatarMenuBubbleView);
 };