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

Unified Diff: webkit/plugins/ppapi/ppapi_plugin_instance.h

Issue 11091005: Update PluginInstance for decrypt-and-decode video. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 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
Index: webkit/plugins/ppapi/ppapi_plugin_instance.h
diff --git a/webkit/plugins/ppapi/ppapi_plugin_instance.h b/webkit/plugins/ppapi/ppapi_plugin_instance.h
index 10c55f56ead19f3737c46b947e4a6fca37418ce5..729f6bccfc43063741e05ac1a079237db094b533 100644
--- a/webkit/plugins/ppapi/ppapi_plugin_instance.h
+++ b/webkit/plugins/ppapi/ppapi_plugin_instance.h
@@ -261,10 +261,9 @@ class WEBKIT_PLUGINS_EXPORT PluginInstance :
bool CancelKeyRequest(const std::string& session_id);
bool Decrypt(const scoped_refptr<media::DecoderBuffer>& encrypted_buffer,
const media::Decryptor::DecryptCB& decrypt_cb);
- // TODO(xhwang): Update this when we need to support decrypt and decode.
bool DecryptAndDecodeFrame(
const scoped_refptr<media::DecoderBuffer>& encrypted_frame,
- const media::Decryptor::DecryptCB& decrypt_cb);
+ const media::Decryptor::VideoDecodeCB& video_decode_cb);
// There are 2 implementations of the fullscreen interface
// PPB_FlashFullscreen is used by Pepper Flash.
@@ -756,10 +755,17 @@ class WEBKIT_PLUGINS_EXPORT PluginInstance :
scoped_refptr<PPB_URLLoader_Impl> document_loader_;
media::DecryptorClient* decryptor_client_;
+
+ // Request ID for tracking pending content decryption callbacks.
+ // Note that zero indicates an invalid request ID.
uint32_t next_decryption_request_id_;
+
typedef std::map<uint32_t, media::Decryptor::DecryptCB> DecryptionCBMap;
DecryptionCBMap pending_decryption_cbs_;
+ uint32_t pending_video_decode_request_id_;
+ media::Decryptor::VideoDecodeCB pending_video_decode_cb_;
ddorwin 2012/10/08 19:05:57 We have a list for decrypt-only and one for decode
xhwang 2012/10/08 20:50:51 TODO added.
+
DISALLOW_COPY_AND_ASSIGN(PluginInstance);
};
« no previous file with comments | « no previous file | webkit/plugins/ppapi/ppapi_plugin_instance.cc » ('j') | webkit/plugins/ppapi/ppapi_plugin_instance.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698