Apply individual policies for the various parts of device status reports.

chrome/browser/chromeos/stub_cros_settings_provider.cc

Index: chrome/browser/chromeos/stub_cros_settings_provider.cc
diff --git a/chrome/browser/chromeos/stub_cros_settings_provider.cc b/chrome/browser/chromeos/stub_cros_settings_provider.cc
index b4690e765cc221fb75ea688a39a969400ecfc3ee..d5fede385ae5aa5795e172eecb9ce0041b68a80b 100644
--- a/chrome/browser/chromeos/stub_cros_settings_provider.cc
+++ b/chrome/browser/chromeos/stub_cros_settings_provider.cc
@@ -21,6 +21,9 @@ const char* kHandledSettings[] = {
   kAccountsPrefUsers,
   kDeviceOwner,
   kReleaseChannel,
+  kReportDeviceVersionInfo,
+  kReportDeviceActivityTimes,
+  kReportDeviceBootMode,
   kSettingProxyEverywhere,
   kSignedDataRoamingEnabled,
   kStatsReportingPref
@@ -34,6 +37,11 @@ StubCrosSettingsProvider::StubCrosSettingsProvider(
   SetDefaults();
 }
 
+StubCrosSettingsProvider::StubCrosSettingsProvider()
+  : CrosSettingsProvider(CrosSettingsProvider::NotifyObserversCallback()) {
+  SetDefaults();
+}
+
 StubCrosSettingsProvider::~StubCrosSettingsProvider() {
 }
 
@@ -71,6 +79,10 @@ void StubCrosSettingsProvider::SetDefaults() {
   values_.SetBoolean(kAccountsPrefAllowNewUser, true);
   values_.SetBoolean(kAccountsPrefShowUserNamesOnSignIn, true);
   // |kDeviceOwner| will be set to the logged-in user by |UserManager|.
+
+  values_.SetBoolean(kReportDeviceVersionInfo, false);

[pastarmovj, 2012/01/27 15:39:03]

Are you sure you need those defaults here. In a real world scenario the policy
fields will not be set until the admin changes them on the server. I don't mind
having those here but you should be sure to work correctly if those are actually
not present in the policy blob.

[Patrick Dubroy, 2012/01/27 16:18:12]

No, you're right. Done.

+  values_.SetBoolean(kReportDeviceActivityTimes, false);
+  values_.SetBoolean(kReportDeviceBootMode, false);
 }
 
 }  // namespace chromeos