Index: net/http/http_basic_stream.cc |
=================================================================== |
--- net/http/http_basic_stream.cc (revision 88789) |
+++ net/http/http_basic_stream.cc (working copy) |
@@ -4,6 +4,7 @@ |
#include "net/http/http_basic_stream.h" |
+#include "base/metrics/histogram.h" |
#include "base/stringprintf.h" |
#include "net/base/io_buffer.h" |
#include "net/base/net_errors.h" |
@@ -13,6 +14,10 @@ |
#include "net/http/http_util.h" |
#include "net/socket/client_socket_handle.h" |
+namespace { |
+int g_socket_reuse_policy = -1; |
+} |
+ |
namespace net { |
HttpBasicStream::HttpBasicStream(ClientSocketHandle* connection, |
@@ -34,6 +39,7 @@ |
request_info_ = request_info; |
parser_.reset(new HttpStreamParser(connection_.get(), request_info, |
read_buf_, net_log)); |
+ bytes_read_offset_ = connection_->socket()->NumBytesRead(); |
return OK; |
} |
@@ -50,6 +56,7 @@ |
request_line_ = base::StringPrintf("%s %s HTTP/1.1\r\n", |
request_info_->method.c_str(), |
path.c_str()); |
+ response_ = response; |
return parser_->SendRequest(request_line_, headers, request_body, response, |
callback); |
} |
@@ -119,4 +126,45 @@ |
return false; |
} |
+void HttpBasicStream::LogNumRttVsBytesMetrics() const { |
+ DCHECK_GE(g_socket_reuse_policy, 0); |
+ DCHECK_LE(g_socket_reuse_policy, 3); |
+ |
+ int64 total_bytes_read = connection_->socket()->NumBytesRead(); |
+ int64 bytes_received = total_bytes_read - bytes_read_offset_; |
+ int64 num_kb = bytes_received / 1024; |
+ double rtt = connection_->socket()->GetConnectTimeMicros(); |
+ rtt /= 1000.0; |
+ |
+ if (num_kb < 1024 && rtt > 0) { // Ignore responses > 1MB |
+ base::TimeDelta duration = base::Time::Now() - |
+ response_->request_time; |
+ double num_rtt = static_cast<double>(duration.InMilliseconds()) / rtt; |
+ int64 num_rtt_scaled = (4 * num_rtt); |
+ |
+ static const char* kGroups[] = { "last_accessed_socket", "warmest_socket", |
willchan no longer on Chromium
2011/06/14 12:01:22
You didn't fix this.
Gagan
2011/06/14 18:25:02
added the extra const u wanted
|
+ "warm_socket", "manual" }; |
+ int bucket = (num_kb / 5) * 5; |
+ const std::string histogram(StringPrintf("Net.Num_RTT_vs_KB_%s_%dKB", |
+ kGroups[g_socket_reuse_policy], |
+ bucket)); |
+ base::Histogram* counter = base::Histogram::FactoryGet( |
+ histogram, 0, 1000, 2, base::Histogram::kUmaTargetedHistogramFlag); |
+ DCHECK_EQ(histogram, counter->histogram_name()); |
+ counter->Add(num_rtt_scaled); |
+ |
+ LOG(ERROR) << request_line_ |
+ << "\nrtt = " << rtt << "\tnum_rtt = " << num_rtt |
+ << "\tnum_kb = " << num_kb |
+ << "\ttotal bytes = " << total_bytes_read |
+ << "\thistogram = " << histogram; |
+ } |
+} |
+ |
+// static |
+// NOTE: 'policy' should be a valid ClientSocketReusePolicy enum value. |
+void HttpBasicStream::SetSocketReusePolicy(int policy) { |
+ g_socket_reuse_policy = policy; |
+} |
+ |
} // namespace net |