HTTP client errors shouldn't be sent to both the connection and the input stream

[nex3]

Currently, if an error happens in an HttpClientConnection, it's passed to the onError callbacks of both the connection itself and the InputStream of the HttpClientResponse, should such a thing exist. Here's a simple reproduction:

    void main() {
      var client = new HttpClient();
      var url = new Uri.fromString("http://pub.dartlang.org");
      var connection = client.openUrl("GET", url);
      connection.onError = (e) => print("connection error: $e");
      connection.onRequest = (request) {
        request.outputStream.onError = (e) => print("request stream error: $e");
        request.outputStream.close();
      };
      connection.onResponse = (response) {
        response.inputStream.onData = response.inputStream.read;
        response.inputStream.onError = (e) => print("response stream error: $e");
        response.inputStream.onClosed = () => print("response stream closed");

        print("response headers received");
        client.shutdown(force: true);
      };
    }

[nex3]

My preferred behavior would be for the error to be passed (only) to HttpClientConnection up the call to onResponse, at which point it should be passed (only) to HttpClientResponse.inputStream.onError.

[madsager]

Set owner to @sgjesse.

[sgjesse]

Fixed in https://code.google.com/p/dart/source/detail?r=15570.

---

Added Fixed label.

[kevmoo]

Removed Area-IO label.
Added Area-Library, Library-IO labels.