Implement addStream for HttpClientRequest/HttpResponse and propegate all write-errors from the sock…

tests/standalone/io/http_shutdown_test.dart

Index: tests/standalone/io/http_shutdown_test.dart
diff --git a/tests/standalone/io/http_shutdown_test.dart b/tests/standalone/io/http_shutdown_test.dart
index 60c320008f05b82fed762bdc62c9a7f52006002b..8c91797d584cb41d0b73cf0e61c9ce2e06adcf8f 100644
--- a/tests/standalone/io/http_shutdown_test.dart
+++ b/tests/standalone/io/http_shutdown_test.dart
@@ -53,6 +53,7 @@ void test2(int totalConnections, int outputStreamWrites) {
           for (int i = 0; i < outputStreamWrites; i++) {
             request.addString("Hello, world!");
           }
+          request.done.catchError((_) {});

[Søren Gjesse, 2013/02/27 15:07:43]

I think we should add default handling of this so that if there is no error
handler on the done future something default will happen. Developers will forget
this and there HTTP servers will crash. Not for this CL, but lets think about
it.

[Anders Johnsen, 2013/02/28 11:39:02]

Yes, we need to decide on what should happen on this one. Let's discuss offline
for another CL.

           return request.close();
         })
         .then((HttpClientResponse response) {
@@ -150,6 +151,7 @@ void test5(int totalConnections) {
               (_) { },
               onDone: () {
                 request.response.close();
+                request.response.done.catchError((e) {});
               },
               onError: (error) { });
         },
@@ -166,6 +168,7 @@ void test5(int totalConnections) {
             // TODO(sgjesse): Make this test work with
             //request.response instead of request.close() return
             //return request.response;
+            request.done.catchError((e) {});
             return request.close();
           })
         .then((response) { })