[PPAPI] It is now possible to pass filesystems from JavaScript to NaCl modules.

content/browser/renderer_host/pepper/pepper_renderer_connection.cc

Index: content/browser/renderer_host/pepper/pepper_renderer_connection.cc
diff --git a/content/browser/renderer_host/pepper/pepper_renderer_connection.cc b/content/browser/renderer_host/pepper/pepper_renderer_connection.cc
index e4c8aa71dc11f3b39f042c320407ac70383cc7f6..dbbdf92049b8a58709eea675a52b500c764e6c64 100644
--- a/content/browser/renderer_host/pepper/pepper_renderer_connection.cc
+++ b/content/browser/renderer_host/pepper/pepper_renderer_connection.cc
@@ -10,6 +10,7 @@
 #include "content/common/pepper_renderer_instance_data.h"
 #include "content/common/view_messages.h"
 #include "content/browser/renderer_host/pepper/pepper_file_ref_host.h"
+#include "content/browser/renderer_host/pepper/pepper_file_system_browser_host.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/common/content_client.h"
 #include "ipc/ipc_message_macros.h"
@@ -97,24 +98,40 @@ void PepperRendererConnection::OnMsgCreateResourceHostsFromHost(
     DLOG(ERROR) << "Invalid plugin process ID.";
   } else {
     for (size_t i = 0; i < nested_msgs.size(); ++i) {
-      // FileRef_CreateExternal is only permitted from the renderer. Because of
-      // this, we handle this message here and not in
-      // content_browser_pepper_host_factory.cc.
+      const IPC::Message& nested_msg = nested_msgs[i];
       scoped_ptr<ppapi::host::ResourceHost> resource_host;
       if (host->IsValidInstance(instance)) {
-        if (nested_msgs[i].type() == PpapiHostMsg_FileRef_CreateExternal::ID) {
+        if (nested_msg.type() == PpapiHostMsg_FileRef_CreateExternal::ID) {
+          // FileRef_CreateExternal is only permitted from the renderer. Because
+          // of this, we handle this message here and not in
+          // content_browser_pepper_host_factory.cc.
           base::FilePath external_path;
           if (ppapi::UnpackMessage<PpapiHostMsg_FileRef_CreateExternal>(
-              nested_msgs[i], &external_path)) {
+              nested_msg, &external_path)) {

[raymes, 2013/10/22 05:59:21]

nit: I have a feeling this should be indented 4 spaces but I'm not certain (and
below also)

[Matt Giuca, 2013/10/23 08:05:54]

Done.

             resource_host.reset(new PepperFileRefHost(
                 host, instance, params.pp_resource(), external_path));
           }
+        } else if (nested_msg.type() ==
+                   PpapiHostMsg_FileSystem_CreateFromRenderer::ID) {
+          // Similarly, FileSystem_CreateFromRenderer is only permitted from the
+          // renderer.
+          std::string root_url;
+          PP_FileSystemType file_system_type;
+          if (ppapi::UnpackMessage<PpapiHostMsg_FileSystem_CreateFromRenderer>(
+              nested_msg, &root_url, &file_system_type)) {
+            resource_host.reset(
+                new PepperFileSystemBrowserHost(host,
+                                                instance,
+                                                params.pp_resource(),
+                                                GURL(root_url),
+                                                file_system_type));
+          }
         }
       }
 
       if (!resource_host.get()) {
         resource_host = host->GetPpapiHost()->CreateResourceHost(
-            params, instance, nested_msgs[i]);
+            params, instance, nested_msg);
       }
 
       if (resource_host.get()) {