Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6585)

Unified Diff: chrome/browser/chromeos/login/users/mock_user_manager.cc

Issue 1412813003: This CL replaces user_manager::UserID with AccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@468875--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Create-AccountID-structure-part2--user_names
Patch Set: Update after review. Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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();
}
}

Powered by Google App Engine
This is Rietveld 408576698