Prevent the Online Reauth window from opening twice.

chrome/browser/ui/cocoa/profiles/user_manager_mac.mm

Index: chrome/browser/ui/cocoa/profiles/user_manager_mac.mm
diff --git a/chrome/browser/ui/cocoa/profiles/user_manager_mac.mm b/chrome/browser/ui/cocoa/profiles/user_manager_mac.mm
index 3a34663b15e1c9b00cebcb53b63c959d42c25f08..0942d4d2feb2532da383a1ef11e3ce76125c8ced 100644
--- a/chrome/browser/ui/cocoa/profiles/user_manager_mac.mm
+++ b/chrome/browser/ui/cocoa/profiles/user_manager_mac.mm
@@ -365,15 +365,18 @@ class ReauthDialogDelegate : public UserManager::ReauthDialogObserver,
 }
 
 - (void)showReauthDialogWithProfile:(Profile*)profile email:(std::string)email {
-  reauth_window_controller_.reset(
-      [[ReauthDialogWindowController alloc]
-          initWithProfile:profile
-                    email:email
-              webContents:webContents_.get()]);
+  if (!reauth_window_controller_.get()) {

[groby-ooo-7-16, 2015/10/19 20:09:31]

Nit: no .get() needed

[anthonyvd, 2015/10/20 19:40:49]

Done.

+    reauth_window_controller_.reset(
+        [[ReauthDialogWindowController alloc]
+            initWithProfile:profile
+                      email:email
+                webContents:webContents_.get()]);
+  }
 }
 
 - (void)closeReauthDialog {
   [reauth_window_controller_ close];
+  reauth_window_controller_.reset();

[groby-ooo-7-16, 2015/10/19 20:09:31]

Q: Doesn't reset() imply -close?

[anthonyvd, 2015/10/20 19:40:49]

It does, thanks for noticing!

 }
 
 @end