PageLoadMetricsObservers observe individual page loads

chrome/browser/page_load_metrics/page_load_metrics_initialize.cc

Index: chrome/browser/page_load_metrics/page_load_metrics_initialize.cc
diff --git a/chrome/browser/page_load_metrics/page_load_metrics_initialize.cc b/chrome/browser/page_load_metrics/page_load_metrics_initialize.cc
index 9dcbb11d82735f240b3f1a638bc278ac31578251..891d9150f6b2740e85c21c8ff36e423146fdfc6d 100644
--- a/chrome/browser/page_load_metrics/page_load_metrics_initialize.cc
+++ b/chrome/browser/page_load_metrics/page_load_metrics_initialize.cc
@@ -13,38 +13,28 @@
 #include "components/rappor/rappor_service.h"
 #include "content/public/browser/web_contents.h"
 
-namespace {
-
-void RegisterPageLoadMetricsObservers(
-    page_load_metrics::PageLoadMetricsObservable* metrics) {
-  // Attach observers scoped to the web contents here.
-
-  // This is a self-destruct class, and will delete itself when triggered by
-  // OnPageLoadMetricsGoingAway.
-  metrics->AddObserver(new FromGWSPageLoadMetricsObserver(metrics));
-  metrics->AddObserver(
-      new google_captcha_observer::GoogleCaptchaObserver(metrics));
-  // StaleWhileRevalidateMetricsObserver also deletes itself from
-  // OnPageLoadMetricsGoingAway.
-  // TODO(ricea): Remove this in April 2016 or before. crbug.com/348877
-  metrics->AddObserver(
-      new chrome::StaleWhileRevalidateMetricsObserver(metrics));
-}
-
-}  // namespace
-
 namespace chrome {
 
 void InitializePageLoadMetricsForWebContents(
     content::WebContents* web_contents) {
-  RegisterPageLoadMetricsObservers(
-      page_load_metrics::MetricsWebContentsObserver::CreateForWebContents(
-          web_contents,
-          make_scoped_ptr(new PageLoadMetricsEmbedderInterfaceImpl())));
+  page_load_metrics::MetricsWebContentsObserver::CreateForWebContents(
+      web_contents,
+      make_scoped_ptr(new PageLoadMetricsEmbedderInterfaceImpl()));
 }
 
 PageLoadMetricsEmbedderInterfaceImpl::~PageLoadMetricsEmbedderInterfaceImpl() {}
 
+void PageLoadMetricsEmbedderInterfaceImpl::RegisterObservers(
+    page_load_metrics::PageLoadTracker* tracker) {
+  // These classes are owned by the metrics.
+  tracker->AddObserver(make_scoped_ptr(new FromGWSPageLoadMetricsObserver()));
+  tracker->AddObserver(
+      make_scoped_ptr(new google_captcha_observer::GoogleCaptchaObserver()));
+  // TODO(ricea): Remove this in April 2016 or before. crbug.com/348877
+  tracker->AddObserver(
+      make_scoped_ptr(new chrome::StaleWhileRevalidateMetricsObserver()));
+}
+
 rappor::RapporService*
 PageLoadMetricsEmbedderInterfaceImpl::GetRapporService() {
   // During the browser process shutdown path, calling this getter can