Fix CreateNetworks to stop it from signaling duplicate networks changed events

webrtc/base/network.h

Index: webrtc/base/network.h
diff --git a/webrtc/base/network.h b/webrtc/base/network.h
index ab3a88dc7dc0c4dd511d5da7aee4b7c035975657..759a618dba94f88494e468f95e25fcc400c54b62 100644
--- a/webrtc/base/network.h
+++ b/webrtc/base/network.h
@@ -99,8 +99,8 @@ class NetworkManager {
   // TODO(guoweis): remove this body when chromium implements this.
   virtual void GetAnyAddressNetworks(NetworkList* networks) {}
 
-  // Dumps a list of networks available to LS_INFO.
-  virtual void DumpNetworks(bool include_ignored) {}
+  // Dumps the current list of networks in the network manager.
+  virtual void DumpNetworks() {}
 
   struct Stats {
     int ipv4_network_count;
@@ -173,8 +173,7 @@ class BasicNetworkManager : public NetworkManagerBase,
   void StartUpdating() override;
   void StopUpdating() override;
 
-  // Logs the available networks.
-  void DumpNetworks(bool include_ignored) override;
+  void DumpNetworks() override;
 
   // MessageHandler interface.
   void OnMessage(Message* msg) override;
@@ -319,6 +318,9 @@ class Network {
   int preference() const { return preference_; }
   void set_preference(int preference) { preference_ = preference; }
 
+  bool active() const { return active_; }
+  void set_active(bool active) { active_ = active; }

[pthatcher1, 2015/12/08 19:40:23]

Can you put some comments/documentation about what it means for a Network to be
active/inactive?

[honghaiz3, 2015/12/10 19:26:18]

Done.

+
   // Debugging description of this network
   std::string ToString() const;
 
@@ -333,6 +335,7 @@ class Network {
   bool ignored_;
   AdapterType type_;
   int preference_;
+  bool active_ = true;
 
   friend class NetworkManager;
 };