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

Unified Diff: content/renderer/media/renderer_gpu_video_accelerator_factories.h

Issue 1306693002: media: Convert I420 VideoFrame to UYVY GpuMemoryBuffer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: s/CHECK_EQ/DCHECK_EQ. Created 5 years, 4 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 | « no previous file | content/renderer/media/renderer_gpu_video_accelerator_factories.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/renderer_gpu_video_accelerator_factories.h
diff --git a/content/renderer/media/renderer_gpu_video_accelerator_factories.h b/content/renderer/media/renderer_gpu_video_accelerator_factories.h
index 981fc656a46ae0998c77e9ed9dd390f36741df85..b1dfc5c5c3f1928fddcee1d057fe2818f9e9b8e2 100644
--- a/content/renderer/media/renderer_gpu_video_accelerator_factories.h
+++ b/content/renderer/media/renderer_gpu_video_accelerator_factories.h
@@ -49,6 +49,7 @@ class CONTENT_EXPORT RendererGpuVideoAcceleratorFactories
const scoped_refptr<ContextProviderCommandBuffer>& context_provider,
bool enable_gpu_memory_buffer_video_frames,
unsigned image_texture_target,
+ media::VideoPixelFormat video_frame_output_format,
bool enable_video_accelerator);
bool IsGpuVideoAcceleratorEnabled() override;
@@ -75,6 +76,7 @@ class CONTENT_EXPORT RendererGpuVideoAcceleratorFactories
bool IsTextureRGSupported() override;
bool ShouldUseGpuMemoryBuffersForVideoFrames() const override;
unsigned ImageTextureTarget() override;
+ media::VideoPixelFormat VideoFrameOutputFormat() override;
gpu::gles2::GLES2Interface* GetGLES2Interface() override;
scoped_ptr<base::SharedMemory> CreateSharedMemory(size_t size) override;
scoped_refptr<base::SingleThreadTaskRunner> GetTaskRunner() override;
@@ -92,6 +94,7 @@ class CONTENT_EXPORT RendererGpuVideoAcceleratorFactories
const scoped_refptr<ContextProviderCommandBuffer>& context_provider,
bool enable_gpu_memory_buffer_video_frames,
unsigned image_texture_target,
+ media::VideoPixelFormat video_frame_output_format_,
bool enable_video_accelerator);
~RendererGpuVideoAcceleratorFactories() override;
@@ -109,9 +112,12 @@ class CONTENT_EXPORT RendererGpuVideoAcceleratorFactories
scoped_refptr<GpuChannelHost> gpu_channel_host_;
scoped_refptr<ContextProviderCommandBuffer> context_provider_;
- // Wheter gpu memory buffers should be used to hold video frames data.
+ // Whether gpu memory buffers should be used to hold video frames data.
bool enable_gpu_memory_buffer_video_frames_;
const unsigned image_texture_target_;
+ // Pixel format of the hardware video frames created when GpuMemoryBuffers
+ // video frames are enabled.
+ const media::VideoPixelFormat video_frame_output_format_;
// Whether video acceleration encoding/decoding should be enabled.
const bool video_accelerator_enabled_;
« no previous file with comments | « no previous file | content/renderer/media/renderer_gpu_video_accelerator_factories.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698