Add policies to control power management on the Chrome OS login screen

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 "base/logging.h"
 #include "base/values.h"
@@ skipping 559 matching lines @@
 
   if (policy.has_attestation_settings()) {
     if (policy.attestation_settings().has_attestation_enabled()) {
       policies->Set(key::kAttestationEnabledForDevice,
                     POLICY_LEVEL_MANDATORY,
                     POLICY_SCOPE_MACHINE,
                     Value::CreateBooleanValue(
                         policy.attestation_settings().attestation_enabled()));
     }
   }
+
+  if (policy.has_login_screen_power_management()) {
+    const em::LoginScreenPowerManagementProto& container(
+        policy.login_screen_power_management());
+    if (container.has_login_screen_power_management()) {
+      policies->Set(key::kDeviceLoginScreenPowerManagement,
+                    POLICY_LEVEL_MANDATORY,
+                    POLICY_SCOPE_MACHINE,
+                    Value::CreateStringValue(

[Mattias Nissler (ping if slow), 2013/07/08 17:36:57]

It'd be nice to pass a dictionary value here if possible.

[bartfab (slow), 2013/07/09 10:02:33]

Right now, string is the better choice. To get a dict, you have to parse the
JSON string from the protobuf. If you try parsing here and the string is
invalid, you get no policy at all and no error message in about:policy to tell
you what went wrong. If it remains a string here, the parsing happens later and
any parsing errors show up in about:policy.

There is also a second reason to leave it as a string: The 'dict' user policy
type is actually aliased to a string for now and only gets parsed in the
ConfigurationPolicyHandler. Since the JSON power management device policy is
meant to be the first step toward JSON power management policy throughout, it
should be consistent with what a 'dict' JSON power management user policy will
look like. If both are expressed as strings, the same code can be used to parse
them. Only once the 'dict' user policy type switches to actually being a
DictionaryValue should the device policy switch as well.

+                        container.login_screen_power_management()));
+    }
+  }
 }
 
 }  // namespace
 
 void DecodeDevicePolicy(const em::ChromeDeviceSettingsProto& policy,
                         PolicyMap* policies,
                         EnterpriseInstallAttributes* install_attributes) {
   // Decode the various groups of policies.
   DecodeLoginPolicies(policy, policies);
   DecodeKioskPolicies(policy, policies, install_attributes);
   DecodeNetworkPolicies(policy, policies, install_attributes);
   DecodeReportingPolicies(policy, policies);
   DecodeAutoUpdatePolicies(policy, policies);
   DecodeAccessibilityPolicies(policy, policies);
   DecodeGenericPolicies(policy, policies);
 }
 
 }  // namespace policy