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

Side by Side Diff: chrome/browser/page_load_metrics/observers/prerender_page_load_metrics_observer.cc

Issue 2423383002: [Prerender] first contentful paint histograms. (Closed)
Patch Set: comments Created 4 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
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/page_load_metrics/observers/prerender_page_load_metrics _observer.h"
6
7 #include "base/memory/ptr_util.h"
8 #include "chrome/browser/prerender/prerender_manager.h"
9 #include "chrome/browser/prerender/prerender_manager_factory.h"
10 #include "content/public/browser/web_contents.h"
11
12 // static
13 std::unique_ptr<page_load_metrics::PageLoadMetricsObserver>
14 PrerenderPageLoadMetricsObserver::CreateIfNeeded(
15 content::WebContents* web_contents) {
16 prerender::PrerenderManager* manager =
17 prerender::PrerenderManagerFactory::GetForBrowserContext(
18 web_contents->GetBrowserContext());
19 if (!manager)
20 return nullptr;
21
22 if (manager->PageLoadMetricsObserverDisabledForTesting())
23 return nullptr;
24
25 return base::WrapUnique<page_load_metrics::PageLoadMetricsObserver>(
26 new PrerenderPageLoadMetricsObserver(manager, web_contents));
27 }
28
29 PrerenderPageLoadMetricsObserver::PrerenderPageLoadMetricsObserver(
30 prerender::PrerenderManager* manager,
31 content::WebContents* web_contents)
32 : prerender_manager_(manager),
33 web_contents_(web_contents),
34 was_hidden_(false) {
35 DCHECK(prerender_manager_);
36 }
37
38 page_load_metrics::PageLoadMetricsObserver::ObservePolicy
39 PrerenderPageLoadMetricsObserver::OnStart(
40 content::NavigationHandle* navigation_handle,
41 const GURL& currently_committed_url,
42 bool started_in_foreground) {
43 DCHECK(!started_in_foreground);
44
45 start_ticks_ = navigation_handle->NavigationStart();
46 return CONTINUE_OBSERVING;
47 }
48
49 void PrerenderPageLoadMetricsObserver::OnFirstContentfulPaint(
50 const page_load_metrics::PageLoadTiming& timing,
51 const page_load_metrics::PageLoadExtraInfo& extra_info) {
52 DCHECK(!start_ticks_.is_null());
53 prerender_manager_->RecordPerceivedFirstContentfulPaint(
Bryan McQuade 2016/11/23 13:51:23 optional: it turns out that, given your implementa
54 web_contents_, start_ticks_ + *timing.first_contentful_paint,
55 was_hidden_);
56 }
57
58 page_load_metrics::PageLoadMetricsObserver::ObservePolicy
59 PrerenderPageLoadMetricsObserver::OnHidden(
60 const page_load_metrics::PageLoadTiming& timing,
61 const page_load_metrics::PageLoadExtraInfo& extra_info) {
62 was_hidden_ = true;
63 return CONTINUE_OBSERVING;
64 }
65
66 void PrerenderPageLoadMetricsObserver::SetNavigationStartTicksForTesting(
67 base::TimeTicks ticks) {
68 start_ticks_ = ticks;
69 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698