Optimize the processing of test options.

tests/standalone/test_config.dart

 // Copyright (c) 2011, 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.
 
 #library("standalone_test_config");
 
 #import("../../tools/testing/dart/test_runner.dart");
 #import("../../tools/testing/dart/status_file_parser.dart");
 
 class StandaloneTestSuite {
@@ skipping 77 matching lines @@
                             shellPath,
                             options,
                             configuration["timeout"],
                             completeHandler,
                             expectations));
       }        
     }
   }
 
   Map testOptions(String filename) {
+    // Since '.*' does not match a newline these RegExps can be used
+    // on the entire contents of files instead of individual lines.
     RegExp testOptionsRegExp = const RegExp(@"// VMOptions=(.*)");
     RegExp dartOptionsRegExp = const RegExp(@"// DartOptions=(.*)");
+
+    // Read the entire file into a byte buffer and transform it to a
+    // String. This will treat the file as ascii but the only parts
+    // we are interested in will be ascii in any case.
     File file = new File(filename);
-    FileInputStream fileStream = file.openInputStream();
-    StringInputStream lines = new StringInputStream(fileStream);
+    file.openSync();
+    List chars = new List(file.lengthSync());
+    var offset = 0;
+    while (offset != chars.length) {
+      offset += file.readListSync(chars, offset, chars.length - offset);
+    }
+    file.closeSync();
+    String contents = new String.fromCharCodes(chars);
+    chars = null;
 
+    // Find the options in the file.
     List<List> result = new List<List>();
     List<String> dartOptions;
-    String line;
-    while ((line = lines.readLine()) != null) {
-      Match match = testOptionsRegExp.firstMatch(line);
-      if (match != null) {
-        result.add(match[1].split(' ').filter((e) => e != ''));
+    bool isNegative = false;
+
+    Iterable<Match> matches = testOptionsRegExp.allMatches(contents);
+    for (var match in matches) {
+      result.add(match[1].split(' ').filter((e) => e != ''));
+    }
+
+    matches = dartOptionsRegExp.allMatches(contents);
+    for (var match in matches) {
+      if (dartOptions != null) {
+        throw new Exception(
+            'More than one "// DartOptions=" line in test $filename');
       }
+      dartOptions = match[1].split(' ').filter((e) => e != '');
+    }
 
-      match = dartOptionsRegExp.firstMatch(line);
-      if (match != null) {
-        if (dartOptions != null) {
-          throw new Exception(
-              'More than one "// DartOptions=" line in test $filename');
-        }
-        dartOptions = match[1].split(' ').filter((e) => e != '');
-      }
-    }
-    return {"vmOptions": result, "dartOptions": dartOptions};
+    return { "vmOptions": result, "dartOptions": dartOptions };
   }
   
   void completeHandler(TestCase testCase) {
   }
 }