Record Mobile Operator ID in UMA report. This is Android only for now.

net/base/network_change_notifier.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/base/network_change_notifier.h"
 
 #include "base/metrics/histogram.h"
+#include "base/metrics/sparse_histogram.h"
 #include "base/synchronization/lock.h"
 #include "base/threading/thread_checker.h"
 #include "build/build_config.h"
 #include "net/base/net_util.h"
 #include "net/base/network_change_notifier_factory.h"
 #include "net/dns/dns_config_service.h"
 #include "net/url_request/url_request.h"
 #include "url/gurl.h"
 
+#if defined(OS_ANDROID)
+#include "net/android/network_library.h"
+#endif
+
 #if defined(OS_WIN)
 #include "net/base/network_change_notifier_win.h"
 #elif defined(OS_LINUX) && !defined(OS_CHROMEOS)
 #include "net/base/network_change_notifier_linux.h"
 #elif defined(OS_MACOSX)
 #include "net/base/network_change_notifier_mac.h"
 #endif
 
 namespace net {
 
@@ skipping 191 matching lines @@
             base::TimeDelta::FromSeconds(5)) {
           // We can compare this sum with the sum of NCN.OfflineDataRecv.
           UMA_HISTOGRAM_COUNTS_10000(
               "NCN.OfflineDataRecvAny5sBeforeOnline",
               offline_packets_received_);
         }
 
         UMA_HISTOGRAM_MEDIUM_TIMES("NCN.OfflineDataRecvUntilOnline",
                                    now - last_offline_packet_received_);
       }
+
+#if defined(OS_ANDROID)
+      if (type == NetworkChangeNotifier::CONNECTION_2G ||
+          type == NetworkChangeNotifier::CONNECTION_3G ||
+          type == NetworkChangeNotifier::CONNECTION_4G) {
+        std::string mcc_mnc = android::GetTelephonyNetworkOperator();
+        if (!mcc_mnc.empty()) {
+          int value = atoi(mcc_mnc.c_str());

[Alexei Svitkine (slow), 2014/04/29 16:35:44]

Nit: Use StringToUint() from base/strings/string_number_conversions.h instead.

If you use that and check its return value, you don't need the empty() check
either.

[bolian, 2014/04/29 17:36:44]

Done.

+          if (value > 0) {
+            UMA_HISTOGRAM_SPARSE_SLOWLY(
+                "NCN.NetworkOperatorMCCMNC_ConnectionChange", value);
+          }
+        }
+      }
+#endif
+
     } else {
       UMA_HISTOGRAM_MEDIUM_TIMES("NCN.OfflineChange", state_duration);
     }
     UMA_HISTOGRAM_MEDIUM_TIMES(
         "NCN.IPAddressChangeToConnectionTypeChange",
         now - last_ip_address_change_);
 
     offline_packets_received_ = 0;
     bytes_read_since_last_connection_change_ = 0;
     peak_kbps_since_last_connection_change_ = 0;
@@ skipping 504 matching lines @@
   DCHECK(g_network_change_notifier);
   g_network_change_notifier = NULL;
 }
 
 NetworkChangeNotifier::DisableForTest::~DisableForTest() {
   DCHECK(!g_network_change_notifier);
   g_network_change_notifier = network_change_notifier_;
 }
 
 }  // namespace net