[Page Load Metrics] Add mojom file to page load metrics.

chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/page_load_metrics/observers/amp_page_load_metrics_observer.h"
 
 #include <string>
 
 #include "base/macros.h"
 #include "base/optional.h"
 #include "base/time/time.h"
 #include "chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.h"
 #include "chrome/common/page_load_metrics/page_load_timing.h"
 #include "url/gurl.h"
 
 class AMPPageLoadMetricsObserverTest
     : public page_load_metrics::PageLoadMetricsObserverTestHarness {
  public:
   AMPPageLoadMetricsObserverTest() {}
 
   void ResetTest() {
+    page_load_metrics::InitPageLoadTimingForTest(&timing_);
     // 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_timing.parse_start = base::TimeDelta::FromSeconds(3);
-    timing_.paint_timing.first_contentful_paint =
+    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);
+    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_);
   }
 
   void RunTest(const GURL& url) {
     NavigateAndCommit(url);
     SimulateTimingUpdate(timing_);
 
     // Navigate again to force OnComplete, which happens when a new navigation
     // occurs.
     NavigateAndCommit(GURL("http://otherurl.com"));
   }
 
   void ValidateHistograms(bool expect_histograms, const char* view_type) {
     ValidateHistogramsFor(
         "PageLoad.Clients.AMP.DocumentTiming."
         "NavigationToDOMContentLoadedEventFired",
-        view_type, timing_.document_timing.dom_content_loaded_event_start,
+        view_type, timing_.document_timing->dom_content_loaded_event_start,
         expect_histograms);
     ValidateHistogramsFor(
         "PageLoad.Clients.AMP.DocumentTiming.NavigationToFirstLayout",
-        view_type, timing_.document_timing.first_layout, expect_histograms);
+        view_type, timing_.document_timing->first_layout, expect_histograms);
     ValidateHistogramsFor(
         "PageLoad.Clients.AMP.DocumentTiming."
         "NavigationToLoadEventFired",
-        view_type, timing_.document_timing.load_event_start, expect_histograms);
+        view_type, timing_.document_timing->load_event_start,
+        expect_histograms);
     ValidateHistogramsFor(
         "PageLoad.Clients.AMP.PaintTiming."
         "NavigationToFirstContentfulPaint",
-        view_type, timing_.paint_timing.first_contentful_paint,
+        view_type, timing_.paint_timing->first_contentful_paint,
         expect_histograms);
     ValidateHistogramsFor(
         "PageLoad.Clients.AMP.ParseTiming.NavigationToParseStart", view_type,
-        timing_.parse_timing.parse_start, expect_histograms);
+        timing_.parse_timing->parse_start, expect_histograms);
   }
 
   void ValidateHistogramsFor(const std::string& histogram,
                              const char* view_type,
                              const base::Optional<base::TimeDelta>& event,
                              bool expect_histograms) {
     const size_t kTypeOffset = strlen("PageLoad.Clients.AMP.");
     std::string view_type_histogram = histogram;
     view_type_histogram.insert(kTypeOffset, view_type);
     histogram_tester().ExpectTotalCount(histogram, expect_histograms ? 1 : 0);
@@ skipping 12 matching lines @@
             event.value().InMilliseconds()),
         1);
   }
 
  protected:
   void RegisterObservers(page_load_metrics::PageLoadTracker* tracker) override {
     tracker->AddObserver(base::WrapUnique(new AMPPageLoadMetricsObserver()));
   }
 
  private:
-  page_load_metrics::PageLoadTiming timing_;
+  page_load_metrics::mojom::PageLoadTiming timing_;
 
   DISALLOW_COPY_AND_ASSIGN(AMPPageLoadMetricsObserverTest);
 };
 
 TEST_F(AMPPageLoadMetricsObserverTest, AMPViewType) {
   using AMPViewType = AMPPageLoadMetricsObserver::AMPViewType;
 
   struct {
     AMPViewType expected_type;
     const char* url;
@@ skipping 35 matching lines @@
   ResetTest();
   RunTest(GURL("https://news.google.com/news/amp?page"));
   ValidateHistograms(true, "GoogleNews.");
 }
 
 TEST_F(AMPPageLoadMetricsObserverTest, NonAMPPage) {
   ResetTest();
   RunTest(GURL("https://www.google.com/not-amp/page"));
   ValidateHistograms(false, "");
 }