ChromeOS: Fix removeUser button on signin screen.

chrome/browser/chromeos/login/users/chrome_user_manager_impl.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 "chrome/browser/chromeos/login/users/chrome_user_manager_impl.h"
 
 #include <cstddef>
 #include <set>
 
 #include "ash/multi_profile_uma.h"
@@ skipping 488 matching lines @@
 }
 
 bool ChromeUserManagerImpl::AreEphemeralUsersEnabled() const {
   policy::BrowserPolicyConnectorChromeOS* connector =
       g_browser_process->platform_part()->browser_policy_connector_chromeos();
   return GetEphemeralUsersEnabled() &&
          (connector->IsEnterpriseManaged() || GetOwnerAccountId().is_valid());
 }
 
 void ChromeUserManagerImpl::OnUserRemoved(const AccountId& account_id) {
-  RemoveReportingUser(FullyCanonicalize(account_id.GetUserEmail()));
+  RemoveReportingUser(account_id);
 }
 
 const std::string& ChromeUserManagerImpl::GetApplicationLocale() const {
   return g_browser_process->GetApplicationLocale();
 }
 
 PrefService* ChromeUserManagerImpl::GetLocalState() const {
   return g_browser_process ? g_browser_process->local_state() : NULL;
 }
 
@@ skipping 623 matching lines @@
       !system::HasSystemTimezonePolicy()) {
     g_browser_process->platform_part()->GetTimezoneResolver()->Start();
   } else {
     g_browser_process->platform_part()->GetTimezoneResolver()->Stop();
   }
 }
 
 void ChromeUserManagerImpl::SetUserAffiliation(
     const std::string& user_email,
     const AffiliationIDSet& user_affiliation_ids) {
-  std::string canonicalized_email = FullyCanonicalize(user_email);
-  user_manager::User* user =
-      FindUserAndModify(AccountId::FromUserEmail(canonicalized_email));
+  const AccountId& account_id =
+      user_manager::UserManager::GetKnownUserAccountId(user_email,
+                                                       std::string());
+  user_manager::User* user = FindUserAndModify(account_id);
 
   if (user) {
     policy::BrowserPolicyConnectorChromeOS const* const connector =
         g_browser_process->platform_part()->browser_policy_connector_chromeos();
     const bool is_affiliated = chromeos::IsUserAffiliated(
         user_affiliation_ids, connector->GetDeviceAffiliationIDs(),
-        canonicalized_email, connector->GetEnterpriseDomain());
+        account_id.GetUserEmail(), connector->GetEnterpriseDomain());

[achuithb, 2015/11/23 21:17:39]

I assume this will take account_id in the future?

[Alexander Alekseev, 2015/11/24 06:04:11]

Yes.

     user->set_affiliation(is_affiliated);
 
     if (user->GetType() == user_manager::USER_TYPE_REGULAR) {
       if (is_affiliated) {
-        AddReportingUser(canonicalized_email);
+        AddReportingUser(account_id);
       } else {
-        RemoveReportingUser(canonicalized_email);
+        RemoveReportingUser(account_id);
       }
     }
   }
 }
 
 bool ChromeUserManagerImpl::ShouldReportUser(const std::string& user_id) const {
   const base::ListValue& reporting_users =
       *(GetLocalState()->GetList(kReportingUsers));
   base::StringValue user_id_value(FullyCanonicalize(user_id));
   return !(reporting_users.Find(user_id_value) == reporting_users.end());
 }
 
-void ChromeUserManagerImpl::AddReportingUser(const std::string& user_id) {
+void ChromeUserManagerImpl::AddReportingUser(const AccountId& account_id) {
   ListPrefUpdate users_update(GetLocalState(), kReportingUsers);
   users_update->AppendIfNotPresent(
-      new base::StringValue(FullyCanonicalize(user_id)));
+      new base::StringValue(account_id.GetUserEmail()));
 }
 
-void ChromeUserManagerImpl::RemoveReportingUser(const std::string& user_id) {
+void ChromeUserManagerImpl::RemoveReportingUser(const AccountId& account_id) {
   ListPrefUpdate users_update(GetLocalState(), kReportingUsers);
-  users_update->Remove(base::StringValue(FullyCanonicalize(user_id)), NULL);
+  users_update->Remove(
+      base::StringValue(FullyCanonicalize(account_id.GetUserEmail())), NULL);

[achuithb, 2015/11/23 21:17:39]

Is FullyCanonicalize necessary here?

[Alexander Alekseev, 2015/11/24 06:04:11]

I left it here in order not to miss this fact later, when kReportingUsers will
be migrated.

 }
 
 }  // namespace chromeos