Added idl file for Pepper 3D.

ppapi/c/ppb_graphics_3d.h

Index: ppapi/c/ppb_graphics_3d.h
===================================================================
--- ppapi/c/ppb_graphics_3d.h	(revision 120596)
+++ ppapi/c/ppb_graphics_3d.h	(working copy)
@@ -2,127 +2,181 @@
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
+
+/* From ppb_graphics_3d.idl modified Tue Feb 07 11:38:46 2012. */
+
 #ifndef PPAPI_C_PPB_GRAPHICS_3D_H_
 #define PPAPI_C_PPB_GRAPHICS_3D_H_
 
-#include "ppapi/c/pp_graphics_3d.h"
-
 #include "ppapi/c/pp_bool.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_stdint.h"
 
-/* Example usage from plugin code:
+#define PPB_GRAPHICS_3D_INTERFACE_1_0 "PPB_Graphics3D;1.0"
+#define PPB_GRAPHICS_3D_INTERFACE PPB_GRAPHICS_3D_INTERFACE_1_0
+
+/**
+ * @file
+ * Defines the <code>PPB_Graphics3D</code> struct representing a 3D graphics
+ * context within the browser.
+ */
+
+
+/* Add 3D graphics enums */
+#include "ppapi/c/pp_graphics_3d.h"
+
+/**
+ * @addtogroup Interfaces
+ * @{
+ */
+/**
+ * <code>PPB_Graphics3D</code> defines the interface for a 3D graphics context.
+ * <strong>Example usage from plugin code:</strong>
  *
- * // Setup.
+ * <strong>Setup:</strong>
+ * <code>
  * PP_Resource context;
  * int32_t attribs[] = {PP_GRAPHICS3DATTRIB_WIDTH, 800,
  *                      PP_GRAPHICS3DATTRIB_HEIGHT, 800,
  *                      PP_GRAPHICS3DATTRIB_NONE};
  * context = g3d->Create(instance, attribs, &context);
  * inst->BindGraphics(instance, context);
+ * </code>
  *
- * // Present one frame.
+ * <strong>Present one frame:</strong>
+ * <code>
  * gles2->Clear(context, GL_COLOR_BUFFER);
  * g3d->SwapBuffers(context);
+ * </code>
  *
- * // Shutdown.
+ * <strong>Shutdown:</strong>
+ * <code>
  * core->ReleaseResource(context);
+ * </code>
  */
-
-#define PPB_GRAPHICS_3D_INTERFACE_1_0 "PPB_Graphics3D;1.0"
-#define PPB_GRAPHICS_3D_INTERFACE PPB_GRAPHICS_3D_INTERFACE_1_0
-
 struct PPB_Graphics3D_1_0 {
   /**
-   * Retrieves the maximum supported value for the given attribute.
+   * GetAttribMaxValue() retrieves the maximum supported value for the
+   * given attribute. This function may be used to check if a particular
+   * attribute value is supported before attempting to create a context.
    *
-   * This function may be used to check if a particular attribute value is
-   * supported before attempting to create a context.
+   * @param[in] instance The module instance.
+   * @param[in] attribute The attribute for which maximum value is queried.
    * Attributes that can be queried for include:
-   * - PP_GRAPHICS3DATTRIB_ALPHA_SIZE
-   * - PP_GRAPHICS3DATTRIB_BLUE_SIZE
-   * - PP_GRAPHICS3DATTRIB_GREEN_SIZE
-   * - PP_GRAPHICS3DATTRIB_RED_SIZE
-   * - PP_GRAPHICS3DATTRIB_DEPTH_SIZE
-   * - PP_GRAPHICS3DATTRIB_STENCIL_SIZE
-   * - PP_GRAPHICS3DATTRIB_SAMPLES
-   * - PP_GRAPHICS3DATTRIB_SAMPLE_BUFFERS
-   * - PP_GRAPHICS3DATTRIB_WIDTH
-   * - PP_GRAPHICS3DATTRIB_HEIGHT
+   * - <code>PP_GRAPHICS3DATTRIB_ALPHA_SIZE</code>
+   * - <code>PP_GRAPHICS3DATTRIB_BLUE_SIZE</code>
+   * - <code>PP_GRAPHICS3DATTRIB_GREEN_SIZE</code>
+   * - <code>PP_GRAPHICS3DATTRIB_RED_SIZE</code>
+   * - <code>PP_GRAPHICS3DATTRIB_DEPTH_SIZE</code>
+   * - <code>PP_GRAPHICS3DATTRIB_STENCIL_SIZE</code>
+   * - <code>PP_GRAPHICS3DATTRIB_SAMPLES</code>
+   * - <code>PP_GRAPHICS3DATTRIB_SAMPLE_BUFFERS</code>
+   * - <code>PP_GRAPHICS3DATTRIB_WIDTH</code>
+   * - <code>PP_GRAPHICS3DATTRIB_HEIGHT</code>
+   * @param[out] value The maximum supported value for <code>attribute</code>
    *
-   * On failure the following error codes may be returned:
-   * - PP_ERROR_BADRESOURCE if instance is invalid.
-   * - PP_ERROR_BADARGUMENT if attribute is invalid or value is NULL
+   * @return Returns <code>PP_TRUE</code> on success or the following on error:
+   * - <code>PP_ERROR_BADRESOURCE</code> if <code>instance</code> is invalid
+   * - <code>PP_ERROR_BADARGUMENT</code> if <code>attribute</code> is invalid
+   *   or <code>value</code> is 0
    */
   int32_t (*GetAttribMaxValue)(PP_Resource instance,
-                               int32_t attribute, int32_t* value);
-
+                               int32_t attribute,
+                               int32_t* value);
   /**
-   * Creates and initializes a rendering context and returns a handle to it.
+   * Create() creates and initializes a 3D rendering context.
    * The returned context is off-screen to start with. It must be attached to
-   * a plugin instance using PPB_Instance::BindGraphics to draw on the web page.
+   * a plugin instance using <code>PPB_Instance::BindGraphics</code> to draw
+   * on the web page.
    *
-   * If share_context is not NULL, then all shareable data, as defined
-   * by the client API (note that for OpenGL and OpenGL ES, shareable data
-   * excludes texture objects named 0) will be shared by share_context, all
-   * other contexts share_context already shares with, and the newly created
-   * context. An arbitrary number of PPB_Graphics3D can share data in
-   * this fashion.
+   * @param[in] instance The module instance.
    *
-   * attrib_list specifies a list of attributes for the context. It is a list
-   * of attribute name-value pairs in which each attribute is immediately
-   * followed by the corresponding desired value. The list is terminated with
-   * PP_GRAPHICS3DATTRIB_NONE. The attrib_list may be NULL or empty
-   * (first attribute is PP_GRAPHICS3DATTRIB_NONE). If an attribute is not
-   * specified in attrib_list, then the default value is used (it is said to
-   * be specified implicitly).
+   * @param[in] share_context The 3D context with which the created context
+   * would share resources. If <code>share_context</code> is not 0, then all
+   * shareable data, as defined by the client API (note that for OpenGL and
+   * OpenGL ES, shareable data excludes texture objects named 0) will be shared
+   * by <code>share_context<code>, all other contexts <code>share_context</code>
+   * already shares with, and the newly created context. An arbitrary number of
+   * <code>PPB_Graphics3D</code> can share data in this fashion.
    *
+   * @param[out] attrib_list specifies a list of attributes for the context.
+   * It is a list of attribute name-value pairs in which each attribute is
+   * immediately followed by the corresponding desired value. The list is
+   * terminated with <code>PP_GRAPHICS3DATTRIB_NONE</code>.
+   * The <code>attrib_list<code> may be 0 or empty (first attribute is
+   * <code>PP_GRAPHICS3DATTRIB_NONE</code>). If an attribute is not
+   * specified in <code>attrib_list</code>, then the default value is used
+   * (it is said to be specified implicitly).
    * Attributes for the context are chosen according to an attribute-specific
    * criteria. Attributes can be classified into two categories:
    * - AtLeast: The attribute value in the returned context meets or exceeds
-   *            the value specified in attrib_list.
+   *            the value specified in <code>attrib_list</code>.
    * - Exact: The attribute value in the returned context is equal to
-   *          the value specified in attrib_list.
+   *          the value specified in <code>attrib_list</code>.
    *
-   * Attributes that can be specified in attrib_list include:
-   * - PP_GRAPHICS3DATTRIB_ALPHA_SIZE: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_BLUE_SIZE: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_GREEN_SIZE: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_RED_SIZE: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_DEPTH_SIZE: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_STENCIL_SIZE: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_SAMPLES: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_SAMPLE_BUFFERS: Category: AtLeast Default: 0.
-   * - PP_GRAPHICS3DATTRIB_WIDTH: Category: Exact Default: 0.
-   * - PP_GRAPHICS3DATTRIB_HEIGHT: Category: Exact Default: 0.
-   * - PP_GRAPHICS3DATTRIB_SWAP_BEHAVIOR:
+   * Attributes that can be specified in <code>attrib_list</code> include:
+   * - <code>PP_GRAPHICS3DATTRIB_ALPHA_SIZE</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_BLUE_SIZE</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_GREEN_SIZE</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_RED_SIZE</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_DEPTH_SIZE</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_STENCIL_SIZE</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_SAMPLES</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_SAMPLE_BUFFERS</code>:
+   *   Category: AtLeast Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_WIDTH</code>:
+   *   Category: Exact Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_HEIGHT</code>:
+   *   Category: Exact Default: 0.
+   * - <code>PP_GRAPHICS3DATTRIB_SWAP_BEHAVIOR</code>:
    *   Category: Exact Default: Implementation defined.
    *
-   * On failure NULL resource is returned.
+   * @return A <code>PP_Resource</code> containing the 3D graphics context if
+   * successful or 0 if unsuccessful.
    */
   PP_Resource (*Create)(PP_Instance instance,
                         PP_Resource share_context,
-                        const int32_t* attrib_list);
-
+                        const int32_t attrib_list[]);
   /**
-   * Returns PP_TRUE if the given resource is a valid PPB_Graphics3D,
-   * PP_FALSE if it is an invalid resource or is a resource of another type.
+   * IsGraphics3D() determines if the given resource is a valid
+   * <code>Graphics3D</code> context.
+   *
+   * @param[in] resource A <code>Graphics3D</code> context resource.
+   *
+   * @return PP_TRUE if the given resource is a valid <code>Graphics3D</code>,
+   * <code>PP_FALSE</code> if it is an invalid resource or is a resource of
+   * another type.
    */
   PP_Bool (*IsGraphics3D)(PP_Resource resource);
-
   /**
-   * Retrieves the value for each attribute in attrib_list. The list
-   * has the same structure as described for PPB_Graphics3D::Create.
-   * It is both input and output structure for this function.
+   * GetAttribs() retrieves the value for each attribute in
+   * <code>attrib_list</code>.
    *
-   * All attributes specified in PPB_Graphics3D.Create can be queried for.
-   * On failure the following error codes may be returned:
-   * - PP_ERROR_BADRESOURCE if context is invalid.
-   * - PP_ERROR_BADARGUMENT if attrib_list is NULL or any attribute in the
-   *   attrib_list is not a valid attribute.
+   * @param[in] context The 3D graphics context.
+   * @param[in,out] attrib_list The list of attributes that are queried.
+   * <code>attrib_list</code> has the same structure as described for
+   * <code>PPB_Graphics3D::Create</code>. It is both input and output
+   * structure for this function. All attributes specified in
+   * <code>PPB_Graphics3D::Create</code> can be queried for.
    *
-   * Example usage: To get the values for rgb bits in the color buffer,
-   * this function must be called as following:
+   * @return Returns <code>PP_OK</code> on success or:
+   * - <code>PP_ERROR_BADRESOURCE</code> if context is invalid
+   * - <code>PP_ERROR_BADARGUMENT</code> if attrib_list is 0 or any attribute
+   *   in the <code>attrib_list</code> is not a valid attribute.
+   *
+   * <strong>Example usage:</strong> To get the values for rgb bits in the
+   * color buffer, this function must be called as following:
+   * <code>
    * int attrib_list[] = {PP_GRAPHICS3DATTRIB_RED_SIZE, 0,
    *                      PP_GRAPHICS3DATTRIB_GREEN_SIZE, 0,
    *                      PP_GRAPHICS3DATTRIB_BLUE_SIZE, 0,
@@ -131,24 +185,29 @@
    * int red_bits = attrib_list[1];
    * int green_bits = attrib_list[3];
    * int blue_bits = attrib_list[5];
+   * </code>
    */
-  int32_t (*GetAttribs)(PP_Resource context, int32_t* attrib_list);
-
+  int32_t (*GetAttribs)(PP_Resource context, int32_t attrib_list[]);
   /**
-   * Sets the values for each attribute in attrib_list. The list
-   * has the same structure as described for PPB_Graphics3D.Create.
+   * SetAttribs() sets the values for each attribute in
+   * <code>attrib_list</code>.
    *
+   * @param[in] context The 3D graphics context.
+   * @param[in] attrib_list The list of attributes whose values need to be set.
+   * <code>attrib_list</code> has the same structure as described for
+   * <code>PPB_Graphics3D::Create</code>.
    * Attributes that can be specified are:
-   * - PP_GRAPHICS3DATTRIB_SWAP_BEHAVIOR
+   * - <code>PP_GRAPHICS3DATTRIB_SWAP_BEHAVIOR</code>
    *
-   * On failure the following error codes may be returned:
-   * - PP_ERROR_BADRESOURCE if context is invalid.
-   * - PP_ERROR_BADARGUMENT if attrib_list is NULL or any attribute in the
-   *   attrib_list is not a valid attribute.
+   * @return Returns <code>PP_OK</code> on success or:
+   * - <code>PP_ERROR_BADRESOURCE</code> if <code>context</code> is invalid.
+   * - <code>PP_ERROR_BADARGUMENT</code> if <code>attrib_list</code> is 0 or
+   *   any attribute in the <code>attrib_list</code> is not a valid attribute.
    */
-  int32_t (*SetAttribs)(PP_Resource context, int32_t* attrib_list);
-
+  int32_t (*SetAttribs)(PP_Resource context, const int32_t attrib_list[]);

[alokp, 2012/02/09 17:56:43]

We are changing the function signature in this case. Although the change does
not affect binary compatibility, does it still warrant a rev change?

[brettw, 2012/02/09 18:15:29]

I'd say don't bother revving the interface if binary compat is the same. It will
be a big pain to update everything.

   /**
+   * GetError() returns the current state of the given 3D context.
+   *
    * The recoverable error conditions that have no side effect are
    * detected and returned immediately by all functions in this interface.
    * In addition the implementation may get into a fatal state while
@@ -162,39 +221,43 @@
    * drawing a frame is expensive and you want to verify the result of
    * ResizeBuffers before attemptimg to draw a frame.
    *
-   * The following error codes may be returned:
-   * - PP_ERROR_NOMEMORY
-   * - PP_ERROR_CONTEXT_LOST
+   * @param[in] The 3D graphics context.
+   * @return Returns:
+   * - <code>PP_OK</code> if no error
+   * - <code>PP_ERROR_NOMEMORY</code>
+   * - <code>PP_ERROR_CONTEXT_LOST</code>
    */
   int32_t (*GetError)(PP_Resource context);
-
   /**
-   * Resizes the backing surface for context.
+   * ResizeBuffers() resizes the backing surface for context.
    *
-   * On failure the following error codes may be returned:
-   * - PP_ERROR_BADRESOURCE if context is invalid.
-   * - PP_ERROR_BADARGUMENT if the value specified for width or height
-   *   is less than zero.
+   * If the surface could not be resized due to insufficient resources,
+   * <code>PP_ERROR_NOMEMORY</code> error is returned on the next
+   * <code>SwapBuffers</code> callback.
    *
-   * If the surface could not be resized due to insufficient resources,
-   * PP_ERROR_NOMEMORY error is returned on the next SwapBuffers callback.
+   * @param[in] context The 3D graphics context.
+   * @param[in] width The width of the backing surface.
+   * @param[in] height The height of the backing surface.
+   * @return Returns <code>PP_OK</code> on success or:
+   * - <code>PP_ERROR_BADRESOURCE</code> if context is invalid.
+   * - <code>PP_ERROR_BADARGUMENT</code> if the value specified for
+   *   <code>width</code> or <code>height</code> is less than zero.
    */
-  int32_t (*ResizeBuffers)(PP_Resource context,
-                           int32_t width, int32_t height);
-
+  int32_t (*ResizeBuffers)(PP_Resource context, int32_t width, int32_t height);
   /**
-   * Makes the contents of the color buffer available for compositing.
-   * This function has no effect on off-screen surfaces - ones not bound
-   * to any plugin instance. The contents of ancillary buffers are always
-   * undefined after calling SwapBuffers. The contents of the color buffer are
-   * undefined if the value of the PP_GRAPHICS3DATTRIB_SWAP_BEHAVIOR attribute
-   * of context is not PP_GRAPHICS3DATTRIB_BUFFER_PRESERVED.
+   * SwapBuffers() makes the contents of the color buffer available for
+   * compositing. This function has no effect on off-screen surfaces - ones not
+   * bound to any plugin instance. The contents of ancillary buffers are always
+   * undefined after calling <code>SwapBuffers</code>. The contents of the color
+   * buffer are undefined if the value of the
+   * <code>PP_GRAPHICS3DATTRIB_SWAP_BEHAVIOR</code> attribute of context is not
+   * <code>PP_GRAPHICS3DATTRIB_BUFFER_PRESERVED</code>.
    *
-   * SwapBuffers runs in asynchronous mode. Specify a callback function and the
-   * argument for that callback function. The callback function will be executed
-   * on the calling thread after the color buffer has been composited with
-   * rest of the html page. While you are waiting for a SwapBuffers callback,
-   * additional calls to SwapBuffers will fail.
+   * <code>SwapBuffers</code> runs in asynchronous mode. Specify a callback
+   * function and the argument for that callback function. The callback function
+   * will be executed on the calling thread after the color buffer has been
+   * composited with rest of the html page. While you are waiting for a
+   * SwapBuffers callback, additional calls to SwapBuffers will fail.
    *
    * Because the callback is executed (or thread unblocked) only when the
    * plugin's current state is actually on the screen, this function provides a
@@ -206,18 +269,27 @@
    * If the context gets into an unrecoverable error condition while
    * processing a command, the error code will be returned as the argument
    * for the callback. The callback may return the following error codes:
-   * - PP_ERROR_NOMEMORY
-   * - PP_ERROR_CONTEXT_LOST
+   * - <code>PP_ERROR_NOMEMORY</code>
+   * - <code>PP_ERROR_CONTEXT_LOST</code>
    * Note that the same error code may also be obtained by calling GetError.
    *
-   * On failure SwapBuffers may return the following error codes:
-   * - PP_ERROR_BADRESOURCE if context is invalid.
-   * - PP_ERROR_BADARGUMENT if callback is invalid.
+   * @param[in] context The 3D graphics context.
+   * @param[in] callback The callback that will executed when
+   * <code>SwapBuffers</code> completes.
+   *
+   * @return Returns PP_OK on success or:
+   * - <code>PP_ERROR_BADRESOURCE</code> if context is invalid.
+   * - <code>PP_ERROR_BADARGUMENT</code> if callback is invalid.
+   *
    */
   int32_t (*SwapBuffers)(PP_Resource context,
                          struct PP_CompletionCallback callback);
 };
 
 typedef struct PPB_Graphics3D_1_0 PPB_Graphics3D;
+/**
+ * @}
+ */
 
 #endif  /* PPAPI_C_PPB_GRAPHICS_3D_H_ */
+