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

Unified Diff: chrome/common/metrics/proto/system_profile.proto

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: addressed comments. 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/common/metrics/proto/system_profile.proto
diff --git a/chrome/common/metrics/proto/system_profile.proto b/chrome/common/metrics/proto/system_profile.proto
index fd339ec5d26db48c249c9d30b0d7827b1cc0f50a..7cc559487b29e6ed52bcc0496270860173b613a3 100644
--- a/chrome/common/metrics/proto/system_profile.proto
+++ b/chrome/common/metrics/proto/system_profile.proto
@@ -287,6 +287,11 @@ message SystemProfileProto {
}
// The physical layer mode of the associated wifi access point, if any.
optional WifiPHYLayerProtocol wifi_phy_layer_protocol = 4;
+
+ // MCC+MNC (mobile country code + mobile network code) of the
+ // the current registered mobile network operator.
+ // If there is no mobile network, the value is the empty string.
Alexei Svitkine (slow) 2014/04/23 17:58:49 How about when it changes during the the course of
bolian 2014/04/24 00:59:24 Is MetricsLog::RecordEnvironment eventually called
+ optional string mcc_mnc = 5;
}
optional Network network = 13;

Powered by Google App Engine
This is Rietveld 408576698