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

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

Issue 1921653002: Enable -Winconsistent-missing-override flag. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase. Created 4 years, 7 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 | « webrtc/build/common.gypi ('k') | webrtc/common_video/i420_buffer_pool.cc » ('j') | 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) 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 10
(...skipping 377 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 int time_offset_ms = real_capture_timestamp - estimated_capture_timestamp; 388 int time_offset_ms = real_capture_timestamp - estimated_capture_timestamp;
389 time_offset_ms = time_offset_ms / 90; 389 time_offset_ms = time_offset_ms / 90;
390 std::stringstream ss; 390 std::stringstream ss;
391 ss << time_offset_ms; 391 ss << time_offset_ms;
392 392
393 webrtc::test::PrintResult( 393 webrtc::test::PrintResult(
394 "capture_ntp_time", "", "real - estimated", ss.str(), "ms", true); 394 "capture_ntp_time", "", "real - estimated", ss.str(), "ms", true);
395 EXPECT_TRUE(std::abs(time_offset_ms) < threshold_ms_); 395 EXPECT_TRUE(std::abs(time_offset_ms) < threshold_ms_);
396 } 396 }
397 397
398 virtual Action OnSendRtp(const uint8_t* packet, size_t length) { 398 Action OnSendRtp(const uint8_t* packet, size_t length) override {
399 rtc::CritScope lock(&crit_); 399 rtc::CritScope lock(&crit_);
400 RTPHeader header; 400 RTPHeader header;
401 EXPECT_TRUE(parser_->Parse(packet, length, &header)); 401 EXPECT_TRUE(parser_->Parse(packet, length, &header));
402 402
403 if (!rtp_start_timestamp_set_) { 403 if (!rtp_start_timestamp_set_) {
404 // Calculate the rtp timestamp offset in order to calculate the real 404 // Calculate the rtp timestamp offset in order to calculate the real
405 // capture time. 405 // capture time.
406 uint32_t first_capture_timestamp = 406 uint32_t first_capture_timestamp =
407 90 * static_cast<uint32_t>(capturer_->first_frame_capture_time()); 407 90 * static_cast<uint32_t>(capturer_->first_frame_capture_time());
408 rtp_start_timestamp_ = header.timestamp - first_capture_timestamp; 408 rtp_start_timestamp_ = header.timestamp - first_capture_timestamp;
(...skipping 284 matching lines...) Expand 10 before | Expand all | Expand 10 after
693 int encoder_inits_; 693 int encoder_inits_;
694 uint32_t last_set_bitrate_; 694 uint32_t last_set_bitrate_;
695 VideoSendStream* send_stream_; 695 VideoSendStream* send_stream_;
696 VideoEncoderConfig encoder_config_; 696 VideoEncoderConfig encoder_config_;
697 } test; 697 } test;
698 698
699 RunBaseTest(&test); 699 RunBaseTest(&test);
700 } 700 }
701 701
702 } // namespace webrtc 702 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/build/common.gypi ('k') | webrtc/common_video/i420_buffer_pool.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698