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

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
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 395 matching lines...) Expand 10 before | Expand all | Expand 10 after
406 int time_offset_ms = real_capture_timestamp - estimated_capture_timestamp; 406 int time_offset_ms = real_capture_timestamp - estimated_capture_timestamp;
407 time_offset_ms = time_offset_ms / 90; 407 time_offset_ms = time_offset_ms / 90;
408 std::stringstream ss; 408 std::stringstream ss;
409 ss << time_offset_ms; 409 ss << time_offset_ms;
410 410
411 webrtc::test::PrintResult( 411 webrtc::test::PrintResult(
412 "capture_ntp_time", "", "real - estimated", ss.str(), "ms", true); 412 "capture_ntp_time", "", "real - estimated", ss.str(), "ms", true);
413 EXPECT_TRUE(std::abs(time_offset_ms) < threshold_ms_); 413 EXPECT_TRUE(std::abs(time_offset_ms) < threshold_ms_);
414 } 414 }
415 415
416 virtual Action OnSendRtp(const uint8_t* packet, size_t length) { 416 Action OnSendRtp(const uint8_t* packet, size_t length) override {
417 rtc::CritScope lock(&crit_); 417 rtc::CritScope lock(&crit_);
418 RTPHeader header; 418 RTPHeader header;
419 EXPECT_TRUE(parser_->Parse(packet, length, &header)); 419 EXPECT_TRUE(parser_->Parse(packet, length, &header));
420 420
421 if (!rtp_start_timestamp_set_) { 421 if (!rtp_start_timestamp_set_) {
422 // Calculate the rtp timestamp offset in order to calculate the real 422 // Calculate the rtp timestamp offset in order to calculate the real
423 // capture time. 423 // capture time.
424 uint32_t first_capture_timestamp = 424 uint32_t first_capture_timestamp =
425 90 * static_cast<uint32_t>(capturer_->first_frame_capture_time()); 425 90 * static_cast<uint32_t>(capturer_->first_frame_capture_time());
426 rtp_start_timestamp_ = header.timestamp - first_capture_timestamp; 426 rtp_start_timestamp_ = header.timestamp - first_capture_timestamp;
(...skipping 284 matching lines...) Expand 10 before | Expand all | Expand 10 after
711 int encoder_inits_; 711 int encoder_inits_;
712 uint32_t last_set_bitrate_; 712 uint32_t last_set_bitrate_;
713 VideoSendStream* send_stream_; 713 VideoSendStream* send_stream_;
714 VideoEncoderConfig encoder_config_; 714 VideoEncoderConfig encoder_config_;
715 } test; 715 } test;
716 716
717 RunBaseTest(&test); 717 RunBaseTest(&test);
718 } 718 }
719 719
720 } // namespace webrtc 720 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698