tree 56fbcdea0285337167e84dcc497582f4b53fd502
parent b294e48bdfa1ccc73ab8c463b8d05fe7e84e922b
author Adam Barth <abarth@google.com> 1525901589 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> 1525901589 +0000

Stop leaking file descriptors for HTTP clients

Previously, we would wait for a Future to terminate before destroying
the underlying raw socket. If that future never terminated, we would
leak the underlying file descriptor.

Now, the "force" codepath in close destroys the socket immediately
(similar to what it did previously for active sockets) and the non-force
codepath has a timeout, as requested by a TODO comment.

Change-Id: I021a93a40e4708ce93b4f1ae6c3f7289764bd69f
Reviewed-on: https://dart-review.googlesource.com/54442
Reviewed-by: Zach Anderson <zra@google.com>
Commit-Queue: Adam Barth <abarth@google.com>