Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(92)

Side by Side Diff: net/http/http_basic_stream.cc

Issue 6990036: Deciding best connection to schedule requests on based on cwnd and idle time (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 9 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/http/http_basic_stream.h" 5 #include "net/http/http_basic_stream.h"
6 6
7 #include "base/metrics/histogram.h"
7 #include "base/stringprintf.h" 8 #include "base/stringprintf.h"
8 #include "net/base/io_buffer.h" 9 #include "net/base/io_buffer.h"
9 #include "net/base/net_errors.h" 10 #include "net/base/net_errors.h"
10 #include "net/http/http_request_headers.h" 11 #include "net/http/http_request_headers.h"
11 #include "net/http/http_request_info.h" 12 #include "net/http/http_request_info.h"
12 #include "net/http/http_stream_parser.h" 13 #include "net/http/http_stream_parser.h"
13 #include "net/http/http_util.h" 14 #include "net/http/http_util.h"
14 #include "net/socket/client_socket_handle.h" 15 #include "net/socket/client_socket_handle.h"
15 16
16 namespace net { 17 namespace net {
(...skipping 10 matching lines...) Expand all
27 28
28 HttpBasicStream::~HttpBasicStream() {} 29 HttpBasicStream::~HttpBasicStream() {}
29 30
30 int HttpBasicStream::InitializeStream(const HttpRequestInfo* request_info, 31 int HttpBasicStream::InitializeStream(const HttpRequestInfo* request_info,
31 const BoundNetLog& net_log, 32 const BoundNetLog& net_log,
32 CompletionCallback* callback) { 33 CompletionCallback* callback) {
33 DCHECK(!parser_.get()); 34 DCHECK(!parser_.get());
34 request_info_ = request_info; 35 request_info_ = request_info;
35 parser_.reset(new HttpStreamParser(connection_.get(), request_info, 36 parser_.reset(new HttpStreamParser(connection_.get(), request_info,
36 read_buf_, net_log)); 37 read_buf_, net_log));
38 bytes_read_offset_ = connection_->socket()->NumBytesRead();
37 return OK; 39 return OK;
38 } 40 }
39 41
40 42
41 int HttpBasicStream::SendRequest(const HttpRequestHeaders& headers, 43 int HttpBasicStream::SendRequest(const HttpRequestHeaders& headers,
42 UploadDataStream* request_body, 44 UploadDataStream* request_body,
43 HttpResponseInfo* response, 45 HttpResponseInfo* response,
44 CompletionCallback* callback) { 46 CompletionCallback* callback) {
45 DCHECK(parser_.get()); 47 DCHECK(parser_.get());
46 DCHECK(request_info_); 48 DCHECK(request_info_);
47 const std::string path = using_proxy_ ? 49 const std::string path = using_proxy_ ?
48 HttpUtil::SpecForRequest(request_info_->url) : 50 HttpUtil::SpecForRequest(request_info_->url) :
49 HttpUtil::PathForRequest(request_info_->url); 51 HttpUtil::PathForRequest(request_info_->url);
50 request_line_ = base::StringPrintf("%s %s HTTP/1.1\r\n", 52 request_line_ = base::StringPrintf("%s %s HTTP/1.1\r\n",
51 request_info_->method.c_str(), 53 request_info_->method.c_str(),
52 path.c_str()); 54 path.c_str());
55 response_ = response;
53 return parser_->SendRequest(request_line_, headers, request_body, response, 56 return parser_->SendRequest(request_line_, headers, request_body, response,
54 callback); 57 callback);
55 } 58 }
56 59
57 uint64 HttpBasicStream::GetUploadProgress() const { 60 uint64 HttpBasicStream::GetUploadProgress() const {
58 return parser_->GetUploadProgress(); 61 return parser_->GetUploadProgress();
59 } 62 }
60 63
61 int HttpBasicStream::ReadResponseHeaders(CompletionCallback* callback) { 64 int HttpBasicStream::ReadResponseHeaders(CompletionCallback* callback) {
62 return parser_->ReadResponseHeaders(callback); 65 return parser_->ReadResponseHeaders(callback);
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 115
113 void HttpBasicStream::GetSSLCertRequestInfo( 116 void HttpBasicStream::GetSSLCertRequestInfo(
114 SSLCertRequestInfo* cert_request_info) { 117 SSLCertRequestInfo* cert_request_info) {
115 parser_->GetSSLCertRequestInfo(cert_request_info); 118 parser_->GetSSLCertRequestInfo(cert_request_info);
116 } 119 }
117 120
118 bool HttpBasicStream::IsSpdyHttpStream() const { 121 bool HttpBasicStream::IsSpdyHttpStream() const {
119 return false; 122 return false;
120 } 123 }
121 124
125 void HttpBasicStream::LogRttVsBytesMetrics() const {
126 int bytes_received = connection_->socket()->NumBytesRead() -
127 bytes_read_offset_;
128 int num_kb = bytes_received / 1024;
129 double rtt = connection_->socket()->GetRTTInMs();
130
131 if (num_kb < 1024 && rtt > 0) { // Ignore responses > 1MB
132 base::TimeDelta duration = base::Time::Now() -
133 response_->request_time;
134 double num_rtt = static_cast<double>(duration.InMilliseconds()) / rtt;
135 int num_rtt_scaled = (4 * num_rtt);
136 int val = 1024 * num_rtt_scaled + num_kb; // Supports ~500 RTT of 1418 B.
137 HISTOGRAM_CUSTOM_COUNTS("Net.RTT_vs_KB", val, 4096, 3048000, 1000);
138 LOG(ERROR) << request_line_
139 << "\nrtt = " << rtt << "\tnum_rtt = " << num_rtt
140 << "\tnum_kb = " << num_kb << "\tval = " << val;
141 }
142 }
143
122 } // namespace net 144 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698