Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(620)

Unified Diff: net/url_request/url_request_http_job.cc

Issue 1921613003: Count successes of link rel=prefetch (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nit Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 51b690824301e8a7bcdb5cbd1f2333d4469858f6..7fd6f5e60a269aef65e649838865f4edc78e3a4e 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -1620,7 +1620,10 @@ void URLRequestHttpJob::RecordPerfHistograms(CompletionCause reason) {
"Net.HttpJob.TotalTimeCached.Secure.NotQuic", total_time);
}
}
- } else {
+ if (response_info_->unused_since_prefetch) {
+ UMA_HISTOGRAM_COUNTS("Net.Prefetch.HitBytes", prefilter_bytes_read());
+ }
gavinp 2016/04/25 15:39:05 Nit: I don't believe you need these braces.
jkarlin 2016/04/25 15:46:14 Done.
+ } else {
UMA_HISTOGRAM_TIMES("Net.HttpJob.TotalTimeNotCached", total_time);
if (is_https_google) {
if (used_quic) {

Powered by Google App Engine
This is Rietveld 408576698