Index: tools/testing/dart/test_runner.dart |
diff --git a/tools/testing/dart/test_runner.dart b/tools/testing/dart/test_runner.dart |
index 356b664819872c0e5394b56312276e5176a9889d..9833dd4517673cba2a8c7ff568746c927af63a9b 100644 |
--- a/tools/testing/dart/test_runner.dart |
+++ b/tools/testing/dart/test_runner.dart |
@@ -107,7 +107,7 @@ class Command { |
// TODO(efortuna): Remove this when fixed (Issue 1306). |
executable = executable.replaceAll('/', '\\'); |
} |
- commandLine = "$executable ${Strings.join(arguments, ' ')}"; |
+ commandLine = "$executable ${arguments.join(' ')}"; |
} |
String toString() => commandLine; |
@@ -196,7 +196,7 @@ class DumpRenderTreeCommand extends Command { |
if (needDartFlags) { |
env = new Map.from(io.Platform.environment); |
if (needDartFlags) { |
- env['DART_FLAGS'] = Strings.join(dartFlags, " "); |
+ env['DART_FLAGS'] = dartFlags.join(" "); |
} |
} |
@@ -305,7 +305,7 @@ class TestCase { |
newCommands.add(newCommand); |
// If there are extra spaces inside the prefix or suffix, this fails. |
String expected = |
- '$prefix ${c.executable} $suffix ${Strings.join(c.arguments, ' ')}'; |
+ '$prefix ${c.executable} $suffix ${c.arguments.join(' ')}'; |
Expect.stringEquals(expected.trim(), newCommand.commandLine); |
} |
commands = newCommands; |
@@ -1152,7 +1152,7 @@ class BatchRunnerProcess { |
} |
String _createArgumentsLine(List<String> arguments) { |
- return Strings.join(arguments, ' ').concat('\n'); |
+ return arguments.join(' ').concat('\n'); |
} |
void _reportResult() { |
@@ -1287,7 +1287,7 @@ class BatchRunnerProcess { |
callback(); |
}).catchError((e) { |
print("Process error:"); |
- print(" Command: $_executable ${Strings.join(_batchArguments, ' ')}"); |
+ print(" Command: $_executable ${_batchArguments.join(' ')}"); |
print(" Error: $e"); |
// If there is an error starting a batch process, chances are that |
// it will always fail. So rather than re-trying a 1000+ times, we |
@@ -1464,7 +1464,7 @@ class ProcessQueue { |
}; |
}).catchError((e) { |
print("Error starting process:"); |
- print(" Command: $cmd ${Strings.join(arg, ' ')}"); |
+ print(" Command: $cmd ${arg.join(' ')}"); |
print(" Error: $e"); |
// TODO(ahe): How to report this as a test failure? |
exit(1); |
@@ -1580,10 +1580,10 @@ class ProcessQueue { |
TestCase test = _tests.removeFirst(); |
if (_listTests) { |
var fields = [test.displayName, |
- Strings.join(new List.from(test.expectedOutcomes), ','), |
+ test.expectedOutcomes.join(','), |
test.isNegative.toString()]; |
fields.addAll(test.commands.last.arguments); |
- print(Strings.join(fields, '\t')); |
+ print(fields.join('\t')); |
return; |
} |
if (test.usesWebDriver && _needsSelenium && !_isSeleniumAvailable || (test |