Make Graphics3D::SwapBuffers take a completion callback

ppapi/c/dev/ppb_graphics_3d_dev.h

 // Copyright (c) 2010 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.
 
 #ifndef PPAPI_C_DEV_PPB_GRAPHICS_3D_DEV_H_
 #define PPAPI_C_DEV_PPB_GRAPHICS_3D_DEV_H_
 
 #include "ppapi/c/pp_bool.h"
+#include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/pp_module.h"
 #include "ppapi/c/pp_resource.h"
 #include "ppapi/c/pp_stdint.h"
 
 // Example usage from plugin code:
 //
 // PP_Resource context = device->Create(module, config, contextAttribList);
 // CHECK(context);
 //
 // // Present one frame.
 // CHECK(device->MakeCurrent(context));
 // glClear(GL_COLOR_BUFFER);
 // CHECK(device->MakeCurrent(NULL));
-// CHECK(device->SwapBuffers(context));
+// CHECK(device->SwapBuffers(context, callback));
 //
 // // Shutdown.
 // core->ReleaseResource(context);
 
-#define PPB_GRAPHICS_3D_DEV_INTERFACE "PPB_Graphics3D(Dev);0.3"
+#define PPB_GRAPHICS_3D_DEV_INTERFACE "PPB_Graphics3D(Dev);0.4"
 
 // These are the same error codes as used by EGL.
 enum {
   PP_GRAPHICS_3D_ERROR_SUCCESS         = 0x3000,
   PP_GRAPHICS_3D_ERROR_NOT_INITIALIZED = 0x3001,
   PP_GRAOHICS_3D_ERROR_BAD_CONTEXT     = 0x3006,
   PP_GRAPHICS_3D_ERROR_BAD_PARAMETER   = 0x300C,
   PP_GRAPHICS_3D_ERROR_CONTEXT_LOST    = 0x300E
 };
 
@@ skipping 43 matching lines @@
   // on success, PP_FALSE on failure.
   PP_Bool (*MakeCurent)(PP_Resource context);
 
   // Returns the calling thread's current context or NULL if no context is
   // current.
   PP_Resource (*GetCurrentContext)();
 
   // Snapshots the rendered frame and makes it available for composition with
   // the rest of the page. The alpha channel is used for translucency effects.
   // One means fully opaque. Zero means fully transparent. Any thread.
+  // The callback will be called when SwapBuffers completes. While a SwapBuffers
+  // call is pending, all subsequent SwapBuffers calls will fail. Specifying a
+  // NULL callback means blocking but this is not legal on the main thread.
   // TODO(apatrick): premultiplied alpha or linear alpha? Premultiplied alpha is
   // better for correct alpha blending effect. Most existing OpenGL code assumes
   // linear. I could convert from linear to premultiplied during the copy from
   // back-buffer to offscreen "front-buffer".
-  PP_Bool (*SwapBuffers)(PP_Resource context);
+  PP_Bool (*SwapBuffers)(PP_Resource context,
+                         struct PP_CompletionCallback callback);
 
   // Returns the current error for this thread. This is not associated with a
   // particular context. It is distinct from the GL error returned by
   // glGetError.
   uint32_t (*GetError)();
 };
 
 #endif  // PPAPI_C_DEV_PPB_GRAPHICS_3D_DEV_H_