Don't close the server until we have received the error.

Don't close the server until we have received the error.

This avoids a scenerio where server.close would destroy the socket before we would receive the error, if the validator stream is paused from the socket.

BUG=https://code.google.com/p/dart/issues/detail?id=8865

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

[Søren Gjesse, 2013-03-01 11:51:19]

lgtm

https://codereview.chromium.org/12385041/diff/1/tests/standalone/io/http_cont...
File tests/standalone/io/http_content_length_test.dart (right):

https://codereview.chromium.org/12385041/diff/1/tests/standalone/io/http_cont...
tests/standalone/io/http_content_length_test.dart:95: Expect.isTrue(e.error is
HttpException, "[$e] [${e.error}]");
Long line (not your fault).

[Anders Johnsen, 2013-03-01 12:26:54]

Landing.

https://codereview.chromium.org/12385041/diff/1/tests/standalone/io/http_cont...
File tests/standalone/io/http_content_length_test.dart (right):

https://codereview.chromium.org/12385041/diff/1/tests/standalone/io/http_cont...
tests/standalone/io/http_content_length_test.dart:95: Expect.isTrue(e.error is
HttpException, "[$e] [${e.error}]");
On 2013/03/01 11:51:19, Søren Gjesse wrote:
> Long line (not your fault).

Done.

[Anders Johnsen, 2013-03-01 12:29:22]

Committed patchset #2 manually as r19299 (presubmit successful).