Add a switch to redetermine role when ICE restarts.

webrtc/api/peerconnectionfactory.h

Index: webrtc/api/peerconnectionfactory.h
diff --git a/webrtc/api/peerconnectionfactory.h b/webrtc/api/peerconnectionfactory.h
index 377ad73ec8c197b78250b847b6ccc304ed9f09ef..7684b9536297ce7d3e8d4c4c7a97f34f6fa67b19 100644
--- a/webrtc/api/peerconnectionfactory.h
+++ b/webrtc/api/peerconnectionfactory.h
@@ -91,7 +91,8 @@ class PeerConnectionFactory : public PeerConnectionFactoryInterface {
   virtual webrtc::MediaControllerInterface* CreateMediaController(
       const cricket::MediaConfig& config) const;
   virtual cricket::TransportController* CreateTransportController(
-      cricket::PortAllocator* port_allocator);
+      cricket::PortAllocator* port_allocator,
+      bool redetermine_role_on_ice_restart);

[pthatcher1, 2016/08/29 22:18:07]

Can we make this a struct instead of a bool?  Then when we add new things over
time, we just add them to the struct rather than adding more and more arguments.

[honghaiz3, 2016/08/29 23:16:59]

Can we delay using a struct until we need to add a second argument?
The chance is that we may not need a second argument at all, which will make the
struct look weird. 
Plus, I think our plan is to get rid of this argument once we can safely switch
to "not redetermine the role" in Chrome. In fact, I think it might be better to
use a setter method instead of changing the constructor because of this reason,
but we have already made changes to change the constructor.  

[pthatcher1, 2016/08/29 23:27:42]

Good point about removing it soon.  

   virtual rtc::Thread* signaling_thread();
   virtual rtc::Thread* worker_thread();
   virtual rtc::Thread* network_thread();