diff options
author | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-07 22:58:45 +0000 |
---|---|---|
committer | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-07 22:58:45 +0000 |
commit | 784e49c439ec14101f1d41fc2504a44211f959e4 (patch) | |
tree | fb95fdf399acbfa52d8e161920f79d9739a10157 /chrome/browser/errorpage_uitest.cc | |
parent | da90173fabbe17da8377e0819cb50e61d4f7ad7c (diff) | |
download | chromium_src-784e49c439ec14101f1d41fc2504a44211f959e4.zip chromium_src-784e49c439ec14101f1d41fc2504a44211f959e4.tar.gz chromium_src-784e49c439ec14101f1d41fc2504a44211f959e4.tar.bz2 |
Coverity: Check return of TabProxy::GoForward, GoBack, etc
CID=5709,5708,5706,5705,5704,5703,5702,5701,5700
BUG=none
TEST=ran the tests
Review URL: http://codereview.chromium.org/256067
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28340 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/errorpage_uitest.cc')
-rw-r--r-- | chrome/browser/errorpage_uitest.cc | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/chrome/browser/errorpage_uitest.cc b/chrome/browser/errorpage_uitest.cc index 1dcae04..3ee8fec 100644 --- a/chrome/browser/errorpage_uitest.cc +++ b/chrome/browser/errorpage_uitest.cc @@ -56,7 +56,7 @@ TEST_F(ErrorPageTest, DNSError_GoBack1) { NavigateToURLBlockUntilNavigationsComplete(test_url, 2); EXPECT_TRUE(WaitForTitleMatching(L"Mock Link Doctor")); - GetActiveTab()->GoBack(); + EXPECT_TRUE(GetActiveTab()->GoBack()); EXPECT_TRUE(WaitForTitleMatching(L"Title Of Awesomeness")); } @@ -80,9 +80,9 @@ TEST_F(ErrorPageTest, DNSError_GoBack2) { // The first navigation should fail, and the second one should be the error // page. - GetActiveTab()->GoBackBlockUntilNavigationsComplete(2); + EXPECT_TRUE(GetActiveTab()->GoBackBlockUntilNavigationsComplete(2)); EXPECT_TRUE(WaitForTitleMatching(L"Mock Link Doctor")); - GetActiveTab()->GoBack(); + EXPECT_TRUE(GetActiveTab()->GoBack()); EXPECT_TRUE(WaitForTitleMatching(L"Title Of Awesomeness")); } @@ -107,12 +107,12 @@ TEST_F(ErrorPageTest, DNSError_GoBack2AndForward) { // The first navigation should fail, and the second one should be the error // page. - GetActiveTab()->GoBackBlockUntilNavigationsComplete(2); + EXPECT_TRUE(GetActiveTab()->GoBackBlockUntilNavigationsComplete(2)); EXPECT_TRUE(WaitForTitleMatching(L"Mock Link Doctor")); - GetActiveTab()->GoBack(); + EXPECT_TRUE(GetActiveTab()->GoBack()); // The first navigation should fail, and the second one should be the error // page. - GetActiveTab()->GoForwardBlockUntilNavigationsComplete(2); + EXPECT_TRUE(GetActiveTab()->GoForwardBlockUntilNavigationsComplete(2)); EXPECT_TRUE(WaitForTitleMatching(L"Mock Link Doctor")); } @@ -137,14 +137,14 @@ TEST_F(ErrorPageTest, DNSError_GoBack2Forward2) { // The first navigation should fail, and the second one should be the error // page. - GetActiveTab()->GoBackBlockUntilNavigationsComplete(2); + EXPECT_TRUE(GetActiveTab()->GoBackBlockUntilNavigationsComplete(2)); EXPECT_TRUE(WaitForTitleMatching(L"Mock Link Doctor")); - GetActiveTab()->GoBack(); + EXPECT_TRUE(GetActiveTab()->GoBack()); // The first navigation should fail, and the second one should be the error // page. - GetActiveTab()->GoForwardBlockUntilNavigationsComplete(2); + EXPECT_TRUE(GetActiveTab()->GoForwardBlockUntilNavigationsComplete(2)); EXPECT_TRUE(WaitForTitleMatching(L"Mock Link Doctor")); - GetActiveTab()->GoForward(); + EXPECT_TRUE(GetActiveTab()->GoForward()); EXPECT_TRUE(WaitForTitleMatching(L"Title Of Awesomeness")); } @@ -171,7 +171,7 @@ TEST_F(ErrorPageTest, IFrameDNSError_GoBack) { NavigateToURL(URLRequestMockHTTPJob::GetMockUrl(L"title2.html")); NavigateToURL(URLRequestMockHTTPJob::GetMockUrl(L"iframe_dns_error.html")); - GetActiveTab()->GoBack(); + EXPECT_TRUE(GetActiveTab()->GoBack()); EXPECT_TRUE(WaitForTitleMatching(L"Title Of Awesomeness")); } @@ -188,8 +188,8 @@ TEST_F(ErrorPageTest, IFrameDNSError_GoBackAndForward) { NavigateToURL(URLRequestMockHTTPJob::GetMockUrl(L"title2.html")); NavigateToURL(URLRequestMockHTTPJob::GetMockUrl(L"iframe_dns_error.html")); - GetActiveTab()->GoBack(); - GetActiveTab()->GoForward(); + EXPECT_TRUE(GetActiveTab()->GoBack()); + EXPECT_TRUE(GetActiveTab()->GoForward()); EXPECT_TRUE(WaitForTitleMatching(L"Blah")); } @@ -241,7 +241,7 @@ TEST_F(ErrorPageTest, Page404_GoBack) { URLRequestMockHTTPJob::GetMockUrl(L"page404.html"), 2); EXPECT_TRUE(WaitForTitleMatching(L"Mock Link Doctor")); - GetActiveTab()->GoBack(); + EXPECT_TRUE(GetActiveTab()->GoBack()); EXPECT_TRUE(WaitForTitleMatching(L"Title Of Awesomeness")); } |