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

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: . 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
« no previous file with comments | « chrome/browser/metrics/metrics_network_observer.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..0a3b7e9ae6b395c267ff5bf6c727c1bcfffd0342 100644
--- a/chrome/common/metrics/proto/system_profile.proto
+++ b/chrome/common/metrics/proto/system_profile.proto
@@ -287,6 +287,14 @@ message SystemProfileProto {
}
// The physical layer mode of the associated wifi access point, if any.
optional WifiPHYLayerProtocol wifi_phy_layer_protocol = 4;
+
+ // Network Operator Information.
+ message NetworkOperator {
+ // MCC+MNC (mobile country code + mobile network code) of the
+ // the current registered operator.
Alexei Svitkine (slow) 2014/04/23 16:54:11 What is the "current registered operator"? Can th
bolian 2014/04/23 17:29:39 I removed the message type since there is only one
+ optional string id = 1;
Alexei Svitkine (slow) 2014/04/23 16:54:11 Should this field be called mmc_mnc? I think that'
bolian 2014/04/23 17:29:39 Done. Changed id to mcc_mnc (you mean mcc_mnc, rig
+ }
+ optional NetworkOperator network_operator = 5;
}
optional Network network = 13;
« no previous file with comments | « chrome/browser/metrics/metrics_network_observer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698