diff options
| author | rvargas <rvargas@chromium.org> | 2015-02-23 16:28:11 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-02-24 00:29:56 +0000 |
| commit | 2f70a1515d5f3e6ea3525c079743bfbcb0818b98 (patch) | |
| tree | 6ac997a1c4ccbd2d62855a25e7eaec27bed5f7f9 /apps | |
| parent | 9426fb80a0cc758ed9686c48bb7642d23c229cfa (diff) | |
| download | chromium_src-2f70a1515d5f3e6ea3525c079743bfbcb0818b98.zip chromium_src-2f70a1515d5f3e6ea3525c079743bfbcb0818b98.tar.gz chromium_src-2f70a1515d5f3e6ea3525c079743bfbcb0818b98.tar.bz2 | |
Remove base::WaitForSingleProcess
BUG=417532
Review URL: https://codereview.chromium.org/938453002
Cr-Commit-Position: refs/heads/master@{#317690}
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/load_and_launch_browsertest.cc | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/apps/load_and_launch_browsertest.cc b/apps/load_and_launch_browsertest.cc index 9432420..fc95dae 100644 --- a/apps/load_and_launch_browsertest.cc +++ b/apps/load_and_launch_browsertest.cc @@ -62,8 +62,10 @@ IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, ASSERT_TRUE(process.IsValid()); ASSERT_TRUE(launched_listener.WaitUntilSatisfied()); - ASSERT_TRUE(base::WaitForSingleProcess(process.Handle(), - TestTimeouts::action_timeout())); + int exit_code; + ASSERT_TRUE(process.WaitForExitWithTimeout(TestTimeouts::action_timeout(), + &exit_code)); + ASSERT_EQ(0, exit_code); } // TODO(jackhou): Enable this test once it works on OSX. It currently does not @@ -102,8 +104,10 @@ IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, ASSERT_TRUE(process.IsValid()); ASSERT_TRUE(launched_listener.WaitUntilSatisfied()); - ASSERT_TRUE(base::WaitForSingleProcess(process.Handle(), - TestTimeouts::action_timeout())); + int exit_code; + ASSERT_TRUE(process.WaitForExitWithTimeout(TestTimeouts::action_timeout(), + &exit_code)); + ASSERT_EQ(0, exit_code); } namespace { |