Convert to new enter method

ppapi/thunk/ppb_layer_compositor_thunk.cc

Index: ppapi/thunk/ppb_layer_compositor_thunk.cc
diff --git a/ppapi/thunk/ppb_layer_compositor_thunk.cc b/ppapi/thunk/ppb_layer_compositor_thunk.cc
index df03a43eb9d0a4820911f008851a234d3112d593..98007d3553321d0f58c8be1fd9a63d910e66d296 100644
--- a/ppapi/thunk/ppb_layer_compositor_thunk.cc
+++ b/ppapi/thunk/ppb_layer_compositor_thunk.cc
@@ -4,7 +4,6 @@
 
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/c/dev/ppb_layer_compositor_dev.h"
-#include "ppapi/thunk/common.h"
 #include "ppapi/thunk/enter.h"
 #include "ppapi/thunk/thunk.h"
 #include "ppapi/thunk/ppb_layer_compositor_api.h"
@@ -46,7 +45,7 @@ void MarkAsDirty(PP_Resource compositor, PP_Resource layer) {
 
 int32_t SwapBuffers(PP_Resource compositor,
                     struct PP_CompletionCallback callback) {
-  return MayForceCallback(callback, PP_ERROR_FAILED);
+  return PP_ERROR_NOINTERFACE;

[yzshen1, 2012/01/25 19:11:01]

This breaks the convention of how |callback| is used.
I think it is better to still use the new enter as everyone else.

[brettw, 2012/01/25 19:54:12]

The problem is that since there's no implementation for this interface, there is
no "API" object to request, so there is nothing to "enter".

I'm OK leaking a callback if you use an interface we haven't implemented yet.

[yzshen1, 2012/01/25 20:06:26]

Okay. I am okay to leave it unchanged.

I just thought that even if there is no implementation, EnterResource will fail
gracefully (right?). And we can then use return
enter.SetResult(PP_ERROR_NOINTERFACE).


On 2012/01/25 19:54:12, brettw wrote:

[brettw, 2012/01/26 05:17:40]

Yeah, but EnterResource is a template, so I have to say EnterResource of
something. I don't want to say some random other resource class. I think
probably EnterResource<anything> would work, but that also seems slightly
misleading.

[yzshen1, 2012/01/27 18:25:47]

[Again, I am fine with the current approach.]
I thought PPB_LayerCompositor_API could be used in this case. Am I missing
anything?

On 2012/01/26 05:17:40, brettw wrote:

 }
 
 const PPB_LayerCompositor_Dev g_ppb_layer_compositor_thunk = {