test.dart: fix for dartium.

tools/testing/dart/test_suite.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 enumerating and preparing tests.
  *
  * This library includes:
  *
  * - Creating tests by listing all the Dart files in certain directories,
@@ skipping 494 matching lines @@
       }
       htmlTest.writeStringSync(GetHtmlContents(
           filename,
           '$filePrefix$dartDir/client/testing/unittest/test_controller.js',
           scriptType,
           filePrefix + scriptPath));
       htmlTest.closeSync();
 
       // Construct the command(s) that compile all the inputs needed by the
       // browser test. For dartium, this will be noop commands.
-      List<Command> commands = [_compileCommand(
-          dartWrapperFilename, compiledDartWrapperFilename,
-          component, tempDir.path, vmOptions)];
+      List<Command> commands = [];
+      if (component != 'dartium') {
+        commands.add(_compileCommand(
+            dartWrapperFilename, compiledDartWrapperFilename,
+            component, tempDir.path, vmOptions));
 
-      // some tests require compiling multiple input scripts.
-      List<String> otherScripts = optionsFromFile['otherScripts'];
-      for (String name in otherScripts) {
-        int end = filename.lastIndexOf('/');
-        if (end == -1) {
-          print('Warning: error processing "OtherScripts" of $filename.');
-          print('Skipping test ($testName).');
-          return;
+        // some tests require compiling multiple input scripts.
+        List<String> otherScripts = optionsFromFile['otherScripts'];
+        for (String name in otherScripts) {
+          int end = filename.lastIndexOf('/');
+          if (end == -1) {
+            print('Warning: error processing "OtherScripts" of $filename.');
+            print('Skipping test ($testName).');
+            return;
+          }
+          String dir = filename.substring(0, end);
+          end = name.lastIndexOf('.dart');
+          if (end == -1) {
+            print('Warning: error processing "OtherScripts" in $filename.');
+            print('Skipping test ($testName).');
+            return;
+          }
+          String compiledName = '${name.substring(0, end)}.js';
+          commands.add(_compileCommand(
+              '$dir/$name', '${tempDir.path}/$compiledName',
+              component, tempDir.path, vmOptions));
         }
-        String dir = filename.substring(0, end);
-        end = name.lastIndexOf('.dart');
-        if (end == -1) {
-          print('Warning: error processing "OtherScripts" in $filename.');
-          print('Skipping test ($testName).');
-          return;
-        }
-        String compiledName = '${name.substring(0, end)}.js';
-        commands.add(_compileCommand(
-            '$dir/$name', '${tempDir.path}/$compiledName',
-            component, tempDir.path, vmOptions));
       }
 
       // Construct the command that executes the browser test
       List<String> args;
       if (component == 'webdriver') {
         args = ['$dartDir/tools/testing/run_selenium.py',
             '--browser=${configuration["browser"]}',
             '--timeout=${configuration["timeout"] - 2}',
             '--out=$htmlPath'];
       } else {
@@ skipping 43 matching lines @@
         String libdir = configuration['froglib'];
         if (libdir == '') {
           libdir = '$dartDir/frog/lib';
         }
         args.addAll(['--libdir=$libdir',
                              '--compile-only',
                              '--out=$outputFile']);
         args.addAll(vmOptions);
         args.add(inputFile);
         break;
-      case 'dartium':
-        // No compilation phase.
-        args = null;
-        break;
       default:
         Expect.fail('unimplemented component $component');
     }
     return new Command(executable, args);
   }
 
   bool get requiresCleanTemporaryDirectory() =>
       configuration['component'] == 'dartc' ||
       configuration['component'] == 'chromium';
 
@@ skipping 556 matching lines @@
  * $noCrash tests are expected to be flaky but not crash
  * $pass tests are expected to pass
  * $failOk tests are expected to fail that we won't fix
  * $fail tests are expected to fail that we should fix
  * $crash tests are expected to crash that we should fix
  * $timeout tests are allowed to timeout
 """;
     print(report);
    }
 }