[HttpJob] Record bytes read for requests

net/url_request/url_request_http_job.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/url_request/url_request_http_job.h"
 
 #include <vector>
 
 #include "base/base_switches.h"
 #include "base/bind.h"
@@ skipping 1484 matching lines @@
           UMA_HISTOGRAM_MEDIUM_TIMES(
               "Net.HttpJob.TotalTimeNotCached.Secure.Quic", total_time);
         } else {
           UMA_HISTOGRAM_MEDIUM_TIMES(
               "Net.HttpJob.TotalTimeNotCached.Secure.NotQuic", total_time);
         }
       }
     }
   }
 
-  if (request_info_.load_flags & LOAD_PREFETCH && !request_->was_cached())
-    UMA_HISTOGRAM_COUNTS("Net.Prefetch.PrefilterBytesReadFromNetwork",
-                         prefilter_bytes_read());
+  UMA_HISTOGRAM_CUSTOM_COUNTS("Net.HttpJob.PrefilterBytesRead",
+                              prefilter_bytes_read(), 1, 50000000, 50);
+
+  if (request_->was_cached()) {

[mmenke, 2016/12/02 14:58:49]

Can we just grab this from response_info_ instead?  I'd really like to reduce
the dependencies of URLRequestJob classes on the URLRequest, since the
URLRequest uses the URLRequestJob, not the other way around.

[jkarlin, 2016/12/02 15:06:23]

Done.

+    UMA_HISTOGRAM_CUSTOM_COUNTS("Net.HttpJob.PrefilterBytesRead.Cache",
+                                prefilter_bytes_read(), 1, 50000000, 50);
+  } else {
+    UMA_HISTOGRAM_CUSTOM_COUNTS("Net.HttpJob.PrefilterBytesRead.Net",
+                                prefilter_bytes_read(), 1, 50000000, 50);
+
+    if (request_info_.load_flags & LOAD_PREFETCH) {
+      UMA_HISTOGRAM_COUNTS("Net.Prefetch.PrefilterBytesReadFromNetwork",
+                           prefilter_bytes_read());
+    }
+  }
 
   start_time_ = base::TimeTicks();
 }
 
 void URLRequestHttpJob::DoneWithRequest(CompletionCause reason) {
   if (done_)
     return;
   done_ = true;
 
   // Notify NetworkQualityEstimator.
@@ skipping 20 matching lines @@
   awaiting_callback_ = false;
 
   // Notify NetworkQualityEstimator.
   NetworkQualityEstimator* network_quality_estimator =
       request()->context()->network_quality_estimator();
   if (network_quality_estimator)
     network_quality_estimator->NotifyURLRequestDestroyed(*request());
 }
 
 }  // namespace net