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

chrome/browser/metrics/metrics_network_observer.cc

 // Copyright (c) 2013 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 "chrome/browser/metrics/metrics_network_observer.h"
 
 #include "base/compiler_specific.h"
 #include "base/task_runner_util.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "content/public/browser/browser_thread.h"
 
+#if defined(OS_ANDROID)
+#include "content/common/android/device_telephony_info.h"
+#endif
+
+#if defined(OS_ANDROID)
+namespace {
+
+  std::string GetAndroidNetworkOperatorID() {
+    content::DeviceTelephonyInfo device_info;
+    return device_info.GetNetworkOperator();

[bengr, 2014/04/25 20:39:41]

Sorry to have not caught this in your last CL, but why is DeviceTelephonyInfo a
class? It holds no state.

[bolian, 2014/04/25 23:33:05]

It is existing code. I added a method. Anyway, it is reverted in the new CL.

+  }
+
+}
+#endif
 
 MetricsNetworkObserver::MetricsNetworkObserver()
     : weak_ptr_factory_(this),
       connection_type_is_ambiguous_(false),
       wifi_phy_layer_protocol_is_ambiguous_(false),
       wifi_phy_layer_protocol_(net::WIFI_PHY_LAYER_PROTOCOL_UNKNOWN) {
   net::NetworkChangeNotifier::AddConnectionTypeObserver(this);
   connection_type_ = net::NetworkChangeNotifier::GetConnectionType();
   ProbeWifiPHYLayerProtocol();
+  GetNetworkOperatorID();
 }
 
 MetricsNetworkObserver::~MetricsNetworkObserver() {
   net::NetworkChangeNotifier::RemoveConnectionTypeObserver(this);
 }
 
 void MetricsNetworkObserver::Reset() {
   connection_type_is_ambiguous_ = false;
   connection_type_ = net::NetworkChangeNotifier::GetConnectionType();
   wifi_phy_layer_protocol_is_ambiguous_ = false;
 }
 
 void MetricsNetworkObserver::OnConnectionTypeChanged(
     net::NetworkChangeNotifier::ConnectionType type) {
   if (type == net::NetworkChangeNotifier::CONNECTION_NONE)
     return;
   if (type != connection_type_ &&
       connection_type_ != net::NetworkChangeNotifier::CONNECTION_NONE) {
     connection_type_is_ambiguous_ = true;
   }
   connection_type_ = type;
 
   ProbeWifiPHYLayerProtocol();
+  GetNetworkOperatorID();
 }
 
 SystemProfileProto::Network::ConnectionType
 MetricsNetworkObserver::connection_type() const {
   switch (connection_type_) {
     case net::NetworkChangeNotifier::CONNECTION_NONE:
     case net::NetworkChangeNotifier::CONNECTION_UNKNOWN:
       return SystemProfileProto::Network::CONNECTION_UNKNOWN;
     case net::NetworkChangeNotifier::CONNECTION_ETHERNET:
       return SystemProfileProto::Network::CONNECTION_ETHERNET;
@@ skipping 43 matching lines @@
 
 void MetricsNetworkObserver::OnWifiPHYLayerProtocolResult(
     net::WifiPHYLayerProtocol mode) {
   if (wifi_phy_layer_protocol_ != net::WIFI_PHY_LAYER_PROTOCOL_UNKNOWN &&
       mode != wifi_phy_layer_protocol_) {
     wifi_phy_layer_protocol_is_ambiguous_ = true;
   }
   wifi_phy_layer_protocol_ = mode;
 }
 
+void MetricsNetworkObserver::GetNetworkOperatorID() {
+#if defined(OS_ANDROID)
+  PostTaskAndReplyWithResult(
+      content::BrowserThread::GetBlockingPool(),
+      FROM_HERE,
+      base::Bind(&GetAndroidNetworkOperatorID),
+      base::Bind(&MetricsNetworkObserver::OnNetworkOperatorID,
+                 weak_ptr_factory_.GetWeakPtr()));
+#endif
+}
+
+void MetricsNetworkObserver::OnNetworkOperatorID(std::string id) {

[bengr, 2014/04/25 20:39:41]

const std::string&

[bolian, 2014/04/25 23:33:05]

reverted in the new CL.

+  network_operator_id_ = id;
+}