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

Unified Diff: chrome/renderer/media/ipc_video_decoder.h

Issue 3335014: Added FakeGlVideoDecodeEngine to exercise the IPC protocol for hardware video decoding (Closed)
Patch Set: compile man... Created 10 years, 3 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
Index: chrome/renderer/media/ipc_video_decoder.h
diff --git a/chrome/renderer/media/ipc_video_decoder.h b/chrome/renderer/media/ipc_video_decoder.h
index 4754ef5f430950d7da1756fc307d3cb737d28267..0309ccde767dbff21fbd901e5c4c026037ce3af2 100644
--- a/chrome/renderer/media/ipc_video_decoder.h
+++ b/chrome/renderer/media/ipc_video_decoder.h
@@ -48,7 +48,6 @@ class IpcVideoDecoder : public media::VideoDecoder,
virtual void OnEmptyBufferDone(scoped_refptr<media::Buffer> buffer);
virtual void OnFillBufferDone(scoped_refptr<media::VideoFrame> frame);
virtual void OnDeviceError();
-
virtual bool ProvidesBuffer();
private:
@@ -93,8 +92,15 @@ class IpcVideoDecoder : public media::VideoDecoder,
// and destruction of the context.
ggl::Context* ggl_context_;
+ // Handle to the hardware video decoder. This object will use IPC to
+ // communicate with the decoder in the GPU process.
scoped_refptr<GpuVideoDecoderHost> gpu_video_decoder_host_;
+ // Texture that contains the video frame.
+ // TODO(hclam): Instead of one texture, we should have a set of textures
+ // as requested by the hardware video decode engine in the GPU process.
+ unsigned int texture_;
+
DISALLOW_COPY_AND_ASSIGN(IpcVideoDecoder);
};

Powered by Google App Engine
This is Rietveld 408576698