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

Unified Diff: webrtc/video/rtp_video_stream_receiver.cc

Issue 3000273002: Reverse |rtx_payload_types| map, and rename. (Closed)
Patch Set: Created 3 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
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 6064e6971557932d3cd8ffea8bb6cf680ca68af4..d0490a6c8b5a773b2659cde4ccd8b927ae44a167 100644
--- a/webrtc/video/rtp_video_stream_receiver.cc
+++ b/webrtc/video/rtp_video_stream_receiver.cc
@@ -148,9 +148,9 @@ RtpVideoStreamReceiver::RtpVideoStreamReceiver(
if (config_.rtp.rtx_ssrc) {
rtp_payload_registry_.SetRtxSsrc(config_.rtp.rtx_ssrc);
- for (const auto& kv : config_.rtp.rtx_payload_types) {
+ for (const auto& kv : config_.rtp.media_pt_by_rtx_pt) {
RTC_DCHECK(kv.second != 0);
danilchap 2017/08/21 14:11:26 change DCHECK to validate rtx payload type (media
nisse-webrtc 2017/08/22 07:11:01 Done, also changed to RTC_DCHECK_NE.
- rtp_payload_registry_.SetRtxPayloadType(kv.second, kv.first);
+ rtp_payload_registry_.SetRtxPayloadType(kv.first, kv.second);
}
}

Powered by Google App Engine
This is Rietveld 408576698