Move the concept of multiple route into Network

webrtc/p2p/client/basicportallocator.cc

Index: webrtc/p2p/client/basicportallocator.cc
diff --git a/webrtc/p2p/client/basicportallocator.cc b/webrtc/p2p/client/basicportallocator.cc
index 3f5aa0a1ff396cbb512d65166d46c54c598b70b9..f3ff6e7dde1123b8735f9d105291036477d264ad 100644
--- a/webrtc/p2p/client/basicportallocator.cc
+++ b/webrtc/p2p/client/basicportallocator.cc
@@ -307,6 +307,10 @@ void BasicPortAllocatorSession::DoAllocate() {
   // instead of specific NIC. This is to ensure the same routing for http
   // traffic by OS is also used here to avoid any local or public IP leakage
   // during stun process.
+  if (allocator_->network_manager()->adapter_enumeration_disabled()) {

[juberti1, 2015/08/13 20:45:58]

It seems odd to set a flag based on some other condition. I would suggest doing
this as 

if (flags() & PORTALLOCATOR_DISABLE_ADAPTER_ENUMERATION ||
   allocator_->network_manager()->IsAdapterEnumerationDisabled())

[guoweis_webrtc, 2015/08/13 20:54:14]

There are other places in the code which depends on this flag. If the pattern of
setting the flag inside the webrtc code base is discouraged, I'll modify these
other places as well.

[juberti1, 2015/08/13 22:10:11]

I see. In that case this seems like the best approach; just add a comment to say
that if the network manager has disabled enumeration, we just act as if the PA
flag has been set. Or would it be possible to remove the PA flag and just use
the NetworkManager setting?

+    set_flags(flags() | PORTALLOCATOR_DISABLE_ADAPTER_ENUMERATION);
+  }
+
   if (flags() & PORTALLOCATOR_DISABLE_ADAPTER_ENUMERATION) {
     allocator_->network_manager()->GetAnyAddressNetworks(&networks);
   } else {