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

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

Issue 2481013007: Improve tracking of user initiated page loads. (Closed)
Patch Set: fix tests Created 4 years, 1 month 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_observer.h" 5 #include "chrome/browser/page_load_metrics/page_load_metrics_observer.h"
6 6
7 namespace page_load_metrics { 7 namespace page_load_metrics {
8 8
9 PageLoadExtraInfo::PageLoadExtraInfo( 9 PageLoadExtraInfo::PageLoadExtraInfo(
10 const base::Optional<base::TimeDelta>& first_background_time, 10 const base::Optional<base::TimeDelta>& first_background_time,
11 const base::Optional<base::TimeDelta>& first_foreground_time, 11 const base::Optional<base::TimeDelta>& first_foreground_time,
12 bool started_in_foreground, 12 bool started_in_foreground,
13 bool user_gesture, 13 bool user_initiated,
14 const GURL& committed_url, 14 const GURL& committed_url,
15 const GURL& start_url, 15 const GURL& start_url,
16 UserAbortType abort_type, 16 UserAbortType abort_type,
17 bool abort_user_initiated,
18 const base::Optional<base::TimeDelta>& time_to_abort, 17 const base::Optional<base::TimeDelta>& time_to_abort,
19 int num_cache_requests, 18 int num_cache_requests,
20 int num_network_requests, 19 int num_network_requests,
21 const PageLoadMetadata& metadata) 20 const PageLoadMetadata& metadata)
22 : first_background_time(first_background_time), 21 : first_background_time(first_background_time),
23 first_foreground_time(first_foreground_time), 22 first_foreground_time(first_foreground_time),
24 started_in_foreground(started_in_foreground), 23 started_in_foreground(started_in_foreground),
25 user_gesture(user_gesture), 24 user_initiated(user_initiated),
26 committed_url(committed_url), 25 committed_url(committed_url),
27 start_url(start_url), 26 start_url(start_url),
28 abort_type(abort_type), 27 abort_type(abort_type),
29 abort_user_initiated(abort_user_initiated),
30 time_to_abort(time_to_abort), 28 time_to_abort(time_to_abort),
31 num_cache_requests(num_cache_requests), 29 num_cache_requests(num_cache_requests),
32 num_network_requests(num_network_requests), 30 num_network_requests(num_network_requests),
33 metadata(metadata) {} 31 metadata(metadata) {}
34 32
35 PageLoadExtraInfo::PageLoadExtraInfo(const PageLoadExtraInfo& other) = default; 33 PageLoadExtraInfo::PageLoadExtraInfo(const PageLoadExtraInfo& other) = default;
36 34
37 PageLoadExtraInfo::~PageLoadExtraInfo() {} 35 PageLoadExtraInfo::~PageLoadExtraInfo() {}
38 36
39 FailedProvisionalLoadInfo::FailedProvisionalLoadInfo(base::TimeDelta interval, 37 FailedProvisionalLoadInfo::FailedProvisionalLoadInfo(base::TimeDelta interval,
(...skipping 30 matching lines...) Expand all
70 } 68 }
71 69
72 PageLoadMetricsObserver::ObservePolicy 70 PageLoadMetricsObserver::ObservePolicy
73 PageLoadMetricsObserver::FlushMetricsOnAppEnterBackground( 71 PageLoadMetricsObserver::FlushMetricsOnAppEnterBackground(
74 const PageLoadTiming& timing, 72 const PageLoadTiming& timing,
75 const PageLoadExtraInfo& extra_info) { 73 const PageLoadExtraInfo& extra_info) {
76 return CONTINUE_OBSERVING; 74 return CONTINUE_OBSERVING;
77 } 75 }
78 76
79 } // namespace page_load_metrics 77 } // namespace page_load_metrics
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698