Add browser path options to testing scripts.

tools/testing/dart/test_options.dart

 // Copyright (c) 2013, 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 test_options_parser;
 
 import "dart:io";
 import "dart:math";
 import "drt_updater.dart";
 import "test_suite.dart";
@@ skipping 225 matching lines @@
               ['--drt'],
               [],
               ''),
           new _TestOptionSpecification(
               'dartium',
               'Path to Dartium Chrome executable',
               ['--dartium'],
               [],
               ''),
           new _TestOptionSpecification(
+              'firefox',
+              'Path to firefox browser executable',
+              ['--firefox'],
+              [],
+              ''),
+          new _TestOptionSpecification(
+              'chrome',
+              'Path to chrome browser executable',
+              ['--chrome'],
+              [],
+              ''),
+          new _TestOptionSpecification(
+              'safari',
+              'Path to safari browser executable',
+              ['--safari'],
+              [],
+              ''),
+          new _TestOptionSpecification(
               'use_sdk',
               '''Use compiler or runtime from the SDK.
 
 Normally, the compiler or runtimes in PRODUCT_DIR is tested, with this
 option, the compiler or runtime in PRODUCT_DIR/dart-sdk/bin is tested.
 
 Note: currently only implemented for dart2js.''',
               ['--use-sdk'],
               [],
               false,
@@ skipping 221 matching lines @@
      config['_reproducing_arguments_'] =
          _constructReproducingCommandArguments(config);
     }
     return result.isEmpty ? null : result;
   }
 
   // For printing out reproducing command lines, we don't want to add these
   // options.
   Set<String>  _blacklistedOptions = new Set<String>.from([
     'progress', 'failure-summary', 'step_name', 'report', 'tasks', 'verbose',
-    'time', 'dart', 'drt', 'dartium', 'build_directory', 'append_logs',
-    'local_ip', 'shard', 'shards',
+    'time', 'dart', 'drt', 'dartium', 'firefox', 'build_directory',

[Bill Hesse, 2013/11/05 18:21:39]

Add the two other ones here.

+    'append_logs', 'local_ip', 'shard', 'shards',
   ]);
 
   List<String> _constructReproducingCommandArguments(Map config) {
     var arguments = new List<String>();
     for (var configKey in config.keys) {
       if (!_blacklistedOptions.contains(configKey)) {
         for (var option in _options) {
           var configValue = config[configKey];
           // We only include entries of [conf] if we find an option for it.
           if (configKey == option.name && configValue != option.defaultValue) {
@@ skipping 281 matching lines @@
         return option;
       }
     }
     print('Unknown test option $name');
     exit(1);
   }
 
 
   List<_TestOptionSpecification> _options;
 }