Display local signin error without browser and record the path of selected profile in user manager.

chrome/browser/ui/webui/signin/user_manager_screen_handler.cc

Index: chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
diff --git a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
index a56bbe86f0f4b62d786d9c086bb32c4b41c3cbb3..620a8e5e8556184023d8e86063ea6479694aa930 100644
--- a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
+++ b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
@@ -469,11 +469,25 @@ void UserManagerScreenHandler::HandleAuthenticatedLaunchUser(
     }
   }
 
-  // In order to support the upgrade case where we have a local hash but no
-  // password token, the user perform a full online reauth.
-  UserManager::ShowReauthDialog(web_ui()->GetWebContents()->GetBrowserContext(),
-                                email_address_,
+  content::BrowserContext* browser_context =
+      web_ui()->GetWebContents()->GetBrowserContext();
+
+// In order to support the upgrade case where we have a local hash but no
+// password token, the user perform a full online reauth.
+// TODO(zmin): Remove the condition for MACOSX once user_manager_mac.cc is
+// updated.
+#if !defined(OS_MACOSX)
+  if (!email_address_.empty()) {
+    UserManager::ShowReauthDialog(browser_context, email_address_,
+                                  signin_metrics::Reason::REASON_UNLOCK);
+  } else {
+    // Fresh sign in via user manager without existed email address.

[anthonyvd, 2016/09/22 18:38:46]

nit: existed -> existing

[zmin, 2016/09/22 19:14:56]

Done.

+    UserManager::ShowSigninDialog(browser_context, profile_path);
+  }
+#else
+  UserManager::ShowReauthDialog(browser_context, email_address_,
                                 signin_metrics::Reason::REASON_UNLOCK);
+#endif
 }

[Roger Tawa OOO till Jul 10th, 2016/09/22 18:59:13]

Don't you need to check the "force" flag in ::HandleLaunchUser() and make sure
to show the sign in dialog there too?  Or is that change planned for a follow up
CL?

[zmin, 2016/09/22 19:14:56]

Yes, this CL is focusing on the error dialog. I change the code here just
because the refactor in the UserManager.

 
 void UserManagerScreenHandler::HandleRemoveUser(const base::ListValue* args) {