Add support for passing in vm options to the testing script.

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 1449 matching lines @@
              "numStaticTypeAnnotations": numStaticTypeAnnotations,
              "numCompileTimeAnnotations": numCompileTimeAnnotations };
   }
 
   List<List<String>> getVmOptions(Map optionsFromFile) {
     var COMPILERS = const ['none', 'dart2dart'];
     var RUNTIMES = const ['none', 'vm', 'drt', 'dartium'];
     var needsVmOptions = COMPILERS.contains(configuration['compiler']) &&
                          RUNTIMES.contains(configuration['runtime']);
     if (!needsVmOptions) return [[]];
-    final vmOptions = optionsFromFile['vmOptions'];
+    var vmOptions = new List.from(optionsFromFile['vmOptions']);
+    // Add general vm options that is given to the testing script.

[kustermann, 2013/07/23 13:55:18]

"that is given" -> "that were given"

[ricow1, 2013/07/24 11:32:50]

Done.

+    if (configuration['vm-options'] != null) {
+      for (var list in vmOptions) {
+        list.addAll(configuration['vm-options'].split(" "));

[kustermann, 2013/07/23 13:55:18]

Make sure we don't add empty options (i.e. consider ./test.py --vm-options="a  b
c").

[ricow1, 2013/07/24 11:32:50]

Done.

+      }
+    }

[kustermann, 2013/07/23 13:55:18]

This change is not going to work. I think this function "getVmOptions()" will be
called multiple times with the same argument. So we could end up adding the
options multiple times.

[ricow1, 2013/07/24 11:32:50]

Done.

     if (configuration['compiler'] != 'dart2dart') return vmOptions;
     // Temporary workaround for race in test suite: tests with different
     // vm options are still compiled into the same output file which
     // may lead to reads from empty files.
     return [vmOptions[0]];
   }
 
   /**
    * Read options from a co19 test file.
    *
@@ skipping 485 matching lines @@
  * $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
  * $compileErrorSkip tests are skipped on browsers due to compile-time error
 """;
     print(report);
    }
 }