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

Unified Diff: chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer_unittest.cc

Issue 2806863003: [Page Load Metrics] Structure PageLoadTiming. (Closed)
Patch Set: rebase Created 3 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: chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer_unittest.cc
diff --git a/chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer_unittest.cc b/chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer_unittest.cc
index eae1092739f32a39173fc1a487806da6bae78113..7ab9d7ea98ee2605e3ce1651f69f6166e971d616 100644
--- a/chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer_unittest.cc
+++ b/chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer_unittest.cc
@@ -22,11 +22,12 @@ class AMPPageLoadMetricsObserverTest
// Reset to the default testing state. Does not reset histogram state.
timing_.navigation_start = base::Time::FromDoubleT(1);
timing_.response_start = base::TimeDelta::FromSeconds(2);
- timing_.parse_start = base::TimeDelta::FromSeconds(3);
- timing_.first_contentful_paint = base::TimeDelta::FromSeconds(4);
- timing_.first_image_paint = base::TimeDelta::FromSeconds(5);
- timing_.first_text_paint = base::TimeDelta::FromSeconds(6);
- timing_.load_event_start = base::TimeDelta::FromSeconds(7);
+ timing_.parse_timing.parse_start = base::TimeDelta::FromSeconds(3);
+ timing_.paint_timing.first_contentful_paint =
+ base::TimeDelta::FromSeconds(4);
+ timing_.paint_timing.first_image_paint = base::TimeDelta::FromSeconds(5);
+ timing_.paint_timing.first_text_paint = base::TimeDelta::FromSeconds(6);
+ timing_.document_timing.load_event_start = base::TimeDelta::FromSeconds(7);
PopulateRequiredTimingFields(&timing_);
}
@@ -43,21 +44,22 @@ class AMPPageLoadMetricsObserverTest
ValidateHistogramsFor(
"PageLoad.Clients.AMPCache2.DocumentTiming."
"NavigationToDOMContentLoadedEventFired",
- timing_.dom_content_loaded_event_start, expect_histograms);
+ timing_.document_timing.dom_content_loaded_event_start,
+ expect_histograms);
ValidateHistogramsFor(
"PageLoad.Clients.AMPCache2.DocumentTiming.NavigationToFirstLayout",
- timing_.first_layout, expect_histograms);
+ timing_.document_timing.first_layout, expect_histograms);
ValidateHistogramsFor(
"PageLoad.Clients.AMPCache2.DocumentTiming."
"NavigationToLoadEventFired",
- timing_.load_event_start, expect_histograms);
+ timing_.document_timing.load_event_start, expect_histograms);
ValidateHistogramsFor(
"PageLoad.Clients.AMPCache2.PaintTiming."
"NavigationToFirstContentfulPaint",
- timing_.first_contentful_paint, expect_histograms);
+ timing_.paint_timing.first_contentful_paint, expect_histograms);
ValidateHistogramsFor(
"PageLoad.Clients.AMPCache2.ParseTiming.NavigationToParseStart",
- timing_.parse_start, expect_histograms);
+ timing_.parse_timing.parse_start, expect_histograms);
}
void ValidateHistogramsFor(const std::string& histogram_,

Powered by Google App Engine
This is Rietveld 408576698