[NetInfo] Add MaxBandwidthChanged notification and implement on Android

net/android/network_change_notifier_android.cc

Index: net/android/network_change_notifier_android.cc
diff --git a/net/android/network_change_notifier_android.cc b/net/android/network_change_notifier_android.cc
index 2063e7b017236f549afa1eec9ac88d4829d293d7..05f8a6b884746dcc98344ba53cff21ed8cc7537a 100644
--- a/net/android/network_change_notifier_android.cc
+++ b/net/android/network_change_notifier_android.cc
@@ -97,6 +97,11 @@ class NetworkChangeNotifierAndroid::DnsConfigServiceThread
     NetworkChangeNotifier::NotifyObserversOfConnectionTypeChange();
   }
 
+  static void NotifyMaxBandwidthChangedObservers(double max_bandwidth_mbps) {

[pauljensen, 2014/12/11 16:13:56]

Can we remove this function?  I don't see a reason for it.  Can we just move the
contents into the one caller?  I think NotifyNetworkChangeNotifierObservers()
only exists because it's one caller isn't static so it cannot be merged into it.

[jkarlin, 2014/12/15 19:43:55]

Done.

+    NetworkChangeNotifier::NotifyObserversOfMaxBandwidthChange(
+        max_bandwidth_mbps);
+  }
+
  private:
   scoped_ptr<DnsConfigService> dns_config_service_;
   // Used to detect tunnel state changes.
@@ -122,6 +127,12 @@ void NetworkChangeNotifierAndroid::OnConnectionTypeChanged() {
   DnsConfigServiceThread::NotifyNetworkChangeNotifierObservers();
 }
 
+void NetworkChangeNotifierAndroid::OnMaxBandwidthChanged(
+    double max_bandwidth_mbps) {
+  DnsConfigServiceThread::NotifyMaxBandwidthChangedObservers(
+      max_bandwidth_mbps);
+}
+
 // static
 bool NetworkChangeNotifierAndroid::Register(JNIEnv* env) {
   return NetworkChangeNotifierDelegateAndroid::Register(env);