Change base::LaunchProcess API slightly

Rather than passing the out param process handle via the options,
take it as a function argument.  This simplifies many callers.

BUG=88990

Review URL: http://codereview.chromium.org/7377012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92701 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 2 insertions, 4 deletions

diff --git a/net/tools/crash_cache/crash_cache.cc b/net/tools/crash_cache/crash_cache.cc
index 637daae..f1b4b79 100644
--- a/net/tools/crash_cache/crash_cache.cc
+++ b/net/tools/crash_cache/crash_cache.cc
@@ -49,9 +49,7 @@ int RunSlave(RankCrashes action) {
   cmdline.AppendArg(base::IntToString(action));
 
   base::ProcessHandle handle;
-  base::LaunchOptions options;
-  options.process_handle = &handle;
-  if (!base::LaunchProcess(cmdline, options)) {
+  if (!base::LaunchProcess(cmdline, base::LaunchOptions(), &handle)) {
     printf("Unable to run test %d\n", action);
     return GENERIC;
   }
diff --git a/net/tools/dump_cache/dump_cache.cc b/net/tools/dump_cache/dump_cache.cc
index 4a63d00..ee5a50e 100644
--- a/net/tools/dump_cache/dump_cache.cc
+++ b/net/tools/dump_cache/dump_cache.cc
@@ -97,7 +97,7 @@ int LaunchSlave(const CommandLine& command_line,
 
   // TODO(evanm): remove needless usage of wstring from here and elsewhere.
   new_command_line.AppendSwitchASCII(kPipe, WideToASCII(pipe_number));
-  if (!base::LaunchProcess(new_command_line, base::LaunchOptions())) {
+  if (!base::LaunchProcess(new_command_line, base::LaunchOptions(), NULL)) {
     printf("Unable to launch the needed version of this tool: %ls\n",
            new_program.c_str());
     printf(kUpgradeHelp);