Set network_accessed earlier, when network transaction creates stream.

net/url_request/url_request.cc

Index: net/url_request/url_request.cc
diff --git a/net/url_request/url_request.cc b/net/url_request/url_request.cc
index 6032c4cb4885b6f6124939844a44d4c7e165bf79..39cfbfeadcb31b12a6668c09c73d1950065651c7 100644
--- a/net/url_request/url_request.cc
+++ b/net/url_request/url_request.cc
@@ -768,6 +768,8 @@ void URLRequest::NotifyReceivedRedirect(const RedirectInfo& redirect_info,
 }
 
 void URLRequest::NotifyBeforeNetworkStart(bool* defer) {
+  response_info_.network_accessed = true;

[davidben, 2015/05/04 17:57:19]

Why does this get set in two places and at two completely different layers? The
job overrides response_info_ at various points. Having this NotifyFoo function
go and overwrite whatever happens to be here right now seems clearly wrong.

If the problem is revalidations, the logic should be in HttpCache::Transaction.

+
   if (delegate_ && !notified_before_network_start_) {
     OnCallToDelegate();
     delegate_->OnBeforeNetworkStart(this, defer);