Index: components/user_manager/fake_user_manager.cc |
diff --git a/components/user_manager/fake_user_manager.cc b/components/user_manager/fake_user_manager.cc |
index 4caa1b335f448238966ff2ebd5b792eb179242ac..39f63f1eae4493c0525b1378d7ec5ffa204caa24 100644 |
--- a/components/user_manager/fake_user_manager.cc |
+++ b/components/user_manager/fake_user_manager.cc |
@@ -31,30 +31,28 @@ |
FakeUserManager::FakeUserManager() |
: UserManagerBase(new FakeTaskRunner(), new FakeTaskRunner()), |
- primary_user_(nullptr) {} |
+ primary_user_(NULL), |
+ owner_email_(std::string()) { |
+} |
FakeUserManager::~FakeUserManager() { |
} |
-const user_manager::User* FakeUserManager::AddUser( |
- const AccountId& account_id) { |
- return AddUserWithAffiliation(account_id, false); |
+const user_manager::User* FakeUserManager::AddUser(const std::string& email) { |
+ return AddUserWithAffiliation(email, false); |
} |
const user_manager::User* FakeUserManager::AddUserWithAffiliation( |
- const AccountId& account_id, |
- bool is_affiliated) { |
- user_manager::User* user = user_manager::User::CreateRegularUser(account_id); |
+ const std::string& email, bool is_affiliated) { |
+ user_manager::User* user = user_manager::User::CreateRegularUser(email); |
user->set_affiliation(is_affiliated); |
users_.push_back(user); |
return user; |
} |
-void FakeUserManager::RemoveUserFromList(const AccountId& account_id) { |
+void FakeUserManager::RemoveUserFromList(const std::string& email) { |
user_manager::UserList::iterator it = users_.begin(); |
- // TODO (alemate): Chenge this to GetAccountId(), once a real AccountId is |
- // passed. crbug.com/546876 |
- while (it != users_.end() && (*it)->GetEmail() != account_id.GetUserEmail()) |
+ while (it != users_.end() && (*it)->email() != email) |
++it; |
if (it != users_.end()) { |
delete *it; |
@@ -81,7 +79,7 @@ |
return logged_in_users_; |
} |
-void FakeUserManager::UserLoggedIn(const AccountId& account_id, |
+void FakeUserManager::UserLoggedIn(const std::string& email, |
const std::string& username_hash, |
bool browser_restart) { |
for (user_manager::UserList::const_iterator it = users_.begin(); |
@@ -99,20 +97,17 @@ |
} |
user_manager::User* FakeUserManager::GetActiveUserInternal() const { |
- if (!users_.empty()) { |
- if (active_account_id_.is_valid()) { |
+ if (users_.size()) { |
+ if (!active_user_id_.empty()) { |
for (user_manager::UserList::const_iterator it = users_.begin(); |
it != users_.end(); ++it) { |
- // TODO (alemate): Chenge this to GetAccountId(), once a real AccountId |
- // is |
- // passed. crbug.com/546876 |
- if ((*it)->GetEmail() == active_account_id_.GetUserEmail()) |
+ if ((*it)->email() == active_user_id_) |
return *it; |
} |
} |
return users_[0]; |
} |
- return nullptr; |
+ return NULL; |
} |
const user_manager::User* FakeUserManager::GetActiveUser() const { |
@@ -123,15 +118,14 @@ |
return GetActiveUserInternal(); |
} |
-void FakeUserManager::SwitchActiveUser(const AccountId& account_id) {} |
- |
-void FakeUserManager::SaveUserDisplayName(const AccountId& account_id, |
+void FakeUserManager::SwitchActiveUser(const std::string& email) { |
+} |
+ |
+void FakeUserManager::SaveUserDisplayName(const std::string& username, |
const base::string16& display_name) { |
for (user_manager::UserList::iterator it = users_.begin(); it != users_.end(); |
++it) { |
- // TODO (alemate): Chenge this to GetAccountId(), once a real AccountId is |
- // passed. crbug.com/546876 |
- if ((*it)->GetEmail() == account_id.GetUserEmail()) { |
+ if ((*it)->email() == username) { |
(*it)->set_display_name(display_name); |
return; |
} |
@@ -146,38 +140,36 @@ |
return users_; |
} |
-const AccountId& FakeUserManager::GetOwnerAccountId() const { |
- return owner_account_id_; |
-} |
- |
-bool FakeUserManager::IsKnownUser(const AccountId& account_id) const { |
+const std::string& FakeUserManager::GetOwnerEmail() const { |
+ return owner_email_; |
+} |
+ |
+bool FakeUserManager::IsKnownUser(const std::string& email) const { |
return true; |
} |
const user_manager::User* FakeUserManager::FindUser( |
- const AccountId& account_id) const { |
+ const std::string& email) const { |
const user_manager::UserList& users = GetUsers(); |
for (user_manager::UserList::const_iterator it = users.begin(); |
it != users.end(); ++it) { |
- // TODO (alemate): Chenge this to GetAccountId(), once a real AccountId is |
- // passed. crbug.com/546876 |
- if ((*it)->GetEmail() == account_id.GetUserEmail()) |
+ if ((*it)->email() == email) |
return *it; |
} |
- return nullptr; |
+ return NULL; |
} |
user_manager::User* FakeUserManager::FindUserAndModify( |
- const AccountId& account_id) { |
- return nullptr; |
+ const std::string& email) { |
+ return NULL; |
} |
const user_manager::User* FakeUserManager::GetLoggedInUser() const { |
- return nullptr; |
+ return NULL; |
} |
user_manager::User* FakeUserManager::GetLoggedInUser() { |
- return nullptr; |
+ return NULL; |
} |
const user_manager::User* FakeUserManager::GetPrimaryUser() const { |
@@ -185,12 +177,12 @@ |
} |
base::string16 FakeUserManager::GetUserDisplayName( |
- const AccountId& account_id) const { |
+ const std::string& username) const { |
return base::string16(); |
} |
std::string FakeUserManager::GetUserDisplayEmail( |
- const AccountId& account_id) const { |
+ const std::string& username) const { |
return std::string(); |
} |
@@ -246,7 +238,7 @@ |
} |
bool FakeUserManager::IsUserNonCryptohomeDataEphemeral( |
- const AccountId& account_id) const { |
+ const std::string& email) const { |
return false; |
} |
@@ -264,23 +256,23 @@ |
} |
PrefService* FakeUserManager::GetLocalState() const { |
- return nullptr; |
+ return NULL; |
} |
bool FakeUserManager::IsEnterpriseManaged() const { |
return false; |
} |
-bool FakeUserManager::IsDemoApp(const AccountId& account_id) const { |
- return false; |
-} |
- |
-bool FakeUserManager::IsKioskApp(const AccountId& account_id) const { |
+bool FakeUserManager::IsDemoApp(const std::string& user_id) const { |
+ return false; |
+} |
+ |
+bool FakeUserManager::IsKioskApp(const std::string& user_id) const { |
return false; |
} |
bool FakeUserManager::IsPublicAccountMarkedForRemoval( |
- const AccountId& account_id) const { |
+ const std::string& user_id) const { |
return false; |
} |