Add Chrome OS-specific controlled setting indicators

chrome/test/data/policy/policy_test_cases.json

 {
   "-- Template --": {
     "intro": "Top-level entries map a policy name to its test parameters, described below.",
 
     "os": ["List of operating systems that support this policy. Valid values:", "win", "linux", "mac", "chromeos"],
     "official_only": "Whether this policy exists in official builds only. Defaults to |false| if not specified.",
     "can_be_recommended": "Whether a recommended value may be set for the policy. Defaults to |false| if not specified.",
     "test_policy": "A policy dictionary that should trigger the managed banner in the settings UI. Usually just sets the current policy.",
     "settings_pages": ["A list of settings pages where the banner should be visible when this policy is set."],
     "note": "If the policy affects any preferences, the following array should be specified with one entry per such preference.",
@@ skipping 1517 matching lines @@
 
   "DisableScreenshots": {
     "os": ["win", "linux", "mac", "chromeos"],
     "test_policy": { "DisableScreenshots": true },
     "settings_pages": [],
     "pref_mappings": [
       { "pref": "disable_screenshots" }
     ]
   },
 
-  "----- Chrome OS policies ------------------------------------------------": {
-    "note": "TODO(frankf): Add prefs for these after crosbug.com/28756 is fixed.",
-    "note 2": "TODO(joaodasilva): that bug is now obsolete and these tests can be expanded to chromeos."
-  },
+  "----- Chrome OS policies ------------------------------------------------": {},
 
   "ChromeOsLockOnIdleSuspend": {
     "os": ["chromeos"],
+    "can_be_recommended": true,
     "test_policy": { "ChromeOsLockOnIdleSuspend": true },
-    "settings_pages": ["chrome://settings-frame"]
+    "settings_pages": ["chrome://settings-frame"],
+    "pref_mappings": [
+      { "pref": "settings.enable_screen_lock",
+        "indicator_tests": [
+          { "policy": { "ChromeOsLockOnIdleSuspend": true } }
+        ]
+      }
+    ]
   },
 
   "PolicyRefreshRate": {
     "os": ["chromeos"],
     "test_policy": { "PolicyRefreshRate": 300000 },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "policy.user_refresh_rate",
+        "local_state": true
+      }
+    ]
   },
 
   "OpenNetworkConfiguration": {
     "os": ["chromeos"],
     "test_policy": { "OpenNetworkConfiguration": "" },
     "settings_pages": []
   },
 
   "DriveDisabled": {
     "os": ["chromeos"],
     "test_policy": { "DriveDisabled": true },
-    "settings_pages": []
+    "settings_pages": [],
+
+    "note": "TODO(bartfab) This actually maps to |gdata.disabled|. However, setting the policy in order to test its pref mapping spins up FileBrowserEventRouter which is then unable to shut down cleanly in the browser test environment."

[Joao da Silva, 2012/10/24 16:26:17]

Can't that be fixed? :-)

[bartfab (slow), 2012/10/24 17:33:51]

I worked around the issue and filed crbug.com/157611 for the underlying cause.

   },
 
   "DriveDisabledOverCellular": {
     "os": ["chromeos"],
     "test_policy": { "DriveDisabledOverCellular": true },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "gdata.cellular.disabled" }
+    ]
   },
 
   "PinnedLauncherApps": {
     "os": ["chromeos"],
+    "can_be_recommended": true,
     "test_policy": { "PinnedLauncherApps": [] },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "pinned_launcher_apps" }
+    ]
   },
 
   "ExternalStorageDisabled": {
     "os": ["chromeos"],
     "test_policy": { "ExternalStorageDisabled": true },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "hardware.external_storage_disabled" }
+    ]
   },
 
   "AudioOutputAllowed": {
     "os": ["chromeos"],
     "test_policy": { "AudioOutputAllowed": true },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "hardware.audio_output_enabled",
+        "local_state": true }
+    ]
   },
 
   "AudioCaptureAllowed": {
     "os": ["chromeos"],
     "test_policy": { "AudioCaptureAllowed": true },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "hardware.audio_capture_enabled",
+        "local_state": true }
+    ]
   },
 
   "----- Chrome OS device policies ---------------------------------------": {},
 
   "DevicePolicyRefreshRate": {
     "os": ["chromeos"],
     "test_policy": { "DevicePolicyRefreshRate": 300000 },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "policy.device_refresh_rate",
+        "local_state": true }
+    ]
   },
 
   "ChromeOsReleaseChannel": {
     "os": ["chromeos"],
     "test_policy": { "ChromeOsReleaseChannel": "stable-channel" },
     "settings_pages": []
   },
 
   "ChromeOsReleaseChannelDelegated": {
     "os": ["chromeos"],
@@ skipping 51 matching lines @@
 
   "DeviceDataRoamingEnabled": {
     "os": ["chromeos"],
     "test_policy": { "DeviceDataRoamingEnabled": true },
     "settings_pages": []
   },
 
   "DeviceMetricsReportingEnabled": {
     "os": ["chromeos"],
     "test_policy": { "DeviceMetricsReportingEnabled": true },
-    "settings_pages": []
+    "settings_pages": [],
+    "pref_mappings": [
+      { "pref": "cros.metrics.reportingEnabled",
+        "indicator_test_setup_js": "Preferences.getInstance().addEventListener('alternate_error_pages.enabled', function(event) { Preferences.prefsChangedCallback(['cros.metrics.reportingEnabled', {value: event.value.value, controlledBy: event.value.controlledBy, disabled: event.value.disabled}]); });",
+        "indicator_tests": [
+          { "policy": { "AlternateErrorPagesEnabled": true } }
+        ]
+      }
+    ],
+
+    "note": "TODO(bartfab): The |indicator_test_setup_js| above is a hack that makes |cros.metrics.reportingEnabled| track the status of the entirely unrelated |alternate_error_pages.enabled| pref. This is because cros settings cannot currently be made policy-controlled in browser tests. Remove this hack once that restriction is lifted."
   },
 
   "DeviceEphemeralUsersEnabled": {
     "os": ["chromeos"],
     "test_policy": { "DeviceEphemeralUsersEnabled": true },
     "settings_pages": []
   },
 
   "DeviceIdleLogoutTimeout": {
     "os": ["chromeos"],
@@ skipping 98 matching lines @@
     "test_policy": { "GCFUserDataDir": "${user_name}/test-frame" },
     "settings_pages": []
   },
 
   "AdditionalLaunchParameters": {
     "os": [],
     "test_policy": { "AdditionalLaunchParameters": "--enable-media-stream" },
     "settings_pages": []
   }
 }