Improve the event handling for string input stream

Improve the event handling for string input stream

1. The string input stream now calls the data or line handler if there
   is still data available after returning from the data or line
   handler. These callbacks are scheduled on the event loop.

2. The reading of data is now throtteld so that calls to read or
   readLine will not eagerly try to read from the wrapped input
   stream. After each call to read the data handler is installed if
   no more buffered data is available. Also when the data handler
   is called the data handler on the wrapped input stream is
   un-installed if no more data is required for the current data or
   line handler.

One side effect of this change is that just reading from an input
stream obtained from a file is no longer possible. One will have to
set up a data handler callback of some kind before any calls to read
will return data. We can of cause change this to allow synchronous
usage of an input stream obtained from a file (maybe with an optional
argument to openInputStream).

Fixed a related throtteling issue with the chunked input stream.

R=ager@google.com

BUG=
TEST=

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

[Mads Ager (google), 2011-12-09 08:47:04]

LGTM

However, we should grep through the source base to find uses of this that no
longer work. I know that there is code in the testing infrastructure that will
need updating.

http://codereview.chromium.org/8885032/diff/1/runtime/bin/string_stream.dart
File runtime/bin/string_stream.dart (right):

http://codereview.chromium.org/8885032/diff/1/runtime/bin/string_stream.dart#...
runtime/bin/string_stream.dart:246: if (decodedLine != null &&
decodedLine[decodedLine.length - 1] == '\r') {
Long line.

[Søren Gjesse, 2011-12-09 11:44:32]

Updated the Dart test runner - PTAL

http://codereview.chromium.org/8885032/diff/1/runtime/bin/string_stream.dart
File runtime/bin/string_stream.dart (right):

http://codereview.chromium.org/8885032/diff/1/runtime/bin/string_stream.dart#...
runtime/bin/string_stream.dart:246: if (decodedLine != null &&
decodedLine[decodedLine.length - 1] == '\r') {
On 2011/12/09 08:47:04, Mads Ager wrote:
> Long line.

Done.

[Mads Ager (google), 2011-12-09 13:09:33]

lgtm

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/status_fi...
File tools/testing/dart/status_file_parser.dart (right):

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/status_fi...
tools/testing/dart/status_file_parser.dart:77: String line;
Indent line 77 to 113.

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/test_suit...
File tools/testing/dart/test_suite.dart (right):

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/test_suit...
tools/testing/dart/test_suite.dart:174: List<Future> fut = new List();
fut fut, delete?

[Søren Gjesse, 2011-12-09 13:36:43]

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/status_fi...
File tools/testing/dart/status_file_parser.dart (right):

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/status_fi...
tools/testing/dart/status_file_parser.dart:77: String line;
On 2011/12/09 13:09:33, Mads Ager wrote:
> Indent line 77 to 113.

Done.

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/test_suit...
File tools/testing/dart/test_suite.dart (right):

http://codereview.chromium.org/8885032/diff/3001/tools/testing/dart/test_suit...
tools/testing/dart/test_suite.dart:174: List<Future> fut = new List();
On 2011/12/09 13:09:33, Mads Ager wrote:
> fut fut, delete?

Done.