Make browser tests all run from a server instead of the local filesystem.

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:async";
 // We need to use the 'io' prefix here, otherwise io.exitCode will shadow
 // CommandOutput.exitCode in subclasses of CommandOutput.
 import "dart:io" as io;
 import "dart:isolate";
 import "dart:uri";
 import "status_file_parser.dart";
 import "test_progress.dart";
 import "test_suite.dart";
+import "http_server.dart" as http_server;
 
 const int NO_TIMEOUT = 0;
 const int SLOW_TIMEOUT_MULTIPLIER = 4;
 
 typedef void TestCaseEvent(TestCase testCase);
 typedef void ExitCodeEvent(int exitCode);
 typedef void EnqueueMoreWork(ProcessQueue queue);
 
 
 /**
@@ skipping 139 matching lines @@
    * with the content of [expectedOutputPath].
    * This is used for example for pixel tests, where [expectedOutputPath] points
    * to a *png file.
    */
   io.Path expectedOutputPath;
 
   DumpRenderTreeCommand(String executable,
                         String htmlFile,
                         List<String> options,
                         List<String> dartFlags,
-                        Uri packageRootUri,
                         io.Path this.expectedOutputPath)
       : super(executable,
               _getArguments(options, htmlFile),
-              _getEnvironment(dartFlags, packageRootUri));
+              _getEnvironment(dartFlags));
 
-  static Map _getEnvironment(List<String> dartFlags, Uri packageRootUri) {
+  static Map _getEnvironment(List<String> dartFlags) {
     var needDartFlags = dartFlags != null && dartFlags.length > 0;
-    var needDartPackageRoot = packageRootUri != null;
 
     var env = null;
-    if (needDartFlags || needDartPackageRoot) {
+    if (needDartFlags) {
       env = new Map.from(io.Platform.environment);
       if (needDartFlags) {
         env['DART_FLAGS'] = Strings.join(dartFlags, " ");
       }
-      if (needDartPackageRoot) {
-        env['DART_PACKAGE_ROOT'] = packageRootUri.toString();
-      }
     }
 
     return env;
   }
 
   static List<String> _getArguments(List<String> options, String htmlFile) {
     var arguments = new List.from(options);
     arguments.add(htmlFile);
     return arguments;
   }
@@ skipping 1493 matching lines @@
       if (test.usesWebDriver && _needsSelenium && !_isSeleniumAvailable || (test
           is BrowserTestCase && test.waitingForOtherTest)) {
         // The test is not yet ready to run. Put the test back in
         // the queue.  Avoid spin-polling by using a timeout.
         _tests.add(test);
         new Timer(100, (timer) {_tryRunTest();});  // Don't lose a process.
         return;
       }
       if (_verbose) {
         int i = 1;
+        if (test is BrowserTestCase) {
+          // Additional command for rerunning the steps locally after the fact.
+          print('$i. ${TestUtils.dartTestExecutable.toNativePath()} '
+              '${TestUtils.dartDir().toNativePath()}/tools/testing/dart/'
+              'http_server.dart -m ${test.configuration["mode"]} '
+              '-a ${test.configuration["arch"]} '
+              '-p ${http_server.TestingServerRunner.serverList[0].port} '
+              '-c ${http_server.TestingServerRunner.serverList[1].port}');
+          i++;
+        }
         for (Command command in test.commands) {
           print('$i. ${command.commandLine}');
           i++;
         }
       }
       _progress.start(test);
       TestCaseEvent oldCallback = test.completedHandler;
       void wrapper(TestCase test_arg) {
         _numProcesses--;
         _progress.done(test_arg);
@@ skipping 18 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++;
     }
   }
 }