PPB_CursorControl_Dev.SetCursor: Add support for custom cursor.

webkit/plugins/ppapi/ppapi_plugin_instance.h

Index: webkit/plugins/ppapi/ppapi_plugin_instance.h
diff --git a/webkit/plugins/ppapi/ppapi_plugin_instance.h b/webkit/plugins/ppapi/ppapi_plugin_instance.h
index b847c362d47bca2400b4b2a98e34b2688dd6688a..5408aef2e58555dec431f74412010c53c423dc3d 100644
--- a/webkit/plugins/ppapi/ppapi_plugin_instance.h
+++ b/webkit/plugins/ppapi/ppapi_plugin_instance.h
@@ -130,7 +130,9 @@ class PluginInstance : public base::RefCounted<PluginInstance> {
   PP_Var GetOwnerElementObject();
   bool BindGraphics(PP_Resource graphics_id);
   bool full_frame() const { return full_frame_; }
-  bool SetCursor(PP_CursorType_Dev type);
+  bool SetCursor(PP_CursorType_Dev type,
+                 PP_Resource custom_image,
+                 const PP_Point* hot_spot);

[brettw, 2011/03/23 02:08:02]

I'd pass this as a ref. We pass pointers in the C API because it doesn't have
refs. It will also allow you to remove the NULL checking.

[yzshen1, 2011/03/23 17:24:35]

If |type| is not PP_CURSORTYPE_CUSTOM, |hot_spot| is ignored and can be NULL. (I
have added comments.)
Using a ref will require us to create an PP_Point instance that won't be used.
If you think it is okay, I would like to leave it as it is.
Thanks!

   PP_Var ExecuteScript(PP_Var script, PP_Var* exception);
 
   // PPP_Instance pass-through.