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

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

Issue 9669019: Revert "Revert the revert." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: 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 4bb24ece6c71ae0e33dbf427dd03764724b11ad0..b69f78a8d036c5d49f9e2dd8bf9e30dcf1415aee 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -329,7 +329,6 @@ class StandardTestSuite implements TestSuite {
case 'dartium':
case 'chromium':
case 'frogium':
- case 'frogsh':
case 'legium':
case 'webdriver':
enqueueBrowserTest(filename, testName, optionsFromFile,
@@ -584,13 +583,6 @@ 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':
@@ -683,12 +675,11 @@ class StandardTestSuite implements TestSuite {
return 'application/dart';
case 'chromium':
case 'frogium':
- case 'frogsh':
case 'legium':
case 'webdriver':
return 'text/javascript';
default:
- Expect.fail("Unexpected component '{configuration['component']}'");
+ Expect.fail('Unimplemented component scriptType');
return null;
}
}
@@ -1133,9 +1124,6 @@ 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