CRX FileSystem Pepper private API

chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc

Index: chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
diff --git a/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc b/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
index b90d1f1586a354e725b2c6d32592e98a5790dca8..cdb935debb447b2ac90807140c2a3444b7835a56 100644
--- a/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
+++ b/chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc
@@ -5,6 +5,7 @@
 #include "chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.h"
 
 #include "chrome/browser/renderer_host/pepper/pepper_broker_message_filter.h"
+#include "chrome/browser/renderer_host/pepper/pepper_crx_file_system_message_filter.h"
 #include "chrome/browser/renderer_host/pepper/pepper_flash_browser_host.h"
 #include "chrome/browser/renderer_host/pepper/pepper_flash_clipboard_message_filter.h"
 #include "chrome/browser/renderer_host/pepper/pepper_flash_device_id_host.h"
@@ -52,6 +53,22 @@ scoped_ptr<ResourceHost> ChromeBrowserPepperHostFactory::CreateResourceHost(
             host_->GetPpapiHost(), instance, params.pp_resource(),
             broker_filter));
       }
+      case PpapiHostMsg_Ext_CrxFileSystem_Create::ID: {
+        int render_process_id;
+        int unused_render_view_id;
+        host_->GetRenderViewIDsForInstance(instance,

[yzshen1, 2013/05/06 17:34:57]

- Please check whether it fails.
- [optional] it seems better to have a static Create() for the message filter,
which takes host as parameter and do
GetRenderViewIDsForInstance/GetProfileDataDirectory/GetDocumentURLForInstance
there. So that we don't have to bloat this file.

[victorhsieh, 2013/05/06 18:32:02]

Done.

+                                           &render_process_id,
+                                           &unused_render_view_id);
+        scoped_refptr<ResourceMessageFilter> crxfs_filter(
+            new PepperCrxFileSystemMessageFilter(
+                render_process_id,
+                host_->GetProfileDataDirectory(),
+                host_->GetDocumentURLForInstance(instance)
+            ));
+        return scoped_ptr<ResourceHost>(new MessageFilterHost(
+            host_->GetPpapiHost(), instance, params.pp_resource(),
+            crxfs_filter));
+      }
       case PpapiHostMsg_Talk_Create::ID:
         return scoped_ptr<ResourceHost>(new PepperTalkHost(
             host_, instance, params.pp_resource()));