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

Side by Side Diff: tools/testing/dart/compiler_configuration.dart

Issue 1308743006: Revert "This is a fixed up version of:" (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 5 years, 3 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 unified diff | Download patch
« no previous file with comments | « tests/standalone/no_assert_test.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library compiler_configuration; 5 library compiler_configuration;
6 6
7 import 'dart:io' show 7 import 'dart:io' show
8 Platform; 8 Platform;
9 9
10 import 'runtime_configuration.dart' show 10 import 'runtime_configuration.dart' show
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 bool get hasCompiler => false; 148 bool get hasCompiler => false;
149 149
150 List<String> computeRuntimeArguments( 150 List<String> computeRuntimeArguments(
151 RuntimeConfiguration runtimeConfiguration, 151 RuntimeConfiguration runtimeConfiguration,
152 String buildDir, 152 String buildDir,
153 TestInformation info, 153 TestInformation info,
154 List<String> vmOptions, 154 List<String> vmOptions,
155 List<String> sharedOptions, 155 List<String> sharedOptions,
156 List<String> originalArguments, 156 List<String> originalArguments,
157 CommandArtifact artifact) { 157 CommandArtifact artifact) {
158 List<String> args = []; 158 return <String>[]
159 if (isChecked) {
160 args.add('--enable_asserts');
161 args.add('--enable_type_checks');
162 }
163 return args
164 ..addAll(vmOptions) 159 ..addAll(vmOptions)
165 ..addAll(sharedOptions) 160 ..addAll(sharedOptions)
166 ..addAll(originalArguments); 161 ..addAll(originalArguments);
167 } 162 }
168 } 163 }
169 164
170 /// Common configuration for dart2js-based tools, such as, dart2js 165 /// Common configuration for dart2js-based tools, such as, dart2js
171 class Dart2xCompilerConfiguration extends CompilerConfiguration { 166 class Dart2xCompilerConfiguration extends CompilerConfiguration {
172 final String moniker; 167 final String moniker;
173 static Map<String, List<Uri>> _bootstrapDependenciesCache = 168 static Map<String, List<Uri>> _bootstrapDependenciesCache =
(...skipping 190 matching lines...) Expand 10 before | Expand all | Expand 10 after
364 // shipped SDK, that is the script is not installed in 359 // shipped SDK, that is the script is not installed in
365 // "$buildDir/dart-sdk/bin/" 360 // "$buildDir/dart-sdk/bin/"
366 return '$prefix/dartanalyzer_developer$suffix'; 361 return '$prefix/dartanalyzer_developer$suffix';
367 } 362 }
368 if (useSdk) { 363 if (useSdk) {
369 prefix = '$buildDir/dart-sdk/bin'; 364 prefix = '$buildDir/dart-sdk/bin';
370 } 365 }
371 return '$prefix/dartanalyzer$suffix'; 366 return '$prefix/dartanalyzer$suffix';
372 } 367 }
373 } 368 }
OLDNEW
« no previous file with comments | « tests/standalone/no_assert_test.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698