Change test_runner.dart to always move on in the batch runner if the batch runner crashed.

Change test_runner.dart to always move on in the batch runner if the batch runner crashed.

If the batch runner crashes we close the streams and restart a new
process.

The other alternative is to distinguish between a hard crash or the java
process and a soft crash where we throw an exception. On a soft crash
we can put a token in stderr, wait for stderr to drain and then restart
the process. I don't think that adds much. If there is a soft crash the
crash indication is all that really matters and we need to fix that in
any case before moving on.

R=whesse@google.com,zundel@google.com
BUG=
TEST=

Committed: https://code.google.com/p/dart/source/detail?r=5156

[Bill Hesse, 2012-03-08 09:15:42]

lgtm

[zundel, 2012-03-08 12:22:16]

https://chromiumcodereview.appspot.com/9638008/diff/1/tools/testing/dart/test...
File tools/testing/dart/test_runner.dart (right):

https://chromiumcodereview.appspot.com/9638008/diff/1/tools/testing/dart/test...
tools/testing/dart/test_runner.dart:691: if (_stdoutDrained) _testCompleted();
What about here?

[zundel, 2012-03-08 12:28:29]

https://chromiumcodereview.appspot.com/9638008/diff/1/tools/testing/dart/test...
File tools/testing/dart/test_runner.dart (right):

https://chromiumcodereview.appspot.com/9638008/diff/1/tools/testing/dart/test...
tools/testing/dart/test_runner.dart:691: if (_stdoutDrained) _testCompleted();
On 2012/03/08 12:22:16, zundel wrote:
> What about here?

nm, I guess we just wait for the process to exit and then _reportResult() will
be called from _exitHandler