Only use one timer to handle HttpServer::idleTimeout.

Only use one timer to handle HttpServer::idleTimeout.

This makes us avoid a lot of timer allocations/cancels by default.

BUG=
R=sgjesse@google.com

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

[Søren Gjesse, 2014-02-24 14:14:35]

lgtm

https://codereview.chromium.org/176893003/diff/1/sdk/lib/io/http_impl.dart
File sdk/lib/io/http_impl.dart (right):

https://codereview.chromium.org/176893003/diff/1/sdk/lib/io/http_impl.dart#ne...
sdk/lib/io/http_impl.dart:1905: final Uint8List _headersBuffer = new
Uint8List(_HEADERS_BUFFER_SIZE);
Why the empty lines?

https://codereview.chromium.org/176893003/diff/1/sdk/lib/io/http_impl.dart#ne...
sdk/lib/io/http_impl.dart:2045: void set idleTimeout(Duration duration) {
Update the dartdoc comment to say that it might take up to 2xidleTimeout for the
timeout to actually happen.

[Anders Johnsen, 2014-02-24 14:35:10]

https://codereview.chromium.org/176893003/diff/1/sdk/lib/io/http_impl.dart
File sdk/lib/io/http_impl.dart (right):

https://codereview.chromium.org/176893003/diff/1/sdk/lib/io/http_impl.dart#ne...
sdk/lib/io/http_impl.dart:1905: final Uint8List _headersBuffer = new
Uint8List(_HEADERS_BUFFER_SIZE);
On 2014/02/24 14:14:35, Søren Gjesse wrote:
> Why the empty lines?

Done.

https://codereview.chromium.org/176893003/diff/1/sdk/lib/io/http_impl.dart#ne...
sdk/lib/io/http_impl.dart:2045: void set idleTimeout(Duration duration) {
On 2014/02/24 14:14:35, Søren Gjesse wrote:
> Update the dartdoc comment to say that it might take up to 2xidleTimeout for
the
> timeout to actually happen.

Done.

[Anders Johnsen, 2014-02-24 14:35:31]

Committed patchset #3 manually as r32965 (presubmit successful).