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

Unified Diff: chromeos/login/login_state.cc

Issue 450363002: Simplify LoginState methods. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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: chromeos/login/login_state.cc
diff --git a/chromeos/login/login_state.cc b/chromeos/login/login_state.cc
index eba009a91d29db15457404cc008a41660f75ee3d..9fd0c870c45bb5462cbda06c22cc4c48d5feedec 100644
--- a/chromeos/login/login_state.cc
+++ b/chromeos/login/login_state.cc
@@ -92,27 +92,16 @@ bool LoginState::IsInSafeMode() const {
return logged_in_state_ == LOGGED_IN_SAFE_MODE;
}
-bool LoginState::IsGuestUser() const {
- if (!IsUserLoggedIn())
- return false;
- switch (logged_in_user_type_) {
- case LOGGED_IN_USER_NONE:
- case LOGGED_IN_USER_REGULAR:
- case LOGGED_IN_USER_OWNER:
- case LOGGED_IN_USER_SUPERVISED:
- case LOGGED_IN_USER_KIOSK_APP:
- return false;
- case LOGGED_IN_USER_GUEST:
- case LOGGED_IN_USER_RETAIL_MODE:
- case LOGGED_IN_USER_PUBLIC_ACCOUNT:
- return true;
- }
- NOTREACHED();
- return false;
+bool LoginState::IsGuestSessionUser() const {
+ return logged_in_user_type_ == LOGGED_IN_USER_GUEST;
+}
+
+bool LoginState::IsPublicSessionUser() const {
+ return logged_in_user_type_ == LOGGED_IN_USER_PUBLIC_ACCOUNT;
}
bool LoginState::IsKioskApp() const {
- return logged_in_user_type_ == LoginState::LOGGED_IN_USER_KIOSK_APP;
+ return logged_in_user_type_ == LOGGED_IN_USER_KIOSK_APP;
}
bool LoginState::UserHasNetworkProfile() const {

Powered by Google App Engine
This is Rietveld 408576698