diff options
author | rvargas <rvargas@chromium.org> | 2014-12-12 16:34:58 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-12-13 00:35:30 +0000 |
commit | d7a1017bef32b69489b0e013c1e6122b4bbb5580 (patch) | |
tree | 5c8a00f75e3af459b3f3349c962c0da20f6cc598 /cloud_print | |
parent | ac3c4b6c6a0732560b641e2884db9cfc2cc85259 (diff) | |
download | chromium_src-d7a1017bef32b69489b0e013c1e6122b4bbb5580.zip chromium_src-d7a1017bef32b69489b0e013c1e6122b4bbb5580.tar.gz chromium_src-d7a1017bef32b69489b0e013c1e6122b4bbb5580.tar.bz2 |
Update cloud_print and chrome/service to use the new version of LaunchProcess.
BUG=417532
Review URL: https://codereview.chromium.org/785353003
Cr-Commit-Position: refs/heads/master@{#308206}
Diffstat (limited to 'cloud_print')
-rw-r--r-- | cloud_print/common/win/install_utils.cc | 3 | ||||
-rw-r--r-- | cloud_print/virtual_driver/win/install/setup.cc | 2 | ||||
-rw-r--r-- | cloud_print/virtual_driver/win/port_monitor/port_monitor.cc | 4 |
3 files changed, 4 insertions, 5 deletions
diff --git a/cloud_print/common/win/install_utils.cc b/cloud_print/common/win/install_utils.cc index d4694cf5..31b4377 100644 --- a/cloud_print/common/win/install_utils.cc +++ b/cloud_print/common/win/install_utils.cc @@ -203,8 +203,7 @@ void DeleteProgramDir(const std::string& delete_switch) { base::CommandLine command_line(temp_path); command_line.AppendSwitchPath(delete_switch, installer_source.DirName()); base::LaunchOptions options; - base::ProcessHandle process_handle; - if (!base::LaunchProcess(command_line, options, &process_handle)) { + if (!base::LaunchProcess(command_line, options).IsValid()) { LOG(ERROR) << "Unable to launch child uninstall."; } } diff --git a/cloud_print/virtual_driver/win/install/setup.cc b/cloud_print/virtual_driver/win/install/setup.cc index f4d0c4d..64b68c2 100644 --- a/cloud_print/virtual_driver/win/install/setup.cc +++ b/cloud_print/virtual_driver/win/install/setup.cc @@ -97,7 +97,7 @@ void SpoolerServiceCommand(const char* command) { options.wait = true; options.start_hidden = true; VLOG(0) << command_line.GetCommandLineString(); - base::LaunchProcess(command_line, options, NULL); + base::LaunchProcess(command_line, options); } HRESULT RegisterPortMonitor(bool install, const base::FilePath& install_path) { diff --git a/cloud_print/virtual_driver/win/port_monitor/port_monitor.cc b/cloud_print/virtual_driver/win/port_monitor/port_monitor.cc index 0878bac..a403af4 100644 --- a/cloud_print/virtual_driver/win/port_monitor/port_monitor.cc +++ b/cloud_print/virtual_driver/win/port_monitor/port_monitor.cc @@ -229,7 +229,7 @@ bool LaunchPrintDialog(const base::FilePath& xps_path, command_line.AppendSwitchNative(switches::kCloudPrintJobTitle, job_title); base::LaunchOptions options; options.as_user = primary_token_scoped.Get(); - base::LaunchProcess(command_line, options, NULL); + base::LaunchProcess(command_line, options); return true; } @@ -255,7 +255,7 @@ void LaunchChromeDownloadPage() { base::LaunchOptions options; options.as_user = token_scoped.Get(); - base::LaunchProcess(command_line, options, NULL); + base::LaunchProcess(command_line, options); } // Returns false if the print job is being run in a context |