[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 ec82d75b1a102c43d0d8d2456dca998777f399ac..5cd600d6147b7aed7ed39d09a6d17785b74abe8d 100644
--- a/net/base/network_change_notifier.h
+++ b/net/base/network_change_notifier.h
@@ -242,6 +242,11 @@ class NET_EXPORT NetworkChangeNotifier {
   // MaxBandwidthChangedObserver for more information on what values to expect.
   static double GetMaxBandwidth();
 
+  // 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/.

[pauljensen, 2015/09/11 14:25:37]

Comment doesn't mention units, neither does function name.

[jkarlin, 2015/09/11 20:15:21]

Done.

+  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);
@@ -396,11 +401,6 @@ class NET_EXPORT NetworkChangeNotifier {
   // cheap as it is called often.
   virtual double GetCurrentMaxBandwidth() 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.