diff options
author | rvargas <rvargas@chromium.org> | 2015-01-12 14:23:23 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-01-12 22:25:04 +0000 |
commit | 07b589c1f6c4d0aac24e270a13fa83031bf2038c (patch) | |
tree | 18a5284042b77bb6845194277e8dea006576b636 /content/common/sandbox_mac_diraccess_unittest.mm | |
parent | d44cce5915db087d726ed2e141896401edeb09b7 (diff) | |
download | chromium_src-07b589c1f6c4d0aac24e270a13fa83031bf2038c.zip chromium_src-07b589c1f6c4d0aac24e270a13fa83031bf2038c.tar.gz chromium_src-07b589c1f6c4d0aac24e270a13fa83031bf2038c.tar.bz2 |
MultiProcessTest: Update SpawnChild* to return a Process.
We should not be dealing with raw handles.
BUG=417532
Review URL: https://codereview.chromium.org/843113003
Cr-Commit-Position: refs/heads/master@{#311127}
Diffstat (limited to 'content/common/sandbox_mac_diraccess_unittest.mm')
-rw-r--r-- | content/common/sandbox_mac_diraccess_unittest.mm | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/content/common/sandbox_mac_diraccess_unittest.mm b/content/common/sandbox_mac_diraccess_unittest.mm index 7a3fed8..a403ecd 100644 --- a/content/common/sandbox_mac_diraccess_unittest.mm +++ b/content/common/sandbox_mac_diraccess_unittest.mm @@ -34,14 +34,14 @@ class MacDirAccessSandboxTest : public base::MultiProcessTest { public: bool CheckSandbox(const std::string& directory_to_try) { setenv(kSandboxAccessPathKey, directory_to_try.c_str(), 1); - base::ProcessHandle child_process = SpawnChild("mac_sandbox_path_access"); - if (child_process == base::kNullProcessHandle) { + base::Process child_process = SpawnChild("mac_sandbox_path_access"); + if (!child_process.IsValid()) { LOG(WARNING) << "SpawnChild failed"; return false; } int code = -1; - if (!base::WaitForExitCode(child_process, &code)) { - LOG(WARNING) << "base::WaitForExitCode failed"; + if (!child_process.WaitForExit(&code)) { + LOG(WARNING) << "Process::WaitForExit failed"; return false; } return code == 0; |