This CL replaces user_manager::UserID with AccountId.

components/user_manager/fake_user_manager.h

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..cffe9598c6c47928ef218e1ce83af2f4aaab224e 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& user_id);

[achuithb, 2015/10/23 00:08:52]

account_id, here and below

user_id, email, account, username, all to account_id

[Alexander Alekseev, 2015/10/23 09:11:24]

Done.

 
   // 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& user_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& email);

[achuithb, 2015/10/23 00:08:52]

account_id

[Alexander Alekseev, 2015/10/23 09:11:24]

Done.

 
   // UserManager overrides.
   const user_manager::UserList& GetUsers() const override;
@@ -39,42 +41,42 @@ 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,

[achuithb, 2015/10/23 00:08:52]

account_id

[Alexander Alekseev, 2015/10/23 09:11:24]

Done.

                     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) override;
+  void SaveUserDisplayName(const AccountId& username,
                            const base::string16& display_name) override;
 
   // Not implemented.
-  void UpdateUserAccountData(const std::string& user_id,
+  void UpdateUserAccountData(const AccountId& user_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& GetOwnerId() const override;

[achuithb, 2015/10/23 00:08:52]

GetOwnerAccountID()?

[Alexander Alekseev, 2015/10/23 09:11:24]

Done.

   void SessionStarted() override {}
-  void RemoveUser(const std::string& email,
+  void RemoveUser(const AccountId& account,
                   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) override;
+  bool IsKnownUser(const AccountId& account) const override;
+  const user_manager::User* FindUser(const AccountId& account) const override;
+  user_manager::User* FindUserAndModify(const AccountId& account) 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& username,
       user_manager::User::OAuthTokenStatus oauth_token_status) override {}
-  void SaveForceOnlineSignin(const std::string& user_id,
+  void SaveForceOnlineSignin(const AccountId& user_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& username) const override;
+  void SaveUserDisplayEmail(const AccountId& username,
                             const std::string& display_email) override {}
-  std::string GetUserDisplayEmail(const std::string& username) const override;
+  std::string GetUserDisplayEmail(const AccountId& username) const override;
   bool IsCurrentUserOwner() const override;
   bool IsCurrentUserNew() const override;
   bool IsCurrentUserNonCryptohomeDataEphemeral() const override;
@@ -88,7 +90,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) const override;
   void AddObserver(Observer* obs) override {}
   void RemoveObserver(Observer* obs) override {}
   void AddSessionStateObserver(UserSessionStateObserver* obs) override {}
@@ -101,37 +103,35 @@ 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& user_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 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 KioskAppLoggedIn(const std::string& app_id) override {}
+  void KioskAppLoggedIn(const AccountId& app_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& user_id) override {}
+  void OnUserRemoved(const AccountId& user_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_user_id_;

[achuithb, 2015/10/23 00:08:52]

active_account_id_

[Alexander Alekseev, 2015/10/23 09:11:24]

Done.

 
  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_id_;

[achuithb, 2015/10/23 00:08:52]

owner_account_id_

[Alexander Alekseev, 2015/10/23 09:11:24]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(FakeUserManager);
 };