Bug 1322358: Don't disable m-sections in answer just because they're inactive. r?drno draft
authorByron Campen [:bwc] <docfaraday@gmail.com>
Wed, 25 Jan 2017 16:42:08 -0600
changeset 466380 6e6140968f95342b0429d8197a90d4e2d2021bb9
parent 466286 24d9eb148461bb4789848b9880867c63c783a2ca
child 543419 5424160ffb11ec65d9618b0f027f7fbe56e72712
push id42893
push userbcampen@mozilla.com
push dateWed, 25 Jan 2017 22:47:08 +0000
reviewersdrno
bugs1322358
milestone54.0a1
Bug 1322358: Don't disable m-sections in answer just because they're inactive. r?drno MozReview-Commit-ID: 6YPwEk7bwIG
media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
--- a/media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
+++ b/media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
@@ -1000,21 +1000,16 @@ JsepSessionImpl::CreateAnswerMSection(co
     rv = BindMatchingLocalTrackToAnswer(&msection);
     NS_ENSURE_SUCCESS(rv, rv);
   }
 
   if (remoteMsection.IsSending()) {
     BindMatchingRemoteTrackToAnswer(&msection);
   }
 
-  if (!msection.IsReceiving() && !msection.IsSending()) {
-    mSdpHelper.DisableMsection(sdp, &msection);
-    return NS_OK;
-  }
-
   // Add extmap attributes.
   AddCommonExtmaps(remoteMsection, &msection);
 
   if (msection.GetFormats().empty()) {
     // Could not negotiate anything. Disable m-section.
     mSdpHelper.DisableMsection(sdp, &msection);
   }