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

Side by Side Diff: media/cast/sender/video_encoder_impl_unittest.cc

Issue 661163004: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <vector> 5 #include <vector>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "media/base/video_frame.h" 10 #include "media/base/video_frame.h"
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 protected: 80 protected:
81 VideoEncoderImplTest() { 81 VideoEncoderImplTest() {
82 video_config_ = GetDefaultVideoSenderConfig(); 82 video_config_ = GetDefaultVideoSenderConfig();
83 video_config_.codec = CODEC_VIDEO_VP8; 83 video_config_.codec = CODEC_VIDEO_VP8;
84 gfx::Size size(video_config_.width, video_config_.height); 84 gfx::Size size(video_config_.width, video_config_.height);
85 video_frame_ = media::VideoFrame::CreateFrame( 85 video_frame_ = media::VideoFrame::CreateFrame(
86 VideoFrame::I420, size, gfx::Rect(size), size, base::TimeDelta()); 86 VideoFrame::I420, size, gfx::Rect(size), size, base::TimeDelta());
87 PopulateVideoFrame(video_frame_.get(), 123); 87 PopulateVideoFrame(video_frame_.get(), 123);
88 } 88 }
89 89
90 virtual ~VideoEncoderImplTest() {} 90 ~VideoEncoderImplTest() override {}
91 91
92 virtual void SetUp() override { 92 void SetUp() override {
93 testing_clock_ = new base::SimpleTestTickClock(); 93 testing_clock_ = new base::SimpleTestTickClock();
94 testing_clock_->Advance(base::TimeTicks::Now() - base::TimeTicks()); 94 testing_clock_->Advance(base::TimeTicks::Now() - base::TimeTicks());
95 task_runner_ = new test::FakeSingleThreadTaskRunner(testing_clock_); 95 task_runner_ = new test::FakeSingleThreadTaskRunner(testing_clock_);
96 cast_environment_ = 96 cast_environment_ =
97 new CastEnvironment(scoped_ptr<base::TickClock>(testing_clock_).Pass(), 97 new CastEnvironment(scoped_ptr<base::TickClock>(testing_clock_).Pass(),
98 task_runner_, 98 task_runner_,
99 task_runner_, 99 task_runner_,
100 task_runner_); 100 task_runner_);
101 } 101 }
102 102
103 virtual void TearDown() override { 103 void TearDown() override {
104 video_encoder_.reset(); 104 video_encoder_.reset();
105 task_runner_->RunTasks(); 105 task_runner_->RunTasks();
106 } 106 }
107 107
108 void CreateEncoder() { 108 void CreateEncoder() {
109 test_video_encoder_callback_ = new TestVideoEncoderCallback( 109 test_video_encoder_callback_ = new TestVideoEncoderCallback(
110 video_config_.max_number_of_video_buffers_used != 1); 110 video_config_.max_number_of_video_buffers_used != 1);
111 video_encoder_.reset( 111 video_encoder_.reset(
112 new VideoEncoderImpl(cast_environment_, video_config_)); 112 new VideoEncoderImpl(cast_environment_, video_config_));
113 } 113 }
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 EXPECT_TRUE(video_encoder_->EncodeVideoFrame( 208 EXPECT_TRUE(video_encoder_->EncodeVideoFrame(
209 video_frame_, testing_clock_->NowTicks(), frame_encoded_callback)); 209 video_frame_, testing_clock_->NowTicks(), frame_encoded_callback));
210 task_runner_->RunTasks(); 210 task_runner_->RunTasks();
211 } 211 }
212 212
213 EXPECT_EQ(10, test_video_encoder_callback_->count_frames_delivered()); 213 EXPECT_EQ(10, test_video_encoder_callback_->count_frames_delivered());
214 } 214 }
215 215
216 } // namespace cast 216 } // namespace cast
217 } // namespace media 217 } // namespace media
OLDNEW
« no previous file with comments | « media/cast/sender/external_video_encoder_unittest.cc ('k') | media/cast/sender/video_sender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698