ash: Use enum for SessionController::CycleActiveUser

chrome/browser/ui/ash/session_state_delegate_chromeos_unittest.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 "chrome/browser/ui/ash/session_state_delegate_chromeos.h"
 
 #include <memory>
 #include <string>
 #include <vector>
 
@@ skipping 121 matching lines @@
   FakeChromeUserManager* user_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(SessionStateDelegateChromeOSTest);
 };
 
 // Make sure that cycling one user does not cause any harm.
 TEST_F(SessionStateDelegateChromeOSTest, CyclingOneUser) {
   UserAddedToSession("firstuser@test.com");
 
   EXPECT_EQ("firstuser@test.com", GetActiveUserEmail());
-  session_state_delegate()->CycleActiveUser(
-      ash::SessionStateDelegate::CYCLE_TO_NEXT_USER);
+  session_state_delegate()->CycleActiveUser(ash::CycleUserDirection::NEXT);
   EXPECT_EQ("firstuser@test.com", GetActiveUserEmail());
-  session_state_delegate()->CycleActiveUser(
-      ash::SessionStateDelegate::CYCLE_TO_PREVIOUS_USER);
+  session_state_delegate()->CycleActiveUser(ash::CycleUserDirection::PREVIOUS);
   EXPECT_EQ("firstuser@test.com", GetActiveUserEmail());
 }
 
 // Cycle three users forwards and backwards to see that it works.
 TEST_F(SessionStateDelegateChromeOSTest, CyclingThreeUsers) {
   UserAddedToSession("firstuser@test.com");
   UserAddedToSession("seconduser@test.com");
   UserAddedToSession("thirduser@test.com");
-  const ash::SessionStateDelegate::CycleUser forward =
-      ash::SessionStateDelegate::CYCLE_TO_NEXT_USER;
+  const ash::CycleUserDirection forward = ash::CycleUserDirection::NEXT;
 
   // Cycle forward.
   EXPECT_EQ("firstuser@test.com", GetActiveUserEmail());
   session_state_delegate()->CycleActiveUser(forward);
   EXPECT_EQ("seconduser@test.com", GetActiveUserEmail());
   session_state_delegate()->CycleActiveUser(forward);
   EXPECT_EQ("thirduser@test.com", GetActiveUserEmail());
   session_state_delegate()->CycleActiveUser(forward);
   EXPECT_EQ("firstuser@test.com", GetActiveUserEmail());
 
   // Cycle backwards.
-  const ash::SessionStateDelegate::CycleUser backward =
-      ash::SessionStateDelegate::CYCLE_TO_PREVIOUS_USER;
+  const ash::CycleUserDirection backward = ash::CycleUserDirection::PREVIOUS;
   session_state_delegate()->CycleActiveUser(backward);
   EXPECT_EQ("thirduser@test.com", GetActiveUserEmail());
   session_state_delegate()->CycleActiveUser(backward);
   EXPECT_EQ("seconduser@test.com", GetActiveUserEmail());
   session_state_delegate()->CycleActiveUser(backward);
   EXPECT_EQ("firstuser@test.com", GetActiveUserEmail());
 }
 
 // Make sure MultiProfile disabled by primary user policy.
 TEST_F(SessionStateDelegateChromeOSTest, MultiProfileDisallowedByUserPolicy) {
@@ skipping 103 matching lines @@
   user_manager()->LoginUser(account_id);
   user_profile_->GetPrefs()->SetString(
       prefs::kMultiProfileUserBehavior,
       chromeos::MultiProfileUserController::kBehaviorNotAllowed);
   user_manager()->AddUser(AccountId::FromUserEmail("bb@b.b"));
   EXPECT_EQ(ash::AddUserSessionPolicy::ERROR_NOT_ALLOWED_PRIMARY_USER,
             session_state_delegate()->GetAddUserSessionPolicy());
 }
 
 }  // namespace chromeos