| Index: chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.cc
|
| diff --git a/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.cc b/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.cc
|
| index 6312cdcf970edfe54a0066375c8d71cfd99d33bb..d25d6d220a9335361e5e4899496305854e785a30 100644
|
| --- a/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.cc
|
| +++ b/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.cc
|
| @@ -91,8 +91,7 @@ AdsPageLoadMetricsObserver::OnCommit(
|
| return CONTINUE_OBSERVING;
|
| }
|
|
|
| -page_load_metrics::PageLoadMetricsObserver::ObservePolicy
|
| -AdsPageLoadMetricsObserver::OnDidFinishSubFrameNavigation(
|
| +void AdsPageLoadMetricsObserver::OnDidFinishSubFrameNavigation(
|
| content::NavigationHandle* navigation_handle) {
|
| // Determine if the frame is part of an existing ad, the root of a new ad,
|
| // or a non-ad frame. Once a frame is labled as an ad, it is always
|
| @@ -115,7 +114,7 @@ AdsPageLoadMetricsObserver::OnDidFinishSubFrameNavigation(
|
| "PageLoad.Clients.Ads.Google.Navigations.AdFrameRenavigatedToAd",
|
| FrameIsAd(navigation_handle));
|
| }
|
| - return CONTINUE_OBSERVING;
|
| + return;
|
| }
|
| // This frame was previously not an ad, process it as usual. If it had
|
| // any child frames that were ads, those will still be recorded.
|
| @@ -132,7 +131,7 @@ AdsPageLoadMetricsObserver::OnDidFinishSubFrameNavigation(
|
| // it's a frame from a previous navigation.
|
| RecordParentExistsForSubFrame(false /* parent_exists */);
|
|
|
| - return CONTINUE_OBSERVING;
|
| + return;
|
| }
|
| RecordParentExistsForSubFrame(true /* parent_exists */);
|
|
|
| @@ -147,7 +146,6 @@ AdsPageLoadMetricsObserver::OnDidFinishSubFrameNavigation(
|
| ad_frames_data_[frame_tree_node_id] = ad_data;
|
|
|
| ProcessOngoingNavigationResource(frame_tree_node_id);
|
| - return CONTINUE_OBSERVING;
|
| }
|
|
|
| page_load_metrics::PageLoadMetricsObserver::ObservePolicy
|
|
|