This CL replaces user_manager::UserID with AccountId.

components/user_manager/user.cc

Index: components/user_manager/user.cc
diff --git a/components/user_manager/user.cc b/components/user_manager/user.cc
index defc117308ec1f706098d9a9f2b5d2f069260be3..5860802d20e3dd7132b6b12d231e56e5917ee5e4 100644
--- a/components/user_manager/user.cc
+++ b/components/user_manager/user.cc
@@ -9,6 +9,7 @@
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_restrictions.h"
 #include "chromeos/login/user_names.h"
+#include "components/signin/core/account_id/account_id.h"
 #include "components/user_manager/user_image/default_user_images.h"
 #include "google_apis/gaia/gaia_auth_util.h"
 #include "ui/base/resource/resource_bundle.h"
@@ -34,10 +35,14 @@ bool User::TypeHasGaiaAccount(UserType user_type) {
          user_type == USER_TYPE_CHILD;
 }
 
+const std::string& User::email() const {
+  return account_id_.GetUserEmail();
+}
+
 // Also used for regular supervised users.
 class RegularUser : public User {
  public:
-  explicit RegularUser(const std::string& email);
+  explicit RegularUser(const AccountId& user_id);

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

account_id

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

Done.

   ~RegularUser() override;
 
   // Overridden from User:
@@ -65,7 +70,7 @@ class GuestUser : public User {
 
 class KioskAppUser : public User {
  public:
-  explicit KioskAppUser(const std::string& app_id);
+  explicit KioskAppUser(const AccountId& app_id);
   ~KioskAppUser() override;
 
   // Overridden from User:
@@ -77,7 +82,7 @@ class KioskAppUser : public User {
 
 class SupervisedUser : public User {
  public:
-  explicit SupervisedUser(const std::string& username);
+  explicit SupervisedUser(const AccountId& user_id);
   ~SupervisedUser() override;
 
   // Overridden from User:
@@ -90,7 +95,7 @@ class SupervisedUser : public User {
 
 class PublicAccountUser : public User {
  public:
-  explicit PublicAccountUser(const std::string& email);
+  explicit PublicAccountUser(const AccountId& user_id);
   ~PublicAccountUser() override;
 
   // Overridden from User:
@@ -118,8 +123,9 @@ const gfx::ImageSkia& User::GetImage() const {
   return user_image_.image();
 }
 
-UserID User::GetUserID() const {
-  return gaia::CanonicalizeEmail(gaia::SanitizeEmail(email()));
+AccountId User::GetUserID() const {
+  return AccountId::FromUserEmail(
+      gaia::CanonicalizeEmail(gaia::SanitizeEmail(email())));
 }
 
 void User::SetIsChild(bool is_child) {
@@ -144,7 +150,7 @@ std::string User::GetAccountName(bool use_display_email) const {
   if (use_display_email && !display_email_.empty())
     return GetUserName(display_email_);
   else
-    return GetUserName(email_);
+    return GetUserName(account_id_.GetUserEmail());
 }
 
 bool User::HasDefaultImage() const {
@@ -175,28 +181,28 @@ bool User::is_active() const {
   return is_active_;
 }
 
-User* User::CreateRegularUser(const std::string& email) {
-  return new RegularUser(email);
+User* User::CreateRegularUser(const AccountId& user_id) {
+  return new RegularUser(user_id);
 }
 
 User* User::CreateGuestUser() {
   return new GuestUser;
 }
 
-User* User::CreateKioskAppUser(const std::string& kiosk_app_username) {
-  return new KioskAppUser(kiosk_app_username);
+User* User::CreateKioskAppUser(const AccountId& kiosk_app_user_id) {
+  return new KioskAppUser(kiosk_app_user_id);
 }
 
-User* User::CreateSupervisedUser(const std::string& username) {
-  return new SupervisedUser(username);
+User* User::CreateSupervisedUser(const AccountId& user_id) {
+  return new SupervisedUser(user_id);
 }
 
-User* User::CreatePublicAccountUser(const std::string& email) {
-  return new PublicAccountUser(email);
+User* User::CreatePublicAccountUser(const AccountId& user_id) {
+  return new PublicAccountUser(user_id);
 }
 
-User::User(const std::string& email)
-    : email_(email),
+User::User(const AccountId& account_id)
+    : account_id_(account_id),
       oauth_token_status_(OAUTH_TOKEN_STATUS_UNKNOWN),

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

in class member initialization

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

Done.

       force_online_signin_(false),
       image_index_(USER_IMAGE_INVALID),
@@ -206,8 +212,7 @@ User::User(const std::string& email)
       is_logged_in_(false),
       is_active_(false),
       profile_is_created_(false),
-      is_affiliated_(false){
-}
+      is_affiliated_(false) {}
 
 User::~User() {
 }
@@ -237,10 +242,10 @@ void User::SetStubImage(const UserImage& stub_user_image,
   image_is_loading_ = is_loading;
 }
 
-RegularUser::RegularUser(const std::string& email)
-    : User(email), is_child_(false) {
+RegularUser::RegularUser(const AccountId& account_id)
+    : User(account_id), is_child_(false) {
   set_can_lock(true);
-  set_display_email(email);
+  set_display_email(account_id.GetUserEmail());
 }
 
 RegularUser::~RegularUser() {
@@ -260,7 +265,7 @@ void RegularUser::SetIsChild(bool is_child) {
   is_child_ = is_child;
 }
 
-GuestUser::GuestUser() : User(chromeos::login::kGuestUserName) {
+GuestUser::GuestUser() : User(chromeos::login::GuestAccountId()) {
   set_display_email(std::string());
 }
 
@@ -271,9 +276,9 @@ UserType GuestUser::GetType() const {
   return user_manager::USER_TYPE_GUEST;
 }
 
-KioskAppUser::KioskAppUser(const std::string& kiosk_app_username)
-    : User(kiosk_app_username) {
-  set_display_email(kiosk_app_username);
+KioskAppUser::KioskAppUser(const AccountId& kiosk_app_user_id)
+    : User(kiosk_app_user_id) {
+  set_display_email(kiosk_app_user_id.GetUserEmail());
 }
 
 KioskAppUser::~KioskAppUser() {
@@ -283,7 +288,7 @@ UserType KioskAppUser::GetType() const {
   return user_manager::USER_TYPE_KIOSK_APP;
 }
 
-SupervisedUser::SupervisedUser(const std::string& username) : User(username) {
+SupervisedUser::SupervisedUser(const AccountId& user_id) : User(user_id) {

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

account_id

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

Done.

   set_can_lock(true);
 }
 
@@ -298,8 +303,8 @@ std::string SupervisedUser::display_email() const {
   return base::UTF16ToUTF8(display_name());
 }
 
-PublicAccountUser::PublicAccountUser(const std::string& email) : User(email) {
-}
+PublicAccountUser::PublicAccountUser(const AccountId& account_id)
+    : User(account_id) {}
 
 PublicAccountUser::~PublicAccountUser() {
 }