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

Unified Diff: components/user_manager/fake_user_manager.h

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: Fix Win GN build. Created 5 years, 1 month 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
« no previous file with comments | « components/user_manager/empty_user_info.cc ('k') | components/user_manager/fake_user_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/user_manager/fake_user_manager.h
diff --git a/components/user_manager/fake_user_manager.h b/components/user_manager/fake_user_manager.h
index a40eed6e800818d468aff2d7e88e3b4ae1f2f41e..77f1649f45c689424c85e13b605cfdaaf4ad71c3 100644
--- a/components/user_manager/fake_user_manager.h
+++ b/components/user_manager/fake_user_manager.h
@@ -9,6 +9,7 @@
#include <string>
#include "base/memory/scoped_ptr.h"
+#include "components/signin/core/account_id/account_id.h"
#include "components/user_manager/user.h"
#include "components/user_manager/user_manager_base.h"
@@ -23,15 +24,16 @@ class USER_MANAGER_EXPORT FakeUserManager : public UserManagerBase {
// Create and add a new user. Created user is not affiliated with the domain,
// that owns the device.
- virtual const user_manager::User* AddUser(const std::string& email);
+ virtual const user_manager::User* AddUser(const AccountId& account_id);
// The same as AddUser() but allows to specify user affiliation with the
// domain, that owns the device.
virtual const user_manager::User* AddUserWithAffiliation(
- const std::string& email, bool is_affiliated);
+ const AccountId& account_id,
+ bool is_affiliated);
// Calculates the user name hash and calls UserLoggedIn to login a user.
- void LoginUser(const std::string& email);
+ void LoginUser(const AccountId& account_id);
// UserManager overrides.
const user_manager::UserList& GetUsers() const override;
@@ -39,42 +41,43 @@ class USER_MANAGER_EXPORT FakeUserManager : public UserManagerBase {
const user_manager::UserList& GetLoggedInUsers() const override;
// Set the user as logged in.
- void UserLoggedIn(const std::string& email,
+ void UserLoggedIn(const AccountId& account_id,
const std::string& username_hash,
bool browser_restart) override;
const user_manager::User* GetActiveUser() const override;
user_manager::User* GetActiveUser() override;
- void SwitchActiveUser(const std::string& email) override;
- void SaveUserDisplayName(const std::string& username,
+ void SwitchActiveUser(const AccountId& account_id) override;
+ void SaveUserDisplayName(const AccountId& account_id,
const base::string16& display_name) override;
// Not implemented.
- void UpdateUserAccountData(const std::string& user_id,
+ void UpdateUserAccountData(const AccountId& account_id,
const UserAccountData& account_data) override {}
void Shutdown() override {}
const user_manager::UserList& GetLRULoggedInUsers() const override;
user_manager::UserList GetUnlockUsers() const override;
- const std::string& GetOwnerEmail() const override;
+ const AccountId& GetOwnerAccountId() const override;
void SessionStarted() override {}
- void RemoveUser(const std::string& email,
+ void RemoveUser(const AccountId& account_id,
user_manager::RemoveUserDelegate* delegate) override {}
- void RemoveUserFromList(const std::string& email) override;
- bool IsKnownUser(const std::string& email) const override;
- const user_manager::User* FindUser(const std::string& email) const override;
- user_manager::User* FindUserAndModify(const std::string& email) override;
+ void RemoveUserFromList(const AccountId& account_id) override;
+ bool IsKnownUser(const AccountId& account_id) const override;
+ const user_manager::User* FindUser(
+ const AccountId& account_id) const override;
+ user_manager::User* FindUserAndModify(const AccountId& account_id) override;
const user_manager::User* GetLoggedInUser() const override;
user_manager::User* GetLoggedInUser() override;
const user_manager::User* GetPrimaryUser() const override;
void SaveUserOAuthStatus(
- const std::string& username,
+ const AccountId& account_id,
user_manager::User::OAuthTokenStatus oauth_token_status) override {}
- void SaveForceOnlineSignin(const std::string& user_id,
+ void SaveForceOnlineSignin(const AccountId& account_id,
bool force_online_signin) override {}
- base::string16 GetUserDisplayName(const std::string& username) const override;
- void SaveUserDisplayEmail(const std::string& username,
+ base::string16 GetUserDisplayName(const AccountId& account_id) const override;
+ void SaveUserDisplayEmail(const AccountId& account_id,
const std::string& display_email) override {}
- std::string GetUserDisplayEmail(const std::string& username) const override;
+ std::string GetUserDisplayEmail(const AccountId& account_id) const override;
bool IsCurrentUserOwner() const override;
bool IsCurrentUserNew() const override;
bool IsCurrentUserNonCryptohomeDataEphemeral() const override;
@@ -88,7 +91,7 @@ class USER_MANAGER_EXPORT FakeUserManager : public UserManagerBase {
bool IsLoggedInAsStub() const override;
bool IsSessionStarted() const override;
bool IsUserNonCryptohomeDataEphemeral(
- const std::string& email) const override;
+ const AccountId& account_id) const override;
void AddObserver(Observer* obs) override {}
void RemoveObserver(Observer* obs) override {}
void AddSessionStateObserver(UserSessionStateObserver* obs) override {}
@@ -101,37 +104,36 @@ class USER_MANAGER_EXPORT FakeUserManager : public UserManagerBase {
const std::string& GetApplicationLocale() const override;
PrefService* GetLocalState() const override;
void HandleUserOAuthTokenStatusChange(
- const std::string& user_id,
+ const AccountId& account_id,
user_manager::User::OAuthTokenStatus status) const override {}
bool IsEnterpriseManaged() const override;
- void LoadPublicAccounts(std::set<std::string>* public_sessions_set) override {
- }
+ void LoadPublicAccounts(std::set<AccountId>* public_sessions_set) override {}
void PerformPreUserListLoadingActions() override {}
void PerformPostUserListLoadingActions() override {}
void PerformPostUserLoggedInActions(bool browser_restart) override {}
- bool IsDemoApp(const std::string& user_id) const override;
- bool IsKioskApp(const std::string& user_id) const override;
+ bool IsDemoApp(const AccountId& account_id) const override;
+ bool IsKioskApp(const AccountId& account_id) const override;
bool IsPublicAccountMarkedForRemoval(
- const std::string& user_id) const override;
+ const AccountId& account_id) const override;
void DemoAccountLoggedIn() override {}
- void KioskAppLoggedIn(const std::string& app_id) override {}
+ void KioskAppLoggedIn(const AccountId& kiosk_app_account_id) override {}
void PublicAccountUserLoggedIn(user_manager::User* user) override {}
- void SupervisedUserLoggedIn(const std::string& user_id) override {}
- void OnUserRemoved(const std::string& user_id) override {}
+ void SupervisedUserLoggedIn(const AccountId& account_id) override {}
+ void OnUserRemoved(const AccountId& account_id) override {}
protected:
user_manager::User* primary_user_;
// If set this is the active user. If empty, the first created user is the
// active user.
- std::string active_user_id_;
+ AccountId active_account_id_ = EmptyAccountId();
private:
// We use this internal function for const-correctness.
user_manager::User* GetActiveUserInternal() const;
- // stub, always empty string.
- std::string owner_email_;
+ // stub, always empty.
+ AccountId owner_account_id_ = EmptyAccountId();
DISALLOW_COPY_AND_ASSIGN(FakeUserManager);
};
« no previous file with comments | « components/user_manager/empty_user_info.cc ('k') | components/user_manager/fake_user_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698