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

Side by Side Diff: chrome/browser/page_load_metrics/page_load_metrics_initialize.cc

Issue 1857443002: Plumb experiment flags through page_load_metrics and add a new observer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@page_load_experiments
Patch Set: No need to add Default group to variation config Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/page_load_metrics/page_load_metrics_initialize.h" 5 #include "chrome/browser/page_load_metrics/page_load_metrics_initialize.h"
6 6
7 #include "chrome/browser/page_load_metrics/observers/aborts_page_load_metrics_ob server.h" 7 #include "chrome/browser/page_load_metrics/observers/aborts_page_load_metrics_ob server.h"
8 #include "chrome/browser/page_load_metrics/observers/core_page_load_metrics_obse rver.h" 8 #include "chrome/browser/page_load_metrics/observers/core_page_load_metrics_obse rver.h"
9 #include "chrome/browser/page_load_metrics/observers/document_write_page_load_me trics_observer.h"
9 #include "chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_ observer.h" 10 #include "chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_ observer.h"
10 #include "chrome/browser/page_load_metrics/observers/google_captcha_observer.h" 11 #include "chrome/browser/page_load_metrics/observers/google_captcha_observer.h"
11 #include "chrome/browser/page_load_metrics/observers/stale_while_revalidate_metr ics_observer.h" 12 #include "chrome/browser/page_load_metrics/observers/stale_while_revalidate_metr ics_observer.h"
12 #include "chrome/browser/prerender/prerender_contents.h" 13 #include "chrome/browser/prerender/prerender_contents.h"
13 #include "components/page_load_metrics/browser/metrics_web_contents_observer.h" 14 #include "components/page_load_metrics/browser/metrics_web_contents_observer.h"
14 #include "components/rappor/rappor_service.h" 15 #include "components/rappor/rappor_service.h"
15 #include "content/public/browser/web_contents.h" 16 #include "content/public/browser/web_contents.h"
16 17
17 namespace chrome { 18 namespace chrome {
18 19
19 void InitializePageLoadMetricsForWebContents( 20 void InitializePageLoadMetricsForWebContents(
20 content::WebContents* web_contents) { 21 content::WebContents* web_contents) {
21 page_load_metrics::MetricsWebContentsObserver::CreateForWebContents( 22 page_load_metrics::MetricsWebContentsObserver::CreateForWebContents(
22 web_contents, make_scoped_ptr(new PageLoadMetricsEmbedder())); 23 web_contents, make_scoped_ptr(new PageLoadMetricsEmbedder()));
23 } 24 }
24 25
25 PageLoadMetricsEmbedder::~PageLoadMetricsEmbedder() {} 26 PageLoadMetricsEmbedder::~PageLoadMetricsEmbedder() {}
26 27
27 void PageLoadMetricsEmbedder::RegisterObservers( 28 void PageLoadMetricsEmbedder::RegisterObservers(
28 page_load_metrics::PageLoadTracker* tracker) { 29 page_load_metrics::PageLoadTracker* tracker) {
29 // These classes are owned by the metrics. 30 // These classes are owned by the metrics.
30 tracker->AddObserver(make_scoped_ptr(new AbortsPageLoadMetricsObserver())); 31 tracker->AddObserver(make_scoped_ptr(new AbortsPageLoadMetricsObserver()));
31 tracker->AddObserver(make_scoped_ptr(new CorePageLoadMetricsObserver())); 32 tracker->AddObserver(make_scoped_ptr(new CorePageLoadMetricsObserver()));
32 tracker->AddObserver(make_scoped_ptr(new FromGWSPageLoadMetricsObserver())); 33 tracker->AddObserver(make_scoped_ptr(new FromGWSPageLoadMetricsObserver()));
33 tracker->AddObserver( 34 tracker->AddObserver(
34 make_scoped_ptr(new google_captcha_observer::GoogleCaptchaObserver())); 35 make_scoped_ptr(new google_captcha_observer::GoogleCaptchaObserver()));
35 // TODO(ricea): Remove this in April 2016 or before. crbug.com/348877 36 // TODO(ricea): Remove this in April 2016 or before. crbug.com/348877
36 tracker->AddObserver( 37 tracker->AddObserver(
37 make_scoped_ptr(new chrome::StaleWhileRevalidateMetricsObserver())); 38 make_scoped_ptr(new chrome::StaleWhileRevalidateMetricsObserver()));
39 tracker->AddObserver(
40 make_scoped_ptr(new DocumentWritePageLoadMetricsObserver()));
38 } 41 }
39 42
40 bool PageLoadMetricsEmbedder::IsPrerendering( 43 bool PageLoadMetricsEmbedder::IsPrerendering(
41 content::WebContents* web_contents) { 44 content::WebContents* web_contents) {
42 return prerender::PrerenderContents::FromWebContents(web_contents) != nullptr; 45 return prerender::PrerenderContents::FromWebContents(web_contents) != nullptr;
43 } 46 }
44 47
45 } // namespace chrome 48 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698