Aggregate device activity, and report per-day activity in device status reports.

chrome/browser/policy/device_status_collector.h

Index: chrome/browser/policy/device_status_collector.h
diff --git a/chrome/browser/policy/device_status_collector.h b/chrome/browser/policy/device_status_collector.h
index 1d6d1f0b57d3a5436f1c5de8a78cc39e7b07f6b6..2adbc5fb93e8741762485e5deb25d8f4d6f884e4 100644
--- a/chrome/browser/policy/device_status_collector.h
+++ b/chrome/browser/policy/device_status_collector.h
@@ -51,10 +51,19 @@ class DeviceStatusCollector : public content::NotificationObserver {
   // Callback which receives the results of the idle state check.
   void IdleStateCallback(IdleState state);
 
-  // The maximum number of active periods timestamps to be stored.
-  unsigned int max_stored_active_periods_;
+  // The number of days in the past to store device activity.
+  // This is kept in case device status uploads fail for a number of days.
+  unsigned int max_stored_past_activity_days_;
+
+  // The number of days in the future to store device activity.
+  // When changing the system time and/or timezones, it's possible to record
+  // activity time that is slightly in the future.
+  unsigned int max_stored_future_activity_days_;
 
  private:
+  // Cleans up the locally stored activity periods if there are too many.

[Mattias Nissler (ping if slow), 2012/02/15 14:56:21]

s/if there are too many/that fall outside of the activity collection window/ or
something alike.

[Patrick Dubroy, 2012/02/15 15:37:25]

Done.

+  void PruneStoredActivityPeriods(base::Time base_time);
+
   void AddActivePeriod(base::Time start, base::Time end);
 
   // Callbacks from chromeos::VersionLoader.
@@ -88,9 +97,6 @@ class DeviceStatusCollector : public content::NotificationObserver {
   // The last time an idle state check was performed.
   base::Time last_idle_check_;
 
-  // The idle state the last time it was checked.
-  IdleState last_idle_state_;
-
   base::RepeatingTimer<DeviceStatusCollector> timer_;
 
   chromeos::VersionLoader version_loader_;