diff options
| author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-11 19:54:06 +0000 |
|---|---|---|
| committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-11 19:54:06 +0000 |
| commit | 9b056baffd0aa88a4c6e41b0ca450f56feff78e2 (patch) | |
| tree | cab13a5d271a45d02c66f1a63d61859dfa5ffef6 /chrome/test/test_launcher | |
| parent | 46b55dca90750a2698a4926bd05ae71f9ba027e9 (diff) | |
| download | chromium_src-9b056baffd0aa88a4c6e41b0ca450f56feff78e2.zip chromium_src-9b056baffd0aa88a4c6e41b0ca450f56feff78e2.tar.gz chromium_src-9b056baffd0aa88a4c6e41b0ca450f56feff78e2.tar.bz2 | |
Fix a problem in browser_tests launcher which was interpreting the line
'YOU HAVE x FLAKY TESTS' as a test name.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/543005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35931 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/test_launcher')
| -rw-r--r-- | chrome/test/test_launcher/test_runner.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/chrome/test/test_launcher/test_runner.cc b/chrome/test/test_launcher/test_runner.cc index fa4cc6e..5c3cf25 100644 --- a/chrome/test/test_launcher/test_runner.cc +++ b/chrome/test/test_launcher/test_runner.cc @@ -50,8 +50,10 @@ bool GetTestList(const CommandLine& command_line, for (std::vector<std::string>::const_iterator iter = lines.begin(); iter != lines.end(); ++iter) { std::string line = *iter; - if (line.empty()) - continue; // Just ignore empty lines if any. + + // Ignore empty and metadata lines (like "YOU HAVE x FLAKY TESTS"). + if (line.empty() || line.find(' ') != std::string::npos) + continue; if (line[line.size() - 1] == '.') { // This is a new test case. |