Only change timeout to 100ms after the connection is established.

Only change timeout to 100ms after the connection is established.

BUG=
R=sgjesse@google.com

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

[Søren Gjesse, 2013-08-21 07:39:54]

lgtm with comments

https://codereview.chromium.org/23074008/diff/1/tests/standalone/io/http_serv...
File tests/standalone/io/http_server_idle_timeout_test.dart (right):

https://codereview.chromium.org/23074008/diff/1/tests/standalone/io/http_serv...
tests/standalone/io/http_server_idle_timeout_test.dart:16:
HttpServer.bind("127.0.0.1", 0).then((server) {
Set server.idleTimeout to null here for completeness.

https://codereview.chromium.org/23074008/diff/1/tests/standalone/io/http_serv...
tests/standalone/io/http_server_idle_timeout_test.dart:20:
Socket.connect("127.0.0.1", server.port).then((socket) {
How about waiting with timer until response data has been received? The call to
write could be delayed as well.

[Anders Johnsen, 2013-08-21 07:43:49]

https://codereview.chromium.org/23074008/diff/1/tests/standalone/io/http_serv...
File tests/standalone/io/http_server_idle_timeout_test.dart (right):

https://codereview.chromium.org/23074008/diff/1/tests/standalone/io/http_serv...
tests/standalone/io/http_server_idle_timeout_test.dart:16:
HttpServer.bind("127.0.0.1", 0).then((server) {
On 2013/08/21 07:39:54, Søren Gjesse wrote:
> Set server.idleTimeout to null here for completeness.

Done.

https://codereview.chromium.org/23074008/diff/1/tests/standalone/io/http_serv...
tests/standalone/io/http_server_idle_timeout_test.dart:20:
Socket.connect("127.0.0.1", server.port).then((socket) {
On 2013/08/21 07:39:54, Søren Gjesse wrote:
> How about waiting with timer until response data has been received? The call
to
> write could be delayed as well.

If I wait so long, the timer would already be started with a 'null' timer, thus
being disabled. Moving to server.listen though.

[Anders Johnsen, 2013-08-21 07:44:18]

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