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

Side by Side Diff: webrtc/modules/video_coding/jitter_buffer_unittest.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) 2011 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2011 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 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 } 208 }
209 209
210 void RequestKeyFrame() override { ++keyframe_requests_; } 210 void RequestKeyFrame() override { ++keyframe_requests_; }
211 211
212 ::testing::NiceMock<ProcessThreadMock> process_thread_mock_; 212 ::testing::NiceMock<ProcessThreadMock> process_thread_mock_;
213 std::vector<uint16_t> nack_sent_; 213 std::vector<uint16_t> nack_sent_;
214 int keyframe_requests_; 214 int keyframe_requests_;
215 215
216 protected: 216 protected:
217 TestBasicJitterBuffer() : scoped_field_trial_(GetParam()) {} 217 TestBasicJitterBuffer() : scoped_field_trial_(GetParam()) {}
218 virtual void SetUp() { 218 void SetUp() override {
219 clock_.reset(new SimulatedClock(0)); 219 clock_.reset(new SimulatedClock(0));
220 jitter_buffer_.reset(new VCMJitterBuffer( 220 jitter_buffer_.reset(new VCMJitterBuffer(
221 clock_.get(), 221 clock_.get(),
222 std::unique_ptr<EventWrapper>(event_factory_.CreateEvent()), 222 std::unique_ptr<EventWrapper>(event_factory_.CreateEvent()),
223 this, 223 this,
224 this)); 224 this));
225 jitter_buffer_->Start(); 225 jitter_buffer_->Start();
226 seq_num_ = 1234; 226 seq_num_ = 1234;
227 timestamp_ = 0; 227 timestamp_ = 0;
228 size_ = 1400; 228 size_ = 1400;
(...skipping 2434 matching lines...) Expand 10 before | Expand all | Expand 10 after
2663 2663
2664 // Stream should be decodable from this point. 2664 // Stream should be decodable from this point.
2665 clock_->AdvanceTimeMilliseconds(kDefaultFramePeriodMs); 2665 clock_->AdvanceTimeMilliseconds(kDefaultFramePeriodMs);
2666 InsertFrame(kVideoFrameDelta); 2666 InsertFrame(kVideoFrameDelta);
2667 EXPECT_TRUE(DecodeCompleteFrame()); 2667 EXPECT_TRUE(DecodeCompleteFrame());
2668 nack_list = jitter_buffer_->GetNackList(&extended); 2668 nack_list = jitter_buffer_->GetNackList(&extended);
2669 EXPECT_EQ(0u, nack_list.size()); 2669 EXPECT_EQ(0u, nack_list.size());
2670 } 2670 }
2671 2671
2672 } // namespace webrtc 2672 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/video_coding/codecs/vp8/vp8_impl.h ('k') | webrtc/p2p/base/faketransportcontroller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698