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

Unified Diff: chrome/browser/chromeos/login/easy_unlock/bootstrap_user_context_initializer.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/easy_unlock/bootstrap_user_context_initializer.cc
diff --git a/chrome/browser/chromeos/login/easy_unlock/bootstrap_user_context_initializer.cc b/chrome/browser/chromeos/login/easy_unlock/bootstrap_user_context_initializer.cc
index 9d66402fec3b6975da2b61d6f9420baebc455ba1..d42c5df3366e0d472e9f732e5610065e0f313e36 100644
--- a/chrome/browser/chromeos/login/easy_unlock/bootstrap_user_context_initializer.cc
+++ b/chrome/browser/chromeos/login/easy_unlock/bootstrap_user_context_initializer.cc
@@ -68,10 +68,9 @@ void BootstrapUserContextInitializer::StartTokenFetch(
}
void BootstrapUserContextInitializer::StartCheckExistingKeys() {
- const std::string& user_id = user_context_.GetUserID();
-
// Use random key for the first time user.
- if (!user_manager::UserManager::Get()->IsKnownUser(user_id)) {
+ if (!user_manager::UserManager::Get()->IsKnownUser(
+ user_context_.GetAccountId())) {
CreateRandomKey();
return;
}
@@ -79,7 +78,7 @@ void BootstrapUserContextInitializer::StartCheckExistingKeys() {
EasyUnlockKeyManager* key_manager =
UserSessionManager::GetInstance()->GetEasyUnlockKeyManager();
key_manager->GetDeviceDataList(
- UserContext(user_id),
+ UserContext(user_context_.GetAccountId()),
base::Bind(&BootstrapUserContextInitializer::OnGetEasyUnlockData,
weak_ptr_factory_.GetWeakPtr()));
}
@@ -99,7 +98,7 @@ void BootstrapUserContextInitializer::OnGetEasyUnlockData(
service->AddObserver(this);
static_cast<EasyUnlockServiceSignin*>(service)
- ->SetCurrentUser(user_context_.GetUserID());
+ ->SetCurrentUser(user_context_.GetAccountId().GetUserEmail());
OnScreenlockStateChanged(service->GetScreenlockState());
}
@@ -203,7 +202,7 @@ void BootstrapUserContextInitializer::OnScreenlockStateChanged(
service->RemoveObserver(this);
service->AttemptAuth(
- user_context_.GetUserID(),
+ user_context_.GetAccountId().GetUserEmail(),
base::Bind(&BootstrapUserContextInitializer::OnEasyUnlockAuthenticated,
weak_ptr_factory_.GetWeakPtr()));
}

Powered by Google App Engine
This is Rietveld 408576698