diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-19 00:27:24 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-19 00:27:24 +0000 |
| commit | 0f5c815c4ce3a1b970ae4b9a47f4a5001e5ec50c (patch) | |
| tree | 0a360bad68bb39d8f87aa7f6136992946264a7d5 /chrome/browser/utility_process_host.cc | |
| parent | 206743c88c222ed96d727da98e16ca19e2aac59e (diff) | |
| download | chromium_src-0f5c815c4ce3a1b970ae4b9a47f4a5001e5ec50c.zip chromium_src-0f5c815c4ce3a1b970ae4b9a47f4a5001e5ec50c.tar.gz chromium_src-0f5c815c4ce3a1b970ae4b9a47f4a5001e5ec50c.tar.bz2 | |
Remove the utility process unit tests to simplify the utility process code a little (helpeful to make it use ChildProcessLauncher). Note that the test isn't needed anymore as a unit test because since it was written, there have been a couple of tests that exercise this code path through browser_tests. They use sandboxed utility processes so it's more equivalent to what the user experiences.
Review URL: http://codereview.chromium.org/402074
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32446 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/utility_process_host.cc')
| -rw-r--r-- | chrome/browser/utility_process_host.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/chrome/browser/utility_process_host.cc b/chrome/browser/utility_process_host.cc index 1ba7068..35bc7f0 100644 --- a/chrome/browser/utility_process_host.cc +++ b/chrome/browser/utility_process_host.cc @@ -104,10 +104,7 @@ bool UtilityProcessHost::StartProcess(const FilePath& exposed_dir) { base::ProcessHandle process; #if defined(OS_WIN) - if (!UseSandbox()) { - // Don't use the sandbox during unit tests. - base::LaunchApp(cmd_line, false, false, &process); - } else if (exposed_dir.empty()) { + if (exposed_dir.empty()) { process = sandbox::StartProcess(&cmd_line); } else { process = sandbox::StartProcessWithAccess(&cmd_line, exposed_dir); |