Chromium Code Reviews| Index: net/base/network_change_notifier.cc |
| diff --git a/net/base/network_change_notifier.cc b/net/base/network_change_notifier.cc |
| index 638964a25d29ad12016b9d20ad64cf82df7ef222..0a0fa4c28efb350a8b081e380073fb4b8338dfed 100644 |
| --- a/net/base/network_change_notifier.cc |
| +++ b/net/base/network_change_notifier.cc |
| @@ -5,6 +5,7 @@ |
| #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" |
| @@ -14,6 +15,10 @@ |
| #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) |
| @@ -253,6 +258,17 @@ class HistogramWatcher |
| if (type != NetworkChangeNotifier::CONNECTION_NONE) { |
| UMA_HISTOGRAM_MEDIUM_TIMES("NCN.NetworkOnlineChange", |
| SinceLast(&last_network_change_)); |
| +#if defined(OS_ANDROID) |
| + std::string mcc_mnc = android::GetTelephonyNetworkOperator(); |
| + // MCC and MNC codes are each 3 digits. |
|
Ilya Sherman
2014/04/26 02:29:12
From scanning the Wikipedia article, it looked lik
bolian
2014/04/28 18:07:55
My bad. Now, I think I should only check the retur
|
| + if (mcc_mnc.length() == 6) { |
| + int value = atoi(mcc_mnc.c_str()); |
| + if (value> 0) { |
| + UMA_HISTOGRAM_SPARSE_SLOWLY("NCN.NetworkOperatorMCCMNC", value); |
| + } |
| + } |
| +#endif |
| + |
| } else { |
| UMA_HISTOGRAM_MEDIUM_TIMES("NCN.NetworkOfflineChange", |
| SinceLast(&last_network_change_)); |