Index: webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc |
diff --git a/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc b/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc |
index b5d739e13f91fe7d12ff1a22dfe57e8068f26921..395836106c5225effd6d48c18ed92a90d2baf556 100644 |
--- a/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc |
+++ b/webrtc/logging/rtc_event_log/rtc_event_log_unittest_helper.cc |
@@ -259,73 +259,41 @@ void RtcEventLogTestHelper::VerifyVideoReceiveStreamConfig( |
void RtcEventLogTestHelper::VerifyVideoSendStreamConfig( |
const ParsedRtcEventLog& parsed_log, |
size_t index, |
- const VideoSendStream::Config& config) { |
+ const rtclog::StreamConfig& config) { |
const rtclog::Event& event = parsed_log.events_[index]; |
ASSERT_TRUE(IsValidBasicEvent(event)); |
ASSERT_EQ(rtclog::Event::VIDEO_SENDER_CONFIG_EVENT, event.type()); |
const rtclog::VideoSendConfig& sender_config = event.video_sender_config(); |
- // Check SSRCs. |
- ASSERT_EQ(static_cast<int>(config.rtp.ssrcs.size()), |
- sender_config.ssrcs_size()); |
- for (int i = 0; i < sender_config.ssrcs_size(); i++) { |
- EXPECT_EQ(config.rtp.ssrcs[i], sender_config.ssrcs(i)); |
- } |
+ |
+ EXPECT_EQ(config.local_ssrc, sender_config.ssrcs(0)); |
+ EXPECT_EQ(config.rtx_ssrc, sender_config.rtx_ssrcs(0)); |
+ |
// Check header extensions. |
- ASSERT_EQ(static_cast<int>(config.rtp.extensions.size()), |
+ ASSERT_EQ(static_cast<int>(config.rtp_extensions.size()), |
sender_config.header_extensions_size()); |
for (int i = 0; i < sender_config.header_extensions_size(); i++) { |
ASSERT_TRUE(sender_config.header_extensions(i).has_name()); |
ASSERT_TRUE(sender_config.header_extensions(i).has_id()); |
const std::string& name = sender_config.header_extensions(i).name(); |
int id = sender_config.header_extensions(i).id(); |
- EXPECT_EQ(config.rtp.extensions[i].id, id); |
- EXPECT_EQ(config.rtp.extensions[i].uri, name); |
- } |
- // Check RTX settings. |
- ASSERT_EQ(static_cast<int>(config.rtp.rtx.ssrcs.size()), |
- sender_config.rtx_ssrcs_size()); |
- for (int i = 0; i < sender_config.rtx_ssrcs_size(); i++) { |
- EXPECT_EQ(config.rtp.rtx.ssrcs[i], sender_config.rtx_ssrcs(i)); |
- } |
- if (sender_config.rtx_ssrcs_size() > 0) { |
- ASSERT_TRUE(sender_config.has_rtx_payload_type()); |
- EXPECT_EQ(config.rtp.rtx.payload_type, sender_config.rtx_payload_type()); |
+ EXPECT_EQ(config.rtp_extensions[i].id, id); |
+ EXPECT_EQ(config.rtp_extensions[i].uri, name); |
} |
// Check encoder. |
ASSERT_TRUE(sender_config.has_encoder()); |
ASSERT_TRUE(sender_config.encoder().has_name()); |
ASSERT_TRUE(sender_config.encoder().has_payload_type()); |
- EXPECT_EQ(config.encoder_settings.payload_name, |
- sender_config.encoder().name()); |
- EXPECT_EQ(config.encoder_settings.payload_type, |
+ EXPECT_EQ(config.codecs[0].payload_name, sender_config.encoder().name()); |
+ EXPECT_EQ(config.codecs[0].payload_type, |
sender_config.encoder().payload_type()); |
+ EXPECT_EQ(config.codecs[0].rtx_payload_type, |
+ sender_config.rtx_payload_type()); |
+ |
// Check consistency of the parser. |
- VideoSendStream::Config parsed_config(nullptr); |
+ rtclog::StreamConfig parsed_config; |
parsed_log.GetVideoSendConfig(index, &parsed_config); |
- // Check SSRCs |
- EXPECT_EQ(config.rtp.ssrcs.size(), parsed_config.rtp.ssrcs.size()); |
- for (size_t i = 0; i < config.rtp.ssrcs.size(); i++) { |
- EXPECT_EQ(config.rtp.ssrcs[i], parsed_config.rtp.ssrcs[i]); |
- } |
- // Check header extensions. |
- EXPECT_EQ(config.rtp.extensions.size(), parsed_config.rtp.extensions.size()); |
- for (size_t i = 0; i < parsed_config.rtp.extensions.size(); i++) { |
- EXPECT_EQ(config.rtp.extensions[i].uri, |
- parsed_config.rtp.extensions[i].uri); |
- EXPECT_EQ(config.rtp.extensions[i].id, parsed_config.rtp.extensions[i].id); |
- } |
- // Check RTX settings. |
- EXPECT_EQ(config.rtp.rtx.ssrcs.size(), parsed_config.rtp.rtx.ssrcs.size()); |
- for (size_t i = 0; i < config.rtp.rtx.ssrcs.size(); i++) { |
- EXPECT_EQ(config.rtp.rtx.ssrcs[i], parsed_config.rtp.rtx.ssrcs[i]); |
- } |
- EXPECT_EQ(config.rtp.rtx.payload_type, parsed_config.rtp.rtx.payload_type); |
- // Check encoder. |
- EXPECT_EQ(config.encoder_settings.payload_name, |
- parsed_config.encoder_settings.payload_name); |
- EXPECT_EQ(config.encoder_settings.payload_type, |
- parsed_config.encoder_settings.payload_type); |
+ VerifyStreamConfigsAreEqual(config, parsed_config); |
} |
void RtcEventLogTestHelper::VerifyAudioReceiveStreamConfig( |