Chromium Code Reviews| Index: chrome/browser/chromeos/login/users/mock_user_manager.cc |
| diff --git a/chrome/browser/chromeos/login/users/mock_user_manager.cc b/chrome/browser/chromeos/login/users/mock_user_manager.cc |
| index 0e79f653b2c2480f152826e8634aa0dff21f33ff..523e0f6b0711fd847f9f67fc2e55217f90304dbf 100644 |
| --- a/chrome/browser/chromeos/login/users/mock_user_manager.cc |
| +++ b/chrome/browser/chromeos/login/users/mock_user_manager.cc |
| @@ -56,8 +56,9 @@ user_manager::UserList MockUserManager::GetUnlockUsers() const { |
| return user_list_; |
| } |
| -const std::string& MockUserManager::GetOwnerEmail() const { |
| - return GetLoggedInUser()->email(); |
| +const AccountId& MockUserManager::GetOwnerAccountId() const { |
| + temporary_owner_account_id_ = GetLoggedInUser()->GetAccountId(); |
| + return temporary_owner_account_id_; |
| } |
| const user_manager::User* MockUserManager::GetActiveUser() const { |
| @@ -81,7 +82,7 @@ MultiProfileUserController* MockUserManager::GetMultiProfileUserController() { |
| } |
| UserImageManager* MockUserManager::GetUserImageManager( |
| - const std::string& user_id) { |
| + const AccountId& account_id) { |
| return NULL; |
| } |
| @@ -90,43 +91,44 @@ SupervisedUserManager* MockUserManager::GetSupervisedUserManager() { |
| } |
| // Creates a new User instance. |
| -void MockUserManager::SetActiveUser(const std::string& email) { |
| +void MockUserManager::SetActiveUser(const AccountId& account_id) { |
| ClearUserList(); |
| - AddUser(email); |
| + AddUser(account_id); |
| } |
| UserFlow* MockUserManager::GetCurrentUserFlow() const { |
| return user_flow_.get(); |
| } |
| -UserFlow* MockUserManager::GetUserFlow(const std::string&) const { |
| +UserFlow* MockUserManager::GetUserFlow(const AccountId&) const { |
| return user_flow_.get(); |
| } |
| user_manager::User* MockUserManager::CreatePublicAccountUser( |
| - const std::string& email) { |
| + const AccountId& account_id) { |
| ClearUserList(); |
| - user_manager::User* user = user_manager::User::CreatePublicAccountUser(email); |
| + user_manager::User* user = |
| + user_manager::User::CreatePublicAccountUser(account_id); |
| user_list_.push_back(user); |
| ProfileHelper::Get()->SetProfileToUserMappingForTesting(user); |
| return user_list_.back(); |
| } |
| user_manager::User* MockUserManager::CreateKioskAppUser( |
| - const std::string& email) { |
| + const AccountId& account_id) { |
| ClearUserList(); |
| - user_list_.push_back(user_manager::User::CreateKioskAppUser(email)); |
| + user_list_.push_back(user_manager::User::CreateKioskAppUser(account_id)); |
| ProfileHelper::Get()->SetProfileToUserMappingForTesting(user_list_.back()); |
| return user_list_.back(); |
| } |
| -void MockUserManager::AddUser(const std::string& email) { |
| - AddUserWithAffiliation(email, false); |
| +void MockUserManager::AddUser(const AccountId& account_id) { |
| + AddUserWithAffiliation(account_id, false); |
| } |
| -void MockUserManager::AddUserWithAffiliation(const std::string& email, |
| +void MockUserManager::AddUserWithAffiliation(const AccountId& account_id, |
| bool is_affiliated) { |
| - user_manager::User* user = user_manager::User::CreateRegularUser(email); |
| + user_manager::User* user = user_manager::User::CreateRegularUser(account_id); |
| user->set_affiliation(is_affiliated); |
| user_list_.push_back(user); |
| ProfileHelper::Get()->SetProfileToUserMappingForTesting(user); |
| @@ -142,7 +144,7 @@ void MockUserManager::ClearUserList() { |
| bool MockUserManager::ShouldReportUser(const std::string& user_id) const { |
| for (const auto& user : user_list_) { |
| - if (user->GetUserID() == user_id) { |
| + if (user->email() == user_id) { |
|
achuithb
2015/10/28 23:11:45
drop {}
Alexander Alekseev
2015/10/29 02:00:41
Done.
|
| return user->is_affiliated(); |
| } |
| } |