Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6500)

Unified Diff: chrome/browser/chromeos/system/tray_accessibility_browsertest.cc

Issue 2041233005: Moves ash::user::LoginStatus to ash/common (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix test Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/system/tray_accessibility_browsertest.cc
diff --git a/chrome/browser/chromeos/system/tray_accessibility_browsertest.cc b/chrome/browser/chromeos/system/tray_accessibility_browsertest.cc
index d79b6a7fcee6eef53c25db2f606e1107a1087437..bc7b1070cf85c4e02b4c08b361d3f2da7c484274 100644
--- a/chrome/browser/chromeos/system/tray_accessibility_browsertest.cc
+++ b/chrome/browser/chromeos/system/tray_accessibility_browsertest.cc
@@ -140,13 +140,11 @@ class TrayAccessibilityTest
return menu_item_view != NULL;
}
- void SetLoginStatus(ash::user::LoginStatus status) {
+ void SetLoginStatus(ash::LoginStatus status) {
tray()->UpdateAfterLoginStatusChange(status);
}
- ash::user::LoginStatus GetLoginStatus() {
- return tray()->login_;
- }
+ ash::LoginStatus GetLoginStatus() { return tray()->login_; }
bool CreateDetailedMenu() {
tray()->PopupDetailedView(0, false);
@@ -268,18 +266,18 @@ class TrayAccessibilityTest
};
IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, LoginStatus) {
- EXPECT_EQ(ash::user::LOGGED_IN_NONE, GetLoginStatus());
+ EXPECT_EQ(ash::LoginStatus::NOT_LOGGED_IN, GetLoginStatus());
user_manager::UserManager::Get()->UserLoggedIn(
AccountId::FromUserEmail("owner@invalid.domain"), "owner@invalid.domain",
true);
user_manager::UserManager::Get()->SessionStarted();
- EXPECT_EQ(ash::user::LOGGED_IN_USER, GetLoginStatus());
+ EXPECT_EQ(ash::LoginStatus::USER, GetLoginStatus());
}
IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, ShowTrayIcon) {
- SetLoginStatus(ash::user::LOGGED_IN_NONE);
+ SetLoginStatus(ash::LoginStatus::NOT_LOGGED_IN);
// Confirms that the icon is invisible before login.
EXPECT_FALSE(IsTrayIconVisible());
@@ -492,7 +490,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, ShowMenuWithShowMenuOption) {
}
IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, ShowMenuWithShowOnLoginScreen) {
- SetLoginStatus(ash::user::LOGGED_IN_NONE);
+ SetLoginStatus(ash::LoginStatus::NOT_LOGGED_IN);
// Confirms that the menu is visible.
EXPECT_TRUE(CanCreateMenuItem());
@@ -597,7 +595,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, KeepMenuVisibilityOnLockScreen) {
EXPECT_TRUE(CanCreateMenuItem());
// Locks the screen.
- SetLoginStatus(ash::user::LOGGED_IN_LOCKED);
+ SetLoginStatus(ash::LoginStatus::LOCKED);
EXPECT_TRUE(CanCreateMenuItem());
// Disables high contrast mode.
@@ -608,7 +606,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, KeepMenuVisibilityOnLockScreen) {
}
IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, ClickDetailMenu) {
- SetLoginStatus(ash::user::LOGGED_IN_USER);
+ SetLoginStatus(ash::LoginStatus::USER);
// Confirms that the check item toggles the spoken feedback.
EXPECT_FALSE(AccessibilityManager::Get()->IsSpokenFeedbackEnabled());
@@ -668,7 +666,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, ClickDetailMenu) {
}
IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, CheckMarksOnDetailMenu) {
- SetLoginStatus(ash::user::LOGGED_IN_NONE);
+ SetLoginStatus(ash::LoginStatus::NOT_LOGGED_IN);
// At first, all of the check is unchecked.
EXPECT_TRUE(CreateDetailedMenu());
@@ -825,7 +823,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, CheckMarksOnDetailMenu) {
CloseDetailMenu();
// Autoclick is disabled on login screen.
- SetLoginStatus(ash::user::LOGGED_IN_USER);
+ SetLoginStatus(ash::LoginStatus::USER);
// Enabling autoclick.
AccessibilityManager::Get()->EnableAutoclick(true);
@@ -851,7 +849,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, CheckMarksOnDetailMenu) {
}
IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, CheckMenuVisibilityOnDetailMenu) {
- SetLoginStatus(ash::user::LOGGED_IN_NONE);
+ SetLoginStatus(ash::LoginStatus::NOT_LOGGED_IN);
EXPECT_TRUE(CreateDetailedMenu());
EXPECT_TRUE(IsSpokenFeedbackMenuShownOnDetailMenu());
EXPECT_TRUE(IsHighContrastMenuShownOnDetailMenu());
@@ -863,7 +861,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, CheckMenuVisibilityOnDetailMenu) {
EXPECT_FALSE(IsSettingsShownOnDetailMenu());
CloseDetailMenu();
- SetLoginStatus(ash::user::LOGGED_IN_USER);
+ SetLoginStatus(ash::LoginStatus::USER);
EXPECT_TRUE(CreateDetailedMenu());
EXPECT_TRUE(IsSpokenFeedbackMenuShownOnDetailMenu());
EXPECT_TRUE(IsHighContrastMenuShownOnDetailMenu());
@@ -875,7 +873,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, CheckMenuVisibilityOnDetailMenu) {
EXPECT_TRUE(IsSettingsShownOnDetailMenu());
CloseDetailMenu();
- SetLoginStatus(ash::user::LOGGED_IN_LOCKED);
+ SetLoginStatus(ash::LoginStatus::LOCKED);
EXPECT_TRUE(CreateDetailedMenu());
EXPECT_TRUE(IsSpokenFeedbackMenuShownOnDetailMenu());
EXPECT_TRUE(IsHighContrastMenuShownOnDetailMenu());
@@ -892,7 +890,7 @@ IN_PROC_BROWSER_TEST_P(TrayAccessibilityTest, CheckMenuVisibilityOnDetailMenu) {
ash::test::ShellTestApi test_api(ash::Shell::GetInstance());
test_api.SetSessionStateDelegate(session_state_delegate);
session_state_delegate->SetUserAddingScreenRunning(true);
- SetLoginStatus(ash::user::LOGGED_IN_USER);
+ SetLoginStatus(ash::LoginStatus::USER);
EXPECT_TRUE(CreateDetailedMenu());
EXPECT_TRUE(IsSpokenFeedbackMenuShownOnDetailMenu());
EXPECT_TRUE(IsHighContrastMenuShownOnDetailMenu());
« no previous file with comments | « chrome/browser/chromeos/shutdown_policy_browsertest.cc ('k') | chrome/browser/ui/ash/network_connect_delegate_chromeos.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698