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

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

Issue 219053003: Deprecate automatic generation of precompiled.js. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Updated cf. comments. Created 6 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
Index: tools/testing/dart/compiler_configuration.dart
diff --git a/tools/testing/dart/compiler_configuration.dart b/tools/testing/dart/compiler_configuration.dart
index 891a33bcf4b8e22fb38755613b631fa55ac126cf..9c7f91b5199e1de093f8d7ae9f346bc7d5d5fc56 100644
--- a/tools/testing/dart/compiler_configuration.dart
+++ b/tools/testing/dart/compiler_configuration.dart
@@ -248,21 +248,15 @@ class Dart2jsCompilerConfiguration extends Dart2xCompilerConfiguration {
CommandBuilder commandBuilder,
List arguments,
Map<String, String> environmentOverrides) {
- String normalOutput = '$tempDir/out.js';
- String cspOutput = '$tempDir/out.precompiled.js';
return new CommandArtifact(
<Command>[
this.computeCompilationCommand(
- normalOutput,
+ '$tempDir/out.js',
buildDir,
CommandBuilder.instance,
arguments,
environmentOverrides)],
- // dart2js always produce both out.js and out.precompiled.js. To avoid
- // recompiling the CSP version, we always tell the CompilationCommand
- // to track the timestamp of out.js, but select which one to run based
- // on CSP mode.
- isCsp ? cspOutput : normalOutput,
+ '$tempDir/out.js',
'application/javascript');
}
« no previous file with comments | « sdk/lib/_internal/compiler/implementation/js_emitter/code_emitter_task.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698