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

Side by Side Diff: webrtc/video/video_quality_test.cc

Issue 2493723002: Explicitly use RTX for RED in VideoQualityTest and video_loopback. (Closed)
Patch Set: Rebase Created 4 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 #include "webrtc/video/video_quality_test.h" 10 #include "webrtc/video/video_quality_test.h"
(...skipping 1335 matching lines...) Expand 10 before | Expand all | Expand 10 after
1346 video_receive_configs_[stream_id].renderer = loopback_video.get(); 1346 video_receive_configs_[stream_id].renderer = loopback_video.get();
1347 if (params_.audio.enabled && params_.audio.sync_video) 1347 if (params_.audio.enabled && params_.audio.sync_video)
1348 video_receive_configs_[stream_id].sync_group = kSyncGroup; 1348 video_receive_configs_[stream_id].sync_group = kSyncGroup;
1349 1349
1350 video_send_config_.suspend_below_min_bitrate = 1350 video_send_config_.suspend_below_min_bitrate =
1351 params_.video.suspend_below_min_bitrate; 1351 params_.video.suspend_below_min_bitrate;
1352 1352
1353 if (params.video.fec) { 1353 if (params.video.fec) {
1354 video_send_config_.rtp.ulpfec.red_payload_type = kRedPayloadType; 1354 video_send_config_.rtp.ulpfec.red_payload_type = kRedPayloadType;
1355 video_send_config_.rtp.ulpfec.ulpfec_payload_type = kUlpfecPayloadType; 1355 video_send_config_.rtp.ulpfec.ulpfec_payload_type = kUlpfecPayloadType;
1356 video_send_config_.rtp.ulpfec.red_rtx_payload_type = kRtxRedPayloadType;
1356 video_receive_configs_[stream_id].rtp.ulpfec.red_payload_type = 1357 video_receive_configs_[stream_id].rtp.ulpfec.red_payload_type =
1357 kRedPayloadType; 1358 kRedPayloadType;
1358 video_receive_configs_[stream_id].rtp.ulpfec.ulpfec_payload_type = 1359 video_receive_configs_[stream_id].rtp.ulpfec.ulpfec_payload_type =
1359 kUlpfecPayloadType; 1360 kUlpfecPayloadType;
1361 video_receive_configs_[stream_id].rtp.ulpfec.red_rtx_payload_type =
1362 kRtxRedPayloadType;
1360 } 1363 }
1361 1364
1362 if (params_.screenshare.enabled) 1365 if (params_.screenshare.enabled)
1363 SetupScreenshare(); 1366 SetupScreenshare();
1364 1367
1365 video_send_stream_ = call->CreateVideoSendStream( 1368 video_send_stream_ = call->CreateVideoSendStream(
1366 video_send_config_.Copy(), video_encoder_config_.Copy()); 1369 video_send_config_.Copy(), video_encoder_config_.Copy());
1367 video_receive_stream = call->CreateVideoReceiveStream( 1370 video_receive_stream = call->CreateVideoReceiveStream(
1368 video_receive_configs_[stream_id].Copy()); 1371 video_receive_configs_[stream_id].Copy());
1369 CreateCapturer(); 1372 CreateCapturer();
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
1445 std::ostringstream str; 1448 std::ostringstream str;
1446 str << receive_logs_++; 1449 str << receive_logs_++;
1447 std::string path = 1450 std::string path =
1448 params_.video.encoded_frame_base_path + "." + str.str() + ".recv.ivf"; 1451 params_.video.encoded_frame_base_path + "." + str.str() + ".recv.ivf";
1449 stream->EnableEncodedFrameRecording(rtc::CreatePlatformFile(path), 1452 stream->EnableEncodedFrameRecording(rtc::CreatePlatformFile(path),
1450 10000000); 1453 10000000);
1451 } 1454 }
1452 } 1455 }
1453 1456
1454 } // namespace webrtc 1457 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698