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

Unified Diff: chrome/browser/chromeos/device_settings_provider.cc

Issue 9348105: Aggregate device activity, and report per-day activity in device status reports. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add a few more unit tests. Created 8 years, 10 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/device_settings_provider.cc
diff --git a/chrome/browser/chromeos/device_settings_provider.cc b/chrome/browser/chromeos/device_settings_provider.cc
index abfbb7a282b544a6e71f94492879a9b4b7c29f55..da02ca7ce7202ec4b95c97b47976100291fd8d8c 100644
--- a/chrome/browser/chromeos/device_settings_provider.cc
+++ b/chrome/browser/chromeos/device_settings_provider.cc
@@ -431,10 +431,8 @@ void DeviceSettingsProvider::UpdateValuesCache() {
new_values_cache.SetBoolean(kReportDeviceVersionInfo,
pol.device_reporting().report_version_info());
}
- if (pol.device_reporting().has_report_activity_times()) {
- new_values_cache.SetBoolean(kReportDeviceActivityTimes,
- pol.device_reporting().report_activity_times());
- }
+ // TODO(dubroy): Re-add device activity time policy here when the UI
+ // to notify the user has been implemented (http://crosbug.com/26252).
Mattias Nissler (ping if slow) 2012/02/15 14:56:21 Umm... I guess you want to change this :)
Patrick Dubroy 2012/02/15 15:37:25 N/A.
if (pol.device_reporting().has_report_boot_mode()) {
new_values_cache.SetBoolean(kReportDeviceBootMode,
pol.device_reporting().report_boot_mode());
« no previous file with comments | « no previous file | chrome/browser/policy/device_status_collector.h » ('j') | chrome/browser/policy/device_status_collector.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698