[chromedriver] Remove extra call to HttpServer::Close() and increase TestHttpServer buffer limit

[chromedriver] Remove extra call to HttpServer::Close() and increase TestHttpServer buffer limit

BUG=
TBR=stgao@chromium.org

Committed: https://crrev.com/92354ccd6e5d1123685cb384a3e08425d801017d
Cr-Commit-Position: refs/heads/master@{#295871}

[samuong, 2014-09-20 00:55:47]

samuong@chromium.org changed reviewers:
+ stgao@chromium.org

[samuong, 2014-09-20 01:04:36]

PTAL. This should fix the following tests on our waterfall:

- TextHandlingTest.testReadALargeAmountOfData
- SyncWebSocketImplTest.SendReceiveLarge
- WebSocketTest.SendReceiveLarge

[stgao, 2014-09-20 02:10:04]

https://codereview.chromium.org/590653002/diff/1/chrome/test/chromedriver/ser...
File chrome/test/chromedriver/server/chromedriver_server.cc (left):

https://codereview.chromium.org/590653002/diff/1/chrome/test/chromedriver/ser...
chrome/test/chromedriver/server/chromedriver_server.cc:90:
server_->Close(connection_id);
Why we don't have to close the connection now? Is it automatically closed after
async write?
If so, please add a comment here explain why explicit close is not needed any
more.

[stgao, 2014-09-20 06:19:59]

Hi Sam,

Once my comment is addressed, please feel free to do a TBR to get this CL
committed through CQ.
I will take a look later.

Thanks,
Shuotao Gao

[samuong, 2014-09-20 06:27:11]

samuong@chromium.org changed reviewers:
- stgao@chromium.org

[samuong, 2014-09-20 06:27:20]

The CQ bit was checked by samuong@chromium.org

[commit-bot: I haz the power, 2014-09-20 06:27:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/590653002/20001

[commit-bot: I haz the power, 2014-09-20 13:21:47]

Committed patchset #2 (id:20001) as e7ea1919961b9cc9b8ae4506f48085c058656667

[commit-bot: I haz the power, 2014-09-20 13:22:21]

Patchset 2 (id:??) landed as
https://crrev.com/92354ccd6e5d1123685cb384a3e08425d801017d
Cr-Commit-Position: refs/heads/master@{#295871}