Add stdin_handle and stdout_handle in base::LaunchOptions.

base/process_util.h

Index: base/process_util.h
diff --git a/base/process_util.h b/base/process_util.h
index 2805e421607e43ff028485351cb1a46801673606..c079674339e604be8a26f4c91ec6b58f1a9da126 100644
--- a/base/process_util.h
+++ b/base/process_util.h
@@ -240,6 +240,9 @@ struct LaunchOptions {
 #if defined(OS_WIN)
                     start_hidden(false), inherit_handles(false), as_user(NULL),
                     empty_desktop_name(false), job_handle(NULL),
+                    stdin_handle(NULL),
+                    stdout_handle(NULL),
+                    stderr_handle(NULL),
                     force_breakaway_from_job_(false)
 #else
                     environ(NULL), fds_to_remap(NULL), maximize_rlimits(NULL),
@@ -282,6 +285,15 @@ struct LaunchOptions {
   // the job object fails.
   HANDLE job_handle;
 
+  // Handles for the redirection of stdin, stdout and stderr. The handles must
+  // be inheritable. If either |stdout_handle| or |stderr_handle| is set then
+  // |stdin_handle| must be set too (i.e. stdin must always be redirected if any

[rvargas (doing something else), 2012/12/06 00:52:16]

Why not just say that if any handle must be inherited all three handles must be
set?. Messing stderr (or stdout) sounds almost as bad as reading my input :)

[Sergey Ulanov, 2012/12/07 23:11:56]

Done.

+  // other stream is redirected). The |inherit_handles| flag must be set to
+  // true if any of these handles is set.
+  HANDLE stdin_handle;
+  HANDLE stdout_handle;
+  HANDLE stderr_handle;
+
   // If set to true, ensures that the child process is launched with the
   // CREATE_BREAKAWAY_FROM_JOB flag which allows it to breakout of the parent
   // job if any.