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

Unified Diff: ash/common/session/session_state_delegate.cc

Issue 2512693003: ash: Replace SessionStateDelegate::CanAddUserToMultiProfile (Closed)
Patch Set: for #2 Created 4 years, 1 month 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
« no previous file with comments | « ash/common/session/session_state_delegate.h ('k') | ash/common/session/session_types.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/session/session_state_delegate.cc
diff --git a/ash/common/session/session_state_delegate.cc b/ash/common/session/session_state_delegate.cc
index fc1a94978eaa1fb71443547239523f9f5c3551d7..b4be363fa48c73c4b169bd9c6f0072b7609e312d 100644
--- a/ash/common/session/session_state_delegate.cc
+++ b/ash/common/session/session_state_delegate.cc
@@ -10,19 +10,14 @@ bool SessionStateDelegate::IsInSecondaryLoginScreen() const {
return GetSessionState() == session_manager::SessionState::LOGIN_SECONDARY;
}
-bool SessionStateDelegate::CanAddUserToMultiProfile(
- SessionStateDelegate::AddUserError* error) const {
- if (!IsMultiProfileAllowedByPrimaryUserPolicy()) {
- if (error)
- *error = ADD_USER_ERROR_NOT_ALLOWED_PRIMARY_USER;
- return false;
- }
- if (NumberOfLoggedInUsers() >= GetMaximumNumberOfLoggedInUsers()) {
- if (error)
- *error = ADD_USER_ERROR_MAXIMUM_USERS_REACHED;
- return false;
- }
- return true;
+AddUserSessionPolicy SessionStateDelegate::GetAddUserSessionPolicy() const {
+ if (!IsMultiProfileAllowedByPrimaryUserPolicy())
+ return AddUserSessionPolicy::ERROR_NOT_ALLOWED_PRIMARY_USER;
+
+ if (NumberOfLoggedInUsers() >= GetMaximumNumberOfLoggedInUsers())
+ return AddUserSessionPolicy::ERROR_MAXIMUM_USERS_REACHED;
+
+ return AddUserSessionPolicy::ALLOWED;
}
} // namespace ash
« no previous file with comments | « ash/common/session/session_state_delegate.h ('k') | ash/common/session/session_types.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698