Refactor FileIO to the 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 c41757b36107b5527dd994c8804759b7759e114c..1f573eb510ec3b170217c5fcab4f38dc8242f04a 100644
--- a/content/public/renderer/renderer_ppapi_host.h
+++ b/content/public/renderer/renderer_ppapi_host.h
@@ -40,6 +40,7 @@ class WebPluginContainer;
 
 namespace webkit {
 namespace ppapi {
+class PluginDelegate;
 class PluginInstance;
 class PluginModule;
 }
@@ -80,6 +81,11 @@ class RendererPpapiHost {
   virtual webkit::ppapi::PluginInstance* GetPluginInstance(
       PP_Instance instance) const = 0;
 
+  // Returns the PluginDelegate for the given plugin instance, or NULL if the

[raymes, 2012/11/26 22:30:54]

I would change "plugin instance" -> "PP_Instance" in order not to confuse it
with a PluginInstance.

[victorhsieh, 2012/11/27 09:44:42]

Deleted

+  // instance is invalid.
+  virtual webkit::ppapi::PluginDelegate* GetDelegateForInstance(

[raymes, 2012/11/27 06:54:28]

The plugin delegate shouldn't be needed for new-style resources. The resource
host should be able to access everything it needs directly (more comments in the
host). I would suggest removing this function. You can still get to the delegate
if needed via the plugininstance.

[victorhsieh, 2012/11/27 09:44:42]

Done.

+      PP_Instance instance) const = 0;
+
   // Returns the RenderView for the given plugin instance, or NULL if the
   // instance is invalid.
   virtual RenderView* GetRenderViewForInstance(PP_Instance instance) const = 0;
@@ -126,4 +132,3 @@ class RendererPpapiHost {
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_RENDERER_PPAPI_HOST_H_
-