[NetInfo] Browser changes to support connection.downlinkMax

net/base/network_change_notifier.h

 // Copyright (c) 2012 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.
 
 #ifndef NET_BASE_NETWORK_CHANGE_NOTIFIER_H_
 #define NET_BASE_NETWORK_CHANGE_NOTIFIER_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 224 matching lines @@
   // the internet, the connection type is CONNECTION_WIFI.
   static ConnectionType GetConnectionType();
 
   // Returns a theoretical upper limit on download bandwidth, potentially based
   // on underlying connection type, signal strength, or some other signal. The
   // default mapping of connection type to maximum bandwidth is provided in the
   // NetInfo spec: http://w3c.github.io/netinfo/. See documentation in the
   // 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);
 
 #if defined(OS_LINUX)
   // Returns the AddressTrackerLinux if present.
   static const internal::AddressTrackerLinux* GetAddressTracker();
 #endif
 
   // Convenience method to determine if the user is offline.
@@ skipping 134 matching lines @@
   // TODO(szym): Retrieve AddressMap from NetworkState. http://crbug.com/144212
   virtual const internal::AddressTrackerLinux*
       GetAddressTrackerInternal() const;
 #endif
 
   // See the description of NetworkChangeNotifier::GetMaxBandwidth().
   // Implementations must be thread-safe. Implementations must also be
   // 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.
   static void NotifyObserversOfIPAddressChange();
   static void NotifyObserversOfConnectionTypeChange();
   static void NotifyObserversOfDNSChange();
   static void NotifyObserversOfInitialDNSConfigRead();
   static void NotifyObserversOfNetworkChange(ConnectionType type);
   static void NotifyObserversOfMaxBandwidthChange(ConnectionType type,
                                                   double max_bandwidth_mbps);
@@ skipping 43 matching lines @@
 
   // Set true to disable non-test notifications (to prevent flakes in tests).
   static bool test_notifications_only_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkChangeNotifier);
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NETWORK_CHANGE_NOTIFIER_H_