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

Unified Diff: chrome/browser/metrics/metrics_network_observer.h

Issue 246553003: Record Mobile Operator ID in UMA report. This is Android only for now. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@operator_id
Patch Set: . Created 6 years, 8 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/metrics/metrics_network_observer.h
diff --git a/chrome/browser/metrics/metrics_network_observer.h b/chrome/browser/metrics/metrics_network_observer.h
index bcec85e00e539d682742c01b0cc4d487cc2f99f6..e59e60e905bda7150f3552a35f6a80369eaaaa0f 100644
--- a/chrome/browser/metrics/metrics_network_observer.h
+++ b/chrome/browser/metrics/metrics_network_observer.h
@@ -39,6 +39,10 @@ class MetricsNetworkObserver
return wifi_phy_layer_protocol_is_ambiguous_;
}
+ std::string network_operator_id() const {
+ return network_operator_id_;
+ }
+
SystemProfileProto::Network::WifiPHYLayerProtocol
wifi_phy_layer_protocol() const;
@@ -49,6 +53,9 @@ class MetricsNetworkObserver
// net::GetWifiPHYLayerProtocol.
void OnWifiPHYLayerProtocolResult(net::WifiPHYLayerProtocol mode);
+ void GetNetworkOperatorID();
+ void OnNetworkOperatorID(std::string);
+
base::WeakPtrFactory<MetricsNetworkObserver> weak_ptr_factory_;
// True if |connection_type_| changed during the lifetime of the log.
@@ -62,6 +69,8 @@ class MetricsNetworkObserver
// net::GetWifiPHYLayerProtocol.
net::WifiPHYLayerProtocol wifi_phy_layer_protocol_;
+ std::string network_operator_id_;
+
DISALLOW_COPY_AND_ASSIGN(MetricsNetworkObserver);
};

Powered by Google App Engine
This is Rietveld 408576698