Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(223)

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_sender.cc

Issue 2066473002: RtpRtcp: Remove the SetSendREDPayloadType and SendREDPayloadType methods (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@remove-red3
Patch Set: rebase Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_sender.h ('k') | webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/rtp_rtcp/source/rtp_sender.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
index b918a908a7d03d0420b1a4e0e08201f1834e4c91..d4749f9f5ed8230b20e7d134a59ac8618a9bc918 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
@@ -392,16 +392,6 @@ int32_t RTPSender::CheckPayloadType(int8_t payload_type,
LOG(LS_ERROR) << "Invalid payload_type " << payload_type;
return -1;
}
- if (audio_configured_) {
- int8_t red_pl_type = -1;
- if (audio_->RED(&red_pl_type) == 0) {
- // We have configured RED.
- if (red_pl_type == payload_type) {
- // And it's a match...
- return 0;
- }
- }
- }
if (payload_type_ == payload_type) {
if (!audio_configured_) {
*video_type = video_->VideoCodecType();
@@ -1624,20 +1614,6 @@ int32_t RTPSender::SetAudioLevel(uint8_t level_d_bov) {
return audio_->SetAudioLevel(level_d_bov);
}
-int32_t RTPSender::SetRED(int8_t payload_type) {
- if (!audio_configured_) {
- return -1;
- }
- return audio_->SetRED(payload_type);
-}
-
-int32_t RTPSender::RED(int8_t *payload_type) const {
- if (!audio_configured_) {
- return -1;
- }
- return audio_->RED(payload_type);
-}
-
RtpVideoCodecTypes RTPSender::VideoCodecType() const {
assert(!audio_configured_ && "Sender is an audio stream!");
return video_->VideoCodecType();
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_sender.h ('k') | webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698