Don't reopen avatar bubble view on second click

chrome/browser/ui/views/bubble/bubble.h

 // Copyright (c) 2011 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_BUBBLE_BUBBLE_H_
 #define CHROME_BROWSER_UI_VIEWS_BUBBLE_BUBBLE_H_
 #pragma once
 
+#include "base/observer_list.h"
 #include "views/bubble/bubble_border.h"
 #include "ui/base/animation/animation_delegate.h"
 #include "views/accelerator.h"
 #include "views/view.h"
 
 #if defined(USE_AURA)
 #include "views/widget/native_widget_aura.h"
 #elif defined(OS_WIN)
 #include "views/widget/native_widget_win.h"
 #elif defined(TOUCH_UI)
@@ skipping 60 matching lines @@
 #elif defined(OS_WIN)
     : public views::NativeWidgetWin,
 #elif defined(TOUCH_UI)
     : public views::NativeWidgetViews,
 #elif defined(TOOLKIT_USES_GTK)
     : public views::NativeWidgetGtk,
 #endif
       public views::AcceleratorTarget,
       public ui::AnimationDelegate {
  public:
+  class Observer {
+   public:
+    // See BubbleDelegate::BubbleClosing for when this is called.
+    virtual void OnBubbleClosing(Bubble* bubble, bool closed_by_escape) = 0;

[Peter Kasting, 2011/10/05 01:22:51]

Nit: You don't seem to use either of these args, so why not omit them for now? 
We can add them later if needed.

[sail, 2011/10/05 01:32:10]

Done.

+  };
+
   // Shows the Bubble.
   // |parent| is set as the parent window.
   // |contents| are the contents shown in the bubble.
   // |position_relative_to| is a rect in screen coordinates at which the Bubble
   // will point.
   // Show() takes ownership of |contents| and deletes the created Bubble when
   // another window is activated. You can explicitly close the bubble by
   // invoking Close().
   // |arrow_location| specifies preferred bubble alignment.
   // You may provide an optional |delegate| to:
@@ skipping 38 matching lines @@
   virtual void Close();
 
   // Overridden from ui::AnimationDelegate:
   virtual void AnimationEnded(const ui::Animation* animation);
   virtual void AnimationProgressed(const ui::Animation* animation);
 
 #ifdef UNIT_TEST
   views::View* contents() const { return contents_; }
 #endif
 
+  void AddObserver(Observer* obs) {
+    observer_list_.AddObserver(obs);
+  }
+
+  void RemoveObserver(Observer* obs) {
+    observer_list_.RemoveObserver(obs);
+  }
+
   static const SkColor kBackgroundColor;
 
  protected:
   Bubble();
 #if defined(OS_CHROMEOS)
   Bubble(views::Widget::InitParams::Type type,
          bool show_while_screen_is_locked);
 #endif
   virtual ~Bubble();
 
@@ skipping 79 matching lines @@
   bool show_while_screen_is_locked_;
 #endif
 
   gfx::Rect position_relative_to_;
   views::BubbleBorder::ArrowLocation arrow_location_;
 
   views::View* contents_;
 
   bool accelerator_registered_;
 
+  ObserverList<Observer> observer_list_;
+
   DISALLOW_COPY_AND_ASSIGN(Bubble);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_BUBBLE_BUBBLE_H_