commit ad1ec2c02af19d65b1302db59a1cebb1c9465516
author Thieu Le <thieule@chromium.org> Thu Jan 05 23:39:48 2012 +0000
committer Gerrit <chrome-bot@google.com> Fri Jan 06 11:15:33 2012 -0800
tree 2094fd8b29c02378205b54ff51e9c0971735e9a9
parent 8a3188dbc04245c17a4729d16a632547ce4bf585

shill: UMA stats Network.Shill.WiFi.Security and Network.Shill.WiFi.PhyMode

Add code to send WiFi security and physical mode UMA metrics.

BUG=chromium-os:24441
TEST=Unit tests

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

wifi_service.cc

diff --git a/wifi_service.cc b/wifi_service.cc
index 4882ccf..dd9d1c7 100644
--- a/wifi_service.cc
+++ b/wifi_service.cc
@@ -299,6 +299,22 @@
                                    technology()),
       Metrics::WiFiFrequencyToChannel(frequency_),
       Metrics::kMetricNetworkChannelMax);
+
+  DCHECK(physical_mode_ < Metrics::kWiFiNetworkPhyModeMax);
+  metrics()->SendEnumToUMA(
+      metrics()->GetFullMetricName(Metrics::kMetricNetworkPhyMode,
+                                   technology()),
+      static_cast<Metrics::WiFiNetworkPhyMode>(physical_mode_),
+      Metrics::kWiFiNetworkPhyModeMax);
+
+  Metrics::WiFiSecurity security_uma =
+      Metrics::WiFiSecurityStringToEnum(security_);
+  DCHECK(security_uma != Metrics::kWiFiSecurityUnknown);
+  metrics()->SendEnumToUMA(
+      metrics()->GetFullMetricName(Metrics::kMetricNetworkSecurity,
+                                   technology()),
+      security_uma,
+      Metrics::kMetricNetworkSecurityMax);
 }
 
 // private methods