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

Unified Diff: tools/testing/dart/test_options.dart

Issue 10809069: dart2dart compiler support. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 5 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 | « no previous file | tools/testing/dart/test_suite.dart » ('j') | tools/testing/dart/test_suite.dart » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/test_options.dart
diff --git a/tools/testing/dart/test_options.dart b/tools/testing/dart/test_options.dart
index 8c6f8526c5d4f55b482542b2d2dfbff559651a31..a814a870886eede5aa32b7d5aaee72e23cea3016 100644
--- a/tools/testing/dart/test_options.dart
+++ b/tools/testing/dart/test_options.dart
@@ -69,6 +69,9 @@ is 'dart file.dart' and you specify special command
none: Do not compile the Dart code (run native Dart code on the VM).
(only valid with the following runtimes: vm, drt)
+ dart2dart: Compile Dart code to Dart code
+ (only valid with the following runtimes: vm, drt)
+
dart2js: Compile dart code to JavaScript by running dart2js (leg).
(only valid with the following runtimes: same as frog)
@@ -79,7 +82,7 @@ is 'dart file.dart' and you specify special command
frog compiler. (only valid with the following runtimes: d8,
drt, chrome, safari, ie, firefox, opera, none (compile only))''',
['-c', '--compiler'],
- ['none', 'frog', 'dart2js', 'dartc'],
+ ['none', 'dart2dart', 'frog', 'dart2js', 'dartc'],
'none'),
new _TestOptionSpecification(
'runtime',
@@ -403,6 +406,7 @@ Note: currently only implemented for dart2js.''',
isValid = config['runtime'] == 'none';
break;
case 'none':
+ case 'dart2dart':
Bill Hesse 2012/07/24 11:58:59 See below. const ['vm', 'drt', 'dartium'].some(equ
Anton Muhin 2012/07/25 11:19:57 Done.
isValid = (const ['vm', 'drt',
'dartium']).indexOf(config['runtime']) >= 0;
}
@@ -421,7 +425,8 @@ Note: currently only implemented for dart2js.''',
print("Error: shard index is ${config['shard']} out of "
"${config['shards']} shards");
}
- if (config['runtime'] == 'dartium' && config['compiler'] == 'none' &&
+ if (config['runtime'] == 'dartium' &&
Bill Hesse 2012/07/24 11:58:59 Please put a TODO(1030) here, to fix this when Col
+ const ['none', 'dart2dart'].indexOf(config['compiler']) >= 0 &&
config['checked']) {
// TODO(vsm): Set the DART_FLAGS environment appropriately when
// invoking Selenium to support checked mode. It's not clear
« no previous file with comments | « no previous file | tools/testing/dart/test_suite.dart » ('j') | tools/testing/dart/test_suite.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698