CRX FileSystem Pepper private API

chrome/browser/renderer_host/pepper/chrome_browser_pepper_host_factory.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #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"
 #include "chrome/browser/renderer_host/pepper/pepper_talk_host.h"
 #include "content/public/browser/browser_ppapi_host.h"
 #include "ppapi/host/message_filter_host.h"
 #include "ppapi/host/ppapi_host.h"
 #include "ppapi/host/resource_host.h"
 #include "ppapi/proxy/ppapi_messages.h"
 #include "ppapi/shared_impl/ppapi_permissions.h"
@@ skipping 27 matching lines @@
   if (host_->GetPpapiHost()->permissions().HasPermission(
           ppapi::PERMISSION_PRIVATE)) {
     switch (message.type()) {
       case PpapiHostMsg_Broker_Create::ID: {
         scoped_refptr<ResourceMessageFilter> broker_filter(
             new PepperBrokerMessageFilter(instance, host_));
         return scoped_ptr<ResourceHost>(new MessageFilterHost(
             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()));
     }
   }
 
   // Flash interfaces.
   if (host_->GetPpapiHost()->permissions().HasPermission(
           ppapi::PERMISSION_FLASH)) {
     switch (message.type()) {
       case PpapiHostMsg_Flash_Create::ID:
         return scoped_ptr<ResourceHost>(new PepperFlashBrowserHost(
             host_, instance, params.pp_resource()));
       case PpapiHostMsg_FlashClipboard_Create::ID: {
         scoped_refptr<ResourceMessageFilter> clipboard_filter(
             new PepperFlashClipboardMessageFilter);
         return scoped_ptr<ResourceHost>(new MessageFilterHost(
             host_->GetPpapiHost(), instance, params.pp_resource(),
             clipboard_filter));
       }
       case PpapiHostMsg_FlashDeviceID_Create::ID:
         return scoped_ptr<ResourceHost>(new PepperFlashDeviceIDHost(
             host_, instance, params.pp_resource()));
     }
   }
   return scoped_ptr<ResourceHost>();
 }
 
 }  // namespace chrome