Remove enrolled device check for saving login password.

Allow managed users logging into their own device to use their
login password for automatically authenticating to networks.

BUG=chromium:386606
TEST=run unit tests, manual test on kevin

Change-Id: I023472e12abaa39a03ea445358f68a7c1afa9391
Reviewed-on: https://chromium-review.googlesource.com/1058598
Reviewed-by: Mattias Nissler <[email protected]>
Reviewed-by: Achuith Bhandarkar <[email protected]>
Commit-Queue: May Lippert <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#563155}(cherry picked from commit ea3a9c7)
Reviewed-on: https://chromium-review.googlesource.com/1081070
Reviewed-by: May Lippert <[email protected]>
Cr-Commit-Position: refs/branch-heads/3440@{#66}
Cr-Branched-From: 010ddcf-refs/heads/master@{#561733}

Author: May Lippert

chrome/browser/chromeos/login/session/user_session_manager.cc

@@ -1082,17 +1082,6 @@ void UserSessionManager::StartCrosSession() {
 }
 
 void UserSessionManager::OnUserNetworkPolicyParsed(bool send_password) {
-  // Sanity check that we only send the password for enterprise users. See
-  // https://crbug.com/386606.
-  const bool is_enterprise_managed = g_browser_process->platform_part()
-                                         ->browser_policy_connector_chromeos()
-                                         ->IsEnterpriseManaged();
-  if (!is_enterprise_managed) {
-    LOG(WARNING) << "Attempting to save user password for non enterprise user.";
-    user_context_.GetMutablePasswordKey()->ClearSecret();
-    return;
-  }
-
   if (send_password) {
     if (user_context_.GetPasswordKey()->GetSecret().size() > 0) {
       DBusThreadManager::Get()->GetSessionManagerClient()->SaveLoginPassword(

chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc

@@ -747,13 +747,9 @@ void GaiaScreenHandler::HandleCompleteAuthentication(
   UserContext user_context(user_type,
                            GetAccountId(email, gaia_id, AccountType::GOOGLE));
   user_context.SetKey(Key(password));
-  // Only save the password for enterprise users. See https://crbug.com/386606.
-  const bool is_enterprise_managed = g_browser_process->platform_part()
-                                         ->browser_policy_connector_chromeos()
-                                         ->IsEnterpriseManaged();
-  if (is_enterprise_managed) {
-    user_context.SetPasswordKey(Key(password));
-  }
+  // Save the user's plaintext password for possible authentication to a
+  // network. See https://crbug.com/386606 for details.
+  user_context.SetPasswordKey(Key(password));
   user_context.SetAuthCode(auth_code);
   user_context.SetAuthFlow(using_saml
                                ? UserContext::AUTH_FLOW_GAIA_WITH_SAML

chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc

@@ -1229,13 +1229,9 @@ void SigninScreenHandler::HandleAuthenticateUser(const AccountId& account_id,
     user_context = UserContext(*user);
   }
   user_context.SetKey(Key(password));
-  // Only save the password for enterprise users. See https://crbug.com/386606.
-  const bool is_enterprise_managed = g_browser_process->platform_part()
-                                         ->browser_policy_connector_chromeos()
-                                         ->IsEnterpriseManaged();
-  if (is_enterprise_managed) {
-    user_context.SetPasswordKey(Key(password));
-  }
+  // Save the user's plaintext password for possible authentication to a
+  // network. See https://crbug.com/386606 for details.
+  user_context.SetPasswordKey(Key(password));
   user_context.SetIsUsingPin(authenticated_by_pin);
   if (account_id.GetAccountType() == AccountType::ACTIVE_DIRECTORY &&
       (user_context.GetUserType() !=