[HttpJob] Record bytes read for requests

net/url_request/url_request_http_job.cc

Index: net/url_request/url_request_http_job.cc
diff --git a/net/url_request/url_request_http_job.cc b/net/url_request/url_request_http_job.cc
index 6934c78006ce18b4cebbc351cbadcdc51345e17d..4bfae6b3cb90e78ca8c62e48f54102e1005cc008 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -1502,9 +1502,21 @@ void URLRequestHttpJob::RecordPerfHistograms(CompletionCause reason) {
     }
   }
 
-  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();
 }