Add a continual_gathering_policy in PeerConnection RTCConfiguration

webrtc/p2p/base/p2ptransportchannel.h

Index: webrtc/p2p/base/p2ptransportchannel.h
diff --git a/webrtc/p2p/base/p2ptransportchannel.h b/webrtc/p2p/base/p2ptransportchannel.h
index 51426239db1b46f9260e6a52a35cff682e3c5648..08f9350b89ae4adc9be1db05fc27a06cc6bf3b13 100644
--- a/webrtc/p2p/base/p2ptransportchannel.h
+++ b/webrtc/p2p/base/p2ptransportchannel.h
@@ -74,9 +74,9 @@ class P2PTransportChannel : public TransportChannelImpl,
     return gathering_state_;
   }
   void AddRemoteCandidate(const Candidate& candidate) override;
-  // Sets the receiving timeout in milliseconds.
+  // Sets the receiving timeout and continue_getting_ports.
   // This also sets the check_receiving_delay proportionally.
-  void SetReceivingTimeout(int receiving_timeout_ms) override;
+  void SetIceConfig(const IceConfig& config) override;
 
   // From TransportChannel:
   int SendPacket(const char* data,
@@ -250,6 +250,7 @@ class P2PTransportChannel : public TransportChannelImpl,
   int check_receiving_delay_;
   int receiving_timeout_;
   uint32 last_ping_sent_ms_ = 0;
+  bool continue_getting_ports_ = false;

[pthatcher1, 2015/09/25 17:37:47]

Why not store the config instead of storing both receiving_timeout_ and
continue_getting_ports_?

[honghaiz3, 2015/09/25 20:10:20]

The config is used to pass the infomation down. 
But this two values are used completely separate here. 
It is more convenient to have two variables when using them.
Plus, when setting we need to check the value of timeout but not the other var.
If we just blindly save the whole ice_config, then we need to check the
receiving time everytime when using it. 
Renamed it 

 
   RTC_DISALLOW_COPY_AND_ASSIGN(P2PTransportChannel);
 };