Avatar Bubble doesn't re-pop when trying to click it away.

chrome/browser/ui/views/profiles/new_avatar_button.h

 // Copyright 2014 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_PROFILES_NEW_AVATAR_BUTTON_H_
 #define CHROME_BROWSER_UI_VIEWS_PROFILES_NEW_AVATAR_BUTTON_H_
 
 #include "chrome/browser/profiles/profile_info_cache_observer.h"
 #include "components/signin/core/browser/signin_error_controller.h"
 #include "ui/views/controls/button/menu_button.h"
@@ skipping 10 matching lines @@
     THEMED_BUTTON,   // Used in a themed browser window.
     NATIVE_BUTTON,    // Used in a native aero or metro window.
   };
 
   NewAvatarButton(views::ButtonListener* listener,
                   const base::string16& profile_name,
                   AvatarButtonStyle button_style,
                   Browser* browser);
   virtual ~NewAvatarButton();
 
+  virtual bool OnMousePressed(const ui::MouseEvent& event) OVERRIDE;

[msw, 2014/08/20 21:12:36]

nit: add comment "// views::MenuButton:"

[Mike Lerman, 2014/08/21 15:15:36]

Done.

+
+  // If this is the second click on the avatar button then the avatar button was
+  // showing on the pressed event and is hidden now. Prevent the bubble from
+  // reshowing.
+  bool IsMouseReleaseActionSuppressed();
+
  private:
   friend class NewAvatarMenuButtonTest;
   friend class ProfileChooserViewBrowserTest;
   FRIEND_TEST_ALL_PREFIXES(NewAvatarMenuButtonTest, SignOut);
   FRIEND_TEST_ALL_PREFIXES(ProfileChooserViewBrowserTest, ViewProfileUMA);
 
   // ProfileInfoCacheObserver:
   virtual void OnProfileAdded(const base::FilePath& profile_path) OVERRIDE;
   virtual void OnProfileWasRemoved(
       const base::FilePath& profile_path,
       const base::string16& profile_name) OVERRIDE;
   virtual void OnProfileNameChanged(
       const base::FilePath& profile_path,
       const base::string16& old_profile_name) OVERRIDE;
   virtual void OnProfileSupervisedUserIdChanged(
       const base::FilePath& profile_path) OVERRIDE;
 
   // SigninErrorController::Observer:
   virtual void OnErrorChanged() OVERRIDE;
 
   // Called when the profile info cache has changed, which means we might
   // have to re-display the profile name.
   void UpdateAvatarButtonAndRelayoutParent();
 
   Browser* browser_;
 
+  // This is used to check if the profile chooser view was showing during the

[msw, 2014/08/20 21:12:36]

nit: s/profile chooser view/bubble/, ditto below for "profile chooser".

[Mike Lerman, 2014/08/21 15:15:36]

Done.

+  // mouse pressed event. If this is true then the mouse released event is
+  // ignored to prevent the profile chooser from reshowing.
+  bool suppress_mouse_released_action_;
+
   DISALLOW_COPY_AND_ASSIGN(NewAvatarButton);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_PROFILES_NEW_AVATAR_BUTTON_H_