Create policy ReportArcStatus

chrome/browser/chromeos/policy/device_status_collector.cc

Index: chrome/browser/chromeos/policy/device_status_collector.cc
diff --git a/chrome/browser/chromeos/policy/device_status_collector.cc b/chrome/browser/chromeos/policy/device_status_collector.cc
index 48025385d0c65fdeff8d3f1879b4b7fb50509e09..56bbbf3304f8a887063487f756461113a74ef75d 100644
--- a/chrome/browser/chromeos/policy/device_status_collector.cc
+++ b/chrome/browser/chromeos/policy/device_status_collector.cc
@@ -472,6 +472,8 @@ DeviceStatusCollector::DeviceStatusCollector(
   running_kiosk_app_subscription_ = cros_settings_->AddSettingsObserver(
       chromeos::kReportRunningKioskApp, callback);
 
+  report_arc_status_pref_.Init(prefs::kReportArcStatus, local_state_, callback);
+
   // Fetch the current values of the policies.
   UpdateReportingSettings();
 
@@ -498,6 +500,7 @@ DeviceStatusCollector::~DeviceStatusCollector() {
 void DeviceStatusCollector::RegisterPrefs(PrefRegistrySimple* registry) {
   registry->RegisterDictionaryPref(prefs::kDeviceActivityTimes,
                                    new base::DictionaryValue);
+  registry->RegisterBooleanPref(prefs::kReportArcStatus, true);
 }
 
 void DeviceStatusCollector::CheckIdleState() {
@@ -551,8 +554,7 @@ void DeviceStatusCollector::UpdateReportingSettings() {
     report_kiosk_session_status_ = true;
   }
 
-  // TODO(phweiss): Create policy to control this, and turn it on by default.
-  report_android_status_ = false;
+  report_android_status_ = local_state_->GetBoolean(prefs::kReportArcStatus);

[bartfab (slow), 2016/10/10 14:40:14]

If this is a per-profile pref, you should not be accessing it via local state
but via a profile.

[phweiss, 2016/10/10 14:46:00]

Changing the documentation to "not per-profile".

 
   if (!report_hardware_status_) {
     ClearCachedResourceUsage();