| Index: tools/testing/dart/test_runner.dart
|
| ===================================================================
|
| --- tools/testing/dart/test_runner.dart (revision 28573)
|
| +++ tools/testing/dart/test_runner.dart (working copy)
|
| @@ -894,6 +894,11 @@
|
| time,
|
| compilationSkipped);
|
|
|
| + bool didFail(TestCase testCase) {
|
| + return _getOutcome() != Expectation.PASS;
|
| + }
|
| +
|
| +
|
| bool get _browserTestFailure {
|
| // We should not need to convert back and forward.
|
| var output = decodeUtf8(super.stdout);
|
| @@ -2116,14 +2121,15 @@
|
| }
|
| }
|
|
|
| - if (command is BrowserTestCommand) {
|
| - // We do not re-run tests on the new browser controller, since it should
|
| - // not be as flaky as selenium.
|
| - return false;
|
| - } else if (command is SeleniumTestCommand) {
|
| - // Selenium tests can be flaky. Try re-running.
|
| + // Selenium tests can be flaky. Try re-running.
|
| + if (command is SeleniumTestCommand) {
|
| return true;
|
| }
|
| +
|
| + // We currently rerun dartium tests, see issue 14074
|
| + if (command is BrowserTestCommand && command.displayName == 'dartium') {
|
| + return true;
|
| + }
|
| }
|
| return false;
|
| }
|
|
|