diff options
| author | cpu@google.com <cpu@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-01 19:43:53 +0000 |
|---|---|---|
| committer | cpu@google.com <cpu@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-01 19:43:53 +0000 |
| commit | d5037e2c63dd298a2907459538684b0d1cdb065d (patch) | |
| tree | a3de59fc6f57f5f54895ba6850b209959f1cd3a4 /chrome | |
| parent | 6f4747945c7563275faaef7a8a3f80dece9358f9 (diff) | |
| download | chromium_src-d5037e2c63dd298a2907459538684b0d1cdb065d.zip chromium_src-d5037e2c63dd298a2907459538684b0d1cdb065d.tar.gz chromium_src-d5037e2c63dd298a2907459538684b0d1cdb065d.tar.bz2 | |
Coverity underground warfare: check return code
- Check return code consistently
- CID 4171
BUG = none
TEST = non
Review URL: http://codereview.chromium.org/118041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17344 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/browser/browser_uitest.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/chrome/browser/browser_uitest.cc b/chrome/browser/browser_uitest.cc index 3b2437a..adeea1f 100644 --- a/chrome/browser/browser_uitest.cc +++ b/chrome/browser/browser_uitest.cc @@ -219,7 +219,7 @@ TEST_F(BrowserTest, NullOpenerRedirectForksProcess) { // Make sure that a new tab has been created and that we have a new renderer // process for it. - tab->NavigateToURLAsync(fork_url); + ASSERT_TRUE(tab->NavigateToURLAsync(fork_url)); PlatformThread::Sleep(action_timeout_ms()); ASSERT_EQ(orig_process_count + 1, GetBrowserProcessCount()); int new_tab_count = -1; @@ -260,7 +260,7 @@ TEST_F(BrowserTest, OtherRedirectsDontForkProcess) { "w.document.location=\"http://localhost:1337\";})()"); // Make sure that a new tab but not new process has been created. - tab->NavigateToURLAsync(dont_fork_url); + ASSERT_TRUE(tab->NavigateToURLAsync(dont_fork_url)); PlatformThread::Sleep(action_timeout_ms()); ASSERT_EQ(orig_process_count, GetBrowserProcessCount()); int new_tab_count = -1; @@ -272,7 +272,7 @@ TEST_F(BrowserTest, OtherRedirectsDontForkProcess) { "document.location=\"http://localhost:1337\";})()"); // Make sure that no new process has been created. - tab->NavigateToURLAsync(dont_fork_url2); + ASSERT_TRUE(tab->NavigateToURLAsync(dont_fork_url2)); PlatformThread::Sleep(action_timeout_ms()); ASSERT_EQ(orig_process_count, GetBrowserProcessCount()); } |