New NaCl zygote implementation 2

base/process_util.h

Index: base/process_util.h
diff --git a/base/process_util.h b/base/process_util.h
index 3e758d4defdd3ad4707f2532b5258b83e12875a6..9f66669eac73334cb7f11ca1ae167b5e2051b866 100644
--- a/base/process_util.h
+++ b/base/process_util.h
@@ -286,6 +286,15 @@ BASE_API bool LaunchAppInNewProcessGroup(
     bool wait,
     ProcessHandle* process_handle);
 
+#if defined(OS_LINUX)
+// Similar to LaunchApp variants above except uses clone(.. clone_flags ..)
+// rather than fork(). This is useful for work inside the setuid sandbox.
+BASE_API bool LaunchAppWithClone(const std::vector<std::string>& argv,
+                                 const file_handle_mapping_vector& fds_to_remap,

[darin (slow to review), 2011/06/24 16:32:19]

not your bug to fix, but "file_handle_mapping_vector" should of course be
FileHandleMappingVector to conform to the style guide.  i'm not sure why it was
named improperly :(

[Brad Chen, 2011/06/24 16:52:12]

Will fix on a separate CL.
On 2011/06/24 16:32:19, darin wrote:

+                                 bool wait, ProcessHandle* process_handle,
+                                 int clone_flags);
+#endif
+
 // AlterEnvironment returns a modified environment vector, constructed from the
 // given environment and the list of changes given in |changes|. Each key in
 // the environment is matched against the first element of the pairs. In the