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

Unified Diff: chrome/browser/ui/webui/signin/user_manager_screen_handler.cc

Issue 2411033006: Display local signin error without browser and record the path of selected profile in user manager … (Closed)
Patch Set: tapted's comments Created 4 years, 2 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
« no previous file with comments | « chrome/browser/ui/webui/signin/signin_error_ui.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 c34c0545d581c08c2d18acfe9f120e07313cd216..c02ef40ba9db2568ee469cdd5e6f282bccbaaf3b 100644
--- a/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
+++ b/chrome/browser/ui/webui/signin/user_manager_screen_handler.cc
@@ -472,22 +472,15 @@ void UserManagerScreenHandler::HandleAuthenticatedLaunchUser(
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()) {
+ // In order to support the upgrade case where we have a local hash but no
+ // password token, the user must perform a full online reauth.
UserManager::ShowReauthDialog(browser_context, email_address_,
signin_metrics::Reason::REASON_UNLOCK);
} else {
// Fresh sign in via user manager without existing email address.
UserManager::ShowSigninDialog(browser_context, profile_path);
}
-#else
- UserManager::ShowReauthDialog(browser_context, email_address_,
- signin_metrics::Reason::REASON_UNLOCK);
-#endif
}
void UserManagerScreenHandler::HandleRemoveUser(const base::ListValue* args) {
« no previous file with comments | « chrome/browser/ui/webui/signin/signin_error_ui.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698