diff --git a/modules/rtp_rtcp/include/rtp_payload_registry.h b/modules/rtp_rtcp/include/rtp_payload_registry.h index 1f8a0a4213..a0790058c1 100644 --- a/modules/rtp_rtcp/include/rtp_payload_registry.h +++ b/modules/rtp_rtcp/include/rtp_payload_registry.h @@ -46,12 +46,6 @@ class RTPPayloadRegistry { int32_t ReceivePayloadType(const VideoCodec& video_codec, int8_t* payload_type) const; - bool RtxEnabled() const; - - void SetRtxSsrc(uint32_t ssrc); - - bool GetRtxSsrc(uint32_t* ssrc) const; - bool IsRed(const RTPHeader& header) const; int GetPayloadTypeFrequency(uint8_t payload_type) const; @@ -100,12 +94,6 @@ class RTPPayloadRegistry { std::map payload_type_map_; int8_t last_received_payload_type_; int8_t last_received_media_payload_type_; - bool rtx_; - uint32_t ssrc_rtx_; - // Only warn once per payload type, if an RTX packet is received but - // no associated payload type found in |rtx_payload_type_map_|. - std::set payload_types_with_suppressed_warnings_ - RTC_GUARDED_BY(crit_sect_); // As a first step in splitting this class up in separate cases for audio and // video, DCHECK that no instance is used for both audio and video. diff --git a/modules/rtp_rtcp/source/rtp_payload_registry.cc b/modules/rtp_rtcp/source/rtp_payload_registry.cc index 4e1ef200b4..7a6887fa33 100644 --- a/modules/rtp_rtcp/source/rtp_payload_registry.cc +++ b/modules/rtp_rtcp/source/rtp_payload_registry.cc @@ -104,9 +104,7 @@ bool IsPayloadTypeValid(int8_t payload_type) { RTPPayloadRegistry::RTPPayloadRegistry() : last_received_payload_type_(-1), - last_received_media_payload_type_(-1), - rtx_(false), - ssrc_rtx_(0) {} + last_received_media_payload_type_(-1) {} RTPPayloadRegistry::~RTPPayloadRegistry() = default; @@ -261,23 +259,6 @@ int32_t RTPPayloadRegistry::ReceivePayloadType(const VideoCodec& video_codec, return -1; } -bool RTPPayloadRegistry::RtxEnabled() const { - rtc::CritScope cs(&crit_sect_); - return rtx_; -} - -void RTPPayloadRegistry::SetRtxSsrc(uint32_t ssrc) { - rtc::CritScope cs(&crit_sect_); - ssrc_rtx_ = ssrc; - rtx_ = true; -} - -bool RTPPayloadRegistry::GetRtxSsrc(uint32_t* ssrc) const { - rtc::CritScope cs(&crit_sect_); - *ssrc = ssrc_rtx_; - return rtx_; -} - bool RTPPayloadRegistry::IsRed(const RTPHeader& header) const { rtc::CritScope cs(&crit_sect_); auto it = payload_type_map_.find(header.payloadType); diff --git a/video/rtp_video_stream_receiver.cc b/video/rtp_video_stream_receiver.cc index 7379facc86..5e00627891 100644 --- a/video/rtp_video_stream_receiver.cc +++ b/video/rtp_video_stream_receiver.cc @@ -138,11 +138,6 @@ RtpVideoStreamReceiver::RtpVideoStreamReceiver( : kDefaultMaxReorderingThreshold; rtp_receive_statistics_->SetMaxReorderingThreshold(max_reordering_threshold); - if (config_.rtp.rtx_ssrc) { - // Needed for rtp_payload_registry_.RtxEnabled(). - rtp_payload_registry_.SetRtxSsrc(config_.rtp.rtx_ssrc); - } - if (IsUlpfecEnabled()) { VideoCodec ulpfec_codec = {}; ulpfec_codec.codecType = kVideoCodecULPFEC; @@ -605,7 +600,7 @@ bool RtpVideoStreamReceiver::IsPacketInOrder(const RTPHeader& header) const { bool RtpVideoStreamReceiver::IsPacketRetransmitted(const RTPHeader& header, bool in_order) const { // Retransmissions are handled separately if RTX is enabled. - if (rtp_payload_registry_.RtxEnabled()) + if (config_.rtp.rtx_ssrc != 0) return false; StreamStatistician* statistician = rtp_receive_statistics_->GetStatistician(header.ssrc);