From 7923c790ae1fe65b26cc08bbaf043c28d47e3cc7 Mon Sep 17 00:00:00 2001 From: "phajdan.jr@chromium.org" Date: Fri, 6 Aug 2010 20:38:23 +0000 Subject: GTTF: Replace DCHECK with EXPECT_TRUE in UITestBase. We should avoid crashing entire ui_tests in case of failure. With EXPECT_TRUE we still see it, but tests continue to run. TBR=huanr Review URL: http://codereview.chromium.org/2868116 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55279 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/test/ui/ui_test.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'chrome/test') diff --git a/chrome/test/ui/ui_test.cc b/chrome/test/ui/ui_test.cc index a80718f..59f25bb 100644 --- a/chrome/test/ui/ui_test.cc +++ b/chrome/test/ui/ui_test.cc @@ -890,7 +890,7 @@ bool UITestBase::CloseBrowser(BrowserProxy* browser, if (*application_closed) { // Let's wait until the process dies (if it is not gone already). bool success = base::WaitForSingleProcess(process_, base::kNoTimeout); - DCHECK(success); + EXPECT_TRUE(success); } return result; -- cgit v1.1