don't retry if the browser is dumprendertree?

tools/testing/dart/test_runner.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * Classes and methods for executing tests.
  *
  * This module includes:
  * - Managing parallel execution of tests, including timeout checks.
  * - Evaluating the output of each test as pass/fail/crash/timeout.
@@ skipping 145 matching lines @@
  * [TestCase] this is the output of.
  */
 class TestOutput {
   TestCase testCase;
   int exitCode;
   bool timedOut;
   bool failed = false;
   List<String> stdout;
   List<String> stderr;
   Duration time;
-  /** 
-   * Set to true if we encounter a condition in the output that indicates we 
-   * need to rerun this test. 
+  /**
+   * Set to true if we encounter a condition in the output that indicates we
+   * need to rerun this test.
    */
   bool requestRetry;
 
   TestOutput(this.testCase, this.exitCode, this.timedOut, this.stdout,
              this.stderr, this.time) {
     testCase.output = this;
     requestRetry = false;
   }
 
   String get result() =>
@@ skipping 73 matching lines @@
   void exitHandler(int exitCode) {
     new TestOutput(testCase, exitCode, timedOut, stdout,
                    stderr, new Date.now().difference(startTime));
     process.close();
     timeoutTimer.cancel();
     if (testCase.output.unexpectedOutput && testCase.configuration['verbose']) {
       print(testCase.displayName);
       for (var line in testCase.output.stderr) print(line);
       for (var line in testCase.output.stdout) print(line);
     }
-    if (testCase is BrowserTestCase && testCase.output.unexpectedOutput &&
-        testCase.numRetries > 0) {
+    if (testCase.configuration['component'] == 'webdriver' &&
+        testCase.output.unexpectedOutput && testCase.numRetries > 0) {
       // Selenium tests can be flaky. Try rerunning.
       testCase.output.requestRetry = true;
     }
     if (testCase.output.requestRetry) {
       testCase.output.requestRetry = false;
       this.timedOut = false;
       testCase.dynamic.numRetries--;
       print("Potential flake. Re-running " + testCase.displayName);
       this.start();
     } else {
@@ skipping 231 matching lines @@
   Queue<TestCase> _tests;
   ProgressIndicator _progress;
   String _temporaryDirectory;
   // For dartc batch processing we keep a list of batch processes.
   List<DartcBatchRunnerProcess> _batchProcesses;
   // Cache information about test cases per test suite. For multiple
   // configurations there is no need to repeatedly search the file
   // system, generate tests, and search test files for options.
   Map<String, List<TestInformation>> _testCache;
   /**
-   * String indicating the browser used to run the tests. Empty if no browser 
+   * String indicating the browser used to run the tests. Empty if no browser
    * used.
    */
   String browserUsed;
 
   ProcessQueue(int this._maxProcesses,
                String progress,
                Date startTime,
                bool printTiming,
                Function this._enqueueMoreWork,
                [bool this._verbose = false,
@@ skipping 30 matching lines @@
       throw new Exception(
           'Test suite requires temporary directory. Not supported on Windows.');
     }
     var tempDir = new Directory('');
     tempDir.createTempSync();
     _temporaryDirectory = tempDir.path;
     return _temporaryDirectory;
   }
 
   /**
-   * Sometimes Webdriver doesn't close every browser window when it's done 
+   * Sometimes Webdriver doesn't close every browser window when it's done
    * with a test. At the end of all tests we clear out any neglected processes
-   * that are still running. 
+   * that are still running.
    */
   void killZombieBrowsers() {
     String chromeName = 'chrome';
     if (new Platform().operatingSystem() == 'macos') {
       chromeName = 'Google\ Chrome';
     }
-    Map<String, List<String>> processNames = {'ie': ['iexplore'], 'safari': 
+    Map<String, List<String>> processNames = {'ie': ['iexplore'], 'safari':
         ['Safari'], 'ff': ['firefox'], 'chrome': ['chromedriver', chromeName]};
     for (String name in processNames[browserUsed]) {
       Process process = null;
       if (new Platform().operatingSystem() == 'windows') {
         process = new Process.start(
             'C:\\Windows\\System32\\taskkill.exe', ['/F', '/IM', name + '.exe',
             '/T']);
       } else {
         process = new Process.start('killall', ['-9', name]);
       }
@@ skipping 115 matching lines @@
           test.displayName != 'dartc/junit_tests') {
         _ensureDartcBatchRunnersStarted(test.executablePath);
         _getDartcBatchRunnerProcess().startTest(test);
       } else {
         new RunningProcess(test).start();
       }
       _numProcesses++;
     }
   }
 }