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

Unified Diff: chrome/browser/page_load_metrics/page_load_metrics_util.cc

Issue 2359523003: Remove the Timing2 histograms that are deprecated. (Closed)
Patch Set: remove WasParseInForeground Created 4 years, 3 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
« no previous file with comments | « chrome/browser/page_load_metrics/page_load_metrics_util.h ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/page_load_metrics/page_load_metrics_util.cc
diff --git a/chrome/browser/page_load_metrics/page_load_metrics_util.cc b/chrome/browser/page_load_metrics/page_load_metrics_util.cc
index 2c0b31d4be9dd454fcf07f66b88290b5db6eeb36..1203a2d778bb074ecbe03490806d22ab957e0472 100644
--- a/chrome/browser/page_load_metrics/page_load_metrics_util.cc
+++ b/chrome/browser/page_load_metrics/page_load_metrics_util.cc
@@ -19,16 +19,4 @@ bool WasStartedInForegroundOptionalEventInForeground(
event.value() <= info.first_background_time.value());
}
-bool WasParseInForeground(const base::Optional<base::TimeDelta>& parse_start,
- const base::Optional<base::TimeDelta>& parse_stop,
- const PageLoadExtraInfo& info) {
- if (!parse_start) {
- return false;
- }
- const bool incomplete_parse_in_foreground =
- !parse_stop && info.started_in_foreground && !info.first_background_time;
-
- return incomplete_parse_in_foreground ||
- WasStartedInForegroundOptionalEventInForeground(parse_stop, info);
-}
} // namespace page_load_metrics
« no previous file with comments | « chrome/browser/page_load_metrics/page_load_metrics_util.h ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698