Chromium Code Reviews| Index: chrome/browser/chromeos/login/users/fake_chrome_user_manager.cc |
| diff --git a/chrome/browser/chromeos/login/users/fake_chrome_user_manager.cc b/chrome/browser/chromeos/login/users/fake_chrome_user_manager.cc |
| index 7191f0c10b7bc0106f7e5bdc304d78bdae1ae85f..e38afbfdc57fe5a096c580e7e96311839039b7f3 100644 |
| --- a/chrome/browser/chromeos/login/users/fake_chrome_user_manager.cc |
| +++ b/chrome/browser/chromeos/login/users/fake_chrome_user_manager.cc |
| @@ -19,25 +19,25 @@ class FakeSupervisedUserManager; |
| FakeChromeUserManager::FakeChromeUserManager() |
| : supervised_user_manager_(new FakeSupervisedUserManager), |
| + owner_id_(EmptyAccountId()), |
|
achuithb
2015/10/23 00:08:51
in class member initialization
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| bootstrap_manager_(NULL), |
| - multi_profile_user_controller_(NULL) { |
| -} |
| + multi_profile_user_controller_(NULL) {} |
| FakeChromeUserManager::~FakeChromeUserManager() { |
| } |
| const user_manager::User* FakeChromeUserManager::AddUser( |
| - const std::string& email) { |
| - return AddUserWithAffiliation(email, false); |
| + const AccountId& account) { |
|
achuithb
2015/10/23 00:08:51
account_id? here and below
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| + return AddUserWithAffiliation(account, false); |
| } |
| const user_manager::User* FakeChromeUserManager::AddUserWithAffiliation( |
| - const std::string& email, |
| + const AccountId& account, |
| bool is_affiliated) { |
| - user_manager::User* user = user_manager::User::CreateRegularUser(email); |
| + user_manager::User* user = user_manager::User::CreateRegularUser(account); |
| user->set_affiliation(is_affiliated); |
| user->set_username_hash( |
| - ProfileHelper::GetUserIdHashByUserIdForTesting(email)); |
| + ProfileHelper::GetUserIdHashByUserIdForTesting(account.GetUserEmail())); |
| user->SetStubImage(user_manager::UserImage( |
| *ResourceBundle::GetSharedInstance().GetImageSkiaNamed( |
| IDR_PROFILE_PICTURE_LOADING)), |
| @@ -47,10 +47,11 @@ const user_manager::User* FakeChromeUserManager::AddUserWithAffiliation( |
| } |
| const user_manager::User* FakeChromeUserManager::AddPublicAccountUser( |
| - const std::string& email) { |
| - user_manager::User* user = user_manager::User::CreatePublicAccountUser(email); |
| + const AccountId& account) { |
| + user_manager::User* user = |
| + user_manager::User::CreatePublicAccountUser(account); |
| user->set_username_hash( |
| - ProfileHelper::GetUserIdHashByUserIdForTesting(email)); |
| + ProfileHelper::GetUserIdHashByUserIdForTesting(account.GetUserEmail())); |
| user->SetStubImage(user_manager::UserImage( |
| *ResourceBundle::GetSharedInstance().GetImageSkiaNamed( |
| IDR_PROFILE_PICTURE_LOADING)), |
| @@ -60,16 +61,17 @@ const user_manager::User* FakeChromeUserManager::AddPublicAccountUser( |
| } |
| void FakeChromeUserManager::AddKioskAppUser( |
| - const std::string& kiosk_app_username) { |
| + const AccountId& kiosk_app_user_id) { |
| user_manager::User* user = |
| - user_manager::User::CreateKioskAppUser(kiosk_app_username); |
| - user->set_username_hash( |
| - ProfileHelper::GetUserIdHashByUserIdForTesting(kiosk_app_username)); |
| + user_manager::User::CreateKioskAppUser(kiosk_app_user_id); |
| + user->set_username_hash(ProfileHelper::GetUserIdHashByUserIdForTesting( |
| + kiosk_app_user_id.GetUserEmail())); |
| users_.push_back(user); |
| } |
| -void FakeChromeUserManager::LoginUser(const std::string& email) { |
| - UserLoggedIn(email, ProfileHelper::GetUserIdHashByUserIdForTesting(email), |
| +void FakeChromeUserManager::LoginUser(const AccountId& account) { |
| + UserLoggedIn(account, ProfileHelper::GetUserIdHashByUserIdForTesting( |
| + account.GetUserEmail()), |
| false /* browser_restart */); |
| } |
| @@ -87,58 +89,57 @@ SupervisedUserManager* FakeChromeUserManager::GetSupervisedUserManager() { |
| } |
| UserImageManager* FakeChromeUserManager::GetUserImageManager( |
| - const std::string& /* user_id */) { |
| + const AccountId& /* user_id */) { |
|
achuithb
2015/10/23 00:08:51
account_id
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| return nullptr; |
| } |
| -void FakeChromeUserManager::SetUserFlow(const std::string& email, |
| +void FakeChromeUserManager::SetUserFlow(const AccountId& account, |
| UserFlow* flow) { |
| - ResetUserFlow(email); |
| - specific_flows_[email] = flow; |
| + ResetUserFlow(account); |
| + specific_flows_[account] = flow; |
| } |
| UserFlow* FakeChromeUserManager::GetCurrentUserFlow() const { |
| if (!IsUserLoggedIn()) |
| return GetDefaultUserFlow(); |
| - return GetUserFlow(GetLoggedInUser()->email()); |
| + return GetUserFlow(GetLoggedInUser()->GetUserID()); |
| } |
| -UserFlow* FakeChromeUserManager::GetUserFlow(const std::string& email) const { |
| - FlowMap::const_iterator it = specific_flows_.find(email); |
| +UserFlow* FakeChromeUserManager::GetUserFlow(const AccountId& account) const { |
| + FlowMap::const_iterator it = specific_flows_.find(account); |
| if (it != specific_flows_.end()) |
| return it->second; |
| return GetDefaultUserFlow(); |
| } |
| -void FakeChromeUserManager::ResetUserFlow(const std::string& email) { |
| - FlowMap::iterator it = specific_flows_.find(email); |
| +void FakeChromeUserManager::ResetUserFlow(const AccountId& account) { |
| + FlowMap::iterator it = specific_flows_.find(account); |
| if (it != specific_flows_.end()) { |
| delete it->second; |
| specific_flows_.erase(it); |
| } |
| } |
| -void FakeChromeUserManager::SwitchActiveUser(const std::string& email) { |
| - active_user_id_ = email; |
| +void FakeChromeUserManager::SwitchActiveUser(const AccountId& account) { |
| + active_user_id_ = account; |
| ProfileHelper::Get()->ActiveUserHashChanged( |
| - ProfileHelper::GetUserIdHashByUserIdForTesting(email)); |
| + ProfileHelper::GetUserIdHashByUserIdForTesting(account.GetUserEmail())); |
| if (!users_.empty() && !active_user_id_.empty()) { |
| for (user_manager::User* user : users_) |
| - user->set_is_active(user->email() == active_user_id_); |
| + user->set_is_active(user->GetUserID() == active_user_id_); |
| } |
| } |
| -const std::string& FakeChromeUserManager::GetOwnerEmail() const { |
| - return owner_email_; |
| +const AccountId& FakeChromeUserManager::GetOwnerId() const { |
| + return owner_id_; |
| } |
| void FakeChromeUserManager::SessionStarted() { |
| } |
| void FakeChromeUserManager::RemoveUser( |
| - const std::string& email, |
| - user_manager::RemoveUserDelegate* delegate) { |
| -} |
| + const AccountId& account, |
| + user_manager::RemoveUserDelegate* delegate) {} |
| user_manager::UserList |
| FakeChromeUserManager::GetUsersAllowedForSupervisedUsersCreation() const { |
| @@ -181,15 +182,14 @@ UserFlow* FakeChromeUserManager::GetDefaultUserFlow() const { |
| } |
| bool FakeChromeUserManager::FindKnownUserPrefs( |
| - const user_manager::UserID& user_id, |
| + const AccountId& user_id, |
| const base::DictionaryValue** out_value) { |
| return false; |
| } |
| void FakeChromeUserManager::UpdateKnownUserPrefs( |
| - const user_manager::UserID& user_id, |
| + const AccountId& user_id, |
| const base::DictionaryValue& values, |
| - bool clear) { |
| -} |
| + bool clear) {} |
| } // namespace chromeos |