Non-SFI NaCl: Batch-open resource files

components/nacl/common/nacl_types.h

 // 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.
 
 #ifndef COMPONENTS_NACL_COMMON_NACL_TYPES_H_
 #define COMPONENTS_NACL_COMMON_NACL_TYPES_H_
 
 #include <string>
+#include <utility>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/shared_memory.h"
 #include "base/process/process_handle.h"
 #include "build/build_config.h"
 #include "ipc/ipc_channel.h"
 #include "ipc/ipc_platform_file.h"
 
 #if defined(OS_POSIX)
@@ skipping 36 matching lines @@
   // (not user-provided).
   kPNaClTranslatorProcessType,
   kNumNaClProcessTypes
 };
 
 // Parameters sent to the NaCl process when we start it.
 struct NaClStartParams {
   NaClStartParams();
   ~NaClStartParams();
 
+  struct ResourceFileInfo {

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

Move above?
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Declaration_O...

Or, maybe declare at outside of NaClStartParams?

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

Done. Let me keep this inside the class to avoid name conflict with the one
below.

+    ResourceFileInfo();
+    ResourceFileInfo(IPC::PlatformFileForTransit file,
+                     const base::FilePath& file_path,
+                     const std::string& file_key);
+    ~ResourceFileInfo();
+
+    IPC::PlatformFileForTransit file;
+    base::FilePath file_path;
+    std::string file_key;
+  };
+
+  void CheckNumOfDescriptors();
+
   IPC::PlatformFileForTransit nexe_file;
   // Used only as a key for validation caching.
   base::FilePath nexe_file_path_metadata;
 
+  std::vector<ResourceFileInfo> resource_files;
   std::vector<FileDescriptor> handles;
   FileDescriptor debug_stub_server_bound_socket;
 
   bool validation_cache_enabled;
   std::string validation_cache_key;
   // Chrome version string. Sending the version string over IPC avoids linkage
   // issues in cases where NaCl is not compiled into the main Chromium
   // executable or DLL.
   std::string version;
 
   bool enable_debug_stub;
   bool enable_ipc_proxy;
 
   NaClAppProcessType process_type;
 
   // For NaCl <-> renderer crash information reporting.
   base::SharedMemoryHandle crash_info_shmem_handle;
 
   // NOTE: Any new fields added here must also be added to the IPC
   // serialization in nacl_messages.h and (for POD fields) the constructor
-  // in nacl_types.cc.
+  // in nacl_types.cc. Any new fields added for passing file handles must
+  // also be checked in NaClStartParams::CheckNumOfDescriptors().
 };
 
 // Parameters sent to the browser process to have it launch a NaCl process.
 //
 // If you change this, you will also need to update the IPC serialization in
 // nacl_host_messages.h.
 struct NaClLaunchParams {
+  struct ResourceFileInfo;
   NaClLaunchParams();
-  NaClLaunchParams(const std::string& manifest_url,
-                   const IPC::PlatformFileForTransit& nexe_file,
-                   uint64_t nexe_token_lo,
-                   uint64_t nexe_token_hi,
-                   int render_view_id,
-                   uint32 permission_bits,
-                   bool uses_nonsfi_mode,
-                   NaClAppProcessType process_type);
+  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,
+      int render_view_id,
+      uint32 permission_bits,
+      bool uses_nonsfi_mode,
+      NaClAppProcessType process_type);
   ~NaClLaunchParams();
 
+  struct ResourceFileInfo {

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

Move to above?

http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Declaration_O...

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

Done.

+    ResourceFileInfo();
+    ResourceFileInfo(IPC::PlatformFileForTransit file,
+                     uint64_t file_token_lo,
+                     uint64_t file_token_hi,
+                     const std::string& file_key);
+    ~ResourceFileInfo();
+
+    IPC::PlatformFileForTransit file;

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

These three fields looks as same as NaClOpenExecutableResult::FileInfo.

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

Done.

+    uint64_t file_token_lo;
+    uint64_t file_token_hi;
+    // This corresponds to a key in the "files" section of the NMF file.
+    std::string file_key;
+  };
+
   std::string manifest_url;
   // On Windows, the HANDLE passed here is valid in the renderer's context.
   // It's the responsibility of the browser to duplicate this handle properly
   // for passing it to the plugin.
   IPC::PlatformFileForTransit nexe_file;
   uint64_t nexe_token_lo;
   uint64_t nexe_token_hi;
+  std::vector<ResourceFileInfo> resource_files_info;
 
   int render_view_id;
   uint32 permission_bits;
   bool uses_nonsfi_mode;
 
   NaClAppProcessType process_type;
 };
 
 struct NaClLaunchResult {
   NaClLaunchResult();
@@ skipping 20 matching lines @@
   // For plugin <-> renderer ManifestService communication.
   IPC::ChannelHandle manifest_service_ipc_channel_handle;
 
   base::ProcessId plugin_pid;
   int plugin_child_id;
 
   // For NaCl <-> renderer crash information reporting.
   base::SharedMemoryHandle crash_info_shmem_handle;
 };
 
+struct NaClOpenExecutableResult {
+  struct FileInfo;
+  NaClOpenExecutableResult();
+  NaClOpenExecutableResult(const FileInfo& file_info,
+                           const std::vector<FileInfo>& resource_files_info);
+  ~NaClOpenExecutableResult();
+
+  struct FileInfo {
+    FileInfo();
+    FileInfo(IPC::PlatformFileForTransit file,
+             uint64_t file_token_lo,
+             uint64_t file_token_hi);
+    ~FileInfo();
+
+    IPC::PlatformFileForTransit file;
+    uint64_t file_token_lo;
+    uint64_t file_token_hi;
+  };
+
+  FileInfo file_info;
+  std::vector<FileInfo> resource_files_info;
+};
+
 }  // namespace nacl
 
 #endif  // COMPONENTS_NACL_COMMON_NACL_TYPES_H_