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

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

Issue 1913693002: Histogram the amount of time spent on HTTP(S) page loads (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add comments 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/observers/stale_while_revalidate_metr ics_observer.h" 5 #include "chrome/browser/page_load_metrics/observers/stale_while_revalidate_metr ics_observer.h"
6 6
7 #include "components/page_load_metrics/browser/page_load_metrics_util.h" 7 #include "components/page_load_metrics/browser/page_load_metrics_util.h"
8 #include "components/page_load_metrics/common/page_load_timing.h" 8 #include "components/page_load_metrics/common/page_load_timing.h"
9 #include "content/public/browser/navigation_handle.h" 9 #include "content/public/browser/navigation_handle.h"
10 #include "net/base/stale_while_revalidate_experiment_domains.h" 10 #include "net/base/stale_while_revalidate_experiment_domains.h"
11 11
12 namespace chrome { 12 namespace chrome {
13 13
14 StaleWhileRevalidateMetricsObserver::StaleWhileRevalidateMetricsObserver() 14 StaleWhileRevalidateMetricsObserver::StaleWhileRevalidateMetricsObserver()
15 : is_interesting_domain_(false) {} 15 : is_interesting_domain_(false) {}
16 16
17 void StaleWhileRevalidateMetricsObserver::OnCommit( 17 void StaleWhileRevalidateMetricsObserver::OnCommit(
18 content::NavigationHandle* navigation_handle) { 18 content::NavigationHandle* navigation_handle,
19 bool started_in_foreground) {
19 is_interesting_domain_ = net::IsHostInStaleWhileRevalidateExperimentDomain( 20 is_interesting_domain_ = net::IsHostInStaleWhileRevalidateExperimentDomain(
20 navigation_handle->GetURL().host()); 21 navigation_handle->GetURL().host());
21 } 22 }
22 23
23 void StaleWhileRevalidateMetricsObserver::OnComplete( 24 void StaleWhileRevalidateMetricsObserver::OnComplete(
24 const page_load_metrics::PageLoadTiming& timing, 25 const page_load_metrics::PageLoadTiming& timing,
25 const page_load_metrics::PageLoadExtraInfo& extra_info) { 26 const page_load_metrics::PageLoadExtraInfo& extra_info) {
26 using page_load_metrics::WasStartedInForegroundEventInForeground; 27 using page_load_metrics::WasStartedInForegroundEventInForeground;
27 28
28 if (!is_interesting_domain_) 29 if (!is_interesting_domain_)
(...skipping 16 matching lines...) Expand all
45 if (WasStartedInForegroundEventInForeground(timing.first_text_paint, 46 if (WasStartedInForegroundEventInForeground(timing.first_text_paint,
46 extra_info)) { 47 extra_info)) {
47 PAGE_LOAD_HISTOGRAM( 48 PAGE_LOAD_HISTOGRAM(
48 "PageLoad.Clients.StaleWhileRevalidateExperiment.Timing2." 49 "PageLoad.Clients.StaleWhileRevalidateExperiment.Timing2."
49 "NavigationToFirstTextPaint", 50 "NavigationToFirstTextPaint",
50 timing.first_text_paint); 51 timing.first_text_paint);
51 } 52 }
52 } 53 }
53 54
54 } // namespace chrome 55 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698