commit b84ba34aec59d083980d840fb845a3e2bea893cb
author Thieu Le <thieule@chromium.org> Fri Mar 02 15:15:19 2012 -0800
committer Gerrit <chrome-bot@google.com> Mon Mar 05 15:52:48 2012 -0800
tree 497fd2768016241c7dbb771d7d71c814f0e03527
parent f20994fa34cea805b8ab1140542285bbf0a1faf4

shill: Add support for Network.*.TimeResumeToReady

BUG=chromium-os:24440
TEST=Unit tests, network_WiFiManager, manually suspend/resume system

Change-Id: Ica618f2c713bac92f609d7894b16b0dd93af1645
Reviewed-on: https://gerrit.chromium.org/gerrit/17263
Commit-Ready: Thieu Le <thieule@chromium.org>
Reviewed-by: Thieu Le <thieule@chromium.org>
Tested-by: Thieu Le <thieule@chromium.org>

wifi_service.cc

diff --git a/wifi_service.cc b/wifi_service.cc
index f3dbe56..8b9250f 100644
--- a/wifi_service.cc
+++ b/wifi_service.cc
@@ -334,9 +334,8 @@
                                             Service::kStateConfiguring);
 }
 
-void WiFiService::SendPostReadyStateMetrics() const {
-  // TODO(thieule): Send physical mode and security metrics.
-  // crosbug.com/24441
+void WiFiService::SendPostReadyStateMetrics(
+    int64 time_resume_to_ready_milliseconds) const {
   metrics()->SendEnumToUMA(
       metrics()->GetFullMetricName(Metrics::kMetricNetworkChannel,
                                    technology()),
@@ -358,6 +357,16 @@
                                    technology()),
       security_uma,
       Metrics::kMetricNetworkSecurityMax);
+
+  if (time_resume_to_ready_milliseconds > 0) {
+    metrics()->SendToUMA(
+        metrics()->GetFullMetricName(
+            Metrics::kMetricTimeResumeToReadyMilliseconds, technology()),
+        time_resume_to_ready_milliseconds,
+        Metrics::kTimerHistogramMillisecondsMin,
+        Metrics::kTimerHistogramMillisecondsMax,
+        Metrics::kTimerHistogramNumBuckets);
+  }
 }
 
 // private methods