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

Unified Diff: chrome/browser/chromeos/login/auth/chrome_login_performer.cc

Issue 1412813003: This CL replaces user_manager::UserID with AccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@468875--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Create-AccountID-structure-part2--user_names
Patch Set: Fix Win GN build. Created 5 years, 1 month 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/chromeos/login/auth/chrome_login_performer.cc
diff --git a/chrome/browser/chromeos/login/auth/chrome_login_performer.cc b/chrome/browser/chromeos/login/auth/chrome_login_performer.cc
index e913bc17c8765c93c69a049b38ecbf5188d5979f..e912445898288b10383a7b6267ed449bb75b9c89 100644
--- a/chrome/browser/chromeos/login/auth/chrome_login_performer.cc
+++ b/chrome/browser/chromeos/login/auth/chrome_login_performer.cc
@@ -19,6 +19,7 @@
#include "chrome/browser/chromeos/policy/device_local_account_policy_service.h"
#include "chrome/browser/chromeos/profiles/profile_helper.h"
#include "chrome/browser/chromeos/settings/cros_settings.h"
+#include "components/signin/core/account_id/account_id.h"
namespace chromeos {
@@ -130,7 +131,8 @@ bool ChromeLoginPerformer::UseExtendedAuthenticatorForSupervisedUser(
const UserContext& user_context) {
SupervisedUserAuthentication* authentication =
ChromeUserManager::Get()->GetSupervisedUserManager()->GetAuthentication();
- return authentication->GetPasswordSchema(user_context.GetUserID()) ==
+ return authentication->GetPasswordSchema(
+ user_context.GetAccountId().GetUserEmail()) ==
SupervisedUserAuthentication::SCHEMA_SALT_HASHED;
}
@@ -143,12 +145,15 @@ UserContext ChromeLoginPerformer::TransformSupervisedKey(
void ChromeLoginPerformer::SetupSupervisedUserFlow(const std::string& user_id) {
SupervisedUserLoginFlow* new_flow = new SupervisedUserLoginFlow(user_id);
- new_flow->SetHost(ChromeUserManager::Get()->GetUserFlow(user_id)->host());
- ChromeUserManager::Get()->SetUserFlow(user_id, new_flow);
+ new_flow->SetHost(ChromeUserManager::Get()
+ ->GetUserFlow(AccountId::FromUserEmail(user_id))
+ ->host());
+ ChromeUserManager::Get()->SetUserFlow(AccountId::FromUserEmail(user_id),
+ new_flow);
}
void ChromeLoginPerformer::SetupEasyUnlockUserFlow(const std::string& user_id) {
- ChromeUserManager::Get()->SetUserFlow(user_id,
+ ChromeUserManager::Get()->SetUserFlow(AccountId::FromUserEmail(user_id),
new EasyUnlockUserLoginFlow(user_id));
}

Powered by Google App Engine
This is Rietveld 408576698