Migrate Graphics2D to new design.

content/public/renderer/renderer_ppapi_host.h

Index: content/public/renderer/renderer_ppapi_host.h
diff --git a/content/public/renderer/renderer_ppapi_host.h b/content/public/renderer/renderer_ppapi_host.h
index 8fda76982f07f6c5106275496178ab10286253f6..9c18e9891f2fde1346026defbefc46c2d0ebc937 100644
--- a/content/public/renderer/renderer_ppapi_host.h
+++ b/content/public/renderer/renderer_ppapi_host.h
@@ -6,6 +6,7 @@
 #define CONTENT_PUBLIC_RENDERER_RENDERER_PPAPI_HOST_H_
 
 #include "ppapi/c/pp_instance.h"
+#include "webkit/plugins/ppapi/plugin_delegate.h"
 
 namespace ppapi {
 namespace host {
@@ -24,7 +25,7 @@ class RenderView;
 class RendererPpapiHost {
  public:
   // Returns the PpapiHost object.
-  virtual ppapi::host::PpapiHost* GetPpapiHost() = 0;
+  virtual ppapi::host::PpapiHost* GetPpapiHost() const = 0;

[brettw, 2012/10/05 22:38:06]

Did you particularly need this change? The reason I didn't make this const is
that it returns a non-const PpapiHost* which is really internal state of the
RendererPpapiHost object. So semantically, the constness of the getter should
match the constness of the return value in this case (even though it compiles
either way).

[victorhsieh, 2012/10/05 23:26:57]

No, we can remove the const-ness up to RendererPpapiHost::GetPlatformGraphics2D.
 Done.

 
   // Returns true if the given PP_Instance is valid and belongs to the
   // plugin associated with this host.
@@ -34,6 +35,11 @@ class RendererPpapiHost {
   // instance is invalid.
   virtual RenderView* GetRenderViewForInstance(PP_Instance instance) const = 0;
 
+  // Returns the PlatformGraphics2D for the given plugin resource, or NULL if
+  // the resource is invalid.
+  virtual webkit::ppapi::PluginDelegate::PlatformGraphics2D*
+      GetPlatformGraphics2D(PP_Resource resource) const = 0;
+
   // Returns true if the given instance is considered to be currently
   // processing a user gesture or the plugin module has the "override user
   // gesture" flag set (in which case it can always do things normally
@@ -48,4 +54,3 @@ class RendererPpapiHost {
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_RENDERER_PPAPI_HOST_H_
-