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

Unified Diff: media/cast/sender/external_video_encoder.cc

Issue 655713003: Standardize usage of virtual/override/final in media/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/cast/sender/external_video_encoder.h ('k') | media/cast/sender/fake_software_video_encoder.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/cast/sender/external_video_encoder.cc
diff --git a/media/cast/sender/external_video_encoder.cc b/media/cast/sender/external_video_encoder.cc
index a856ed3ba8e89b7b9751ad0f3f779b02844b5a67..6dec102436ed50892f671ca1c5a33ca625e8603c 100644
--- a/media/cast/sender/external_video_encoder.cc
+++ b/media/cast/sender/external_video_encoder.cc
@@ -173,7 +173,7 @@ class LocalVideoEncodeAcceleratorClient
}
protected:
- virtual void NotifyError(VideoEncodeAccelerator::Error error) override {
+ void NotifyError(VideoEncodeAccelerator::Error error) override {
DCHECK(encoder_task_runner_.get());
DCHECK(encoder_task_runner_->RunsTasksOnCurrentThread());
VLOG(1) << "ExternalVideoEncoder NotifyError: " << error;
@@ -185,9 +185,9 @@ class LocalVideoEncodeAcceleratorClient
}
// Called to allocate the input and output buffers.
- virtual void RequireBitstreamBuffers(unsigned int input_count,
- const gfx::Size& input_coded_size,
- size_t output_buffer_size) override {
+ void RequireBitstreamBuffers(unsigned int input_count,
+ const gfx::Size& input_coded_size,
+ size_t output_buffer_size) override {
DCHECK(encoder_task_runner_.get());
DCHECK(encoder_task_runner_->RunsTasksOnCurrentThread());
DCHECK(video_encode_accelerator_);
@@ -202,9 +202,9 @@ class LocalVideoEncodeAcceleratorClient
// Encoder has encoded a frame and it's available in one of out output
// buffers.
- virtual void BitstreamBufferReady(int32 bitstream_buffer_id,
- size_t payload_size,
- bool key_frame) override {
+ void BitstreamBufferReady(int32 bitstream_buffer_id,
+ size_t payload_size,
+ bool key_frame) override {
DCHECK(encoder_task_runner_.get());
DCHECK(encoder_task_runner_->RunsTasksOnCurrentThread());
if (bitstream_buffer_id < 0 ||
@@ -361,7 +361,7 @@ class LocalVideoEncodeAcceleratorClient
friend class base::RefCountedThreadSafe<LocalVideoEncodeAcceleratorClient>;
- virtual ~LocalVideoEncodeAcceleratorClient() {
+ ~LocalVideoEncodeAcceleratorClient() override {
Destroy();
DCHECK(!video_encode_accelerator_);
}
« no previous file with comments | « media/cast/sender/external_video_encoder.h ('k') | media/cast/sender/fake_software_video_encoder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698