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

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

Issue 1902073003: VM: Remove redundant build target dart_product. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 8 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 | « runtime/tests/vm/vm.status ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/runtime_configuration.dart
diff --git a/tools/testing/dart/runtime_configuration.dart b/tools/testing/dart/runtime_configuration.dart
index 4edacc3acd712a7fbcdb377fa8b9e1536ff4e11a..f4d3b620674e06b9f424460078f42e7ec6a4692b 100644
--- a/tools/testing/dart/runtime_configuration.dart
+++ b/tools/testing/dart/runtime_configuration.dart
@@ -48,9 +48,6 @@ class RuntimeConfiguration {
case 'vm':
return new StandaloneDartRuntimeConfiguration();
- case 'dart_product':
- return new DartProductRuntimeConfiguration();
-
case 'dart_precompiled':
return new DartPrecompiledRuntimeConfiguration();
@@ -218,30 +215,6 @@ class StandaloneDartRuntimeConfiguration extends DartVmRuntimeConfiguration {
}
}
-class DartProductRuntimeConfiguration extends DartVmRuntimeConfiguration {
- List<Command> computeRuntimeCommands(
- TestSuite suite,
- CommandBuilder commandBuilder,
- CommandArtifact artifact,
- List<String> arguments,
- Map<String, String> environmentOverrides) {
- String script = artifact.filename;
- String type = artifact.mimeType;
- if (script != null && type != 'application/dart-snapshot') {
- throw "dart_product cannot run files of type '$type'.";
- }
-
- var augmentedArgs = new List();
- augmentedArgs.add("--run-full-snapshot=${artifact.filename}");
Cutch 2016/04/19 19:19:28 In product mode the test harness builds a full sna
Florian Schneider 2016/04/19 20:12:42 Oops. Yes, restored with using 'dart' as a binary.
- augmentedArgs.addAll(arguments);
-
- return <Command>[
- commandBuilder.getVmCommand(suite.dartVmProductBinaryFileName,
- augmentedArgs, environmentOverrides)
- ];
- }
-}
-
class DartPrecompiledRuntimeConfiguration extends DartVmRuntimeConfiguration {
List<Command> computeRuntimeCommands(
TestSuite suite,
« no previous file with comments | « runtime/tests/vm/vm.status ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698