Adds bluetooth connection type to NetworkChangeNotifier.

This is for compliance with NetInfov3 (http://w3c.github.io/netinfo/).

BUG=378785

Review URL: https://codereview.chromium.org/307943004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@274817 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 24 insertions, 6 deletions

diff --git a/net/base/network_change_notifier.cc b/net/base/network_change_notifier.cc
index aef8356..c55be07 100644
--- a/net/base/network_change_notifier.cc
+++ b/net/base/network_change_notifier.cc
@@ -151,6 +151,11 @@ class HistogramWatcher
           UMA_HISTOGRAM_TIMES("NCN.CM.FastestRTTOnNone",
                               fastest_RTT_since_last_connection_change_);
           break;
+        case NetworkChangeNotifier::CONNECTION_BLUETOOTH:
+          UMA_HISTOGRAM_TIMES("NCN.CM.FirstReadOnBluetooth",
+                              first_byte_after_connection_change_);
+          UMA_HISTOGRAM_TIMES("NCN.CM.FastestRTTOnBluetooth",
+                              fastest_RTT_since_last_connection_change_);
       }
     }
     if (peak_kbps_since_last_connection_change_) {
@@ -183,6 +188,10 @@ class HistogramWatcher
           UMA_HISTOGRAM_COUNTS("NCN.CM.PeakKbpsOnNone",
                                peak_kbps_since_last_connection_change_);
           break;
+        case NetworkChangeNotifier::CONNECTION_BLUETOOTH:
+          UMA_HISTOGRAM_COUNTS("NCN.CM.PeakKbpsOnBluetooth",
+                               peak_kbps_since_last_connection_change_);
+          break;
       }
     }
     switch (last_connection_type_) {
@@ -214,6 +223,10 @@ class HistogramWatcher
         UMA_HISTOGRAM_LONG_TIMES("NCN.CM.TimeOnNone", state_duration);
         UMA_HISTOGRAM_COUNTS("NCN.CM.KBTransferedOnNone", kilobytes_read);
         break;
+      case NetworkChangeNotifier::CONNECTION_BLUETOOTH:
+        UMA_HISTOGRAM_LONG_TIMES("NCN.CM.TimeOnBluetooth", state_duration);
+        UMA_HISTOGRAM_COUNTS("NCN.CM.KBTransferedOnBluetooth", kilobytes_read);
+        break;
     }
 
     if (type != NetworkChangeNotifier::CONNECTION_NONE) {
@@ -540,13 +553,14 @@ const char* NetworkChangeNotifier::ConnectionTypeToString(
     "CONNECTION_2G",
     "CONNECTION_3G",
     "CONNECTION_4G",
-    "CONNECTION_NONE"
+    "CONNECTION_NONE",
+    "CONNECTION_BLUETOOTH"
   };
   COMPILE_ASSERT(
       arraysize(kConnectionTypeNames) ==
-          NetworkChangeNotifier::CONNECTION_NONE + 1,
+          NetworkChangeNotifier::CONNECTION_LAST + 1,
       ConnectionType_name_count_mismatch);
-  if (type < CONNECTION_UNKNOWN || type > CONNECTION_NONE) {
+  if (type < CONNECTION_UNKNOWN || type > CONNECTION_LAST) {
     NOTREACHED();
     return "CONNECTION_INVALID";
   }
@@ -625,6 +639,7 @@ bool NetworkChangeNotifier::IsConnectionCellular(ConnectionType type) {
     case CONNECTION_ETHERNET:
     case CONNECTION_WIFI:
     case CONNECTION_NONE:
+    case CONNECTION_BLUETOOTH:
       is_cellular = false;
       break;
   }
diff --git a/net/base/network_change_notifier.h b/net/base/network_change_notifier.h
index 4671e49..6b801bc 100644
--- a/net/base/network_change_notifier.h
+++ b/net/base/network_change_notifier.h
@@ -32,8 +32,8 @@ class AddressTrackerLinux;
 // destroyed on the same thread.
 class NET_EXPORT NetworkChangeNotifier {
  public:
-  // Using the terminology of the Network Information API:
-  // http://www.w3.org/TR/netinfo-api.
+  // This is a superset of the connection types in the NetInfo v3 specification:
+  // http://w3c.github.io/netinfo/.
   enum ConnectionType {
     CONNECTION_UNKNOWN = 0,  // A connection exists, but its type is unknown.
                              // Also used as a default value.
@@ -43,7 +43,8 @@ class NET_EXPORT NetworkChangeNotifier {
     CONNECTION_3G = 4,
     CONNECTION_4G = 5,
     CONNECTION_NONE = 6,     // No connection.
-    CONNECTION_LAST = CONNECTION_NONE
+    CONNECTION_BLUETOOTH = 7,
+    CONNECTION_LAST = CONNECTION_BLUETOOTH
   };
 
   class NET_EXPORT IPAddressObserver {
diff --git a/net/tools/net_watcher/net_watcher.cc b/net/tools/net_watcher/net_watcher.cc
index f0e7e1e..421e695 100644
--- a/net/tools/net_watcher/net_watcher.cc
+++ b/net/tools/net_watcher/net_watcher.cc
@@ -50,6 +50,8 @@ const char* ConnectionTypeToString(
       return "CONNECTION_4G";
     case net::NetworkChangeNotifier::CONNECTION_NONE:
       return "CONNECTION_NONE";
+    case net::NetworkChangeNotifier::CONNECTION_BLUETOOTH:
+      return "CONNECTION_BLUETOOTH";
     default:
       return "CONNECTION_UNEXPECTED";
   }