Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3499)

Unified Diff: chrome/browser/ui/views/profiles/new_avatar_button.cc

Issue 1136693002: Chrome user menu shouldn't close if a tab steals focus (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address msw's comments Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..ab4ca6ff16c76c58a2c7928c83e444cfcc13cc25 100644
--- a/chrome/browser/ui/views/profiles/new_avatar_button.cc
+++ b/chrome/browser/ui/views/profiles/new_avatar_button.cc
@@ -120,8 +120,15 @@ 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 the
+ // current
msw 2015/05/20 00:18:11 nit: fix line wrapping...
gogerald1 2015/05/20 14:06:24 Done.
+ // browser and Hide it if it's showing in another browser.
+ ProfileChooserView::ShowingType show_type =
+ ProfileChooserView::IsShowingInBrowser(browser_);
+ if (show_type == ProfileChooserView::IS_SHOWING)
+ suppress_mouse_released_action_ = true;
+ else if (show_type == ProfileChooserView::IS_SHOWING_IN_ANOTHER_BROWSER)
+ ProfileChooserView::Hide();
return LabelButton::OnMousePressed(event);
}

Powered by Google App Engine
This is Rietveld 408576698