Pepper: PPB_VideoDecoder software-only mode.

ppapi/api/ppb_video_decoder.idl

Index: ppapi/api/ppb_video_decoder.idl
diff --git a/ppapi/api/ppb_video_decoder.idl b/ppapi/api/ppb_video_decoder.idl
index b450633b818267219892a10743d54625417f3b3b..da890f2b3e4b64680784dbd8c28b0cac73410845 100644
--- a/ppapi/api/ppb_video_decoder.idl
+++ b/ppapi/api/ppb_video_decoder.idl
@@ -10,7 +10,8 @@
 [generate_thunk]
 
 label Chrome {
-  [channel=dev] M36 = 0.1
+  [channel=dev] M36 = 0.1,
+  [channel=dev] M39 = 0.2
 };
 
 /**
@@ -90,6 +91,35 @@ interface PPB_VideoDecoder {
       [in] PP_CompletionCallback callback);
 
   /**
+   * Initializes a video decoder resource. This should be called after Create()
+   * and before any other functions.
+   *
+   * @param[in] video_decoder A <code>PP_Resource</code> identifying the video
+   * decoder.
+   * @param[in] graphics3d_context A <code>PPB_Graphics3D</code> resource to use
+   * during decoding.
+   * @param[in] profile A <code>PP_VideoProfile</code> specifying the video
+   * codec profile.
+   * @param[in] hardware_acceleration A <code>PP_HardwareAcceleration</code>

[teravest, 2014/08/22 18:32:37]

nit: The parameter name is acceleration, not hardware_acceleration.

[bbudge, 2014/08/22 21:24:52]

Done.

+   * specifying whether to use a hardware accelerated or a software resource
+   * implementation.
+   * @param[in] callback A <code>PP_CompletionCallback</code> to be called upon
+   * completion.
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   * Returns PP_ERROR_NOTSUPPORTED if video decoding is not available, or the
+   * requested profile is not supported. In this case, the client may call
+   * Initialize() again with different parameters to find a good configuration.

[teravest, 2014/08/22 18:32:37]

Is it safe to call Initialize() again if the call succeeds?

[bbudge, 2014/08/22 21:24:52]

This only works after failing with PP_ERROR_NOTSUPPORTED. Calling Initialize
again after a successful call will always return PP_ERROR_FAILED.

+   */
+  [version = 0.2]
+  int32_t Initialize(
+      [in] PP_Resource video_decoder,
+      [in] PP_Resource graphics3d_context,
+      [in] PP_VideoProfile profile,
+      [in] PP_HardwareAcceleration acceleration,
+      [in] PP_CompletionCallback callback);
+
+  /**
    * Decodes a bitstream buffer. Copies |size| bytes of data from the plugin's
    * |buffer|. The plugin should wait until the decoder signals completion by
    * returning PP_OK or by running |callback| before calling Decode() again.