Chromium Code Reviews| Index: chrome/browser/chromeos/login/session/user_session_manager.cc |
| diff --git a/chrome/browser/chromeos/login/session/user_session_manager.cc b/chrome/browser/chromeos/login/session/user_session_manager.cc |
| index 4da8df1bf1d54a655bb441120b2cba6a845a7dce..f953cf21ea9b2fa74192fde2334dcbfc6fec9da5 100644 |
| --- a/chrome/browser/chromeos/login/session/user_session_manager.cc |
| +++ b/chrome/browser/chromeos/login/session/user_session_manager.cc |
| @@ -450,7 +450,7 @@ void UserSessionManager::StartSession( |
| delegate_ = delegate; |
| start_session_type_ = start_session_type; |
| - VLOG(1) << "Starting session for " << user_context.GetUserID(); |
| + VLOG(1) << "Starting session for " << user_context.GetUserID().GetUserEmail(); |
| PreStartSession(); |
| CreateUserSession(user_context, has_auth_cookies); |
| @@ -758,7 +758,7 @@ void UserSessionManager::OnSessionRestoreStateChanged( |
| if (!connection_error) { |
| // We are in one of "done" states here. |
| user_manager::UserManager::Get()->SaveUserOAuthStatus( |
| - user_manager::UserManager::Get()->GetLoggedInUser()->email(), |
| + user_manager::UserManager::Get()->GetLoggedInUser()->GetUserID(), |
| user_status); |
| } |
| @@ -869,8 +869,8 @@ void UserSessionManager::StoreUserContextDataBeforeProfileIsCreated() { |
| void UserSessionManager::StartCrosSession() { |
| BootTimesRecorder* btl = BootTimesRecorder::Get(); |
| btl->AddLoginTimeMarker("StartSession-Start", false); |
| - DBusThreadManager::Get()->GetSessionManagerClient()-> |
| - StartSession(user_context_.GetUserID()); |
| + DBusThreadManager::Get()->GetSessionManagerClient()->StartSession( |
| + user_context_.GetUserID().GetUserEmail()); |
| btl->AddLoginTimeMarker("StartSession-End", false); |
| } |
| @@ -885,8 +885,8 @@ void UserSessionManager::NotifyUserLoggedIn() { |
| } |
| void UserSessionManager::PrepareProfile() { |
| - bool is_demo_session = |
| - DemoAppLauncher::IsDemoAppSession(user_context_.GetUserID()); |
| + bool is_demo_session = DemoAppLauncher::IsDemoAppSession( |
|
achuithb
2015/10/23 00:08:50
const bool
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| + user_context_.GetUserID().GetUserEmail()); |
| // TODO(nkostylev): Figure out whether demo session is using the right profile |
| // path or not. See https://codereview.chromium.org/171423009 |
| @@ -968,8 +968,8 @@ void UserSessionManager::InitProfilePreferences( |
| if (gaia_id.empty()) { |
| AccountTrackerService* account_tracker = |
| AccountTrackerServiceFactory::GetForProfile(profile); |
| - AccountInfo info = |
| - account_tracker->FindAccountInfoByEmail(user_context.GetUserID()); |
| + AccountInfo info = account_tracker->FindAccountInfoByEmail( |
|
achuithb
2015/10/23 00:08:50
const
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| + user_context.GetUserID().GetUserEmail()); |
| gaia_id = info.gaia; |
| DCHECK(!gaia_id.empty()); |
| } |
| @@ -979,8 +979,8 @@ void UserSessionManager::InitProfilePreferences( |
| // profiles that might not have it set yet). |
| SigninManagerBase* signin_manager = |
| SigninManagerFactory::GetForProfile(profile); |
| - signin_manager->SetAuthenticatedAccountInfo(gaia_id, |
| - user_context.GetUserID()); |
| + signin_manager->SetAuthenticatedAccountInfo( |
| + gaia_id, user_context.GetUserID().GetUserEmail()); |
| // Backfill GAIA ID in user prefs stored in Local State. |
| std::string tmp_gaia_id; |
| @@ -994,7 +994,7 @@ void UserSessionManager::InitProfilePreferences( |
| void UserSessionManager::UserProfileInitialized(Profile* profile, |
| bool is_incognito_profile, |
| - const std::string& user_id) { |
| + const AccountId& user_id) { |
|
achuithb
2015/10/23 00:08:50
account_id
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| // Demo user signed in. |
| if (is_incognito_profile) { |
| profile->OnLogin(); |
| @@ -1021,9 +1021,10 @@ void UserSessionManager::UserProfileInitialized(Profile* profile, |
| // first. |
| bool transfer_saml_auth_cookies_on_subsequent_login = false; |
| if (has_auth_cookies_ && |
| - g_browser_process->platform_part()-> |
| - browser_policy_connector_chromeos()->GetUserAffiliation(user_id) == |
| - policy::USER_AFFILIATION_MANAGED) { |
| + g_browser_process->platform_part() |
| + ->browser_policy_connector_chromeos() |
| + ->GetUserAffiliation(user_id.GetUserEmail()) == |
| + policy::USER_AFFILIATION_MANAGED) { |
| CrosSettings::Get()->GetBoolean( |
| kAccountsPrefTransferSAMLCookies, |
| &transfer_saml_auth_cookies_on_subsequent_login); |
| @@ -1440,7 +1441,7 @@ void UserSessionManager::RestorePendingUserSessions() { |
| DCHECK(!user_already_logged_in); |
| if (!user_already_logged_in) { |
| - UserContext user_context(user_id); |
| + UserContext user_context(AccountId::FromUserEmail(user_id)); |
| user_context.SetUserIDHash(user_id_hash); |
| user_context.SetIsUsingOAuth(false); |
| @@ -1503,7 +1504,7 @@ void UserSessionManager::UpdateEasyUnlockKeys(const UserContext& user_context) { |
| key_manager->RefreshKeys( |
| user_context, *device_list, |
| base::Bind(&UserSessionManager::OnEasyUnlockKeyOpsFinished, AsWeakPtr(), |
| - user_context.GetUserID())); |
| + user_context.GetUserID().GetUserEmail())); |
| } |
| net::URLRequestContextGetter* |
| @@ -1564,8 +1565,8 @@ void UserSessionManager::OnEasyUnlockKeyOpsFinished( |
| if (!easy_unlock_key_ops_finished_callback_.is_null()) |
| easy_unlock_key_ops_finished_callback_.Run(); |
| - const user_manager::User* user = |
| - user_manager::UserManager::Get()->FindUser(user_id); |
| + const user_manager::User* user = user_manager::UserManager::Get()->FindUser( |
| + AccountId::FromUserEmail(user_id)); |
| EasyUnlockService* easy_unlock_service = |
| EasyUnlockService::GetForUser(*user); |
| easy_unlock_service->CheckCryptohomeKeysAndMaybeHardlock(); |
| @@ -1737,7 +1738,7 @@ void UserSessionManager::OnOAuth2TokensFetched(UserContext context) { |
| } |
| } |
| -void UserSessionManager::OnTokenHandleObtained(const user_manager::UserID& id, |
| +void UserSessionManager::OnTokenHandleObtained(const AccountId& id, |
|
achuithb
2015/10/23 00:08:50
account_id
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| bool success) { |
| if (!success) |
| LOG(ERROR) << "OAuth2 token handle fetch failed."; |