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 b1b2c49c5f60fb9edf8e7bbd0a28d2fd7ce9a879..cd38fd5fb545931ab0dbeff072e7502aa1c5de68 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 |
@@ -86,8 +86,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 |
@@ -110,7 +109,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. |
@@ -134,7 +133,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 |