Expose kDeviceLoginScreenDomainAutoComplete through CrosSettings

chrome/browser/chromeos/policy/device_policy_decoder_chromeos.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/policy/device_policy_decoder_chromeos.h"
 
 #include <limits>
 #include <string>
 
 #include "base/callback.h"
@@ skipping 577 matching lines @@
 
     if (container.has_login_screen_default_virtual_keyboard_enabled()) {
       policies->Set(
           key::kDeviceLoginScreenDefaultVirtualKeyboardEnabled,
           POLICY_LEVEL_MANDATORY,
           POLICY_SCOPE_MACHINE,
           new base::FundamentalValue(
               container.login_screen_default_virtual_keyboard_enabled()),
           NULL);
     }
-
-    // The behavior when policy is not set and when it is set to an empty string
-    // is the same. Thus lets add policy to the map only if it is set and its
-    // value is not an empty string.
-    if (container.has_login_screen_domain_auto_complete() &&
-        !container.login_screen_domain_auto_complete().empty()) {
-      policies->Set(
-          key::kDeviceLoginScreenDomainAutoComplete, POLICY_LEVEL_MANDATORY,
-          POLICY_SCOPE_MACHINE,
-          new base::StringValue(container.login_screen_domain_auto_complete()),
-          nullptr);
-    }
   }
 }
 
 void DecodeGenericPolicies(const em::ChromeDeviceSettingsProto& policy,
                            PolicyMap* policies) {
   if (policy.has_device_policy_refresh_rate()) {
     const em::DevicePolicyRefreshRateProto& container(
         policy.device_policy_refresh_rate());
     if (container.has_device_policy_refresh_rate()) {
       policies->Set(
@@ skipping 143 matching lines @@
   if (policy.has_extension_cache_size()) {
     const em::ExtensionCacheSizeProto& container(policy.extension_cache_size());
     if (container.has_extension_cache_size()) {
       policies->Set(
           key::kExtensionCacheSize, POLICY_LEVEL_MANDATORY,
           POLICY_SCOPE_MACHINE,
           DecodeIntegerValue(container.extension_cache_size()).release(),
           nullptr);
     }
   }
+
+  if (policy.has_login_screen_domain_auto_complete()) {
+    const em::LoginScreenDomainAutoCompleteProto& container(
+        policy.login_screen_domain_auto_complete());
+    policies->Set(
+        key::kDeviceLoginScreenDomainAutoComplete, POLICY_LEVEL_MANDATORY,
+        POLICY_SCOPE_MACHINE,
+        new base::StringValue(container.login_screen_domain_auto_complete()),
+        nullptr);
+  }
 }
 
 }  // namespace
 
 void DecodeDevicePolicy(const em::ChromeDeviceSettingsProto& policy,
                         PolicyMap* policies) {
   // TODO(achuith): Remove this once crbug.com/263527 is resolved.
   VLOG(2) << "DecodeDevicePolicy " << policy.SerializeAsString();
 
   // Decode the various groups of policies.
   DecodeLoginPolicies(policy, policies);
   DecodeNetworkPolicies(policy, policies);
   DecodeReportingPolicies(policy, policies);
   DecodeAutoUpdatePolicies(policy, policies);
   DecodeAccessibilityPolicies(policy, policies);
   DecodeGenericPolicies(policy, policies);
 }
 
 }  // namespace policy