diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-24 16:39:13 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-24 16:39:13 +0000 |
| commit | 661452b3f486a43f9b102f46aa6d0a097361718b (patch) | |
| tree | c74f69ede1ef994246accaa4ad823dc6ccbbcd43 /chrome/test/browser | |
| parent | 8beca054430fa0ee3f92065530b3ea919425d3a7 (diff) | |
| download | chromium_src-661452b3f486a43f9b102f46aa6d0a097361718b.zip chromium_src-661452b3f486a43f9b102f46aa6d0a097361718b.tar.gz chromium_src-661452b3f486a43f9b102f46aa6d0a097361718b.tar.bz2 | |
The browser test runner was returning the wrong value on success/failure.
BUG=None
TEST=browser_tests.exe should return 0 on success, 1 on failure.
Review URL: http://codereview.chromium.org/147060
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19128 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/browser')
| -rw-r--r-- | chrome/test/browser/browser_test_runner.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/test/browser/browser_test_runner.cc b/chrome/test/browser/browser_test_runner.cc index fe633fe..b508d12 100644 --- a/chrome/test/browser/browser_test_runner.cc +++ b/chrome/test/browser/browser_test_runner.cc @@ -112,7 +112,7 @@ bool RunTests(const BrowserTestRunnerFactory& browser_test_runner_factory) { printf("%d test%s failed\n", static_cast<int>(failed_tests.size()), failed_tests.size() > 1 ? "s" : ""); if (failed_tests.empty()) - return false; + return true; printf("Failing tests:\n"); for (std::vector<std::string>::const_iterator iter = failed_tests.begin(); @@ -120,7 +120,7 @@ bool RunTests(const BrowserTestRunnerFactory& browser_test_runner_factory) { printf("%s\n", iter->c_str()); } - return true; + return false; } } // namespace |