commit c38ec5a81e6471d0e23730741bdfd20ea4263459
author mbelshe@chromium.org <mbelshe@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Aug 25 23:32:38 2010
committer mbelshe@chromium.org <mbelshe@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Wed Aug 25 23:32:38 2010
tree b7ecb327519634d1392b1ec02ec645ebae59378a
parent 4528428b696fac9d394843801a49a47dd5d83ed8

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

net/http/http_network_transaction.cc

diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index 9abbb08..1dad4169 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -155,7 +155,6 @@
       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 @@
   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 @@
       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 @@
         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 @@
                               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),

net/http/http_network_transaction.h

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 @@
   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_;