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

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

Issue 1537653002: Remove old java-based analyzer from test scripts and status files (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 years 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/test_runner.dart
diff --git a/tools/testing/dart/test_runner.dart b/tools/testing/dart/test_runner.dart
index 97ad17ac578ba7e8539a8034bd6c26fbca429ae8..eeb1d6931c16ef0aec684e5e1082fc36c5f613e8 100644
--- a/tools/testing/dart/test_runner.dart
+++ b/tools/testing/dart/test_runner.dart
@@ -610,7 +610,7 @@ class CommandBuilder {
AnalysisCommand getAnalysisCommand(
String displayName, executable, arguments, environmentOverrides,
- {String flavor: 'dartanalyzer'}) {
+ {String flavor: 'dart2analyzer'}) {
var command = new AnalysisCommand._(
flavor, displayName, executable, arguments, environmentOverrides);
return _getUniqueCommand(command);
@@ -2450,7 +2450,8 @@ class CommandExecutorImpl implements CommandExecutor {
final int maxProcesses;
final int maxBrowserProcesses;
- // For dartanalyzer batch processing we keep a list of batch processes.
+ // For dart2js and analyzer batch processing,
+ // we keep a list of batch processes.
final _batchProcesses = new Map<String, List<BatchRunnerProcess>>();
// We keep a BrowserTestRunner for every configuration.
final _browserTestRunners = new Map<Map, BrowserTestRunner>();

Powered by Google App Engine
This is Rietveld 408576698