Restore the behavior where an ICE restart redetermines the ICE role.

webrtc/p2p/base/transport.h

Index: webrtc/p2p/base/transport.h
diff --git a/webrtc/p2p/base/transport.h b/webrtc/p2p/base/transport.h
index e4753d217d46ea94430a93c2577241e520f3f496..a7ca72ff8ea1ed8979d29486ed1f564572d4b281 100644
--- a/webrtc/p2p/base/transport.h
+++ b/webrtc/p2p/base/transport.h
@@ -314,24 +314,26 @@ class Transport : public sigslot::has_slots<> {
     return false;
   }
 
- protected:
-  // These are called by Create/DestroyChannel above in order to create or
-  // destroy the appropriate type of channel.
-  virtual TransportChannelImpl* CreateTransportChannel(int component) = 0;
-  virtual void DestroyTransportChannel(TransportChannelImpl* channel) = 0;
-
   // The current local transport description, for use by derived classes
-  // when performing transport description negotiation.
+  // when performing transport description negotiation, and possibly used
+  // by the transport controller.
   const TransportDescription* local_description() const {
     return local_description_.get();
   }
 
   // The current remote transport description, for use by derived classes
-  // when performing transport description negotiation.
+  // when performing transport description negotiation, and possibly used
+  // by the transport controller.
   const TransportDescription* remote_description() const {
     return remote_description_.get();
   }
 
+ protected:
+  // These are called by Create/DestroyChannel above in order to create or
+  // destroy the appropriate type of channel.
+  virtual TransportChannelImpl* CreateTransportChannel(int component) = 0;
+  virtual void DestroyTransportChannel(TransportChannelImpl* channel) = 0;
+
   // Pushes down the transport parameters from the local description, such
   // as the ICE ufrag and pwd.
   // Derived classes can override, but must call the base as well.