Expose kDeviceLoginScreenDomainAutoComplete through CrosSettings

chrome/browser/chromeos/settings/device_settings_provider.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/settings/device_settings_provider.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/logging.h"
@@ skipping 33 matching lines @@
     kAccountsPrefDeviceLocalAccounts,
     kAccountsPrefDeviceLocalAccountAutoLoginBailoutEnabled,
     kAccountsPrefDeviceLocalAccountAutoLoginDelay,
     kAccountsPrefDeviceLocalAccountAutoLoginId,
     kAccountsPrefDeviceLocalAccountPromptForNetworkWhenOffline,
     kAccountsPrefEphemeralUsersEnabled,
     kAccountsPrefShowUserNamesOnSignIn,
     kAccountsPrefSupervisedUsersEnabled,
     kAccountsPrefTransferSAMLCookies,
     kAccountsPrefUsers,
+    kAccountsPrefDeviceLoginScreenDomainAutoComplete,
     kAllowRedeemChromeOsRegistrationOffers,
     kAllowedConnectionTypesForUpdate,
     kAttestationForContentProtectionEnabled,
     kDeviceAttestationEnabled,
     kDeviceOwner,
     kHeartbeatEnabled,
     kHeartbeatFrequency,
     kPolicyMissingMitigationMode,
     kReleaseChannel,
     kReleaseChannelDelegated,
@@ skipping 175 matching lines @@
       list->Append(new base::StringValue(*it));
     }
     new_values_cache->SetValue(kStartUpFlags, list);
   }
 
   if (policy.has_saml_settings()) {
     new_values_cache->SetBoolean(
         kAccountsPrefTransferSAMLCookies,
         policy.saml_settings().transfer_saml_cookies());
   }
+
+  // The behavior when policy is not set and when it is set to an empty string
+  // is the same. Thus lets add policy only if it is set and its value is not

[Mattias Nissler (ping if slow), 2015/04/07 12:29:45]

Question: Is there a use case for turning off auto-default to @gmail.com? If so,
the natural way to do this would be to configure an empty auto-completion
domain? Do we know what GAIA does if we give them an empty string as parameter?

[peletskyi, 2015/04/07 13:48:12]

This use case is not mentioned in the design doc go/cros-prd-autocomplete
Maybe more senior developers can provide more information

+  // an empty string.
+  if (policy.has_login_screen_domain_auto_complete() &&
+      policy.login_screen_domain_auto_complete()
+          .has_login_screen_domain_auto_complete() &&
+      !policy.login_screen_domain_auto_complete()
+           .login_screen_domain_auto_complete()
+           .empty()) {
+    new_values_cache->SetString(
+        kAccountsPrefDeviceLoginScreenDomainAutoComplete,
+        policy.login_screen_domain_auto_complete()
+            .login_screen_domain_auto_complete());
+  }
 }
 
 void DecodeNetworkPolicies(
     const em::ChromeDeviceSettingsProto& policy,
     PrefValueMap* new_values_cache) {
   // kSignedDataRoamingEnabled has a default value of false.
   new_values_cache->SetBoolean(
       kSignedDataRoamingEnabled,
       policy.has_data_roaming_enabled() &&
       policy.data_roaming_enabled().has_data_roaming_enabled() &&
@@ skipping 545 matching lines @@
 void DeviceSettingsProvider::AttemptMigration() {
   if (device_settings_service_->HasPrivateOwnerKey()) {
     PrefValueMap::const_iterator i;
     for (i = migration_values_.begin(); i != migration_values_.end(); ++i)
       DoSet(i->first, *i->second);
     migration_values_.Clear();
   }
 }
 
 }  // namespace chromeos