Index: chrome/browser/ui/ash/session_state_delegate_chromeos_unittest.cc |
diff --git a/chrome/browser/ui/ash/session_state_delegate_chromeos_unittest.cc b/chrome/browser/ui/ash/session_state_delegate_chromeos_unittest.cc |
index 04a620f30fa0cd2c7948be0b0d4308440171dd2b..6d58b793d30c6d7807c8d616cb51cb456a93e8e0 100644 |
--- a/chrome/browser/ui/ash/session_state_delegate_chromeos_unittest.cc |
+++ b/chrome/browser/ui/ash/session_state_delegate_chromeos_unittest.cc |
@@ -19,7 +19,6 @@ |
#include "chrome/common/pref_names.h" |
#include "chrome/test/base/testing_browser_process.h" |
#include "chrome/test/base/testing_profile_manager.h" |
-#include "components/signin/core/account_id/account_id.h" |
#include "components/user_manager/user_manager.h" |
#include "content/public/test/test_browser_thread_bundle.h" |
#include "net/cert/x509_certificate.h" |
@@ -82,9 +81,9 @@ |
} |
// Add and log in a user to the session. |
- void UserAddedToSession(std::string user) { |
- user_manager()->AddUser(AccountId::FromUserEmail(user)); |
- user_manager()->LoginUser(AccountId::FromUserEmail(user)); |
+ void UserAddedToSession(const std::string& user) { |
+ user_manager()->AddUser(user); |
+ user_manager()->LoginUser(user); |
} |
// Get the active user. |
@@ -102,13 +101,12 @@ |
new TestingProfileManager(TestingBrowserProcess::GetGlobal())); |
ASSERT_TRUE(profile_manager_->SetUp()); |
- const AccountId account_id(AccountId::FromUserEmail(kUser)); |
- const user_manager::User* user = user_manager()->AddUser(account_id); |
+ const std::string user_email(kUser); |
+ const user_manager::User* user = user_manager()->AddUser(user_email); |
// Note that user profiles are created after user login in reality. |
- user_profile_ = |
- profile_manager_->CreateTestingProfile(account_id.GetUserEmail()); |
- user_profile_->set_profile_name(account_id.GetUserEmail()); |
+ user_profile_ = profile_manager_->CreateTestingProfile(user_email); |
+ user_profile_->set_profile_name(user_email); |
chromeos::ProfileHelper::Get()->SetUserToProfileMappingForTesting( |
user, user_profile_); |
} |
@@ -174,8 +172,8 @@ |
InitForMultiProfile(); |
EXPECT_TRUE( |
session_state_delegate()->IsMultiProfileAllowedByPrimaryUserPolicy()); |
- const AccountId account_id(AccountId::FromUserEmail(kUser)); |
- user_manager()->LoginUser(account_id); |
+ const std::string user_email(kUser); |
+ user_manager()->LoginUser(user_email); |
EXPECT_TRUE( |
session_state_delegate()->IsMultiProfileAllowedByPrimaryUserPolicy()); |
@@ -190,12 +188,11 @@ |
TEST_F(SessionStateDelegateChromeOSTest, |
MultiProfileDisallowedByPolicyCertificates) { |
InitForMultiProfile(); |
- const AccountId account_id(AccountId::FromUserEmail(kUser)); |
- user_manager()->LoginUser(account_id); |
- EXPECT_TRUE( |
- session_state_delegate()->IsMultiProfileAllowedByPrimaryUserPolicy()); |
- policy::PolicyCertServiceFactory::SetUsedPolicyCertificates( |
- account_id.GetUserEmail()); |
+ const std::string user_email(kUser); |
+ user_manager()->LoginUser(user_email); |
+ EXPECT_TRUE( |
+ session_state_delegate()->IsMultiProfileAllowedByPrimaryUserPolicy()); |
+ policy::PolicyCertServiceFactory::SetUsedPolicyCertificates(user_email); |
EXPECT_FALSE( |
session_state_delegate()->IsMultiProfileAllowedByPrimaryUserPolicy()); |
@@ -207,8 +204,8 @@ |
TEST_F(SessionStateDelegateChromeOSTest, |
MultiProfileDisallowedByPrimaryUserCertificatesInMemory) { |
InitForMultiProfile(); |
- const AccountId account_id(AccountId::FromUserEmail(kUser)); |
- user_manager()->LoginUser(account_id); |
+ const std::string user_email(kUser); |
+ user_manager()->LoginUser(user_email); |
EXPECT_TRUE( |
session_state_delegate()->IsMultiProfileAllowedByPrimaryUserPolicy()); |
cert_verifier_.reset(new policy::PolicyCertVerifier(base::Closure())); |
@@ -242,8 +239,8 @@ |
EXPECT_TRUE( |
session_state_delegate()->CanAddUserToMultiProfile(&add_user_error)); |
- const AccountId account_id(AccountId::FromUserEmail(kUser)); |
- user_manager()->LoginUser(account_id); |
+ const std::string user_email(kUser); |
+ user_manager()->LoginUser(user_email); |
while (session_state_delegate()->NumberOfLoggedInUsers() < |
session_state_delegate()->GetMaximumNumberOfLoggedInUsers()) { |
UserAddedToSession("bb@b.b"); |
@@ -263,8 +260,8 @@ |
EXPECT_TRUE( |
session_state_delegate()->CanAddUserToMultiProfile(&add_user_error)); |
- const AccountId account_id(AccountId::FromUserEmail(kUser)); |
- user_manager()->LoginUser(account_id); |
+ const std::string user_email(kUser); |
+ user_manager()->LoginUser(user_email); |
UserAddedToSession("bb@b.b"); |
EXPECT_FALSE( |
session_state_delegate()->CanAddUserToMultiProfile(&add_user_error)); |
@@ -280,12 +277,12 @@ |
EXPECT_TRUE( |
session_state_delegate()->CanAddUserToMultiProfile(&add_user_error)); |
- const AccountId account_id(AccountId::FromUserEmail(kUser)); |
- user_manager()->LoginUser(account_id); |
+ const std::string user_email(kUser); |
+ user_manager()->LoginUser(user_email); |
user_profile_->GetPrefs()->SetString( |
prefs::kMultiProfileUserBehavior, |
chromeos::MultiProfileUserController::kBehaviorNotAllowed); |
- user_manager()->AddUser(AccountId::FromUserEmail("bb@b.b")); |
+ user_manager()->AddUser("bb@b.b"); |
EXPECT_FALSE( |
session_state_delegate()->CanAddUserToMultiProfile(&add_user_error)); |
EXPECT_EQ(ash::SessionStateDelegate::ADD_USER_ERROR_NOT_ALLOWED_PRIMARY_USER, |