commit 5ef2e3911957dada3c9e5e8d42d90d0dd56d1a29
author Luigi Semenzato <semenzato@chromium.org> Tue Apr 15 15:15:02 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Apr 16 17:14:51 2014 +0000
tree 2e8cbaf7acece0cf413ebe152d57cb03f3d289c0
parent a5f4fe6b98ae5d214ecbe6a946d77240b9446409

metrics_daemon: report cumulative daily use.

This had been lost in the previous changes.  Also, the meaning
of DailyUse changes: it is now cumulative daily use.  But we should
revise these stats so I think it's OK for now.

BUG=chromium:339588
TEST=unit tests

Change-Id: I4894fef0ffb237633abcf949c01dd7ea5f5e5d6e
Reviewed-on: https://chromium-review.googlesource.com/195005
Reviewed-by: Daniel Erat <derat@chromium.org>
Tested-by: Luigi Semenzato <semenzato@chromium.org>
Commit-Queue: Luigi Semenzato <semenzato@chromium.org>

metrics/metrics_daemon.cc

diff --git a/metrics/metrics_daemon.cc b/metrics/metrics_daemon.cc
index b9d8cf0..1f20699 100644
--- a/metrics/metrics_daemon.cc
+++ b/metrics/metrics_daemon.cc
@@ -421,6 +421,7 @@
   daily_cycle_->Set(day);
 
   // Daily stats.
+  ReportDailyUse(active_use_seconds);
   SendCrashFrequencySample(any_crashes_daily_count_);
   SendCrashFrequencySample(user_crashes_daily_count_);
   SendCrashFrequencySample(kernel_crashes_daily_count_);
@@ -1125,18 +1126,16 @@
   return true;
 }
 
-// static
-void MetricsDaemon::ReportDailyUse(void* handle, int count) {
-  if (count <= 0)
+void MetricsDaemon::ReportDailyUse(int use_seconds) {
+  if (use_seconds <= 0)
     return;
 
-  MetricsDaemon* daemon = static_cast<MetricsDaemon*>(handle);
-  int minutes = (count + kSecondsPerMinute / 2) / kSecondsPerMinute;
-  daemon->SendSample("Logging.DailyUseTime",
-                     minutes,
-                     1,
-                     kMinutesPerDay,
-                     50);
+  int minutes = (use_seconds + kSecondsPerMinute / 2) / kSecondsPerMinute;
+  SendSample("Logging.DailyUseTime",
+             minutes,
+             1,
+             kMinutesPerDay * 30 * 2,  // cumulative---two months worth
+             50);
 }
 
 void MetricsDaemon::SendSample(const string& name, int sample,