Plumb support for audio sample formats.

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 aba404f8bb414414d6630a170631a64dead10eb8..e171a77dd59001bc6aa641ad25c9ddef439eae1b 100644
--- a/ppapi/api/private/pp_content_decryptor.idl
+++ b/ppapi/api/private/pp_content_decryptor.idl
@@ -140,6 +140,20 @@ enum PP_DecryptedFrameFormat {
 };
 
 /**
+ * <code>PP_DecryptedSampleFormat</code> contains audio sample formats.
+ */
+[assert_size(4)]
+enum PP_DecryptedSampleFormat {
+  PP_DECRYPTEDSAMPLEFORMAT_UNKNOWN = 0,
+  PP_DECRYPTEDSAMPLEFORMAT_U8 = 1,
+  PP_DECRYPTEDSAMPLEFORMAT_S16 = 2,
+  PP_DECRYPTEDSAMPLEFORMAT_S32 = 3,
+  PP_DECRYPTEDSAMPLEFORMAT_F32 = 4,
+  PP_DECRYPTEDSAMPLEFORMAT_PLANAR_S16 = 5,
+  PP_DECRYPTEDSAMPLEFORMAT_PLANAR_F32 = 6
+};
+
+/**
  * The <code>PP_DecryptResult</code> enum contains decryption and decoding
  * result constants.
  */
@@ -235,6 +249,34 @@ struct PP_DecryptedFrameInfo {
 };
 
 /**
+ * <code>PP_DecryptedSampleInfo</code> contains the result of the
+ * decrypt and decode operation on the associated samples, information required
+ * to access the sample data in buffer, and tracking info.
+ */
+[assert_size(32)]

[DaleCurtis, 2013/10/11 01:30:53]

Not sure where the extra 4 bytes comes from, it counts out to 28 by my hand
calculation... maybe padding?

[xhwang, 2013/10/11 22:45:16]

see l.126. you may want to have a 

uint32_t padding;

explicitly to avoid surprise.

[dmichael (off chromium), 2013/10/11 23:00:35]

PP_DecryptTrackingInfo has an int64_t, so it gets 8-byte aligned. I suspect
there's 4 bytes of padding between |data_size| and |tracking_info|.

+struct PP_DecryptedSampleInfo {
+  /**
+   * Result of the decrypt and decode operation.
+   */
+  PP_DecryptResult result;
+
+  /**
+   * Format of the decrypted samples.
+   */
+  PP_DecryptedSampleFormat format;
+
+  /**
+   * Size in bytes of decrypted samples.
+   */
+  uint32_t data_size;

[DaleCurtis, 2013/10/11 01:30:53]

Is this necessary? The size is already attached to the PP_Resource.  I just
followed what PP_DecryptedFrameInfo is doing...

[xhwang, 2013/10/11 22:45:16]

You mean PP_DecryptedBufferInfo? See the comment on l.186.

+
+  /**
+   * Information needed by the client to track the decrypted samples.
+   */
+  PP_DecryptTrackingInfo tracking_info;
+};
+
+/**
  * <code>PP_AudioCodec</code> contains audio codec type constants.
  */
 [assert_size(4)]