diff --git a/dom/media/webrtc/jsep/JsepSessionImpl.cpp b/dom/media/webrtc/jsep/JsepSessionImpl.cpp index fc979c034b7f..8eefba1236a5 100644 --- a/dom/media/webrtc/jsep/JsepSessionImpl.cpp +++ b/dom/media/webrtc/jsep/JsepSessionImpl.cpp @@ -907,9 +907,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); @@ -1143,14 +1141,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 21c4db5f9e16..d66f14d1b82f 100644 --- a/media/webrtc/signaling/gtest/jsep_session_unittest.cpp +++ b/media/webrtc/signaling/gtest/jsep_session_unittest.cpp @@ -4976,9 +4976,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());