Skip to content

Commit

Permalink
Bug 1551854: Don't stomp the mid on a transceiver if the answer has n…
Browse files Browse the repository at this point in the history
…o mid in it. r=mjf

Depends on D37076

Differential Revision: https://phabricator.services.mozilla.com/D37077

--HG--
extra : moz-landing-system : lando
  • Loading branch information
docfaraday committed Jul 8, 2019
1 parent c603cf6 commit 798f1f1
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1446,7 +1446,9 @@ nsresult JsepSessionImpl::UpdateTransceiversFromRemoteDescription(
}

if (!mSdpHelper.MsectionIsDisabled(msection)) {
transceiver->Associate(msection.GetAttributeList().GetMid());
if (msection.GetAttributeList().HasAttribute(SdpAttribute::kMidAttribute)) {
transceiver->Associate(msection.GetAttributeList().GetMid());
}
if (!transceiver->IsAssociated()) {
transceiver->Associate(GetNewMid());
} else {
Expand Down

0 comments on commit 798f1f1

Please sign in to comment.