From ea8eff37370c8caf90b3c952779c91ae7898e944 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Niels=20M=C3=B6ller?= Date: Tue, 5 Jul 2022 15:50:11 +0200 Subject: [PATCH] Delete rtp_sender_ check in ModuleRtpRtcpImpl::SetSendingMediaStatus Bug: webrtc:10198 Change-Id: Ic40cd702717665a70f5aac0833963d467ea71dd0 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/267845 Reviewed-by: Danil Chapovalov Commit-Queue: Niels Moller Cr-Commit-Position: refs/heads/main@{#37452} --- modules/rtp_rtcp/source/rtp_rtcp_impl.cc | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/modules/rtp_rtcp/source/rtp_rtcp_impl.cc b/modules/rtp_rtcp/source/rtp_rtcp_impl.cc index 51b5f68b8c..ae4bf5187a 100644 --- a/modules/rtp_rtcp/source/rtp_rtcp_impl.cc +++ b/modules/rtp_rtcp/source/rtp_rtcp_impl.cc @@ -323,15 +323,8 @@ bool ModuleRtpRtcpImpl::Sending() const { return rtcp_sender_.Sending(); } -// TODO(nisse): This method shouldn't be called for a receive-only -// stream. Delete rtp_sender_ check as soon as all applications are -// updated. void ModuleRtpRtcpImpl::SetSendingMediaStatus(const bool sending) { - if (rtp_sender_) { - rtp_sender_->packet_generator.SetSendingMediaStatus(sending); - } else { - RTC_DCHECK(!sending); - } + rtp_sender_->packet_generator.SetSendingMediaStatus(sending); } bool ModuleRtpRtcpImpl::SendingMedia() const {