Don't tweak buffer size for SPDY

Don't tweak buffer size for SPDY

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=68754

[willchan no longer on Chromium, 2010-12-09 00:56:36]

LGTM

On Wed, Dec 8, 2010 at 4:49 PM, <mbelshe@chromium.org> wrote:

> Reviewers: willchan,
>
> Description:
> Don't tweak buffer size for SPDY
>
> BUG=none
> TEST=none
>
>
> Please review this at http://codereview.chromium.org/5588011/
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     net/spdy/spdy_session.cc
>
>
> Index: net/spdy/spdy_session.cc
> ===================================================================
> --- net/spdy/spdy_session.cc    (revision 68605)
> +++ net/spdy/spdy_session.cc    (working copy)
> @@ -62,17 +62,6 @@
>
>  const int kReadBufferSize = 8 * 1024;
>
> -void AdjustSocketBufferSizes(ClientSocket* socket) {
> -  // Adjust socket buffer sizes.
> -  // SPDY uses one socket, and we want a really big buffer.
> -  // This greatly helps on links with packet loss - we can even
> -  // outperform Vista's dynamic window sizing algorithm.
> -  // TODO(mbelshe): more study.
> -  const int kSocketBufferSize = 512 * 1024;
> -  socket->SetReceiveBufferSize(kSocketBufferSize);
> -  socket->SetSendBufferSize(kSocketBufferSize);
> -}
> -
>  class NetLogSpdySessionParameter : public NetLog::EventParameters {
>  public:
>   NetLogSpdySessionParameter(const HostPortProxyPair& host_pair)
> @@ -294,8 +283,6 @@
>   static base::StatsCounter spdy_sessions("spdy.sessions");
>   spdy_sessions.Increment();
>
> -  AdjustSocketBufferSizes(connection->socket());
> -
>   state_ = CONNECTED;
>   connection_.reset(connection);
>   is_secure_ = is_secure;
>
>
>