Pepper: PPB_VideoEncoder should be 'Dev' only for now.

ppapi/c/ppb_video_encoder.h

 /* Copyright (c) 2015 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
-/* From ppb_video_encoder.idl modified Wed Feb  4 05:24:29 2015. */
+/* From ppb_video_encoder.idl modified Thu Feb  5 04:48:42 2015. */
 
 #ifndef PPAPI_C_PPB_VIDEO_ENCODER_H_
 #define PPAPI_C_PPB_VIDEO_ENCODER_H_
 
 #include "ppapi/c/pp_array_output.h"
 #include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_codecs.h"
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/pp_macros.h"
 #include "ppapi/c/pp_resource.h"
 #include "ppapi/c/pp_size.h"
 #include "ppapi/c/pp_stdint.h"
 #include "ppapi/c/ppb_video_frame.h"
 
-#define PPB_VIDEOENCODER_INTERFACE_0_1 "PPB_VideoEncoder;0.1"
-#define PPB_VIDEOENCODER_INTERFACE PPB_VIDEOENCODER_INTERFACE_0_1

[llandwerlin-old, 2015/02/05 13:05:46]

We might want to keep this define.

[dmichael (off chromium), 2015/02/05 20:58:58]

This is omitted on purpose. It's one of the signals to a plugin author that
they're not using a stable API, and we may break or change it at any time.

-
+#define PPB_VIDEOENCODER_INTERFACE_0_1 "PPB_VideoEncoder;0.1" /* dev */
 /**
  * @file
  * This file defines the <code>PPB_VideoEncoder</code> interface.
  */
 
 
 /**
  * @addtogroup Interfaces
  * @{
  */
@@ skipping 14 matching lines @@
  * - Call RecycleBitstreamBuffer() after consuming the data in the bitstream
  *   buffer.
  * - To destroy the encoder, the plugin should release all of its references to
  *   it. Any pending callbacks will abort before the encoder is destroyed.
  *
  * Available video codecs vary by platform.
  * All: theora, vorbis, vp8.
  * Chrome and ChromeOS: h264.
  * ChromeOS: mpeg4.
  */
-struct PPB_VideoEncoder_0_1 {
+struct PPB_VideoEncoder_0_1 { /* dev */
   /**
    * Creates a new video encoder resource.
    *
    * @param[in] instance A <code>PP_Instance</code> identifying the instance
    * with the video encoder.
    *
    * @return A <code>PP_Resource</code> corresponding to a video encoder if
    * successful or 0 otherwise.
    */
   PP_Resource (*Create)(PP_Instance instance);
@@ skipping 160 matching lines @@
    * callbacks will still run, reporting <code>PP_ERROR_ABORTED</code> . It is
    * not valid to call any encoder functions after a call to this method.
    * <strong>Note:</strong> Destroying the video encoder closes it implicitly,
    * so you are not required to call Close().
    *
    * @param[in] video_encoder A <code>PP_Resource</code> identifying the video
    * encoder.
    */
   void (*Close)(PP_Resource video_encoder);
 };
-
-typedef struct PPB_VideoEncoder_0_1 PPB_VideoEncoder;

[llandwerlin-old, 2015/02/05 13:05:46]

I think we want to keep this typedef too.
(The generator script is annoying :))

[dmichael (off chromium), 2015/02/05 20:58:58]

ditto, this is intentional

 /**
  * @}
  */
 
 #endif  /* PPAPI_C_PPB_VIDEO_ENCODER_H_ */