diff options
| author | mbelshe@chromium.org <mbelshe@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-25 23:32:38 +0000 |
|---|---|---|
| committer | mbelshe@chromium.org <mbelshe@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-25 23:32:38 +0000 |
| commit | c38ec5a81e6471d0e23730741bdfd20ea4263459 (patch) | |
| tree | b7ecb327519634d1392b1ec02ec645ebae59378a /net | |
| parent | 4528428b696fac9d394843801a49a47dd5d83ed8 (diff) | |
| download | chromium_src-c38ec5a81e6471d0e23730741bdfd20ea4263459.zip chromium_src-c38ec5a81e6471d0e23730741bdfd20ea4263459.tar.gz chromium_src-c38ec5a81e6471d0e23730741bdfd20ea4263459.tar.bz2 | |
Remove the reused_socket_ member because we don't really need it.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/3120032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57418 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
| -rw-r--r-- | net/http/http_network_transaction.cc | 9 | ||||
| -rw-r--r-- | net/http/http_network_transaction.h | 3 |
2 files changed, 4 insertions, 8 deletions
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc index 9abbb08..1dad416 100644 --- a/net/http/http_network_transaction.cc +++ b/net/http/http_network_transaction.cc @@ -155,7 +155,6 @@ HttpNetworkTransaction::HttpNetworkTransaction(HttpNetworkSession* session) user_callback_(NULL), session_(session), request_(NULL), - reused_socket_(false), headers_valid_(false), logged_response_time_(false), read_buf_len_(0), @@ -598,7 +597,6 @@ int HttpNetworkTransaction::DoInitStreamComplete(int result) { if (result == OK) { next_state_ = STATE_GENERATE_PROXY_AUTH_TOKEN; - reused_socket_ = stream_->IsConnectionReused(); if (is_https_request()) stream_->GetSSLInfo(&response_.ssl_info); } else { @@ -924,7 +922,8 @@ void HttpNetworkTransaction::LogTransactionConnectedMetrics() { base::TimeDelta::FromMilliseconds(1), base::TimeDelta::FromMinutes(10), 100); - if (!reused_socket_) { + bool reused_socket = stream_->IsConnectionReused(); + if (!reused_socket) { UMA_HISTOGRAM_CLIPPED_TIMES( "Net.Transaction_Connected_New", total_duration, @@ -952,7 +951,7 @@ void HttpNetworkTransaction::LogTransactionConnectedMetrics() { total_duration, base::TimeDelta::FromMilliseconds(1), base::TimeDelta::FromMinutes(10), 100); - if (!reused_socket_) { + if (!reused_socket) { UMA_HISTOGRAM_CLIPPED_TIMES( FieldTrial::MakeName("Net.Transaction_Connected_New", "SpdyImpact"), total_duration, base::TimeDelta::FromMilliseconds(1), @@ -995,7 +994,7 @@ void HttpNetworkTransaction::LogTransactionMetrics() const { total_duration, base::TimeDelta::FromMilliseconds(1), base::TimeDelta::FromMinutes(10), 100); - if (!reused_socket_) { + if (!stream_->IsConnectionReused()) { UMA_HISTOGRAM_CLIPPED_TIMES( "Net.Transaction_Latency_Total_New_Connection_Under_10", total_duration, base::TimeDelta::FromMilliseconds(1), diff --git a/net/http/http_network_transaction.h b/net/http/http_network_transaction.h index 8559885..c4cb8d0 100644 --- a/net/http/http_network_transaction.h +++ b/net/http/http_network_transaction.h @@ -208,9 +208,6 @@ class HttpNetworkTransaction : public HttpTransaction, scoped_refptr<StreamFactory::StreamRequestJob> stream_request_; scoped_ptr<HttpStreamHandle> stream_; - // True if this transaction was serviced on a reused socket. - bool reused_socket_; - // True if we've validated the headers that the stream parser has returned. bool headers_valid_; |