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

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

Index: chrome/browser/ui/views/profiles/profile_chooser_view.cc
diff --git a/chrome/browser/ui/views/profiles/profile_chooser_view.cc b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
index f55258026ec657bf0ba3466211ae3fa8ea3bc099..b02925c14a7091b1d36eec52463f91cc4ee073cd 100644
--- a/chrome/browser/ui/views/profiles/profile_chooser_view.cc
+++ b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
@@ -686,6 +686,14 @@ void ProfileChooserView::ShowView(profiles::BubbleViewMode view_to_display,
   if (view_mode_ != profiles::BUBBLE_VIEW_MODE_PROFILE_CHOOSER)
     tutorial_mode_ = profiles::TUTORIAL_MODE_NONE;
 
+  if (view_mode_ == profiles::BUBBLE_VIEW_MODE_GAIA_SIGNIN ||
+      view_mode_ == profiles::BUBBLE_VIEW_MODE_GAIA_ADD_ACCOUNT ||
+      view_mode_ == profiles::BUBBLE_VIEW_MODE_GAIA_REAUTH ||
+      tutorial_mode_ == profiles::TUTORIAL_MODE_CONFIRM_SIGNIN) {
+    profile_bubble_->set_close_on_deactivate(false);
+  } else
+    profile_bubble_->set_close_on_deactivate(close_on_deactivate_for_testing_);

[Roger Tawa OOO till Jul 10th, 2015/05/14 18:55:03]

If you added { and } to the if block, you also need the { and } around the else
block, even if it's a trivial one line.

[gogerald1, 2015/05/14 21:35:22]

Done.

+
   layout->StartRow(1, 0);
   layout->AddView(sub_view);
   Layout();