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

Unified Diff: chrome/browser/chromeos/login/session/user_session_manager.cc

Issue 1306183007: Switch to id-based affiliation determination (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
Index: chrome/browser/chromeos/login/session/user_session_manager.cc
diff --git a/chrome/browser/chromeos/login/session/user_session_manager.cc b/chrome/browser/chromeos/login/session/user_session_manager.cc
index 7c2aa36bbb7aabb19e729377b6c180ee8ee124f6..f06ccea8c5cb506b6e4b18be9fecf91d6a2d154d 100644
--- a/chrome/browser/chromeos/login/session/user_session_manager.cc
+++ b/chrome/browser/chromeos/login/session/user_session_manager.cc
@@ -1017,14 +1017,14 @@ void UserSessionManager::UserProfileInitialized(Profile* profile,
// authentication cookies set by a SAML IdP on subsequent logins after the
// first.
bool transfer_saml_auth_cookies_on_subsequent_login = false;
- if (has_auth_cookies_ &&
- g_browser_process->platform_part()
- ->browser_policy_connector_chromeos()
- ->GetUserAffiliation(account_id.GetUserEmail()) ==
- policy::USER_AFFILIATION_MANAGED) {
- CrosSettings::Get()->GetBoolean(
- kAccountsPrefTransferSAMLCookies,
- &transfer_saml_auth_cookies_on_subsequent_login);
+ if (has_auth_cookies_) {
+ const user_manager::User* user =
+ user_manager::UserManager::Get()->FindUser(account_id);
+ if (user->is_affiliated()) {
+ CrosSettings::Get()->GetBoolean(
+ kAccountsPrefTransferSAMLCookies,
+ &transfer_saml_auth_cookies_on_subsequent_login);
+ }
}
// Transfers authentication-related data from the profile that was used for

Powered by Google App Engine
This is Rietveld 408576698