Chrome user menu shouldn't close if a tab steals focus

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

Index: chrome/browser/ui/views/profiles/new_avatar_button.cc
diff --git a/chrome/browser/ui/views/profiles/new_avatar_button.cc b/chrome/browser/ui/views/profiles/new_avatar_button.cc
index 0fa981f6ef08fb36f9c2566b4ac52aaac0c5089a..9cd7ab9cee08c514c6e2ad69531f894dd69adbd6 100644
--- a/chrome/browser/ui/views/profiles/new_avatar_button.cc
+++ b/chrome/browser/ui/views/profiles/new_avatar_button.cc
@@ -120,8 +120,13 @@ NewAvatarButton::~NewAvatarButton() {
 }
 
 bool NewAvatarButton::OnMousePressed(const ui::MouseEvent& event) {
-  // Prevent the bubble from being re-shown if it's already showing.
-  suppress_mouse_released_action_ = ProfileChooserView::IsShowing();
+  // Prevent the bubble from being re-shown if it's already showing in current
+  // browser and Hide it if it's showing in another browser.
+  if (ProfileChooserView::IsShowingInBrowser(browser_) == IS_SHOWING)
+    suppress_mouse_released_action_ = true;
+  else if (ProfileChooserView::IsShowingInBrowser(browser_) ==
+           IS_SHOWING_IN_ANOTHER_BROWSER)
+    ProfileChooserView::Hide();

[Roger Tawa OOO till Jul 10th, 2015/05/15 14:29:17]

Nit: need { and } around if and else blocks.
Nit: create a local variable for
ProfileChooserView::IsShowingInBrowser(browser_)?

[gogerald1, 2015/05/19 23:35:12]

Acknowledged. use local variable to save one function call, after that the if
else is just a single sentence, so it will be more clear without {}.

   return LabelButton::OnMousePressed(event);
 }