Index: chrome/browser/chromeos/login/existing_user_controller_auto_login_unittest.cc |
diff --git a/chrome/browser/chromeos/login/existing_user_controller_auto_login_unittest.cc b/chrome/browser/chromeos/login/existing_user_controller_auto_login_unittest.cc |
index 3d6ff7fc0a5b7e1841b072cb602bdcd32651ae9e..ce4be3305ccc87e49a0eae390e71a9c5e909e889 100644 |
--- a/chrome/browser/chromeos/login/existing_user_controller_auto_login_unittest.cc |
+++ b/chrome/browser/chromeos/login/existing_user_controller_auto_login_unittest.cc |
@@ -101,11 +101,12 @@ class ExistingUserControllerAutoLoginTest : public ::testing::Test { |
return existing_user_controller()->auto_login_timer_.get(); |
} |
- const std::string& auto_login_username() const { |
- return existing_user_controller()->public_session_auto_login_username_; |
+ const AccountId& auto_login_account_id() const { |
+ return existing_user_controller()->public_session_auto_login_account_id_; |
} |
- void set_auto_login_username(const std::string& username) { |
- existing_user_controller()->public_session_auto_login_username_ = username; |
+ void set_auto_login_account_id(const AccountId& account_id) { |
+ existing_user_controller()->public_session_auto_login_account_id_ = |
+ account_id; |
} |
int auto_login_delay() const { |
@@ -157,19 +158,19 @@ class ExistingUserControllerAutoLoginTest : public ::testing::Test { |
TEST_F(ExistingUserControllerAutoLoginTest, StartAutoLoginTimer) { |
// Timer shouldn't start until signin screen is ready. |
- set_auto_login_username(auto_login_account_id_.GetUserEmail()); |
+ set_auto_login_account_id(auto_login_account_id_); |
set_auto_login_delay(kAutoLoginDelay2); |
existing_user_controller()->StartPublicSessionAutoLoginTimer(); |
EXPECT_FALSE(auto_login_timer()); |
// Timer shouldn't start if the policy isn't set. |
- set_auto_login_username(""); |
+ set_auto_login_account_id(EmptyAccountId()); |
existing_user_controller()->OnSigninScreenReady(); |
existing_user_controller()->StartPublicSessionAutoLoginTimer(); |
EXPECT_FALSE(auto_login_timer()); |
// Timer shouldn't fire in the middle of a login attempt. |
- set_auto_login_username(auto_login_account_id_.GetUserEmail()); |
+ set_auto_login_account_id(auto_login_account_id_); |
set_is_login_in_progress(true); |
existing_user_controller()->StartPublicSessionAutoLoginTimer(); |
EXPECT_FALSE(auto_login_timer()); |
@@ -185,7 +186,7 @@ TEST_F(ExistingUserControllerAutoLoginTest, StartAutoLoginTimer) { |
TEST_F(ExistingUserControllerAutoLoginTest, StopAutoLoginTimer) { |
existing_user_controller()->OnSigninScreenReady(); |
- set_auto_login_username(auto_login_account_id_.GetUserEmail()); |
+ set_auto_login_account_id(auto_login_account_id_); |
set_auto_login_delay(kAutoLoginDelay2); |
existing_user_controller()->StartPublicSessionAutoLoginTimer(); |
@@ -199,7 +200,7 @@ TEST_F(ExistingUserControllerAutoLoginTest, StopAutoLoginTimer) { |
TEST_F(ExistingUserControllerAutoLoginTest, ResetAutoLoginTimer) { |
existing_user_controller()->OnSigninScreenReady(); |
- set_auto_login_username(auto_login_account_id_.GetUserEmail()); |
+ set_auto_login_account_id(auto_login_account_id_); |
// Timer starts off not running. |
EXPECT_FALSE(auto_login_timer()); |
@@ -233,14 +234,14 @@ TEST_F(ExistingUserControllerAutoLoginTest, ConfigureAutoLogin) { |
ConfigureAutoLogin(); |
EXPECT_FALSE(auto_login_timer()); |
EXPECT_EQ(auto_login_delay(), 0); |
- EXPECT_EQ(auto_login_username(), ""); |
+ EXPECT_EQ(auto_login_account_id(), EmptyAccountId()); |
// Timer shouldn't start when the delay alone is set. |
SetAutoLoginSettings("", kAutoLoginDelay1); |
ConfigureAutoLogin(); |
EXPECT_FALSE(auto_login_timer()); |
EXPECT_EQ(auto_login_delay(), kAutoLoginDelay1); |
- EXPECT_EQ(auto_login_username(), ""); |
+ EXPECT_EQ(auto_login_account_id(), EmptyAccountId()); |
// Timer should start when the account ID is set. |
SetAutoLoginSettings(auto_login_user_id_, kAutoLoginDelay1); |
@@ -250,7 +251,7 @@ TEST_F(ExistingUserControllerAutoLoginTest, ConfigureAutoLogin) { |
EXPECT_EQ(auto_login_timer()->GetCurrentDelay().InMilliseconds(), |
kAutoLoginDelay1); |
EXPECT_EQ(auto_login_delay(), kAutoLoginDelay1); |
- EXPECT_EQ(auto_login_username(), auto_login_account_id_.GetUserEmail()); |
+ EXPECT_EQ(auto_login_account_id(), auto_login_account_id_); |
// Timer should restart when the delay is changed. |
SetAutoLoginSettings(auto_login_user_id_, kAutoLoginDelay2); |
@@ -260,7 +261,7 @@ TEST_F(ExistingUserControllerAutoLoginTest, ConfigureAutoLogin) { |
EXPECT_EQ(auto_login_timer()->GetCurrentDelay().InMilliseconds(), |
kAutoLoginDelay2); |
EXPECT_EQ(auto_login_delay(), kAutoLoginDelay2); |
- EXPECT_EQ(auto_login_username(), auto_login_account_id_.GetUserEmail()); |
+ EXPECT_EQ(auto_login_account_id(), auto_login_account_id_); |
// Timer should stop when the account ID is unset. |
SetAutoLoginSettings("", kAutoLoginDelay2); |
@@ -269,7 +270,7 @@ TEST_F(ExistingUserControllerAutoLoginTest, ConfigureAutoLogin) { |
EXPECT_FALSE(auto_login_timer()->IsRunning()); |
EXPECT_EQ(auto_login_timer()->GetCurrentDelay().InMilliseconds(), |
kAutoLoginDelay2); |
- EXPECT_EQ(auto_login_username(), ""); |
+ EXPECT_EQ(auto_login_account_id(), EmptyAccountId()); |
EXPECT_EQ(auto_login_delay(), kAutoLoginDelay2); |
} |