| Index: base/test/launcher/test_launcher.cc
|
| diff --git a/base/test/launcher/test_launcher.cc b/base/test/launcher/test_launcher.cc
|
| index 8d62afac53a26f9f529c3880a2bb329e95b4478d..c5348eb3f08e3b5ff01d8d306f639ca9ec4cf841 100644
|
| --- a/base/test/launcher/test_launcher.cc
|
| +++ b/base/test/launcher/test_launcher.cc
|
| @@ -461,9 +461,6 @@ void TestLauncher::OnTestFinished(const TestResult& result) {
|
| tests_to_retry_.insert(result.full_name);
|
| }
|
|
|
| - // TODO(phajdan.jr): Flag unreliable unknown results, unless passed on retry.
|
| - // See http://crbug.com/381733 .
|
| -
|
| results_tracker_.AddTestResult(result);
|
|
|
| // TODO(phajdan.jr): Align counter (padding).
|
| @@ -881,6 +878,11 @@ void TestLauncher::OnLaunchTestProcessFinished(
|
| }
|
|
|
| void TestLauncher::OnTestIterationFinished() {
|
| + TestResultsTracker::TestStatusMap tests_by_status(
|
| + results_tracker_.GetTestStatusMapForCurrentIteration());
|
| + if (!tests_by_status[TestResult::TEST_UNKNOWN].empty())
|
| + results_tracker_.AddGlobalTag(kUnreliableResultsTag);
|
| +
|
| // When we retry tests, success is determined by having nothing more
|
| // to retry (everything eventually passed), as opposed to having
|
| // no failures at all.
|
|
|