ChromeOS cryptohome should be able to use gaia id as user identifier.

chrome/browser/chromeos/login/crash_restore_browsertest.cc

 // Copyright 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 <string>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/run_loop.h"
 #include "chrome/browser/chromeos/login/session/user_session_manager.h"
 #include "chrome/browser/chromeos/login/session/user_session_manager_test_api.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chromeos/chromeos_switches.h"
+#include "chromeos/cryptohome/cryptohome_parameters.h"
 #include "chromeos/dbus/cryptohome_client.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/fake_session_manager_client.h"
 #include "chromeos/dbus/session_manager_client.h"
 #include "components/user_manager/user.h"
 #include "components/user_manager/user_manager.h"
 #include "content/public/test/test_utils.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace chromeos {
 
 namespace {
 
 const char kUserId1[] = "user1@example.com";
 const char kUserId2[] = "user2@example.com";
 const char kUserId3[] = "user3@example.com";
 
 }  // namespace
 
 class CrashRestoreSimpleTest : public InProcessBrowserTest {
  protected:
   CrashRestoreSimpleTest() {}
 
   ~CrashRestoreSimpleTest() override {}
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
-    command_line->AppendSwitchASCII(switches::kLoginUser, kUserId1);
+    command_line->AppendSwitchASCII(switches::kLoginUser, cryptohome_id1_.id());
     command_line->AppendSwitchASCII(
         switches::kLoginProfile,
-        CryptohomeClient::GetStubSanitizedUsername(kUserId1));
+        CryptohomeClient::GetStubSanitizedUsername(cryptohome_id1_));
   }
 
   void SetUpInProcessBrowserTestFixture() override {
     // Redirect session_manager DBus calls to FakeSessionManagerClient.
     session_manager_client_ = new FakeSessionManagerClient;
     chromeos::DBusThreadManager::GetSetterForTesting()->SetSessionManagerClient(
         scoped_ptr<SessionManagerClient>(session_manager_client_));
-    session_manager_client_->StartSession(kUserId1);
+    session_manager_client_->StartSession(cryptohome_id1_);
   }
 
   FakeSessionManagerClient* session_manager_client_;
+  const AccountId account_id1_ = AccountId::FromUserEmail(kUserId1);
+  const AccountId account_id2_ = AccountId::FromUserEmail(kUserId2);
+  const AccountId account_id3_ = AccountId::FromUserEmail(kUserId3);
+  const cryptohome::Identification cryptohome_id1_ =
+      cryptohome::Identification(account_id1_);
+  const cryptohome::Identification cryptohome_id2_ =
+      cryptohome::Identification(account_id2_);
+  const cryptohome::Identification cryptohome_id3_ =
+      cryptohome::Identification(account_id3_);
 };
 
 IN_PROC_BROWSER_TEST_F(CrashRestoreSimpleTest, RestoreSessionForOneUser) {
   user_manager::UserManager* user_manager = user_manager::UserManager::Get();
   user_manager::User* user = user_manager->GetActiveUser();
   ASSERT_TRUE(user);
-  EXPECT_EQ(kUserId1, user->email());
-  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(kUserId1),
+  EXPECT_EQ(account_id1_, user->GetAccountId());
+  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(cryptohome_id1_),
             user->username_hash());
   EXPECT_EQ(1UL, user_manager->GetLoggedInUsers().size());
 }
 
 // Observer that keeps track of user sessions restore event.
 class UserSessionRestoreObserver : public UserSessionStateObserver {
  public:
   UserSessionRestoreObserver()
       : running_loop_(false),
         user_sessions_restored_(
@@ skipping 33 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(UserSessionRestoreObserver);
 };
 
 class CrashRestoreComplexTest : public CrashRestoreSimpleTest {
  protected:
   CrashRestoreComplexTest() {}
   ~CrashRestoreComplexTest() override {}
 
   void SetUpInProcessBrowserTestFixture() override {
     CrashRestoreSimpleTest::SetUpInProcessBrowserTestFixture();
-    session_manager_client_->StartSession(kUserId2);
-    session_manager_client_->StartSession(kUserId3);
+    session_manager_client_->StartSession(cryptohome_id2_);
+    session_manager_client_->StartSession(cryptohome_id3_);
   }
 };
 
 IN_PROC_BROWSER_TEST_F(CrashRestoreComplexTest, RestoreSessionForThreeUsers) {
   {
     UserSessionRestoreObserver restore_observer;
     restore_observer.Wait();
   }
 
   chromeos::test::UserSessionManagerTestApi session_manager_test_api(
       chromeos::UserSessionManager::GetInstance());
   session_manager_test_api.SetShouldObtainTokenHandleInTests(false);
 
   DCHECK(UserSessionManager::GetInstance()->UserSessionsRestored());
 
   // User that is last in the user sessions map becomes active. This behavior
   // will become better defined once each user gets a separate user desktop.
   user_manager::UserManager* user_manager = user_manager::UserManager::Get();
   user_manager::User* user = user_manager->GetActiveUser();
   ASSERT_TRUE(user);
-  EXPECT_EQ(kUserId3, user->email());
-  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(kUserId3),
+  EXPECT_EQ(account_id3_, user->GetAccountId());
+  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(cryptohome_id3_),
             user->username_hash());
   const user_manager::UserList& users = user_manager->GetLoggedInUsers();
   ASSERT_EQ(3UL, users.size());
 
   // User that becomes active moves to the beginning of the list.
-  EXPECT_EQ(kUserId3, users[0]->email());
-  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(kUserId3),
+  EXPECT_EQ(account_id3_, users[0]->GetAccountId());
+  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(cryptohome_id3_),
             users[0]->username_hash());
-  EXPECT_EQ(kUserId2, users[1]->email());
-  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(kUserId2),
+  EXPECT_EQ(account_id2_, users[1]->GetAccountId());
+  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(cryptohome_id2_),
             users[1]->username_hash());
-  EXPECT_EQ(kUserId1, users[2]->email());
-  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(kUserId1),
+  EXPECT_EQ(account_id1_, users[2]->GetAccountId());
+  EXPECT_EQ(CryptohomeClient::GetStubSanitizedUsername(cryptohome_id1_),
             users[2]->username_hash());
 }
 
 }  // namespace chromeos