Chromium Code Reviews| Index: content/common/gpu/media/android_video_decode_accelerator_unittest.cc |
| diff --git a/content/common/gpu/media/android_video_decode_accelerator_unittest.cc b/content/common/gpu/media/android_video_decode_accelerator_unittest.cc |
| index e391d11ed4ec509b23692d2d594372d8ee21d659..3ab48ee5f8f6c1174b201f7331688437b3535c6f 100644 |
| --- a/content/common/gpu/media/android_video_decode_accelerator_unittest.cc |
| +++ b/content/common/gpu/media/android_video_decode_accelerator_unittest.cc |
| @@ -34,28 +34,26 @@ class MockVideoDecodeAcceleratorClient |
| : public media::VideoDecodeAccelerator::Client { |
| public: |
| MockVideoDecodeAcceleratorClient() {}; |
| - virtual ~MockVideoDecodeAcceleratorClient() {}; |
| + ~MockVideoDecodeAcceleratorClient() override{}; |
|
Charlie Reis
2015/02/04 20:46:43
nit: Space before braces, here and below.
(Bug in
dcheng
2015/02/04 20:47:58
The issue is the superfluous ;. I've reported this
|
| // VideoDecodeAccelerator::Client implementation. |
| - virtual void ProvidePictureBuffers(uint32 requested_num_of_buffers, |
| - const gfx::Size& dimensions, |
| - uint32 texture_target) override {}; |
| - virtual void DismissPictureBuffer(int32 picture_buffer_id) override {}; |
| - virtual void PictureReady(const media::Picture& picture) override {}; |
| - virtual void NotifyEndOfBitstreamBuffer( |
| - int32 bitstream_buffer_id) override {}; |
| - virtual void NotifyFlushDone() override {}; |
| - virtual void NotifyResetDone() override {}; |
| - virtual void NotifyError( |
| - media::VideoDecodeAccelerator::Error error) override {}; |
| + void ProvidePictureBuffers(uint32 requested_num_of_buffers, |
| + const gfx::Size& dimensions, |
| + uint32 texture_target) override{}; |
| + void DismissPictureBuffer(int32 picture_buffer_id) override{}; |
| + void PictureReady(const media::Picture& picture) override{}; |
| + void NotifyEndOfBitstreamBuffer(int32 bitstream_buffer_id) override{}; |
| + void NotifyFlushDone() override{}; |
| + void NotifyResetDone() override{}; |
| + void NotifyError(media::VideoDecodeAccelerator::Error error) override{}; |
| }; |
| class AndroidVideoDecodeAcceleratorTest : public testing::Test { |
| public: |
| - virtual ~AndroidVideoDecodeAcceleratorTest() {} |
| + ~AndroidVideoDecodeAcceleratorTest() override {} |
| protected: |
| - virtual void SetUp() override { |
| + void SetUp() override { |
| JNIEnv* env = base::android::AttachCurrentThread(); |
| media::RegisterJni(env); |
| // TODO(felipeg): fix GL bindings, so that the decoder can perform GL |