Pepper: Define PPB_VideoEncoder API.

ppapi/c/pp_codecs.h

Index: ppapi/c/pp_codecs.h
diff --git a/ppapi/c/pp_codecs.h b/ppapi/c/pp_codecs.h
index 86d8fb5ea3dc4f04becd9bef041f4cced64d8b9b..15e23fcaf433a55fd5e606b32f45c796df958104 100644
--- a/ppapi/c/pp_codecs.h
+++ b/ppapi/c/pp_codecs.h
@@ -3,11 +3,12 @@
  * found in the LICENSE file.
  */
 
-/* From pp_codecs.idl modified Wed Nov  5 13:38:52 2014. */
+/* From pp_codecs.idl modified Mon Jan 26 16:01:37 2015. */
 
 #ifndef PPAPI_C_PP_CODECS_H_
 #define PPAPI_C_PP_CODECS_H_
 
+#include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_macros.h"
 #include "ppapi/c/pp_point.h"
 #include "ppapi/c/pp_rect.h"
@@ -130,6 +131,52 @@ struct PP_VideoPicture_0_1 {
    */
   struct PP_Size texture_size;
 };
+
+/**
+ * Supported video profile information. See the PPB_VideoEncoder function
+ * GetSupportedProfiles() for more details.
+ */
+struct PP_SupportedVideoProfile {

[dmichael (off chromium), 2015/02/03 23:26:08]

nit: I think whether it's "Supported" is more a property of a value returned by
GetSupportedProfiles than it is the type (Though I'm not sure if we'd use the
type for anything else).

Nonetheless...  a name like PP_VidepProfileDescription might be better?

[llandwerlin-old, 2015/02/04 01:01:52]

Acknowledged.

[bbudge, 2015/02/04 13:56:40]

This tries to follow the VEA api, where the class is VEA::SupportedProfile.
https://code.google.com/p/chromium/codesearch#chromium/src/media/video/video_...

This type may in future be used by VideoDecoder, as there's an outstanding bug
to add similar querying there.
https://code.google.com/p/chromium/issues/detail?id=436213

PP_VideoProfileDescription is a good name too. Done.

+  /**
+   * The codec profile.
+   */
+  PP_VideoProfile profile;
+  /**
+   * Dimensions of the maximum resolution of video frames, in pixels.
+   */
+  struct PP_Size max_resolution;
+  /**
+   * The numerator of the maximum frame rate.
+   */
+  uint32_t max_framerate_numerator;
+  /**
+   * The denominator of the maximum frame rate.
+   */
+  uint32_t max_framerate_denominator;
+  /**
+   * A value indicating if the profile is available in hardware, software, or
+   * both.
+   */
+  PP_HardwareAcceleration acceleration;
+};
+
+/**
+ * Struct describing a bitstream buffer.
+ */
+struct PP_BitstreamBuffer {
+  /**
+   * The size, in bytes, of the bitstream data.
+   */
+  uint32_t size;
+  /**
+   * The base address of the bitstream data.
+   */
+  void* buffer;
+  /**
+   * Whether the buffer represents a key frame.
+   */
+  PP_Bool key_frame;
+};
 /**
  * @}
  */