Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Unified Diff: utils/testrunner/testrunner.dart

Issue 46673003: Remove uses of Options from utils directory. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « utils/testrunner/options.dart ('k') | utils/testrunner/utils.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/testrunner/testrunner.dart
diff --git a/utils/testrunner/testrunner.dart b/utils/testrunner/testrunner.dart
index 0292b2c5f7b7abf3f4204af530a656f83b9c3e15..a700de4835f5786e2573198449dbfa2ec7326703 100755
--- a/utils/testrunner/testrunner.dart
+++ b/utils/testrunner/testrunner.dart
@@ -339,8 +339,7 @@ void sanitizeConfig(Map config, ArgParser parser) {
var pathSep = Platform.pathSeparator;
if (dartsdk == null) {
- var opt = new Options();
- var runner = opt.executable;
+ var runner = Platform.executable;
var idx = runner.indexOf('dart-sdk');
if (idx < 0) {
print("Please use --dartsdk option or run using the dart executable "
@@ -354,7 +353,7 @@ void sanitizeConfig(Map config, ArgParser parser) {
'$dartsdk${pathSep}dart-sdk${pathSep}bin${pathSep}dart2js';
config['dart'] = '$dartsdk${pathSep}dart-sdk${pathSep}bin${pathSep}dart';
config['pub'] = '$dartsdk${pathSep}dart-sdk${pathSep}bin${pathSep}pub';
- config['drt'] =
+ config['drt'] =
'$dartsdk/chromium/Content Shell.app/Contents/MacOS/Content Shell';
} else if (Platform.operatingSystem == 'linux') {
config['dart2js'] =
@@ -378,9 +377,9 @@ void sanitizeConfig(Map config, ArgParser parser) {
config['exclude'] = normalizeFilter(config['exclude']);
}
-main() {
+main(List<String> arguments) {
var optionsParser = getOptionParser();
- var options = loadConfiguration(optionsParser);
+ var options = loadConfiguration(optionsParser, arguments);
if (isSane(options)) {
if (options['list-options']) {
printOptions(optionsParser, options, false, stdout);
« no previous file with comments | « utils/testrunner/options.dart ('k') | utils/testrunner/utils.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698