Add a support for a slow marker in status files.

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.
  */
 #library("test_runner");
 
 #import("dart:io");
 #import("dart:isolate");
 #import("status_file_parser.dart");
 #import("test_progress.dart");
 #import("test_suite.dart");
 
 final int NO_TIMEOUT = 0;
+final int SLOW_TIMEOUT_MULTIPLIER = 4;
 
 /** A command executed as a step in a test case. */
 class Command {
   /** Path to the executable of this command. */
   String executable;
 
   /** Command line arguments to the executable. */
   List<String> arguments;
 
   /** The actual command line that will be executed. */
@@ skipping 86 matching lines @@
         final newCommand = new Command(newExecutablePath, newArguments);
         newCommands.add(newCommand);
         // If there are extra spaces inside the prefix or suffix, this fails.
         Expect.stringEquals('$prefix ${c.commandLine} $suffix'.trim(),
             newCommand.commandLine);
       }
       commands = newCommands;
     }
   }
 
-  int get timeout() => configuration['timeout'];
+  int get timeout() {
+    int timeout = configuration['timeout'];
+    if (expectedOutcomes.contains(SLOW)) {
+      timeout *= SLOW_TIMEOUT_MULTIPLIER;
+    }
+    return timeout;
+  }
 

[Bill Hesse, 2012/08/15 11:41:03]

How about 
  if (expectedOutcomes.contains(SLOW)) {
    return SLOW_TIMEOUT_MULTIPLIER * configuration['timeout'];
} else {
  return configuration['timeout'];
}
or even 
 => expectedOutcomes.contains(SLOW) ?
   ... :
   ... ;

   String get configurationString() {
     final compiler = configuration['compiler'];
     final runtime = configuration['runtime'];
     final mode = configuration['mode'];
     final arch = configuration['arch'];
     return "$compiler-$runtime ${mode}_$arch";
   }
 
   List<String> get batchRunnerArguments() => ['-batch'];
   List<String> get batchTestArguments() => commands.last().arguments;
@@ skipping 1040 matching lines @@
         // the developer doesn't waste his or her time trying to fix a bunch of
         // tests that appear to be broken but were actually just flakes that
         // didn't get retried because there had already been one failure.
         bool allowRetry = _MAX_FAILED_NO_RETRY > _progress.numFailedTests;
         new RunningProcess(test, allowRetry, this).start();
       }
       _numProcesses++;
     }
   }
 }