Update PPP side of Pepper CDM API to support video decoding.

ppapi/api/private/pp_content_decryptor.idl

Index: ppapi/api/private/pp_content_decryptor.idl
diff --git a/ppapi/api/private/pp_content_decryptor.idl b/ppapi/api/private/pp_content_decryptor.idl
index 14d9c1c596ff3fb793fef88a6563ae8671ad7730..287734aacf36120bf5697175ad722e2d0405f99d 100644
--- a/ppapi/api/private/pp_content_decryptor.idl
+++ b/ppapi/api/private/pp_content_decryptor.idl
@@ -114,6 +114,58 @@ struct PP_EncryptedBlockInfo {
 };
 
 /**
+ * <code>PP_DecryptedFrameFormat</code> contains video frame formats.
+ */
+[assert_size(4)]
+enum PP_DecryptedFrameFormat {
+  PP_DECRYPTEDFRAMEFORMAT_UNKNOWN = 0,
+  PP_DECRYPTEDFRAMEFORMAT_EMPTY = 1,
+  PP_DECRYPTEDFRAMEFORMAT_YV12 = 2,
+  PP_DECRYPTEDFRAMEFORMAT_I420 = 3
+};
+
+/**
+ * <code>PP_VideoCodec</code> contains video codec type constants.
+ */
+[assert_size(4)]
+enum PP_VideoCodec {
+  PP_VIDEOCODEC_UNKNOWN = 0,
+  PP_VIDEOCODEC_VP8 = 1
+};
+
+/**
+ * <code>PP_EncryptedVideoFrameInfo</code> contains the information required
+ * to decrypt and decode a video frame.
+ */
+[assert_size(256)]
+struct PP_EncryptedVideoFrameInfo {

[xhwang, 2012/10/01 06:19:52]

Since the decoder has been initialized with something like
InitializeVideoDecoder(config), shouldn't the decode already know the format,
codec and sizes? Do we need to pass these info in every encrypted block/frame?
Can we just use PP_EncryptedBlockInfo for the encrypted video frame?

[Tom Finegan, 2012/10/02 02:42:26]

For playing back adaptive content, won't we need the format information to
determine if the decoder needs to be reinitialized? 

[xhwang, 2012/10/03 22:58:21]

Re this comment and the general comment.

It's an option to hide initialization in DecryptAndDecodeFrame() calls. But in
that case we need all information needed to initialize a video codec in this
structure. I think we are still missing some, e.g. the extra data. From media
pipeline's perspective, we always know when the config changed, so that is not a
problem.

Since this CL is getting large, I agree we can land this CL now with a TODO that
we'll revisit decoder initialization later.

+  /**
+   * The decoded video frame format.
+   */
+  PP_DecryptedFrameFormat format;
+
+  /**
+   * The video frame codec type.
+   */
+  PP_VideoCodec codec;
+
+  /**
+   * Video frame width in pixels.
+   */
+  int32_t width;
+
+  /**
+   * Video frame height in pixels.
+   */
+  int32_t height;
+
+  /** 
+   * Information required to decrypt the frame.
+   */
+  PP_EncryptedBlockInfo encrypted_frame_info;

[ddorwin, 2012/10/01 00:48:10]

This member name is very close to the class name. Should it be encryption_info
or something?

[Tom Finegan, 2012/10/02 02:42:26]

Done.

+};
+
+/**
  * The <code>PP_DecryptResult</code> enum contains decryption and decoding
  * result constants.
  */