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

Side by Side Diff: webrtc/call/call_perf_tests.cc

Issue 1693443002: VideoCaptureInput enforce VideoFrame::render_time to be generated by webrtc clock. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 10 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 unified diff | Download patch
« no previous file with comments | « no previous file | webrtc/video/video_capture_input.h » ('j') | webrtc/video/video_capture_input.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 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 <algorithm> 10 #include <algorithm>
(...skipping 374 matching lines...) Expand 10 before | Expand all | Expand 10 after
385 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithVideoCreatedFirst) { 385 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithVideoCreatedFirst) {
386 TestAudioVideoSync(false, false, DriftingClock::kNoDrift, 386 TestAudioVideoSync(false, false, DriftingClock::kNoDrift,
387 DriftingClock::kNoDrift, DriftingClock::kNoDrift); 387 DriftingClock::kNoDrift, DriftingClock::kNoDrift);
388 } 388 }
389 389
390 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithFec) { 390 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithFec) {
391 TestAudioVideoSync(true, false, DriftingClock::kNoDrift, 391 TestAudioVideoSync(true, false, DriftingClock::kNoDrift,
392 DriftingClock::kNoDrift, DriftingClock::kNoDrift); 392 DriftingClock::kNoDrift, DriftingClock::kNoDrift);
393 } 393 }
394 394
395 // TODO(danilchap): Reenable after adding support for frame capture clock 395 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithVideoNtpDrift) {
396 // that is not in sync with local TickTime clock.
397 TEST_F(CallPerfTest, DISABLED_PlaysOutAudioAndVideoInSyncWithVideoNtpDrift) {
398 TestAudioVideoSync(false, true, DriftingClock::PercentsFaster(10.0f), 396 TestAudioVideoSync(false, true, DriftingClock::PercentsFaster(10.0f),
399 DriftingClock::kNoDrift, DriftingClock::kNoDrift); 397 DriftingClock::kNoDrift, DriftingClock::kNoDrift);
400 } 398 }
401 399
402 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithAudioRtpDrift) { 400 TEST_F(CallPerfTest, PlaysOutAudioAndVideoInSyncWithAudioRtpDrift) {
403 TestAudioVideoSync(false, true, DriftingClock::kNoDrift, 401 TestAudioVideoSync(false, true, DriftingClock::kNoDrift,
404 DriftingClock::kNoDrift, 402 DriftingClock::kNoDrift,
405 DriftingClock::PercentsFaster(30.0f)); 403 DriftingClock::PercentsFaster(30.0f));
406 } 404 }
407 405
(...skipping 388 matching lines...) Expand 10 before | Expand all | Expand 10 after
796 int encoder_inits_; 794 int encoder_inits_;
797 uint32_t last_set_bitrate_; 795 uint32_t last_set_bitrate_;
798 VideoSendStream* send_stream_; 796 VideoSendStream* send_stream_;
799 VideoEncoderConfig encoder_config_; 797 VideoEncoderConfig encoder_config_;
800 } test; 798 } test;
801 799
802 RunBaseTest(&test); 800 RunBaseTest(&test);
803 } 801 }
804 802
805 } // namespace webrtc 803 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | webrtc/video/video_capture_input.h » ('j') | webrtc/video/video_capture_input.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698