[NetInfo] Browser changes to support connection.downlinkMax

net/base/network_change_notifier.h

Index: net/base/network_change_notifier.h
diff --git a/net/base/network_change_notifier.h b/net/base/network_change_notifier.h
index 8206e3911776efe4a36ff65360db09b2b4b77667..83e6360c86f96abda9e317bd494798abb61801c2 100644
--- a/net/base/network_change_notifier.h
+++ b/net/base/network_change_notifier.h
@@ -243,6 +243,11 @@ class NET_EXPORT NetworkChangeNotifier {
   static void GetMaxBandwidthAndConnectionType(double* max_bandwidth_mbps,
                                                ConnectionType* connection_type);
 
+  // Returns a theoretical upper limit (in mbps) on download bandwidth given a

[pauljensen, 2015/09/15 17:39:24]

mbps->Mbps

[jkarlin, 2015/09/15 17:51:49]

Hmm, I believe they're the same. grep wars show 38 files with mbps and 69 with
Mbps. 

[pauljensen, 2015/09/15 17:58:32]

Should be M.  m=milli M=mega
https://en.wikipedia.org/wiki/Metric_prefix
You might want to use Mibit/s to be more accurate.
https://en.wikipedia.org/wiki/Data_rate_units#Mebibit_per_second

[jkarlin, 2015/09/15 18:07:19]

Ah the elusive millibit. Fine fine. Done.

+  // connection subtype. The mapping of connection type to maximum bandwidth is
+  // provided in the NetInfo spec: http://w3c.github.io/netinfo/.
+  static double GetMaxBandwidthForConnectionSubtype(ConnectionSubtype subtype);
+
   // Retrieve the last read DnsConfig. This could be expensive if the system has
   // a large HOSTS file.
   static void GetDnsConfig(DnsConfig* config);
@@ -399,11 +404,6 @@ class NET_EXPORT NetworkChangeNotifier {
       double* max_bandwidth_mbps,
       ConnectionType* connection_type) const;
 
-  // Returns a theoretical upper limit on download bandwidth given a connection
-  // subtype. The mapping of connection type to maximum bandwidth is provided in
-  // the NetInfo spec: http://w3c.github.io/netinfo/.
-  static double GetMaxBandwidthForConnectionSubtype(ConnectionSubtype subtype);
-
   // Broadcasts a notification to all registered observers.  Note that this
   // happens asynchronously, even for observers on the current thread, even in
   // tests.