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

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

Issue 1433893002: Add StaleWhileRevalidateExperiment histograms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@s-w-r-dafsa
Patch Set: Fix compile error. Created 5 years 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/browser_process.h" 7 #include "chrome/browser/browser_process.h"
8 #include "chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_ observer.h" 8 #include "chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_ observer.h"
9 #include "chrome/browser/page_load_metrics/observers/google_captcha_observer.h" 9 #include "chrome/browser/page_load_metrics/observers/google_captcha_observer.h"
10 #include "chrome/browser/page_load_metrics/observers/stale_while_revalidate_metr ics_observer.h"
10 #include "chrome/browser/prerender/prerender_contents.h" 11 #include "chrome/browser/prerender/prerender_contents.h"
11 #include "components/page_load_metrics/browser/metrics_web_contents_observer.h" 12 #include "components/page_load_metrics/browser/metrics_web_contents_observer.h"
12 #include "components/rappor/rappor_service.h" 13 #include "components/rappor/rappor_service.h"
13 #include "content/public/browser/web_contents.h" 14 #include "content/public/browser/web_contents.h"
14 15
15 namespace { 16 namespace {
16 17
17 void RegisterPageLoadMetricsObservers( 18 void RegisterPageLoadMetricsObservers(
18 page_load_metrics::PageLoadMetricsObservable* metrics) { 19 page_load_metrics::PageLoadMetricsObservable* metrics) {
19 // Attach observers scoped to the web contents here. 20 // Attach observers scoped to the web contents here.
20 21
21 // This is a self-destruct class, and will delete itself when triggered by 22 // This is a self-destruct class, and will delete itself when triggered by
22 // OnPageLoadMetricsGoingAway. 23 // OnPageLoadMetricsGoingAway.
23 metrics->AddObserver(new FromGWSPageLoadMetricsObserver(metrics)); 24 metrics->AddObserver(new FromGWSPageLoadMetricsObserver(metrics));
24 metrics->AddObserver( 25 metrics->AddObserver(
25 new google_captcha_observer::GoogleCaptchaObserver(metrics)); 26 new google_captcha_observer::GoogleCaptchaObserver(metrics));
27 // StaleWhileRevalidateMetricsObserver also deletes itself from
28 // OnPageLoadMetricsGoingAway.
29 // TODO(ricea): Remove this in April 2016 or before. crbug.com/348877
30 metrics->AddObserver(
31 new chrome::StaleWhileRevalidateMetricsObserver(metrics));
26 } 32 }
27 33
28 } // namespace 34 } // namespace
29 35
30 namespace chrome { 36 namespace chrome {
31 37
32 void InitializePageLoadMetricsForWebContents( 38 void InitializePageLoadMetricsForWebContents(
33 content::WebContents* web_contents) { 39 content::WebContents* web_contents) {
34 RegisterPageLoadMetricsObservers( 40 RegisterPageLoadMetricsObservers(
35 page_load_metrics::MetricsWebContentsObserver::CreateForWebContents( 41 page_load_metrics::MetricsWebContentsObserver::CreateForWebContents(
36 web_contents, 42 web_contents,
37 make_scoped_ptr(new PageLoadMetricsEmbedderInterfaceImpl()))); 43 make_scoped_ptr(new PageLoadMetricsEmbedderInterfaceImpl())));
38 } 44 }
39 45
40 PageLoadMetricsEmbedderInterfaceImpl::~PageLoadMetricsEmbedderInterfaceImpl() {} 46 PageLoadMetricsEmbedderInterfaceImpl::~PageLoadMetricsEmbedderInterfaceImpl() {}
41 47
42 rappor::RapporService* 48 rappor::RapporService*
43 PageLoadMetricsEmbedderInterfaceImpl::GetRapporService() { 49 PageLoadMetricsEmbedderInterfaceImpl::GetRapporService() {
44 return g_browser_process->rappor_service(); 50 return g_browser_process->rappor_service();
45 } 51 }
46 52
47 bool PageLoadMetricsEmbedderInterfaceImpl::IsPrerendering( 53 bool PageLoadMetricsEmbedderInterfaceImpl::IsPrerendering(
48 content::WebContents* web_contents) { 54 content::WebContents* web_contents) {
49 return prerender::PrerenderContents::FromWebContents(web_contents) != nullptr; 55 return prerender::PrerenderContents::FromWebContents(web_contents) != nullptr;
50 } 56 }
51 57
52 } // namespace chrome 58 } // namespace chrome
OLDNEW
« no previous file with comments | « chrome/browser/page_load_metrics/observers/stale_while_revalidate_metrics_observer.cc ('k') | chrome/browser/ui/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698