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

Unified Diff: chromeos/login/auth/cryptohome_authenticator.cc

Issue 2601283002: Use AccountIdKey to identify Active Directory accounts. (Closed)
Patch Set: rebase Created 3 years, 12 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/dbus/auth_policy_client.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/login/auth/cryptohome_authenticator.cc
diff --git a/chromeos/login/auth/cryptohome_authenticator.cc b/chromeos/login/auth/cryptohome_authenticator.cc
index e12ac3101f9c083600a272612b0a2e706c8b4dc5..f434dcd4d21e0d8364127c15c7d71f83e5d3bfda 100644
--- a/chromeos/login/auth/cryptohome_authenticator.cc
+++ b/chromeos/login/auth/cryptohome_authenticator.cc
@@ -212,17 +212,17 @@ void EnsureCryptohomeMigratedToGaiaId(
scoped_refptr<CryptohomeAuthenticator> resolver,
bool ephemeral,
bool create_if_nonexistent) {
+ if (attempt->user_context.GetAccountId().GetAccountType() ==
+ AccountType::ACTIVE_DIRECTORY) {
+ cryptohome::SetGaiaIdMigrationStatusDone(
+ attempt->user_context.GetAccountId());
+ }
const bool is_gaiaid_migration_started = switches::IsGaiaIdMigrationStarted();
if (!is_gaiaid_migration_started) {
UMACryptohomeMigrationToGaiaId(CryptohomeMigrationToGaiaId::NOT_STARTED);
DoMount(attempt, resolver, ephemeral, create_if_nonexistent);
return;
}
- if (attempt->user_context.GetAccountId().GetAccountType() ==
- AccountType::ACTIVE_DIRECTORY) {
- cryptohome::SetGaiaIdMigrationStatusDone(
- attempt->user_context.GetAccountId());
- }
const bool already_migrated = cryptohome::GetGaiaIdMigrationStatus(
attempt->user_context.GetAccountId());
const bool has_account_key =
« no previous file with comments | « chromeos/dbus/auth_policy_client.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698