Upgrade the windows specific version of LaunchProcess to avoid raw handles.

base/process/launch_win.cc

Index: base/process/launch_win.cc
diff --git a/base/process/launch_win.cc b/base/process/launch_win.cc
index a3303a5e08588669aac9d84cf6f9acdb2473b594..003bc15828118f8fbdd07ce9197750e5140cf0ba 100644
--- a/base/process/launch_win.cc
+++ b/base/process/launch_win.cc
@@ -232,6 +232,17 @@ bool LaunchProcess(const string16& cmdline,
   return true;
 }
 
+// TODO(rvargas) crbug.com/416721: Remove this stub after LaunchProcess is
+// fully migrated to use Process.
+Process LaunchProcess(const string16& cmdline,
+                      const LaunchOptions& options) {
+  win::ScopedHandle process;

[gab, 2014/11/27 13:28:44]

s/process/process_handle
(for naming consistency with everything else in this file and this CL)

[rvargas (doing something else), 2014/12/01 20:50:36]

Done.

+  if (LaunchProcess(cmdline, options, &process))
+    return Process(process.Take());
+
+  return Process();
+}
+
 bool LaunchProcess(const CommandLine& cmdline,

[gab, 2014/11/27 13:28:44]

Is this still needed? (and similarly should that definition be ifdef'ed out in
the header on OS_WIN?)

Or are we not there yet? Perhaps this is actually what you meant by the comment
on line 235, sorry about the noise if that's the case :-)!

[rvargas (doing something else), 2014/12/01 20:50:36]

Yeah, this function is what prevents me from switching right away the contract
of the win-specific override :(. And this will take multiple CLs as this version
is used everywhere.

                    const LaunchOptions& options,
                    ProcessHandle* process_handle) {