Set network_accessed earlier, when network transaction creates stream.

net/http/http_network_transaction.cc

Index: net/http/http_network_transaction.cc
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index c094bdbbae1c626d01774e9d9be5204bdd340d4c..3d64b0f799072cc98d114f06f65a2da2c0d7c847 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -395,10 +395,7 @@ int64 HttpNetworkTransaction::GetTotalReceivedBytes() const {
 void HttpNetworkTransaction::DoneReading() {}
 
 const HttpResponseInfo* HttpNetworkTransaction::GetResponseInfo() const {
-  return ((headers_valid_ && response_.headers.get()) ||
-          response_.ssl_info.cert.get() || response_.cert_request_info.get())
-             ? &response_
-             : NULL;
+  return &response_;

[davidben, 2015/05/14 22:28:14]

With this, except for FailingHttpTransaction, GetResponseInfo never returns
NULL. How about fixing that one and changing the documentation of
HttpTransaction to not return NULL? Also document that it may only be called
after Start() completes.

Also mention in the commit message that we're changing the meaning of
GetResponseInfo slightly.

[Deprecated (see juliatuttle), 2015/05/14 23:52:10]

Done.

 }
 
 LoadState HttpNetworkTransaction::GetLoadState() const {
@@ -736,6 +733,8 @@ int HttpNetworkTransaction::DoCreateStream() {
       FROM_HERE_WITH_EXPLICIT_FUNCTION(
           "424359 HttpNetworkTransaction::DoCreateStream"));
 
+  response_.network_accessed = true;
+
   next_state_ = STATE_CREATE_STREAM_COMPLETE;
   if (ForWebSocketHandshake()) {
     stream_request_.reset(
@@ -974,7 +973,6 @@ int HttpNetworkTransaction::DoSendRequestComplete(int result) {
   send_end_time_ = base::TimeTicks::Now();
   if (result < 0)
     return HandleIOError(result);
-  response_.network_accessed = true;
   next_state_ = STATE_READ_HEADERS;
   return OK;
 }