Skip to content

Commit 1c81585

Browse files
committed
Remove test retrying for Safari, in test scripts.
BUG=dartbug.com/21434 [email protected] Review URL: https://codereview.chromium.org//884073003 git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@43503 260f80e4-7a28-3924-810f-c04153c831b5
1 parent 10ba25d commit 1c81585

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

tools/testing/dart/test_runner.dart

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2622,15 +2622,6 @@ class ReplayingCommandExecutor implements CommandExecutor {
26222622

26232623
bool shouldRetryCommand(CommandOutput output) {
26242624
var command = output.command;
2625-
// We rerun tests on Safari because 6.2 and 7.1 are flaky. Issue 21434.
2626-
if (command is BrowserTestCommand &&
2627-
command.retry &&
2628-
command.browser == 'safari' &&
2629-
output is BrowserControllerTestOutcome &&
2630-
output._rawOutcome != Expectation.PASS) {
2631-
return true;
2632-
}
2633-
26342625
if (!output.successful) {
26352626
List<String> stdout, stderr;
26362627

0 commit comments

Comments
 (0)