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

Unified Diff: chromeos/cryptohome/cryptohome_parameters.cc

Issue 2452053002: Rename AccountId method GetGaiaIdKey() to GetAccountIdKey(). (Closed)
Patch Set: Updated 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 | « chromeos/cryptohome/cryptohome_parameters.h ('k') | chromeos/login/auth/cryptohome_authenticator.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/cryptohome/cryptohome_parameters.cc
diff --git a/chromeos/cryptohome/cryptohome_parameters.cc b/chromeos/cryptohome/cryptohome_parameters.cc
index dcb649eb30fbdfcbb57f4f93c5e89fa39ba0ad26..9ce9c917de6459108320a22d95615be724e16ed0 100644
--- a/chromeos/cryptohome/cryptohome_parameters.cc
+++ b/chromeos/cryptohome/cryptohome_parameters.cc
@@ -23,7 +23,7 @@ const std::string GetCryptohomeId(const AccountId& account_id) {
return account_id.GetUserEmail(); // Migrated
if (GetGaiaIdMigrationStatus(account_id))
- return account_id.GetGaiaIdKey();
+ return account_id.GetAccountIdKey();
return account_id.GetUserEmail(); // Migrated
}
@@ -56,7 +56,7 @@ AccountId Identification::GetAccountId() const {
// A LOT of tests start with --login_user <user>, and not registing this user
// before. So we might have "known_user" entry without gaia_id.
for (const AccountId& known_id : known_account_ids) {
- if (!known_id.GetGaiaId().empty() && known_id.GetGaiaIdKey() == id_) {
+ if (!known_id.GetGaiaId().empty() && known_id.GetAccountIdKey() == id_) {
return known_id;
}
}
« no previous file with comments | « chromeos/cryptohome/cryptohome_parameters.h ('k') | chromeos/login/auth/cryptohome_authenticator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698