Non-SFI NaCl: Batch-open resource files

components/nacl/common/nacl_types.cc

 // Copyright 2013 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 "components/nacl/common/nacl_types.h"
 #include "ipc/ipc_platform_file.h"
 
+#if defined(OS_POSIX)
+#include "ipc/file_descriptor_set_posix.h"
+#endif
+
 namespace nacl {
 
 NaClStartParams::NaClStartParams()
     : nexe_file(IPC::InvalidPlatformFileForTransit()),
       validation_cache_enabled(false),
       enable_debug_stub(false),
       enable_ipc_proxy(false),
       process_type(kUnknownNaClProcessType),
       crash_info_shmem_handle(base::SharedMemory::NULLHandle()) {
 }
 
 NaClStartParams::~NaClStartParams() {
 }
 
+void NaClStartParams::CheckNumOfDescriptors() {
+#if defined(OS_POSIX)

[hidehiko, 2015/01/28 09:05:19]

Isn't this only for Non-SFI? If so, limited to Linux? Otherwise, maybe we should
have some check for Win, too?
Anyway, I'd like to know your concept to limit OS_POSIX here.

[Yusuke Sato, 2015/02/04 02:00:28]

Right, for now, OS_LINUX would suffice. Changed to OS_LINUX with a TODO.

+  // '3' is for |nexe_file|, |debug_stub_server_bound_socket|, and
+  // |crash_info_shmem_handle|.
+  size_t num_descs = handles.size() + resource_files.size() + 3;
+  // If this CHECK fails, either of the following has to be done:
+  // * Increase kMaxDescriptorsPerMessage in ipc/file_descriptor_set_posix.h.
+  // * Or, decrease kMaxPreOpenResourceFiles in
+  //   components/nacl/renderer/ppb_nacl_private_impl.cc.
+  CHECK(num_descs <= FileDescriptorSet::kMaxDescriptorsPerMessage);
+#endif
+}
+
+NaClStartParams::ResourceFileInfo::ResourceFileInfo()
+    : file(IPC::InvalidPlatformFileForTransit()) {
+}
+
+NaClStartParams::ResourceFileInfo::ResourceFileInfo(
+    IPC::PlatformFileForTransit file,
+    const base::FilePath& file_path,
+    const std::string& file_key)
+    : file(file), file_path(file_path), file_key(file_key) {
+}
+
+NaClStartParams::ResourceFileInfo::~ResourceFileInfo() {
+}
+
 NaClLaunchParams::NaClLaunchParams()
     : nexe_file(IPC::InvalidPlatformFileForTransit()),
       nexe_token_lo(0),
       nexe_token_hi(0),
       render_view_id(0),
       permission_bits(0),
       process_type(kUnknownNaClProcessType) {
 }
 
 NaClLaunchParams::NaClLaunchParams(
     const std::string& manifest_url,
     const IPC::PlatformFileForTransit& nexe_file,
     uint64_t nexe_token_lo,
     uint64_t nexe_token_hi,
+    const std::vector<ResourceFileInfo>& resource_files_info,
     int render_view_id,
     uint32 permission_bits,
     bool uses_nonsfi_mode,
     NaClAppProcessType process_type)
     : manifest_url(manifest_url),
       nexe_file(nexe_file),
       nexe_token_lo(nexe_token_lo),
       nexe_token_hi(nexe_token_hi),
+      resource_files_info(resource_files_info),
       render_view_id(render_view_id),
       permission_bits(permission_bits),
       uses_nonsfi_mode(uses_nonsfi_mode),
       process_type(process_type) {
 }
 
 NaClLaunchParams::~NaClLaunchParams() {
 }
 
+NaClLaunchParams::ResourceFileInfo::ResourceFileInfo()
+    : file(IPC::InvalidPlatformFileForTransit()),
+      file_token_lo(0),
+      file_token_hi(0) {
+}
+
+NaClLaunchParams::ResourceFileInfo::ResourceFileInfo(
+    IPC::PlatformFileForTransit file,
+    uint64_t file_token_lo,
+    uint64_t file_token_hi,
+    const std::string& file_key)
+    : file(file),
+      file_token_lo(file_token_lo),
+      file_token_hi(file_token_hi),
+      file_key(file_key) {
+}
+
+NaClLaunchParams::ResourceFileInfo::~ResourceFileInfo() {
+}
+
 NaClLaunchResult::NaClLaunchResult()
     : imc_channel_handle(IPC::InvalidPlatformFileForTransit()),
       ppapi_ipc_channel_handle(),
       trusted_ipc_channel_handle(),
       plugin_pid(base::kNullProcessId),
       plugin_child_id(0),
       crash_info_shmem_handle(base::SharedMemory::NULLHandle()) {
 }
 
 NaClLaunchResult::NaClLaunchResult(
     FileDescriptor imc_channel_handle,
     const IPC::ChannelHandle& ppapi_ipc_channel_handle,
     const IPC::ChannelHandle& trusted_ipc_channel_handle,
     const IPC::ChannelHandle& manifest_service_ipc_channel_handle,
     base::ProcessId plugin_pid,
     int plugin_child_id,
     base::SharedMemoryHandle crash_info_shmem_handle)
     : imc_channel_handle(imc_channel_handle),
       ppapi_ipc_channel_handle(ppapi_ipc_channel_handle),
       trusted_ipc_channel_handle(trusted_ipc_channel_handle),
       manifest_service_ipc_channel_handle(manifest_service_ipc_channel_handle),
       plugin_pid(plugin_pid),
       plugin_child_id(plugin_child_id),
       crash_info_shmem_handle(crash_info_shmem_handle) {
 }
 
 NaClLaunchResult::~NaClLaunchResult() {
 }
 
+NaClOpenExecutableResult::NaClOpenExecutableResult() {
+}
+
+NaClOpenExecutableResult::NaClOpenExecutableResult(
+    const NaClOpenExecutableResult::FileInfo& file_info,
+    const std::vector<NaClOpenExecutableResult::FileInfo>& resource_files_info)
+    : file_info(file_info),
+      resource_files_info(resource_files_info) {
+}
+
+NaClOpenExecutableResult::~NaClOpenExecutableResult() {
+}
+
+NaClOpenExecutableResult::FileInfo::FileInfo()
+    : file(IPC::InvalidPlatformFileForTransit()),
+      file_token_lo(0),
+      file_token_hi(0) {
+}
+
+NaClOpenExecutableResult::FileInfo::FileInfo(IPC::PlatformFileForTransit file,
+                                             uint64_t file_token_lo,
+                                             uint64_t file_token_hi)
+    : file(file),
+      file_token_lo(file_token_lo),
+      file_token_hi(file_token_hi) {
+}
+
+NaClOpenExecutableResult::FileInfo::~FileInfo() {
+}
+
 }  // namespace nacl