Fixing bug that results in incorrect ICE role with ICE lite endpoints.

webrtc/p2p/base/transportcontroller.cc

Index: webrtc/p2p/base/transportcontroller.cc
diff --git a/webrtc/p2p/base/transportcontroller.cc b/webrtc/p2p/base/transportcontroller.cc
index f06a180088608301c7d73560a346b42383b69481..df85a7c3584ffb069c52416c1b36fe35b0ecf9ea 100644
--- a/webrtc/p2p/base/transportcontroller.cc
+++ b/webrtc/p2p/base/transportcontroller.cc
@@ -602,7 +602,11 @@ bool TransportController::SetLocalTransportDescription_n(
   if (redetermine_role_on_ice_restart_ && transport->local_description() &&
       IceCredentialsChanged(transport->local_description()->ice_ufrag,
                             transport->local_description()->ice_pwd,
-                            tdesc.ice_ufrag, tdesc.ice_pwd)) {
+                            tdesc.ice_ufrag, tdesc.ice_pwd) &&
+      // Don't change the ICE role if the remote endpoint is ICE lite; we
+      // should always be controlling in that case.
+      (!transport->remote_description() ||
+       transport->remote_description()->ice_mode != ICEMODE_LITE)) {
     IceRole new_ice_role =
         (action == CA_OFFER) ? ICEROLE_CONTROLLING : ICEROLE_CONTROLLED;
     SetIceRole(new_ice_role);