Index: chrome/browser/chromeos/login/signin/oauth2_browsertest.cc |
diff --git a/chrome/browser/chromeos/login/signin/oauth2_browsertest.cc b/chrome/browser/chromeos/login/signin/oauth2_browsertest.cc |
index b83bb0c22a8d2d137771d0e2220522dafb186334..3c224239332714891d3712c9497ff3cfbaffad00 100644 |
--- a/chrome/browser/chromeos/login/signin/oauth2_browsertest.cc |
+++ b/chrome/browser/chromeos/login/signin/oauth2_browsertest.cc |
@@ -29,6 +29,7 @@ |
#include "chromeos/login/auth/user_context.h" |
#include "components/app_modal/javascript_app_modal_dialog.h" |
#include "components/app_modal/native_app_modal_dialog.h" |
+#include "components/signin/core/account_id/account_id.h" |
#include "components/signin/core/browser/account_tracker_service.h" |
#include "components/signin/core/browser/profile_oauth2_token_service.h" |
#include "components/user_manager/user.h" |
@@ -197,7 +198,9 @@ class OAuth2Test : public OobeBaseTest { |
user_manager::User::OAUTH2_TOKEN_STATUS_VALID); |
// Try login. Primary profile has changed. |
- EXPECT_TRUE(TryToLogin(kTestGaiaId, kTestEmail, kTestAccountPassword)); |
+ EXPECT_TRUE( |
+ TryToLogin(AccountId::FromUserEmailGaiaId(kTestEmail, kTestGaiaId), |
+ kTestAccountPassword)); |
Profile* profile = ProfileManager::GetPrimaryUserProfile(); |
// Wait for the session merge to finish. |
@@ -212,15 +215,13 @@ class OAuth2Test : public OobeBaseTest { |
user_manager::User::OAUTH2_TOKEN_STATUS_VALID); |
} |
- bool TryToLogin(const std::string& gaia_id, |
- const std::string& username, |
- const std::string& password) { |
- if (!AddUserToSession(gaia_id, username, password)) |
+ bool TryToLogin(const AccountId& account_id, const std::string& password) { |
+ if (!AddUserToSession(account_id, password)) |
return false; |
if (const user_manager::User* active_user = |
user_manager::UserManager::Get()->GetActiveUser()) { |
- return active_user->email() == username; |
+ return active_user->GetAccountId() == account_id; |
} |
return false; |
@@ -251,8 +252,7 @@ class OAuth2Test : public OobeBaseTest { |
return OobeBaseTest::profile(); |
} |
- bool AddUserToSession(const std::string& gaia_id, |
- const std::string& username, |
+ bool AddUserToSession(const AccountId& account_id, |
const std::string& password) { |
ExistingUserController* controller = |
ExistingUserController::current_controller(); |
@@ -261,8 +261,8 @@ class OAuth2Test : public OobeBaseTest { |
return false; |
} |
- UserContext user_context(username); |
- user_context.SetGaiaID(gaia_id); |
+ UserContext user_context(account_id); |
+ user_context.SetGaiaID(account_id.GetGaiaId()); |
user_context.SetKey(Key(password)); |
controller->Login(user_context, SigninSpecifics()); |
content::WindowedNotificationObserver( |
@@ -273,7 +273,7 @@ class OAuth2Test : public OobeBaseTest { |
for (user_manager::UserList::const_iterator it = logged_users.begin(); |
it != logged_users.end(); |
++it) { |
- if ((*it)->email() == username) |
+ if ((*it)->GetAccountId() == account_id) |
return true; |
} |
return false; |
@@ -494,7 +494,9 @@ IN_PROC_BROWSER_TEST_F(OAuth2Test, DISABLED_MergeSession) { |
EXPECT_EQ(GetOAuthStatusFromLocalState(account_id), |
user_manager::User::OAUTH2_TOKEN_STATUS_VALID); |
- EXPECT_TRUE(TryToLogin(kTestGaiaId, kTestEmail, kTestAccountPassword)); |
+ EXPECT_TRUE( |
+ TryToLogin(AccountId::FromUserEmailGaiaId(kTestEmail, kTestGaiaId), |
+ kTestAccountPassword)); |
// Wait for the session merge to finish. |
WaitForMergeSessionCompletion(OAuth2LoginManager::SESSION_RESTORE_FAILED); |