| Index: webrtc/video/rtp_video_stream_receiver.cc
|
| diff --git a/webrtc/video/rtp_video_stream_receiver.cc b/webrtc/video/rtp_video_stream_receiver.cc
|
| index 4678b8f0afff36cea95279d27b21812cb3f8a5eb..a14b06504bb6d7d37b8964e646b251b5e033289d 100644
|
| --- a/webrtc/video/rtp_video_stream_receiver.cc
|
| +++ b/webrtc/video/rtp_video_stream_receiver.cc
|
| @@ -105,7 +105,6 @@ RtpVideoStreamReceiver::RtpVideoStreamReceiver(
|
| rtp_receive_statistics_(ReceiveStatistics::Create(clock_)),
|
| ulpfec_receiver_(UlpfecReceiver::Create(config->rtp.remote_ssrc, this)),
|
| receiving_(false),
|
| - restored_packet_in_use_(false),
|
| last_packet_log_ms_(-1),
|
| rtp_rtcp_(CreateRtpRtcpModule(rtp_receive_statistics_.get(),
|
| transport,
|
| @@ -146,12 +145,8 @@ RtpVideoStreamReceiver::RtpVideoStreamReceiver(
|
| 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);
|
| -
|
| - for (const auto& kv : config_.rtp.rtx_associated_payload_types) {
|
| - RTC_DCHECK_NE(kv.first, 0);
|
| - rtp_payload_registry_.SetRtxPayloadType(kv.first, kv.second);
|
| - }
|
| }
|
|
|
| if (IsUlpfecEnabled()) {
|
| @@ -168,11 +163,6 @@ RtpVideoStreamReceiver::RtpVideoStreamReceiver(
|
| strncpy(red_codec.plName, "red", sizeof(red_codec.plName));
|
| red_codec.plType = config_.rtp.ulpfec.red_payload_type;
|
| RTC_CHECK(AddReceiveCodec(red_codec));
|
| - if (config_.rtp.ulpfec.red_rtx_payload_type != -1) {
|
| - rtp_payload_registry_.SetRtxPayloadType(
|
| - config_.rtp.ulpfec.red_rtx_payload_type,
|
| - config_.rtp.ulpfec.red_payload_type);
|
| - }
|
| }
|
|
|
| if (config_.rtp.rtcp_xr.receiver_reference_time_report)
|
| @@ -495,31 +485,7 @@ void RtpVideoStreamReceiver::ParseAndHandleEncapsulatingHeader(
|
| }
|
| ulpfec_receiver_->ProcessReceivedFec();
|
| } else if (rtp_payload_registry_.IsRtx(header)) {
|
| - if (header.headerLength + header.paddingLength == packet_length) {
|
| - // This is an empty packet and should be silently dropped before trying to
|
| - // parse the RTX header.
|
| - return;
|
| - }
|
| - // Remove the RTX header and parse the original RTP header.
|
| - if (packet_length < header.headerLength)
|
| - return;
|
| - if (packet_length > sizeof(restored_packet_))
|
| - return;
|
| - if (restored_packet_in_use_) {
|
| - LOG(LS_WARNING) << "Multiple RTX headers detected, dropping packet.";
|
| - return;
|
| - }
|
| - if (!rtp_payload_registry_.RestoreOriginalPacket(
|
| - restored_packet_, packet, &packet_length, config_.rtp.remote_ssrc,
|
| - header)) {
|
| - LOG(LS_WARNING) << "Incoming RTX packet: Invalid RTP header ssrc: "
|
| - << header.ssrc << " payload type: "
|
| - << static_cast<int>(header.payloadType);
|
| - return;
|
| - }
|
| - restored_packet_in_use_ = true;
|
| - OnRecoveredPacket(restored_packet_, packet_length);
|
| - restored_packet_in_use_ = false;
|
| + LOG(LS_WARNING) << "Unexpected RTX packet on media ssrc";
|
| }
|
| }
|
|
|
|
|