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

Unified Diff: chrome/browser/prerender/prerender_histograms.cc

Issue 2423383002: [Prerender] first contentful paint histograms. (Closed)
Patch Set: comments Created 4 years, 1 month 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: chrome/browser/prerender/prerender_histograms.cc
diff --git a/chrome/browser/prerender/prerender_histograms.cc b/chrome/browser/prerender/prerender_histograms.cc
index 4e97663049a8d3e75b14b55e98a28ec60af7f2cc..18254d3daf891549cecc16214c95f3a3f83eb34e 100644
--- a/chrome/browser/prerender/prerender_histograms.cc
+++ b/chrome/browser/prerender/prerender_histograms.cc
@@ -310,6 +310,23 @@ void PrerenderHistograms::RecordPerceivedPageLoadTime(
}
}
+void PrerenderHistograms::RecordPerceivedFirstContentfulPaintTime(
+ Origin origin,
+ base::TimeDelta perceived_first_contentful_paint) {
+ RecordHistogramTime(
+ GetPerceivedFirstContentfulPaintHistogramName(origin, IsOriginWash()),
+ base::TimeDelta::FromMilliseconds(10), base::TimeDelta::FromMinutes(2),
+ perceived_first_contentful_paint, 50);
+}
+
+void PrerenderHistograms::RecordPerceivedFirstContentfulPaintStatus(
+ Origin origin,
+ bool successful) {
+ RecordHistogramEnum(
+ GetHistogramName(origin, IsOriginWash(), "PerceivedTTFCPRecorded"),
+ successful ? 1 : 0, 2);
+}
+
void PrerenderHistograms::RecordPageLoadTimeNotSwappedIn(
Origin origin,
base::TimeDelta page_load_time,
@@ -452,7 +469,7 @@ void PrerenderHistograms::RecordPrefetchRedirectCount(
RecordHistogramEnum(histogram_name, redirect_count, kMaxRedirectCount);
}
-void PrerenderHistograms::RecordFirstContentfulPaint(
+void PrerenderHistograms::RecordNoStateFirstContentfulPaint(
Origin origin,
bool is_no_store,
base::TimeDelta time,
@@ -467,7 +484,7 @@ void PrerenderHistograms::RecordFirstContentfulPaint(
base::TimeDelta::FromMinutes(30), prefetch_age, 50);
}
- RecordHistogramTime(GetFirstContentfulPaintHistogramName(
+ RecordHistogramTime(GetNoStateFirstContentfulPaintHistogramName(
origin, IsOriginWash(), is_no_store, prefetch_age),
base::TimeDelta::FromMilliseconds(10),
base::TimeDelta::FromMinutes(2), time, 50);
@@ -480,7 +497,7 @@ bool PrerenderHistograms::IsOriginWash() const {
}
// static
-std::string PrerenderHistograms::GetFirstContentfulPaintHistogramName(
+std::string PrerenderHistograms::GetNoStateFirstContentfulPaintHistogramName(
Origin origin,
bool is_wash,
bool is_no_store,
@@ -499,4 +516,11 @@ std::string PrerenderHistograms::GetFirstContentfulPaintHistogramName(
return GetHistogramName(origin, is_wash, histogram_base_name);
}
+// static
+std::string PrerenderHistograms::GetPerceivedFirstContentfulPaintHistogramName(
+ Origin origin,
+ bool is_wash) {
+ return GetHistogramName(origin, is_wash, "PerceivedTTFCP");
+}
+
} // namespace prerender

Powered by Google App Engine
This is Rietveld 408576698