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

Unified Diff: webrtc/logging/rtc_event_log/rtc_event_log_parser.cc

Issue 2646073004: Make RTX pt/apt reconfigurable by calling WebRtcVideoChannel2::SetRecvParameters. (Closed)
Patch Set: Created 3 years, 11 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/logging/rtc_event_log/rtc_event_log_parser.cc
diff --git a/webrtc/logging/rtc_event_log/rtc_event_log_parser.cc b/webrtc/logging/rtc_event_log/rtc_event_log_parser.cc
index ce55a4fb39b740a422eda7b874a8a000f88e4601..b550b751bccab4554e5d47f03355cb158078d771 100644
--- a/webrtc/logging/rtc_event_log/rtc_event_log_parser.cc
+++ b/webrtc/logging/rtc_event_log/rtc_event_log_parser.cc
@@ -312,17 +312,16 @@ void ParsedRtcEventLog::GetVideoReceiveConfig(
RTC_CHECK(receiver_config.has_remb());
config->rtp.remb = receiver_config.remb();
// Get RTX map.
- config->rtp.rtx.clear();
+ config->rtp.rtx_payload_types.clear();
for (int i = 0; i < receiver_config.rtx_map_size(); i++) {
const rtclog::RtxMap& map = receiver_config.rtx_map(i);
RTC_CHECK(map.has_payload_type());
RTC_CHECK(map.has_config());
RTC_CHECK(map.config().has_rtx_ssrc());
RTC_CHECK(map.config().has_rtx_payload_type());
- webrtc::VideoReceiveStream::Config::Rtp::Rtx rtx_pair;
- rtx_pair.ssrc = map.config().rtx_ssrc();
- rtx_pair.payload_type = map.config().rtx_payload_type();
- config->rtp.rtx.insert(std::make_pair(map.payload_type(), rtx_pair));
+ config->rtp.rtx_ssrc = map.config().rtx_ssrc();
terelius 2017/01/23 12:52:01 Check that all RTX-SSRCs are identical?
brandtr 2017/01/24 10:04:40 WDYT about this approach?
+ config->rtp.rtx_payload_types.insert(
+ std::make_pair(map.payload_type(), map.config().rtx_payload_type()));
}
// Get header extensions.
GetHeaderExtensions(&config->rtp.extensions,

Powered by Google App Engine
This is Rietveld 408576698