This CL replaces user_manager::UserID with AccountId.

chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc

Index: chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc
diff --git a/chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc b/chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc
index 9ec75fe5ead89ea4589271d7fc83de69c9d0da16..1055a8ddc1cac7241bb17c7eb10d5dea02372e00 100644
--- a/chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc
+++ b/chrome/browser/ui/webui/options/chromeos/shared_options_browsertest.cc
@@ -86,8 +86,10 @@ const PrefTest kPrefTests[] = {
 class SharedOptionsTest : public LoginManagerTest {
  public:
   SharedOptionsTest()
-    : LoginManagerTest(false),
-      device_settings_provider_(NULL) {
+      : LoginManagerTest(false),
+        device_settings_provider_(NULL),
+        test_owner_account_id_(AccountId::FromUserEmail(kTestOwner)),
+        test_non_owner_account_id_(AccountId::FromUserEmail(kTestNonOwner)) {
     stub_settings_provider_.Set(kDeviceOwner, base::StringValue(kTestOwner));
   }
 
@@ -266,40 +268,43 @@ class SharedOptionsTest : public LoginManagerTest {
   StubAccountSettingsProvider stub_settings_provider_;
   CrosSettingsProvider* device_settings_provider_;
 
+  const AccountId test_owner_account_id_;
+  const AccountId test_non_owner_account_id_;
+
  private:
   DISALLOW_COPY_AND_ASSIGN(SharedOptionsTest);
 };
 
 IN_PROC_BROWSER_TEST_F(SharedOptionsTest, PRE_SharedOptions) {
-  RegisterUser(kTestOwner);
-  RegisterUser(kTestNonOwner);
+  RegisterUser(test_owner_account_id_.GetUserEmail());
+  RegisterUser(test_non_owner_account_id_.GetUserEmail());
   StartupUtils::MarkOobeCompleted();
 }
 
 IN_PROC_BROWSER_TEST_F(SharedOptionsTest, SharedOptions) {
   // Log in the owner first, then add a secondary user.
-  LoginUser(kTestOwner);
+  LoginUser(test_owner_account_id_.GetUserEmail());
   UserAddingScreen::Get()->Start();
   content::RunAllPendingInMessageLoop();
-  AddUser(kTestNonOwner);
+  AddUser(test_non_owner_account_id_.GetUserEmail());
 
   user_manager::UserManager* manager = user_manager::UserManager::Get();
   ASSERT_EQ(2u, manager->GetLoggedInUsers().size());
   {
     SCOPED_TRACE("Checking settings for owner, primary user.");
-    CheckOptionsUI(manager->FindUser(manager->GetOwnerEmail()), true, true);
+    CheckOptionsUI(manager->FindUser(manager->GetOwnerAccountId()), true, true);
   }
   {
     SCOPED_TRACE("Checking settings for non-owner, secondary user.");
-    CheckOptionsUI(manager->FindUser(kTestNonOwner), false, false);
+    CheckOptionsUI(manager->FindUser(test_non_owner_account_id_), false, false);
   }
   // TODO(michaelpg): Add tests for non-primary owner and primary non-owner
   // when the owner-only multiprofile restriction is removed, probably M38.
 }
 
 IN_PROC_BROWSER_TEST_F(SharedOptionsTest, PRE_ScreenLockPreferencePrimary) {
-  RegisterUser(kTestOwner);
-  RegisterUser(kTestNonOwner);
+  RegisterUser(test_owner_account_id_.GetUserEmail());
+  RegisterUser(test_non_owner_account_id_.GetUserEmail());
   StartupUtils::MarkOobeCompleted();
 }
 
@@ -309,14 +314,15 @@ IN_PROC_BROWSER_TEST_F(SharedOptionsTest, PRE_ScreenLockPreferencePrimary) {
 // other signed-in user has enabled this preference, the shared setting
 // indicator explains this.)
 IN_PROC_BROWSER_TEST_F(SharedOptionsTest, ScreenLockPreferencePrimary) {
-  LoginUser(kTestOwner);
+  LoginUser(test_owner_account_id_.GetUserEmail());
   UserAddingScreen::Get()->Start();
   content::RunAllPendingInMessageLoop();
-  AddUser(kTestNonOwner);
+  AddUser(test_non_owner_account_id_.GetUserEmail());
 
   user_manager::UserManager* manager = user_manager::UserManager::Get();
-  const user_manager::User* user1 = manager->FindUser(kTestOwner);
-  const user_manager::User* user2 = manager->FindUser(kTestNonOwner);
+  const user_manager::User* user1 = manager->FindUser(test_owner_account_id_);
+  const user_manager::User* user2 =
+      manager->FindUser(test_non_owner_account_id_);
 
   PrefService* prefs1 =
       ProfileHelper::Get()->GetProfileByUserUnsafe(user1)->GetPrefs();
@@ -369,8 +375,8 @@ IN_PROC_BROWSER_TEST_F(SharedOptionsTest, ScreenLockPreferencePrimary) {
 }
 
 IN_PROC_BROWSER_TEST_F(SharedOptionsTest, PRE_ScreenLockPreferenceSecondary) {
-  RegisterUser(kTestOwner);
-  RegisterUser(kTestNonOwner);
+  RegisterUser(test_owner_account_id_.GetUserEmail());
+  RegisterUser(test_non_owner_account_id_.GetUserEmail());
   StartupUtils::MarkOobeCompleted();
 }
 
@@ -380,14 +386,15 @@ IN_PROC_BROWSER_TEST_F(SharedOptionsTest, PRE_ScreenLockPreferenceSecondary) {
 // other signed-in user has enabled this preference, the shared setting
 // indicator explains this.)
 IN_PROC_BROWSER_TEST_F(SharedOptionsTest, ScreenLockPreferenceSecondary) {
-  LoginUser(kTestOwner);
+  LoginUser(test_owner_account_id_.GetUserEmail());
   UserAddingScreen::Get()->Start();
   content::RunAllPendingInMessageLoop();
-  AddUser(kTestNonOwner);
+  AddUser(test_non_owner_account_id_.GetUserEmail());
 
   user_manager::UserManager* manager = user_manager::UserManager::Get();
-  const user_manager::User* user1 = manager->FindUser(kTestOwner);
-  const user_manager::User* user2 = manager->FindUser(kTestNonOwner);
+  const user_manager::User* user1 = manager->FindUser(test_owner_account_id_);
+  const user_manager::User* user2 =
+      manager->FindUser(test_non_owner_account_id_);
 
   PrefService* prefs1 =
       ProfileHelper::Get()->GetProfileByUserUnsafe(user1)->GetPrefs();