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

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

Issue 2888673002: Add support for counting same-document AMP loads. (Closed)
Patch Set: comment cleanup Created 3 years, 7 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/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

Powered by Google App Engine
This is Rietveld 408576698