Chromium Code Reviews| Index: chrome/browser/chromeos/login/users/chrome_user_manager_impl.h |
| diff --git a/chrome/browser/chromeos/login/users/chrome_user_manager_impl.h b/chrome/browser/chromeos/login/users/chrome_user_manager_impl.h |
| index dbbf3f5be30ab16c2635959a4b7e2b4824029867..d2fea96e21332cae111d2ea4b98e4911f0445ca9 100644 |
| --- a/chrome/browser/chromeos/login/users/chrome_user_manager_impl.h |
| +++ b/chrome/browser/chromeos/login/users/chrome_user_manager_impl.h |
| @@ -27,6 +27,7 @@ |
| #include "chrome/browser/chromeos/policy/device_local_account_policy_service.h" |
| #include "chrome/browser/chromeos/settings/cros_settings.h" |
| #include "chrome/browser/chromeos/settings/device_settings_service.h" |
| +#include "components/signin/core/account_id/account_id.h" |
| #include "components/user_manager/user.h" |
| #include "content/public/browser/notification_observer.h" |
| #include "content/public/browser/notification_registrar.h" |
| @@ -69,12 +70,12 @@ class ChromeUserManagerImpl |
| // UserManagerInterface implementation: |
| BootstrapManager* GetBootstrapManager() override; |
| MultiProfileUserController* GetMultiProfileUserController() override; |
| - UserImageManager* GetUserImageManager(const std::string& user_id) override; |
| + UserImageManager* GetUserImageManager(const AccountId& user_id) override; |
|
achuithb
2015/10/23 00:08:51
account_id here and elsewhere
Alexander Alekseev
2015/10/23 09:11:22
Done.
|
| SupervisedUserManager* GetSupervisedUserManager() override; |
| UserFlow* GetCurrentUserFlow() const override; |
| - UserFlow* GetUserFlow(const std::string& user_id) const override; |
| - void SetUserFlow(const std::string& user_id, UserFlow* flow) override; |
| - void ResetUserFlow(const std::string& user_id) override; |
| + UserFlow* GetUserFlow(const AccountId& user_id) const override; |
| + void SetUserFlow(const AccountId& user_id, UserFlow* flow) override; |
| + void ResetUserFlow(const AccountId& user_id) override; |
| // UserManager implementation: |
| void Shutdown() override; |
| @@ -84,13 +85,13 @@ class ChromeUserManagerImpl |
| user_manager::UserList GetUnlockUsers() const override; |
| void SessionStarted() override; |
| void SaveUserOAuthStatus( |
| - const std::string& user_id, |
| + const AccountId& user_id, |
| user_manager::User::OAuthTokenStatus oauth_token_status) override; |
| - void SaveUserDisplayName(const std::string& user_id, |
| + void SaveUserDisplayName(const AccountId& user_id, |
| const base::string16& display_name) override; |
| bool CanCurrentUserLock() const override; |
| bool IsUserNonCryptohomeDataEphemeral( |
| - const std::string& user_id) const override; |
| + const AccountId& user_id) const override; |
| bool AreSupervisedUsersAllowed() const override; |
| // content::NotificationObserver implementation. |
| @@ -115,7 +116,7 @@ class ChromeUserManagerImpl |
| // UserManagerBase implementation: |
| bool AreEphemeralUsersEnabled() const override; |
| - void OnUserRemoved(const std::string& user_id) override; |
| + void OnUserRemoved(const AccountId& user_id) override; |
| // ChromeUserManager implementation: |
| bool ShouldReportUser(const std::string& user_id) const override; |
| @@ -127,31 +128,30 @@ class ChromeUserManagerImpl |
| const std::string& GetApplicationLocale() const override; |
| PrefService* GetLocalState() const override; |
| void HandleUserOAuthTokenStatusChange( |
| - const std::string& user_id, |
| + const AccountId& user_id, |
| user_manager::User::OAuthTokenStatus status) const override; |
| bool IsEnterpriseManaged() const override; |
| - void LoadPublicAccounts(std::set<std::string>* users_set) override; |
| + void LoadPublicAccounts(std::set<AccountId>* users_set) override; |
| void NotifyOnLogin() override; |
| void NotifyUserAddedToSession(const user_manager::User* added_user, |
| bool user_switch_pending) override; |
| void PerformPreUserListLoadingActions() override; |
| void PerformPostUserListLoadingActions() override; |
| void PerformPostUserLoggedInActions(bool browser_restart) override; |
| - void RemoveNonCryptohomeData(const std::string& user_id) override; |
| - void RemoveUserInternal(const std::string& user_email, |
| + void RemoveNonCryptohomeData(const AccountId& user_id) override; |
| + void RemoveUserInternal(const AccountId& user_email, |
| user_manager::RemoveUserDelegate* delegate) override; |
| - bool IsDemoApp(const std::string& user_id) const override; |
| - bool IsKioskApp(const std::string& user_id) const override; |
| - bool IsPublicAccountMarkedForRemoval( |
| - const std::string& user_id) const override; |
| + bool IsDemoApp(const AccountId& user_id) const override; |
| + bool IsKioskApp(const AccountId& user_id) const override; |
| + bool IsPublicAccountMarkedForRemoval(const AccountId& user_id) const override; |
| void DemoAccountLoggedIn() override; |
| void GuestUserLoggedIn() override; |
| - void KioskAppLoggedIn(const std::string& app_id) override; |
| + void KioskAppLoggedIn(const AccountId& app_id) override; |
| void PublicAccountUserLoggedIn(user_manager::User* user) override; |
| - void RegularUserLoggedIn(const std::string& user_id) override; |
| - void RegularUserLoggedInAsEphemeral(const std::string& user_id) override; |
| - void SupervisedUserLoggedIn(const std::string& user_id) override; |
| - bool HasPendingBootstrap(const std::string& user_id) const override; |
| + void RegularUserLoggedIn(const AccountId& user_id) override; |
| + void RegularUserLoggedInAsEphemeral(const AccountId& user_id) override; |
| + void SupervisedUserLoggedIn(const AccountId& user_id) override; |
| + bool HasPendingBootstrap(const AccountId& user_id) const override; |
| private: |
| friend class SupervisedUserManagerImpl; |
| @@ -159,7 +159,7 @@ class ChromeUserManagerImpl |
| friend class WallpaperManager; |
| friend class WallpaperManagerTest; |
| - typedef base::hash_map<std::string, linked_ptr<UserImageManager> > |
| + typedef base::hash_map<AccountId, linked_ptr<UserImageManager> > |
| UserImageManagerMap; |
| ChromeUserManagerImpl(); |
| @@ -237,7 +237,7 @@ class ChromeUserManagerImpl |
| // Session length limiter. |
| scoped_ptr<SessionLengthLimiter> session_length_limiter_; |
| - typedef std::map<std::string, UserFlow*> FlowMap; |
| + typedef std::map<AccountId, UserFlow*> FlowMap; |
| // Lazy-initialized default flow. |
| mutable scoped_ptr<UserFlow> default_flow_; |