Add device policy to disallow shutdown

chrome/browser/chromeos/settings/device_settings_provider.cc

Index: chrome/browser/chromeos/settings/device_settings_provider.cc
diff --git a/chrome/browser/chromeos/settings/device_settings_provider.cc b/chrome/browser/chromeos/settings/device_settings_provider.cc
index 7c2196f563169d377d386ca0f534efcb83240d41..6ca169fcce89449e87d097469aa8e223808b7302 100644
--- a/chrome/browser/chromeos/settings/device_settings_provider.cc
+++ b/chrome/browser/chromeos/settings/device_settings_provider.cc
@@ -39,48 +39,49 @@ namespace {
 
 // List of settings handled by the DeviceSettingsProvider.
 const char* const kKnownSettings[] = {
-  kAccountsPrefAllowGuest,
-  kAccountsPrefAllowNewUser,
-  kAccountsPrefDeviceLocalAccounts,
-  kAccountsPrefDeviceLocalAccountAutoLoginBailoutEnabled,
-  kAccountsPrefDeviceLocalAccountAutoLoginDelay,
-  kAccountsPrefDeviceLocalAccountAutoLoginId,
-  kAccountsPrefDeviceLocalAccountPromptForNetworkWhenOffline,
-  kAccountsPrefEphemeralUsersEnabled,
-  kAccountsPrefShowUserNamesOnSignIn,
-  kAccountsPrefSupervisedUsersEnabled,
-  kAccountsPrefTransferSAMLCookies,
-  kAccountsPrefUsers,
-  kAllowRedeemChromeOsRegistrationOffers,
-  kAllowedConnectionTypesForUpdate,
-  kAppPack,
-  kAttestationForContentProtectionEnabled,
-  kDeviceAttestationEnabled,
-  kDeviceOwner,
-  kIdleLogoutTimeout,
-  kIdleLogoutWarningDuration,
-  kPolicyMissingMitigationMode,
-  kReleaseChannel,
-  kReleaseChannelDelegated,
-  kReportDeviceActivityTimes,
-  kReportDeviceBootMode,
-  kReportDeviceLocation,
-  kReportDeviceNetworkInterfaces,
-  kReportDeviceUsers,
-  kReportDeviceVersionInfo,
-  kScreenSaverExtensionId,
-  kScreenSaverTimeout,
-  kServiceAccountIdentity,
-  kSignedDataRoamingEnabled,
-  kStartUpFlags,
-  kStartUpUrls,
-  kStatsReportingPref,
-  kSystemTimezonePolicy,
-  kSystemUse24HourClock,
-  kUpdateDisabled,
-  kVariationsRestrictParameter,
-  kDeviceDisabled,
-  kDeviceDisabledMessage,
+    kAccountsPrefAllowGuest,

[Daniel Erat, 2014/12/05 19:06:11]

did clang-format suggest this change?

[cschuet (SLOW), 2014/12/09 08:50:11]

Oh, yes I didn't even pay attention to this. clang-format introduced this
formatting change. Do you want me to revert it?

[Daniel Erat, 2014/12/09 16:00:36]

it's okay with me if you leave it at four. i think that chromium has
historically used two, but i see both styles when i grep -A 2 for '\[\] = {$',
so i'm fine with just going to whatever clang-format says.

+    kAccountsPrefAllowNewUser,
+    kAccountsPrefDeviceLocalAccounts,
+    kAccountsPrefDeviceLocalAccountAutoLoginBailoutEnabled,
+    kAccountsPrefDeviceLocalAccountAutoLoginDelay,
+    kAccountsPrefDeviceLocalAccountAutoLoginId,
+    kAccountsPrefDeviceLocalAccountPromptForNetworkWhenOffline,
+    kAccountsPrefEphemeralUsersEnabled,
+    kAccountsPrefShowUserNamesOnSignIn,
+    kAccountsPrefSupervisedUsersEnabled,
+    kAccountsPrefTransferSAMLCookies,
+    kAccountsPrefUsers,
+    kAllowRedeemChromeOsRegistrationOffers,
+    kAllowedConnectionTypesForUpdate,
+    kAppPack,
+    kAttestationForContentProtectionEnabled,
+    kDeviceAttestationEnabled,
+    kDeviceOwner,
+    kIdleLogoutTimeout,
+    kIdleLogoutWarningDuration,
+    kPolicyMissingMitigationMode,
+    kReleaseChannel,
+    kReleaseChannelDelegated,
+    kReportDeviceActivityTimes,
+    kReportDeviceBootMode,
+    kReportDeviceLocation,
+    kReportDeviceNetworkInterfaces,
+    kReportDeviceUsers,
+    kReportDeviceVersionInfo,
+    kScreenSaverExtensionId,
+    kScreenSaverTimeout,
+    kServiceAccountIdentity,
+    kSignedDataRoamingEnabled,
+    kStartUpFlags,
+    kStartUpUrls,
+    kStatsReportingPref,
+    kSystemTimezonePolicy,
+    kSystemUse24HourClock,
+    kUpdateDisabled,
+    kVariationsRestrictParameter,
+    kDeviceDisabled,
+    kDeviceDisabledMessage,
+    kShutdownAllowed,
 };
 
 bool HasOldMetricsFile() {
@@ -120,6 +121,10 @@ void DecodeLoginPolicies(
   }
 
   new_values_cache->SetBoolean(
+      kShutdownAllowed, !policy.has_shutdown_allowed() ||
+                            policy.shutdown_allowed().shutdown_allowed());

[Daniel Erat, 2014/12/05 19:06:11]

this indenting is weird; please follow the style that's used in the next block:

new_values_cache->SetBoolean(
    kShutdownAllowed,
    !policy.has_shutdown_allowed() ||
    policy.shutdown_allowed().shutdown_allowed());

[cschuet (SLOW), 2014/12/08 14:37:43]

Done.

+
+  new_values_cache->SetBoolean(
       kAccountsPrefAllowGuest,
       !policy.has_guest_mode_enabled() ||
       !policy.guest_mode_enabled().has_guest_mode_enabled() ||