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

Unified Diff: ppapi/api/private/ppb_content_decryptor_private.idl

Issue 11013052: Add PPAPI CDM video decoder initialization. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Decoder init, first pass. 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: ppapi/api/private/ppb_content_decryptor_private.idl
diff --git a/ppapi/api/private/ppb_content_decryptor_private.idl b/ppapi/api/private/ppb_content_decryptor_private.idl
index d73cae1b95373f1c7d5f0ddf121ba6ae6050104f..392e373143accfed6aa87b3017c33988c1ef5a09 100644
--- a/ppapi/api/private/ppb_content_decryptor_private.idl
+++ b/ppapi/api/private/ppb_content_decryptor_private.idl
@@ -9,7 +9,7 @@
* Decryption Modules, not normal plugins.
*/
label Chrome {
- M23 = 0.2
+ M23 = 0.3
Ami GONE FROM CHROMIUM 2012/10/08 18:35:16 is any of this really being merged back to 23? d
Tom Finegan 2012/10/08 23:23:27 I don't think so. Changed to M24.
Ami GONE FROM CHROMIUM 2012/10/09 07:21:12 Policy is "do what brettw says" ;)
};
/**
@@ -146,6 +146,26 @@ interface PPB_ContentDecryptor_Private {
[in] PP_DecryptedBlockInfo decrypted_block_info);
/**
+ * Called after the <code>InitializeAudioDecoder()</code> or
+ * <code>InitializeVideoDecoder()</code> method on the
+ * <code>PPP_ContentDecryptor_Private</code> interface completes to report
+ * decoder intialization status to the browser.
+ *
+ * @param[in] success A <code>PP_Bool</code> that is set to
+ * <code>PP_TRUE</code> when the decoder initialization request associated
+ * with <code>request_id</code> was successful.
+ *
+ * @param[in] request_id The <code>request_id</code> value passed to
+ * <code>InitializeAudioDecoder</code> or <code>InitializeVideoDecoder</code>
+ * in <code>PP_AudioDecoderConfig</code> or
+ * <code>PP_VideoDecoderConfig</code>.
+ */
Ami GONE FROM CHROMIUM 2012/10/08 18:35:16 Doesn't versioning require marking the method as b
Tom Finegan 2012/10/08 23:23:27 Agreed, but media is the only client using this in
Ami GONE FROM CHROMIUM 2012/10/09 07:21:12 I don't understand your statement, but I'll let yo
+ void DecoderInitializeStatus(
xhwang 2012/10/08 17:16:13 This name is a little bit strange. How about Decod
Tom Finegan 2012/10/08 23:23:27 Done.
+ [in] PP_Instance instance,
+ [in] PP_Bool success,
+ [in] uint32_t request_id);
+
+ /**
* Called after the <code>DecryptAndDecode()</code> method on the
* <code>PPP_ContentDecryptor_Private</code> interface completes to deliver
* a decrypted and decoded video frame to the browser for rendering.

Powered by Google App Engine
This is Rietveld 408576698