diff --git a/dom/media/webrtc/jsep/JsepSessionImpl.cpp b/dom/media/webrtc/jsep/JsepSessionImpl.cpp index 5a1f49bf6563..e173819bbea5 100644 --- a/dom/media/webrtc/jsep/JsepSessionImpl.cpp +++ b/dom/media/webrtc/jsep/JsepSessionImpl.cpp @@ -920,9 +920,7 @@ nsresult JsepSessionImpl::SetLocalDescriptionOffer(UniquePtr offer) { std::vector recvTracks; recvTracks.reserve(mTransceivers.size()); for (auto& transceiver : mTransceivers) { - if (transceiver.mJsDirection & sdp::kRecv) { - recvTracks.push_back(&transceiver.mRecvTrack); - } + recvTracks.push_back(&transceiver.mRecvTrack); } JsepTrack::SetUniqueReceivePayloadTypes(recvTracks, true); @@ -1156,14 +1154,9 @@ nsresult JsepSessionImpl::HandleNegotiatedSession( CopyBundleTransports(); std::vector receiveTracks; + receiveTracks.reserve(mTransceivers.size()); for (auto& transceiver : mTransceivers) { - // Do not count payload types for non-active recv tracks as duplicates. If - // we receive an RTP packet with a payload type that is used by both a - // sendrecv and a sendonly m-section, there is no ambiguity; it is for the - // sendrecv m-section. - if (transceiver.mRecvTrack.GetActive()) { - receiveTracks.push_back(&transceiver.mRecvTrack); - } + receiveTracks.push_back(&transceiver.mRecvTrack); } JsepTrack::SetUniqueReceivePayloadTypes(receiveTracks); diff --git a/media/webrtc/signaling/gtest/jsep_session_unittest.cpp b/media/webrtc/signaling/gtest/jsep_session_unittest.cpp index ae457a413bff..66ed1a59d3b4 100644 --- a/media/webrtc/signaling/gtest/jsep_session_unittest.cpp +++ b/media/webrtc/signaling/gtest/jsep_session_unittest.cpp @@ -4990,9 +4990,9 @@ TEST_F(JsepSessionTest, TestUniqueReceivePayloadTypes) { ASSERT_FALSE(IsNull(offerTransceivers[2].mRecvTrack)); ASSERT_TRUE(offerTransceivers[2].mRecvTrack.GetNegotiatedDetails()); ASSERT_THAT(offerTransceivers[2].mRecvTrack.GetUniqueReceivePayloadTypes(), - UnorderedElementsAre(97, 99, 103, 105, 120, 121, 122, 123, 126)); - ASSERT_THAT(offerTransceivers[2].mRecvTrack.GetDuplicateReceivePayloadTypes(), UnorderedElementsAre()); + ASSERT_THAT(offerTransceivers[2].mRecvTrack.GetDuplicateReceivePayloadTypes(), + UnorderedElementsAre(97, 99, 103, 105, 120, 121, 122, 123, 126)); ASSERT_TRUE(IsNull(offerTransceivers[3].mRecvTrack)); ASSERT_TRUE(offerTransceivers[3].mRecvTrack.GetNegotiatedDetails());