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

content/renderer/pepper/host_var_tracker.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 "content/renderer/pepper/host_var_tracker.h"
 
 #include "base/logging.h"
 #include "content/renderer/pepper/host_array_buffer_var.h"
 #include "content/renderer/pepper/host_resource_var.h"
 #include "content/renderer/pepper/npobject_var.h"
@@ skipping 85 matching lines @@
 
 int HostVarTracker::GetLiveNPObjectVarsForInstance(PP_Instance instance) const {
   CheckThreadingPreconditions();
 
   InstanceMap::const_iterator found = instance_map_.find(instance);
   if (found == instance_map_.end())
     return 0;
   return static_cast<int>(found->second->size());
 }
 
+PP_Var HostVarTracker::MakeResourcePPVar(const IPC::Message& creation_message,
+                                         int pending_renderer_id,
+                                         int pending_browser_id,
+                                         PP_Instance instance) {
+  return VarTracker::MakeResourcePPVar(0);

[raymes, 2013/10/30 06:31:51]

Should this be NOTREACHED?

[Matt Giuca, 2013/10/30 07:00:39]

Well, it's technically valid to call this -- on the host side, we always ignore
creation_message. If the pp_resource is 0 and creation_message is not empty,
we'll end up here, and we should just return a PP_Var with 0.

I don't think we *will* ever get here, since why would creation_message be
non-empty when deserializing a resource on the host side? But technically, this
is allowed.

+}
+
 ppapi::ResourceVar* HostVarTracker::MakeResourceVar(PP_Resource pp_resource) {
   return new HostResourceVar(pp_resource);
 }
 
 void HostVarTracker::DidDeleteInstance(PP_Instance instance) {
   CheckThreadingPreconditions();
 
   InstanceMap::iterator found_instance = instance_map_.find(instance);
   if (found_instance == instance_map_.end())
     return;  // Nothing to do.
@@ skipping 51 matching lines @@
   if (it->second.instance != instance)
     return false;
 
   *handle = it->second.handle;
   *size_in_bytes = it->second.size_in_bytes;
   shared_memory_map_.erase(it);
   return true;
 }
 
 }  // namespace content