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

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

Issue 9610002: remove node.js dependency by running the frog compiler in (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: no change Created 8 years, 9 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 | « tools/testing/dart/test_options.dart ('k') | tools/testing/frogpad/frogpad.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/test_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 801cd576812f5998d456c7ec0487288e31a60137..244e60920f8eb0ec56fc92285898147a5c754c22 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -330,6 +330,7 @@ class StandardTestSuite implements TestSuite {
case 'dartium':
case 'chromium':
case 'frogium':
+ case 'frogsh':
case 'legium':
case 'webdriver':
enqueueBrowserTest(filename, testName, optionsFromFile,
@@ -584,6 +585,13 @@ class StandardTestSuite implements TestSuite {
args.add(inputFile);
// TODO(whesse): Add --fatal-type-errors if needed.
break;
+ case 'frogsh':
+ args.clear();
+ args.add('--out=$outputFile');
+ args.add('--frogpad_js=' +
+ '${TestUtils.buildDir(configuration)}/frog/bin/frogpad.js');
+ args.add(inputFile);
+ break;
case 'frogium':
case 'legium':
case 'webdriver':
@@ -676,11 +684,12 @@ class StandardTestSuite implements TestSuite {
return 'application/dart';
case 'chromium':
case 'frogium':
+ case 'frogsh':
case 'legium':
case 'webdriver':
return 'text/javascript';
default:
- Expect.fail('Unimplemented component scriptType');
+ Expect.fail("Unexpected component '{configuration['component']}'");
return null;
}
}
@@ -1124,6 +1133,9 @@ class TestUtils {
return null; // No separate compiler for dartium tests.
}
var name = configuration['frog'];
+ if (configuration['component'] == 'frogsh') {
+ name = '${dartDir()}/tools/testing/frogpad/frogpad.py';
+ }
if (name == '') {
name = '${buildDir(configuration)}/${compilerName(configuration)}';
}
« no previous file with comments | « tools/testing/dart/test_options.dart ('k') | tools/testing/frogpad/frogpad.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698