| Index: chrome/browser/page_load_metrics/page_load_metrics_observer.h
|
| diff --git a/chrome/browser/page_load_metrics/page_load_metrics_observer.h b/chrome/browser/page_load_metrics/page_load_metrics_observer.h
|
| index c84787a443019d30ad4dcad76c606d1878f092f1..b46878c3f65b9f5014985c9f2a18ac535b7225d1 100644
|
| --- a/chrome/browser/page_load_metrics/page_load_metrics_observer.h
|
| +++ b/chrome/browser/page_load_metrics/page_load_metrics_observer.h
|
| @@ -122,8 +122,8 @@ struct PageLoadExtraInfo {
|
| PageEndReason page_end_reason,
|
| UserInitiatedInfo page_end_user_initiated_info,
|
| const base::Optional<base::TimeDelta>& page_end_time,
|
| - const PageLoadMetadata& main_frame_metadata,
|
| - const PageLoadMetadata& child_frame_metadata);
|
| + const mojom::PageLoadMetadata& main_frame_metadata,
|
| + const mojom::PageLoadMetadata& child_frame_metadata);
|
|
|
| // Simplified version of the constructor, intended for use in tests.
|
| static PageLoadExtraInfo CreateForTesting(const GURL& url,
|
| @@ -192,10 +192,10 @@ struct PageLoadExtraInfo {
|
|
|
| // Extra information supplied to the page load metrics system from the
|
| // renderer for the main frame.
|
| - const PageLoadMetadata main_frame_metadata;
|
| + const mojom::PageLoadMetadata main_frame_metadata;
|
|
|
| // PageLoadMetadata for child frames of the current page load.
|
| - const PageLoadMetadata child_frame_metadata;
|
| + const mojom::PageLoadMetadata child_frame_metadata;
|
| };
|
|
|
| // Container for various information about a completed request within a page
|
| @@ -308,7 +308,7 @@ class PageLoadMetricsObserver {
|
| // OnHidden is triggered when a page leaves the foreground. It does not fire
|
| // when a foreground page is permanently closed; for that, listen to
|
| // OnComplete instead.
|
| - virtual ObservePolicy OnHidden(const PageLoadTiming& timing,
|
| + virtual ObservePolicy OnHidden(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info);
|
|
|
| // OnShown is triggered when a page is brought to the foreground. It does not
|
| @@ -333,7 +333,7 @@ class PageLoadMetricsObserver {
|
| // implementers should implement one of the On* callbacks, such as
|
| // OnFirstContentfulPaint or OnDomContentLoadedEventStart. Please email
|
| // loading-dev@chromium.org if you intend to override this method.
|
| - virtual void OnTimingUpdate(const PageLoadTiming& timing,
|
| + virtual void OnTimingUpdate(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| // OnUserInput is triggered when a new user input is passed in to
|
| // web_contents. Contains a TimeDelta from navigation start.
|
| @@ -342,25 +342,25 @@ class PageLoadMetricsObserver {
|
| // The following methods are invoked at most once, when the timing for the
|
| // associated event first becomes available.
|
| virtual void OnDomContentLoadedEventStart(
|
| - const PageLoadTiming& timing,
|
| + const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnLoadEventStart(const PageLoadTiming& timing,
|
| + virtual void OnLoadEventStart(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnFirstLayout(const PageLoadTiming& timing,
|
| + virtual void OnFirstLayout(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnFirstPaint(const PageLoadTiming& timing,
|
| + virtual void OnFirstPaint(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnFirstTextPaint(const PageLoadTiming& timing,
|
| + virtual void OnFirstTextPaint(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnFirstImagePaint(const PageLoadTiming& timing,
|
| + virtual void OnFirstImagePaint(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnFirstContentfulPaint(const PageLoadTiming& timing,
|
| + virtual void OnFirstContentfulPaint(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnFirstMeaningfulPaint(const PageLoadTiming& timing,
|
| + virtual void OnFirstMeaningfulPaint(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnParseStart(const PageLoadTiming& timing,
|
| + virtual void OnParseStart(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
| - virtual void OnParseStop(const PageLoadTiming& timing,
|
| + virtual void OnParseStop(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
|
|
| // Invoked when there is a change in either the main_frame_metadata or the
|
| @@ -393,7 +393,7 @@ class PageLoadMetricsObserver {
|
| //
|
| // The default implementation does nothing, and returns CONTINUE_OBSERVING.
|
| virtual ObservePolicy FlushMetricsOnAppEnterBackground(
|
| - const PageLoadTiming& timing,
|
| + const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info);
|
|
|
| // One of OnComplete or OnFailedProvisionalLoad is invoked for tracked page
|
| @@ -410,7 +410,7 @@ class PageLoadMetricsObserver {
|
| // also want to implement FlushMetricsOnAppEnterBackground, to avoid loss of
|
| // data if the application is killed while in the background (this happens
|
| // frequently on Android).
|
| - virtual void OnComplete(const PageLoadTiming& timing,
|
| + virtual void OnComplete(const mojom::PageLoadTiming& timing,
|
| const PageLoadExtraInfo& extra_info) {}
|
|
|
| // OnFailedProvisionalLoad is invoked for tracked page loads that did not
|
|
|