This CL replaces user_manager::UserID with AccountId.

chromeos/login/auth/user_context.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chromeos/login/auth/user_context.h"
 #include "chromeos/login/user_names.h"
 
 namespace chromeos {
 
 UserContext::UserContext()
-    : is_using_oauth_(true),
+    : account_id_(EmptyAccountId()),

[achuithb, 2015/10/28 23:11:46]

I think you can still use in-class member initialization, and only set the
values that are different from the default in the ctors?

[Alexander Alekseev, 2015/10/29 02:00:42]

I mean that adding a new constructor here without explicit initialization of
account_id is probably an error here.

I hope we would be able to make it constant member at some point.

+      is_using_oauth_(true),
       auth_flow_(AUTH_FLOW_OFFLINE),
-      user_type_(user_manager::USER_TYPE_REGULAR) {
-}
+      user_type_(user_manager::USER_TYPE_REGULAR) {}
 
 UserContext::UserContext(const UserContext& other)
-    : user_id_(other.user_id_),
+    : account_id_(other.account_id_),
       gaia_id_(other.gaia_id_),
       key_(other.key_),
       auth_code_(other.auth_code_),
       refresh_token_(other.refresh_token_),
       access_token_(other.access_token_),
       user_id_hash_(other.user_id_hash_),
       is_using_oauth_(other.is_using_oauth_),
       auth_flow_(other.auth_flow_),
       user_type_(other.user_type_),
       public_session_locale_(other.public_session_locale_),
       public_session_input_method_(other.public_session_input_method_),
       device_id_(other.device_id_),
-      gaps_cookie_(other.gaps_cookie_) {
-}
+      gaps_cookie_(other.gaps_cookie_) {}
 
-UserContext::UserContext(const std::string& user_id)
-    : user_id_(login::CanonicalizeUserID(user_id)),
+UserContext::UserContext(const AccountId& account_id)
+    : account_id_(account_id),
       is_using_oauth_(true),
       auth_flow_(AUTH_FLOW_OFFLINE),
       user_type_(user_manager::USER_TYPE_REGULAR) {
+  account_id_.SetUserEmail(
+      login::CanonicalizeUserID(account_id.GetUserEmail()));
 }
 
 UserContext::UserContext(user_manager::UserType user_type,
                          const std::string& user_id)
-    : is_using_oauth_(true),
+    : account_id_(EmptyAccountId()),
+      is_using_oauth_(true),
       auth_flow_(AUTH_FLOW_OFFLINE),
       user_type_(user_type) {
   if (user_type_ == user_manager::USER_TYPE_REGULAR)
-    user_id_ = login::CanonicalizeUserID(user_id);
+    account_id_ = AccountId::FromUserEmail(login::CanonicalizeUserID(user_id));
   else
-    user_id_ = user_id;
+    account_id_ = AccountId::FromUserEmail(user_id);
 }
 
 UserContext::~UserContext() {
 }
 
 bool UserContext::operator==(const UserContext& context) const {
-  return context.user_id_ == user_id_ && context.gaia_id_ == gaia_id_ &&
+  return context.account_id_ == account_id_ && context.gaia_id_ == gaia_id_ &&
          context.key_ == key_ && context.auth_code_ == auth_code_ &&
          context.refresh_token_ == refresh_token_ &&
          context.access_token_ == access_token_ &&
          context.user_id_hash_ == user_id_hash_ &&
          context.is_using_oauth_ == is_using_oauth_ &&
          context.auth_flow_ == auth_flow_ && context.user_type_ == user_type_ &&
          context.public_session_locale_ == public_session_locale_ &&
          context.public_session_input_method_ == public_session_input_method_;
 }
 
 bool UserContext::operator!=(const UserContext& context) const {
   return !(*this == context);
 }
 
-const std::string& UserContext::GetUserID() const {
-  return user_id_;
+const AccountId& UserContext::GetAccountId() const {
+  return account_id_;
 }
 
 const std::string& UserContext::GetGaiaID() const {
   return gaia_id_;
 }
 
 const Key* UserContext::GetKey() const {
   return &key_;
 }
 
@@ skipping 39 matching lines @@
 
 const std::string& UserContext::GetDeviceId() const {
   return device_id_;
 }
 
 const std::string& UserContext::GetGAPSCookie() const {
   return gaps_cookie_;
 }
 
 bool UserContext::HasCredentials() const {
-  return (!user_id_.empty() && !key_.GetSecret().empty()) ||
+  return (account_id_.is_valid() && !key_.GetSecret().empty()) ||
          !auth_code_.empty();
 }
 
 void UserContext::SetUserID(const std::string& user_id) {

[achuithb, 2015/10/28 23:11:46]

Why not SetAccountId instead?

[Alexander Alekseev, 2015/10/29 02:00:42]

Because this CL is limited to UserManager API.

-  user_id_ = login::CanonicalizeUserID(user_id);
+  account_id_ = AccountId::FromUserEmail(login::CanonicalizeUserID(user_id));
 }
 
 void UserContext::SetGaiaID(const std::string& gaia_id) {
   gaia_id_ = gaia_id;
 }
 
 void UserContext::SetKey(const Key& key) {
   key_ = key;
 }
 
@@ skipping 41 matching lines @@
   gaps_cookie_ = gaps_cookie;
 }
 
 void UserContext::ClearSecrets() {
   key_.ClearSecret();
   auth_code_.clear();
   refresh_token_.clear();
 }
 
 }  // namespace chromeos