Add connection type to NCN::MaxBandwidthChanged

net/base/network_change_notifier_linux.cc

 // 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.
 
 #include "net/base/network_change_notifier_linux.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/threading/thread.h"
 #include "net/base/address_tracker_linux.h"
@@ skipping 64 matching lines @@
   NetworkChangeNotifier::NotifyObserversOfIPAddressChange();
   // When the IP address of a network interface is added/deleted, the
   // connection type may have changed.
   OnLinkChanged();
 }
 
 void NetworkChangeNotifierLinux::Thread::OnLinkChanged() {
   if (last_type_ != GetCurrentConnectionType()) {
     NetworkChangeNotifier::NotifyObserversOfConnectionTypeChange();
     last_type_ = GetCurrentConnectionType();
+    double max_bandwidth =
+        last_type_ == CONNECTION_NONE
+            ? NetworkChangeNotifier::GetMaxBandwidthForConnectionSubtype(
+                  SUBTYPE_NONE)
+            : NetworkChangeNotifier::GetMaxBandwidthForConnectionSubtype(
+                  SUBTYPE_UNKNOWN);

[pauljensen, 2015/09/15 18:17:40]

how about ReallyLongFunctionName(conditon ? SUBTYPE_NONE : SUBTYPE_UNKNOWN)?
rather than duplicating ReallyLongFunctionName

[jkarlin, 2015/09/16 12:12:22]

Good idea. Done.

+    NetworkChangeNotifier::NotifyObserversOfMaxBandwidthChange(last_type_,
+                                                               max_bandwidth);
   }
 }
 
 NetworkChangeNotifierLinux::NetworkChangeNotifierLinux(
     const base::hash_set<std::string>& ignored_interfaces)
     : NetworkChangeNotifier(NetworkChangeCalculatorParamsLinux()),
       notifier_thread_(new Thread(ignored_interfaces)) {
   // We create this notifier thread because the notification implementation
   // needs a MessageLoopForIO, and there's no guarantee that
   // MessageLoop::current() meets that criterion.
@@ skipping 25 matching lines @@
 NetworkChangeNotifierLinux::GetCurrentConnectionType() const {
   return notifier_thread_->GetCurrentConnectionType();
 }
 
 const internal::AddressTrackerLinux*
 NetworkChangeNotifierLinux::GetAddressTrackerInternal() const {
   return notifier_thread_->address_tracker();
 }
 
 }  // namespace net