Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(305)

Unified Diff: chrome/browser/chromeos/policy/device_status_collector.cc

Issue 2396103002: Create policy ReportArcStatus (Closed)
Patch Set: Rebase Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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);
if (!report_hardware_status_) {
ClearCachedResourceUsage();

Powered by Google App Engine
This is Rietveld 408576698