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

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

Issue 9500002: Rename blahHandler to onBlah throughout dart:io. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Renaming Created 8 years, 10 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/test_runner.dart
diff --git a/tools/testing/dart/test_runner.dart b/tools/testing/dart/test_runner.dart
index fe5a2ea8933db1b5e19d109e42798f082badf437..302b889ae68dee967201f03aeb7c2320689e9510 100644
--- a/tools/testing/dart/test_runner.dart
+++ b/tools/testing/dart/test_runner.dart
@@ -360,15 +360,15 @@ class RunningProcess {
command.executable = command.executable.replaceAll('/', '\\');
}
process = new Process.start(command.executable, command.arguments);
- process.exitHandler = exitHandler;
+ process.onExit = exitHandler;
startTime = new Date.now();
InputStream stdoutStream = process.stdout;
InputStream stderrStream = process.stderr;
StringInputStream stdoutStringStream = new StringInputStream(stdoutStream);
StringInputStream stderrStringStream = new StringInputStream(stderrStream);
- stdoutStringStream.lineHandler =
+ stdoutStringStream.onLine =
makeReadHandler(stdoutStringStream, stdout);
- stderrStringStream.lineHandler =
+ stderrStringStream.onLine =
makeReadHandler(stderrStringStream, stderr);
timeoutTimer = new Timer(timeoutHandler, 1000 * testCase.timeout);
}
@@ -416,7 +416,7 @@ class BatchRunnerProcess {
// if needed.
_executable = testCase.commands.last().executable;
_batchArguments = testCase.batchRunnerArguments;
- _process.exitHandler = (exitCode) {
+ _process.onExit = (exitCode) {
_process.close();
_startProcess(() {
doStartTest(testCase);
@@ -431,7 +431,7 @@ class BatchRunnerProcess {
void terminate() {
if (_process !== null) {
bool closed = false;
- _process.exitHandler = (exitCode) {
+ _process.onExit = (exitCode) {
closed = true;
_process.close();
};
@@ -454,8 +454,8 @@ class BatchRunnerProcess {
_startTime = new Date.now();
_testStdout = new List<String>();
_testStderr = new List<String>();
- _stdoutStream.lineHandler = _readOutput(_stdoutStream, _testStdout);
- _stderrStream.lineHandler = _readOutput(_stderrStream, _testStderr);
+ _stdoutStream.onLine = _readOutput(_stdoutStream, _testStdout);
+ _stderrStream.onLine = _readOutput(_stderrStream, _testStderr);
_timer = new Timer(_timeoutHandler, testCase.timeout * 1000);
var line = _createArgumentsLine(testCase.batchTestArguments);
_process.stdin.write(line.charCodes());
@@ -515,7 +515,7 @@ class BatchRunnerProcess {
}
void _timeoutHandler(ignore) {
- _process.exitHandler = (exitCode) {
+ _process.onExit = (exitCode) {
_process.close();
_startProcess(() {
_reportResult(">>> TEST TIMEOUT");
@@ -530,10 +530,10 @@ class BatchRunnerProcess {
_stderrStream = new StringInputStream(_process.stderr);
_testStdout = new List<String>();
_testStderr = new List<String>();
- _stdoutStream.lineHandler = _readOutput(_stdoutStream, _testStdout);
- _stderrStream.lineHandler = _readOutput(_stderrStream, _testStderr);
- _process.exitHandler = _exitHandler;
- _process.startHandler = then;
+ _stdoutStream.onLine = _readOutput(_stdoutStream, _testStdout);
+ _stderrStream.onLine = _readOutput(_stderrStream, _testStderr);
+ _process.onExit = _exitHandler;
+ _process.onStart = then;
}
}
@@ -646,15 +646,15 @@ class ProcessQueue {
}
if (name == processNames[browserUsed].last()) {
- process.exitHandler = (exitCode) {
+ process.onExit = (exitCode) {
process.close();
_progress.allDone();
};
- process.errorHandler = (error) {
+ process.onError = (error) {
_progress.allDone();
};
} else {
- process.exitHandler = (exitCode) {
+ process.onExit = (exitCode) {
process.close();
};
}
@@ -692,7 +692,7 @@ class ProcessQueue {
// implemented, and add Windows support.
var deletion =
new Process.start('/bin/rm', ['-rf', _temporaryDirectory]);
- deletion.exitHandler = (int exitCode) {
+ deletion.onExit = (int exitCode) {
if (exitCode == 0) {
if (!_listTests) { // Output of --list option is used by scripts.
print('\nTemporary directory $_temporaryDirectory deleted.');

Powered by Google App Engine
This is Rietveld 408576698