Improve NetworkStateHandler API

chromeos/network/network_state_handler_observer.h

Index: chromeos/network/network_state_handler_observer.h
diff --git a/chromeos/network/network_state_handler_observer.h b/chromeos/network/network_state_handler_observer.h
index 07a4a7723413ce86fef66d5b5394b819f8656e53..0f097876eee4e656f82f6e2c7038fc2c7f2946ae 100644
--- a/chromeos/network/network_state_handler_observer.h
+++ b/chromeos/network/network_state_handler_observer.h
@@ -34,17 +34,17 @@ class CHROMEOS_EXPORT NetworkStateHandlerObserver {
   // of devices, so they are not passed in the method.
   virtual void DeviceListChanged();
 
-  // The active network changed. |network| will be NULL if there is no longer
-  // an active network.
-  virtual void ActiveNetworkChanged(const NetworkState* network);
+  // The default network changed (includes VPNs) or its connection state
+  // changed. |network| will be NULL if there is no longer a default network.
+  virtual void DefaultNetworkChanged(const NetworkState* network);
 
-  // The state of the active network changed.
-  virtual void ActiveNetworkStateChanged(const NetworkState* network);
+  // The connection state of |network| changed.
+  virtual void NetworkStateChanged(const NetworkState* network);
 
-  // One or more network service properties changed. Note: for the active
-  // network, this will be called in *addition* to ActiveNetworkStateChanged()
-  // if the state property changed.
-  virtual void NetworkServiceChanged(const NetworkState* network);
+  // One or more network service properties changed. Note: this will get

[pneubeck (no reviews), 2012/12/19 19:40:06]

I would still add a reference to |network|
... properties of |network| ...

[stevenjb, 2012/12/19 21:36:50]

Done.

+  // called in *addition* to NetworkStateChanged() when the connection state
+  // property changes. Use this to track properties like wifi strength.
+  virtual void NetworkPropertyChanged(const NetworkState* network);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkStateHandlerObserver);