| Index: webrtc/media/engine/webrtcvideoengine2_unittest.cc
|
| diff --git a/webrtc/media/engine/webrtcvideoengine2_unittest.cc b/webrtc/media/engine/webrtcvideoengine2_unittest.cc
|
| index 644a846e5934b224f6a7db7d3ae60d72287fb8c6..fd416404511f53219debcfa9815723c930407e46 100644
|
| --- a/webrtc/media/engine/webrtcvideoengine2_unittest.cc
|
| +++ b/webrtc/media/engine/webrtcvideoengine2_unittest.cc
|
| @@ -82,10 +82,10 @@ void VerifySendStreamHasRtxTypes(const webrtc::VideoSendStream::Config& config,
|
| EXPECT_TRUE(it != rtx_types.end() &&
|
| it->second == config.rtp.rtx.payload_type);
|
|
|
| - if (config.rtp.fec.red_rtx_payload_type != -1) {
|
| - it = rtx_types.find(config.rtp.fec.red_payload_type);
|
| + if (config.rtp.ulpfec.red_rtx_payload_type != -1) {
|
| + it = rtx_types.find(config.rtp.ulpfec.red_payload_type);
|
| EXPECT_TRUE(it != rtx_types.end() &&
|
| - it->second == config.rtp.fec.red_rtx_payload_type);
|
| + it->second == config.rtp.ulpfec.red_rtx_payload_type);
|
| }
|
| }
|
| } // namespace
|
| @@ -2245,8 +2245,8 @@ TEST_F(WebRtcVideoChannel2Test, SetDefaultSendCodecs) {
|
|
|
| // Make sure NACK and FEC are enabled on the correct payload types.
|
| EXPECT_EQ(1000, config.rtp.nack.rtp_history_ms);
|
| - EXPECT_EQ(default_ulpfec_codec_.id, config.rtp.fec.ulpfec_payload_type);
|
| - EXPECT_EQ(default_red_codec_.id, config.rtp.fec.red_payload_type);
|
| + EXPECT_EQ(default_ulpfec_codec_.id, config.rtp.ulpfec.ulpfec_payload_type);
|
| + EXPECT_EQ(default_red_codec_.id, config.rtp.ulpfec.red_payload_type);
|
|
|
| EXPECT_EQ(1u, config.rtp.rtx.ssrcs.size());
|
| EXPECT_EQ(kRtxSsrcs1[0], config.rtp.rtx.ssrcs[0]);
|
| @@ -2262,8 +2262,8 @@ TEST_F(WebRtcVideoChannel2Test, SetSendCodecsWithoutFec) {
|
| FakeVideoSendStream* stream = AddSendStream();
|
| webrtc::VideoSendStream::Config config = stream->GetConfig().Copy();
|
|
|
| - EXPECT_EQ(-1, config.rtp.fec.ulpfec_payload_type);
|
| - EXPECT_EQ(-1, config.rtp.fec.red_payload_type);
|
| + EXPECT_EQ(-1, config.rtp.ulpfec.ulpfec_payload_type);
|
| + EXPECT_EQ(-1, config.rtp.ulpfec.red_payload_type);
|
| }
|
|
|
| TEST_F(WebRtcVideoChannel2Test,
|
| @@ -2301,14 +2301,14 @@ TEST_F(WebRtcVideoChannel2Test, SetSendCodecsWithoutFecDisablesFec) {
|
| FakeVideoSendStream* stream = AddSendStream();
|
| webrtc::VideoSendStream::Config config = stream->GetConfig().Copy();
|
|
|
| - EXPECT_EQ(kUlpfecCodec.id, config.rtp.fec.ulpfec_payload_type);
|
| + EXPECT_EQ(kUlpfecCodec.id, config.rtp.ulpfec.ulpfec_payload_type);
|
|
|
| parameters.codecs.pop_back();
|
| ASSERT_TRUE(channel_->SetSendParameters(parameters));
|
| stream = fake_call_->GetVideoSendStreams()[0];
|
| ASSERT_TRUE(stream != NULL);
|
| config = stream->GetConfig().Copy();
|
| - EXPECT_EQ(-1, config.rtp.fec.ulpfec_payload_type)
|
| + EXPECT_EQ(-1, config.rtp.ulpfec.ulpfec_payload_type)
|
| << "SetSendCodec without FEC should disable current FEC.";
|
| }
|
|
|
| @@ -2657,20 +2657,22 @@ TEST_F(WebRtcVideoChannel2Test, SetRecvCodecsWithoutFecDisablesFec) {
|
|
|
| FakeVideoReceiveStream* stream = AddRecvStream();
|
|
|
| - EXPECT_EQ(kUlpfecCodec.id, stream->GetConfig().rtp.fec.ulpfec_payload_type);
|
| + EXPECT_EQ(kUlpfecCodec.id,
|
| + stream->GetConfig().rtp.ulpfec.ulpfec_payload_type);
|
|
|
| cricket::VideoRecvParameters recv_parameters;
|
| recv_parameters.codecs.push_back(kVp8Codec);
|
| ASSERT_TRUE(channel_->SetRecvParameters(recv_parameters));
|
| stream = fake_call_->GetVideoReceiveStreams()[0];
|
| ASSERT_TRUE(stream != NULL);
|
| - EXPECT_EQ(-1, stream->GetConfig().rtp.fec.ulpfec_payload_type)
|
| + EXPECT_EQ(-1, stream->GetConfig().rtp.ulpfec.ulpfec_payload_type)
|
| << "SetSendCodec without FEC should disable current FEC.";
|
| }
|
|
|
| TEST_F(WebRtcVideoChannel2Test, SetSendParamsWithoutFecDisablesReceivingFec) {
|
| FakeVideoReceiveStream* stream = AddRecvStream();
|
| - EXPECT_EQ(kUlpfecCodec.id, stream->GetConfig().rtp.fec.ulpfec_payload_type);
|
| + EXPECT_EQ(kUlpfecCodec.id,
|
| + stream->GetConfig().rtp.ulpfec.ulpfec_payload_type);
|
|
|
| cricket::VideoRecvParameters recv_parameters;
|
| recv_parameters.codecs.push_back(kVp8Codec);
|
| @@ -2679,14 +2681,14 @@ TEST_F(WebRtcVideoChannel2Test, SetSendParamsWithoutFecDisablesReceivingFec) {
|
| ASSERT_TRUE(channel_->SetRecvParameters(recv_parameters));
|
| stream = fake_call_->GetVideoReceiveStreams()[0];
|
| ASSERT_TRUE(stream != NULL);
|
| - EXPECT_EQ(kUlpfecCodec.id, stream->GetConfig().rtp.fec.ulpfec_payload_type)
|
| + EXPECT_EQ(kUlpfecCodec.id, stream->GetConfig().rtp.ulpfec.ulpfec_payload_type)
|
| << "FEC should be enabled on the recieve stream.";
|
|
|
| cricket::VideoSendParameters send_parameters;
|
| send_parameters.codecs.push_back(kVp8Codec);
|
| ASSERT_TRUE(channel_->SetSendParameters(send_parameters));
|
| stream = fake_call_->GetVideoReceiveStreams()[0];
|
| - EXPECT_EQ(-1, stream->GetConfig().rtp.fec.ulpfec_payload_type)
|
| + EXPECT_EQ(-1, stream->GetConfig().rtp.ulpfec.ulpfec_payload_type)
|
| << "FEC should have been disabled when we know the other side won't do "
|
| "FEC.";
|
|
|
| @@ -2694,7 +2696,7 @@ TEST_F(WebRtcVideoChannel2Test, SetSendParamsWithoutFecDisablesReceivingFec) {
|
| send_parameters.codecs.push_back(kUlpfecCodec);
|
| ASSERT_TRUE(channel_->SetSendParameters(send_parameters));
|
| stream = fake_call_->GetVideoReceiveStreams()[0];
|
| - EXPECT_EQ(kUlpfecCodec.id, stream->GetConfig().rtp.fec.ulpfec_payload_type)
|
| + EXPECT_EQ(kUlpfecCodec.id, stream->GetConfig().rtp.ulpfec.ulpfec_payload_type)
|
| << "FEC should be enabled on the recieve stream.";
|
| }
|
|
|
|
|