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

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

Issue 11066014: Revert revision 13230 (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 2 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 | « tools/testing/dart/test_options.dart ('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/test_runner.dart
===================================================================
--- tools/testing/dart/test_runner.dart (revision 13230)
+++ tools/testing/dart/test_runner.dart (working copy)
@@ -32,13 +32,10 @@
/** Command line arguments to the executable. */
List<String> arguments;
- /** Environment variables set when running the command */
- Map<String, String> environment;
-
/** The actual command line that will be executed. */
String commandLine;
- Command(this.executable, this.arguments, [this.environment = const {}]) {
+ Command(this.executable, this.arguments) {
if (Platform.operatingSystem == 'windows') {
// Windows can't handle the first command if it is a .bat file or the like
// with the slashes going the other direction.
@@ -636,8 +633,7 @@
}
}
- VoidFunction makeReadHandler(StringInputStream source,
- List<String> destination) {
+ VoidFunction makeReadHandler(StringInputStream source, List<String> destination) {
void handler () {
if (source.closed) return; // TODO(whesse): Remove when bug is fixed.
var line = source.readLine();
@@ -660,11 +656,7 @@
void runCommand(Command command,
void exitHandler(int exitCode)) {
- ProcessOptions options = new ProcessOptions();
- options.environment = command.environment;
- process = Process.start(command.executable,
- command.arguments,
- options);
+ process = Process.start(command.executable, command.arguments);
process.onExit = exitHandler;
process.onError = (e) {
print("Error starting process:");
« no previous file with comments | « tools/testing/dart/test_options.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698