NetworkChangeNotifierChromeos: Handle IPConfig property changes on the default network

chromeos/network/network_state_handler.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 "chromeos/network/network_state_handler.h"
 
 #include "base/bind.h"
 #include "base/format_macros.h"
 #include "base/stl_util.h"
 #include "base/string_util.h"
@@ skipping 415 matching lines @@
   std::string prev_connection_state = network->connection_state();
   if (!network->PropertyChanged(key, value))
     return;
 
   std::string detail = network->name() + "." + key;
   std::string vstr = ValueAsString(value);
   if (!vstr.empty())
     detail += " = " + vstr;
   network_event_log::AddEntry(kLogModule, "NetworkPropertyUpdated", detail);
 
-  if (network->connection_state() != prev_connection_state)
+  if (network->connection_state() != prev_connection_state) {
     OnNetworkConnectionStateChanged(network);
+  }
+  else if (network->path() == default_network_path_ &&
+           key != flimflam::kSignalStrengthProperty) {
+    // WiFi signal strength updates are too noisy, so don't
+    // trigger default network updates for those changes.
+    OnDefaultNetworkChanged();
+  }
+
   NetworkPropertiesUpdated(network);
 }
 
-void NetworkStateHandler::UpdateNetworkServiceIPAddress(
-    const std::string& service_path,
-    const std::string& ip_address) {
-  NetworkState* network = GetModifiableNetworkState(service_path);
-  if (!network)
-    return;
-  std::string detail = network->name() + ".IPAddress = " + ip_address;
-  network_event_log::AddEntry(kLogModule, "NetworkIPChanged", detail);
-  network->set_ip_address(ip_address);
-  NetworkPropertiesUpdated(network);
-}
-
 void NetworkStateHandler::UpdateDeviceProperty(const std::string& device_path,
                                                const std::string& key,
                                                const base::Value& value) {
   DeviceState* device = GetModifiableDeviceState(device_path);
   if (!device)
     return;
   if (!device->PropertyChanged(key, value))
     return;
 
   std::string detail = device->name() + "." + key;
@@ skipping 140 matching lines @@
     return;
   ScanCallbackList& callback_list = scan_complete_callbacks_[type];
   for (ScanCallbackList::iterator iter = callback_list.begin();
        iter != callback_list.end(); ++iter) {
     (*iter).Run();
   }
   scan_complete_callbacks_.erase(type);
 }
 
 }  // namespace chromeos