diff options
author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-14 20:55:30 +0000 |
---|---|---|
committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-14 20:55:30 +0000 |
commit | adda869f22b29e7c43c2d84efdda2919854ed608 (patch) | |
tree | 9798ebe411388bacbd50a5798504f04ace15902b /chrome_frame/test | |
parent | 81f5486a2aa70e059a76b6e0851bb55b2e3daf53 (diff) | |
download | chromium_src-adda869f22b29e7c43c2d84efdda2919854ed608.zip chromium_src-adda869f22b29e7c43c2d84efdda2919854ed608.tar.gz chromium_src-adda869f22b29e7c43c2d84efdda2919854ed608.tar.bz2 |
Clean up users of a deprecated base::LaunchApp API.
BUG=88990
Review URL: http://codereview.chromium.org/7351003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92598 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame/test')
-rw-r--r-- | chrome_frame/test/chrome_frame_test_utils.cc | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/chrome_frame/test/chrome_frame_test_utils.cc b/chrome_frame/test/chrome_frame_test_utils.cc index 3975c70..397a07b 100644 --- a/chrome_frame/test/chrome_frame_test_utils.cc +++ b/chrome_frame/test/chrome_frame_test_utils.cc @@ -159,15 +159,19 @@ base::ProcessHandle LaunchExecutable(const std::wstring& executable, LOG(ERROR) << "Failed to find executable: " << executable; } else { CommandLine cmdline = CommandLine::FromString(path); - if (!base::LaunchApp(cmdline, false, false, &process)) { - LOG(ERROR) << "LaunchApp failed: " << ::GetLastError(); + base::LaunchOptions options; + options.process_handle = &process; + if (!base::LaunchProcess(cmdline, options)) { + LOG(ERROR) << "LaunchProcess failed: " << ::GetLastError(); } } } else { CommandLine cmdline((FilePath(path))); cmdline.AppendArgNative(argument); - if (!base::LaunchApp(cmdline, false, false, &process)) { - LOG(ERROR) << "LaunchApp failed: " << ::GetLastError(); + base::LaunchOptions options; + options.process_handle = &process; + if (!base::LaunchProcess(cmdline, options)) { + LOG(ERROR) << "LaunchProcess failed: " << ::GetLastError(); } } return process; @@ -183,7 +187,9 @@ base::ProcessHandle LaunchChrome(const std::wstring& url) { cmd.AppendArgNative(url); base::ProcessHandle process = NULL; - base::LaunchApp(cmd, false, false, &process); + base::LaunchOptions options; + options.process_handle = &process; + base::LaunchProcess(cmd, options); return process; } @@ -590,8 +596,9 @@ base::ProcessHandle StartCrashService() { DVLOG(1) << "Starting crash_service.exe so you know if a test crashes!"; FilePath crash_service_path = exe_dir.AppendASCII("crash_service.exe"); - if (!base::LaunchApp(crash_service_path.value(), false, false, - &crash_service)) { + base::LaunchOptions options; + options.process_handle = &crash_service; + if (!base::LaunchProcess(crash_service_path.value(), options)) { DLOG(ERROR) << "Couldn't start crash_service.exe"; return NULL; } |