This CL replaces std::string user_id in ash/* with AccountId.

chrome/browser/ui/ash/session_state_delegate_chromeos.cc

 // Copyright (c) 2013 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 "chrome/browser/ui/ash/session_state_delegate_chromeos.h"
 
 #include "ash/content/shell_content_state.h"
 #include "ash/multi_profile_uma.h"
 #include "ash/session/session_state_observer.h"
 #include "ash/system/chromeos/multi_user/user_switch_util.h"
@@ skipping 140 matching lines @@
 }
 
 gfx::ImageSkia SessionStateDelegateChromeos::GetAvatarImageForWindow(
     aura::Window* window) const {
   content::BrowserContext* context =
       ash::ShellContentState::GetInstance()->GetBrowserContextForWindow(window);
   return GetAvatarImageForContext(context);
 }
 
 void SessionStateDelegateChromeos::SwitchActiveUser(
-    const std::string& user_id) {
+    const AccountId& account_id) {
   // Disallow switching to an already active user since that might crash.
   // Also check that we got a user id and not an email address.
-  DCHECK_EQ(user_id,
-            gaia::CanonicalizeEmail(gaia::SanitizeEmail(user_id)));
-  if (user_id == user_manager::UserManager::Get()->GetActiveUser()->email())
+  DCHECK_EQ(
+      account_id.GetUserEmail(),
+      gaia::CanonicalizeEmail(gaia::SanitizeEmail(account_id.GetUserEmail())));
+  if (account_id ==
+      user_manager::UserManager::Get()->GetActiveUser()->GetAccountId())
     return;
-  TryToSwitchUser(user_id);
+  TryToSwitchUser(account_id);
 }
 
 void SessionStateDelegateChromeos::CycleActiveUser(CycleUser cycle_user) {
   // Make sure there is a user to switch to.
   if (NumberOfLoggedInUsers() <= 1)
     return;
 
   const user_manager::UserList& logged_in_users =
       user_manager::UserManager::Get()->GetLoggedInUsers();
 
-  std::string user_id =
-      user_manager::UserManager::Get()->GetActiveUser()->email();
+  AccountId account_id =
+      user_manager::UserManager::Get()->GetActiveUser()->GetAccountId();
 
   // Get an iterator positioned at the active user.
   user_manager::UserList::const_iterator it;
   for (it = logged_in_users.begin();
        it != logged_in_users.end(); ++it) {
-    if ((*it)->email() == user_id)
+    if ((*it)->GetAccountId() == account_id)
       break;
   }
 
   // Active user not found.
   if (it == logged_in_users.end())
     return;
 
   // Get the user's email to select, wrapping to the start/end of the list if
   // necessary.
   switch (cycle_user) {
     case CYCLE_TO_NEXT_USER:
       if (++it == logged_in_users.end())
-        user_id = (*logged_in_users.begin())->email();
+        account_id = (*logged_in_users.begin())->GetAccountId();
       else
-        user_id = (*it)->email();
+        account_id = (*it)->GetAccountId();
       break;
     case CYCLE_TO_PREVIOUS_USER:
       if (it == logged_in_users.begin())
         it = logged_in_users.end();
-      user_id = (*(--it))->email();
+      account_id = (*(--it))->GetAccountId();
       break;
   }
 
-  // Switch using the transformed |user_id|.
-  TryToSwitchUser(user_id);
+  // Switch using the transformed |account_id|.
+  TryToSwitchUser(account_id);
 }
 
 bool SessionStateDelegateChromeos::IsMultiProfileAllowedByPrimaryUserPolicy()
     const {
   return chromeos::MultiProfileUserController::GetPrimaryUserPolicy() ==
          chromeos::MultiProfileUserController::ALLOWED;
 }
 
 void SessionStateDelegateChromeos::AddSessionStateObserver(
     ash::SessionStateObserver* observer) {
   session_state_observer_list_.AddObserver(observer);
 }
 
 void SessionStateDelegateChromeos::RemoveSessionStateObserver(
     ash::SessionStateObserver* observer) {
   session_state_observer_list_.RemoveObserver(observer);
 }
 
 void SessionStateDelegateChromeos::LoggedInStateChanged() {
   SetSessionState(chromeos::LoginState::Get()->IsUserLoggedIn() ?
       SESSION_STATE_ACTIVE : SESSION_STATE_LOGIN_PRIMARY, false);
 }
 
 void SessionStateDelegateChromeos::ActiveUserChanged(
     const user_manager::User* active_user) {
-  FOR_EACH_OBSERVER(ash::SessionStateObserver,
-                    session_state_observer_list_,
-                    ActiveUserChanged(active_user->email()));
+  FOR_EACH_OBSERVER(ash::SessionStateObserver, session_state_observer_list_,
+                    ActiveUserChanged(active_user->GetAccountId()));
 }
 
 void SessionStateDelegateChromeos::UserAddedToSession(
     const user_manager::User* added_user) {
-  FOR_EACH_OBSERVER(ash::SessionStateObserver,
-                    session_state_observer_list_,
-                    UserAddedToSession(added_user->email()));
+  FOR_EACH_OBSERVER(ash::SessionStateObserver, session_state_observer_list_,
+                    UserAddedToSession(added_user->GetAccountId()));
 }
 
 void SessionStateDelegateChromeos::OnUserAddingStarted() {
   SetSessionState(SESSION_STATE_LOGIN_SECONDARY, false);
 }
 
 void SessionStateDelegateChromeos::OnUserAddingFinished() {
   SetSessionState(SESSION_STATE_ACTIVE, false);
 }
 
 void SessionStateDelegateChromeos::SetSessionState(SessionState new_state,
                                                    bool force) {
   if (session_state_ == new_state && !force)
     return;
 
   session_state_ = new_state;
   NotifySessionStateChanged();
 }
 
 void SessionStateDelegateChromeos::NotifySessionStateChanged() {
   FOR_EACH_OBSERVER(ash::SessionStateObserver,
                     session_state_observer_list_,
                     SessionStateChanged(session_state_));
 }
 
-void DoSwitchUser(const std::string& user_id) {
-  user_manager::UserManager::Get()->SwitchActiveUser(
-      AccountId::FromUserEmail(user_id));
+void DoSwitchUser(const AccountId& account_id) {
+  user_manager::UserManager::Get()->SwitchActiveUser(account_id);
 }
 
 void SessionStateDelegateChromeos::TryToSwitchUser(
-    const std::string& user_id) {
-  ash::TrySwitchingActiveUser(base::Bind(&DoSwitchUser, user_id));
+    const AccountId& account_id) {
+  ash::TrySwitchingActiveUser(base::Bind(&DoSwitchUser, account_id));
 }