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

Side by Side Diff: media/video/video_decode_accelerator.cc

Issue 1490333005: Don't require VDAs to return all PictureBuffers at once. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ...because trybots have feelings too. Created 5 years 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 (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "media/video/video_decode_accelerator.h" 5 #include "media/video/video_decode_accelerator.h"
6 6
7 #include <GLES2/gl2.h> 7 #include <GLES2/gl2.h>
8 #include "base/logging.h" 8 #include "base/logging.h"
9 9
10 namespace media { 10 namespace media {
(...skipping 12 matching lines...) Expand all
23 // GPU process subclasses must override this. 23 // GPU process subclasses must override this.
24 LOG(FATAL) << "This should only get called in the GPU process"; 24 LOG(FATAL) << "This should only get called in the GPU process";
25 return false; // not reached 25 return false; // not reached
26 } 26 }
27 27
28 GLenum VideoDecodeAccelerator::GetSurfaceInternalFormat() const { 28 GLenum VideoDecodeAccelerator::GetSurfaceInternalFormat() const {
29 return GL_RGBA; 29 return GL_RGBA;
30 } 30 }
31 31
32 VideoDecodeAccelerator::SupportedProfile::SupportedProfile() 32 VideoDecodeAccelerator::SupportedProfile::SupportedProfile()
33 : profile(media::VIDEO_CODEC_PROFILE_UNKNOWN) { 33 : profile(media::VIDEO_CODEC_PROFILE_UNKNOWN) {}
Pawel Osciak 2015/12/05 00:18:55 Nit: we should probably either change style for al
liberato (no reviews please) 2015/12/07 19:04:39 Done, changed 35 to match.
34 }
35 34
36 VideoDecodeAccelerator::SupportedProfile::~SupportedProfile() { 35 VideoDecodeAccelerator::SupportedProfile::~SupportedProfile() {
37 } 36 }
38 37
38 VideoDecodeAccelerator::Capabilities::Capabilities() : flags(NO_FLAGS) {}
39
40 VideoDecodeAccelerator::Capabilities::~Capabilities() {}
41
39 } // namespace media 42 } // namespace media
40 43
41 namespace std { 44 namespace std {
42 45
43 void default_delete<media::VideoDecodeAccelerator>::operator()( 46 void default_delete<media::VideoDecodeAccelerator>::operator()(
44 media::VideoDecodeAccelerator* vda) const { 47 media::VideoDecodeAccelerator* vda) const {
45 vda->Destroy(); 48 vda->Destroy();
46 } 49 }
47 50
48 } // namespace std 51 } // namespace std
OLDNEW
« media/video/video_decode_accelerator.h ('K') | « media/video/video_decode_accelerator.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698