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

Side by Side Diff: chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer.h

Issue 2874663005: [Page Load Metrics] Add mojom file to page load metrics. (Closed)
Patch Set: Remove unnecessary variable Created 3 years, 7 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 #ifndef CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_FROM_GWS_PAGE_LOAD_METRICS_OB SERVER_H_ 5 #ifndef CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_FROM_GWS_PAGE_LOAD_METRICS_OB SERVER_H_
6 #define CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_FROM_GWS_PAGE_LOAD_METRICS_OB SERVER_H_ 6 #define CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_FROM_GWS_PAGE_LOAD_METRICS_OB SERVER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/optional.h" 9 #include "base/optional.h"
10 #include "chrome/browser/page_load_metrics/page_load_metrics_observer.h" 10 #include "chrome/browser/page_load_metrics/page_load_metrics_observer.h"
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 navigation_initiated_via_link_ = navigation_initiated_via_link; 55 navigation_initiated_via_link_ = navigation_initiated_via_link;
56 } 56 }
57 57
58 void SetNavigationStart(const base::TimeTicks navigation_start) { 58 void SetNavigationStart(const base::TimeTicks navigation_start) {
59 // Should be invoked at most once 59 // Should be invoked at most once
60 DCHECK(navigation_start_.is_null()); 60 DCHECK(navigation_start_.is_null());
61 navigation_start_ = navigation_start; 61 navigation_start_ = navigation_start;
62 } 62 }
63 63
64 // Invoked when metrics for the given page are complete. 64 // Invoked when metrics for the given page are complete.
65 void OnComplete(const page_load_metrics::PageLoadTiming& timing, 65 void OnComplete(const page_load_metrics::mojom::PageLoadTiming& timing,
66 const page_load_metrics::PageLoadExtraInfo& extra_info); 66 const page_load_metrics::PageLoadExtraInfo& extra_info);
67 void OnFailedProvisionalLoad( 67 void OnFailedProvisionalLoad(
68 const page_load_metrics::FailedProvisionalLoadInfo& failed_load_info, 68 const page_load_metrics::FailedProvisionalLoadInfo& failed_load_info,
69 const page_load_metrics::PageLoadExtraInfo& extra_info); 69 const page_load_metrics::PageLoadExtraInfo& extra_info);
70 70
71 void OnDomContentLoadedEventStart( 71 void OnDomContentLoadedEventStart(
72 const page_load_metrics::PageLoadTiming& timing, 72 const page_load_metrics::mojom::PageLoadTiming& timing,
73 const page_load_metrics::PageLoadExtraInfo& extra_info); 73 const page_load_metrics::PageLoadExtraInfo& extra_info);
74 void OnLoadEventStart(const page_load_metrics::PageLoadTiming& timing, 74 void OnLoadEventStart(const page_load_metrics::mojom::PageLoadTiming& timing,
75 const page_load_metrics::PageLoadExtraInfo& extra_info); 75 const page_load_metrics::PageLoadExtraInfo& extra_info);
76 void OnFirstPaintInPage( 76 void OnFirstPaintInPage(
77 const page_load_metrics::PageLoadTiming& timing, 77 const page_load_metrics::mojom::PageLoadTiming& timing,
78 const page_load_metrics::PageLoadExtraInfo& extra_info); 78 const page_load_metrics::PageLoadExtraInfo& extra_info);
79 void OnFirstTextPaintInPage( 79 void OnFirstTextPaintInPage(
80 const page_load_metrics::PageLoadTiming& timing, 80 const page_load_metrics::mojom::PageLoadTiming& timing,
81 const page_load_metrics::PageLoadExtraInfo& extra_info); 81 const page_load_metrics::PageLoadExtraInfo& extra_info);
82 void OnFirstImagePaintInPage( 82 void OnFirstImagePaintInPage(
83 const page_load_metrics::PageLoadTiming& timing, 83 const page_load_metrics::mojom::PageLoadTiming& timing,
84 const page_load_metrics::PageLoadExtraInfo& extra_info); 84 const page_load_metrics::PageLoadExtraInfo& extra_info);
85 void OnFirstContentfulPaintInPage( 85 void OnFirstContentfulPaintInPage(
86 const page_load_metrics::PageLoadTiming& timing, 86 const page_load_metrics::mojom::PageLoadTiming& timing,
87 const page_load_metrics::PageLoadExtraInfo& extra_info); 87 const page_load_metrics::PageLoadExtraInfo& extra_info);
88 void OnParseStart(const page_load_metrics::PageLoadTiming& timing, 88 void OnParseStart(const page_load_metrics::mojom::PageLoadTiming& timing,
89 const page_load_metrics::PageLoadExtraInfo& extra_info); 89 const page_load_metrics::PageLoadExtraInfo& extra_info);
90 void OnParseStop(const page_load_metrics::PageLoadTiming& timing, 90 void OnParseStop(const page_load_metrics::mojom::PageLoadTiming& timing,
91 const page_load_metrics::PageLoadExtraInfo& extra_info); 91 const page_load_metrics::PageLoadExtraInfo& extra_info);
92 void OnUserInput(const blink::WebInputEvent& event); 92 void OnUserInput(const blink::WebInputEvent& event);
93 void FlushMetricsOnAppEnterBackground( 93 void FlushMetricsOnAppEnterBackground(
94 const page_load_metrics::PageLoadTiming& timing, 94 const page_load_metrics::mojom::PageLoadTiming& timing,
95 const page_load_metrics::PageLoadExtraInfo& extra_info); 95 const page_load_metrics::PageLoadExtraInfo& extra_info);
96 96
97 // The methods below are public only for testing. 97 // The methods below are public only for testing.
98 static bool IsGoogleSearchHostname(const GURL& url); 98 static bool IsGoogleSearchHostname(const GURL& url);
99 static bool IsGoogleSearchResultUrl(const GURL& url); 99 static bool IsGoogleSearchResultUrl(const GURL& url);
100 static bool IsGoogleSearchRedirectorUrl(const GURL& url); 100 static bool IsGoogleSearchRedirectorUrl(const GURL& url);
101 bool ShouldLogFailedProvisionalLoadMetrics(); 101 bool ShouldLogFailedProvisionalLoadMetrics();
102 bool ShouldLogPostCommitMetrics(const GURL& url); 102 bool ShouldLogPostCommitMetrics(const GURL& url);
103 bool ShouldLogForegroundEventAfterCommit( 103 bool ShouldLogForegroundEventAfterCommit(
104 const base::Optional<base::TimeDelta>& event, 104 const base::Optional<base::TimeDelta>& event,
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 public: 146 public:
147 FromGWSPageLoadMetricsObserver(); 147 FromGWSPageLoadMetricsObserver();
148 148
149 // page_load_metrics::PageLoadMetricsObserver implementation: 149 // page_load_metrics::PageLoadMetricsObserver implementation:
150 ObservePolicy OnStart(content::NavigationHandle* navigation_handle, 150 ObservePolicy OnStart(content::NavigationHandle* navigation_handle,
151 const GURL& currently_committed_url, 151 const GURL& currently_committed_url,
152 bool started_in_foreground) override; 152 bool started_in_foreground) override;
153 ObservePolicy OnCommit(content::NavigationHandle* navigation_handle) override; 153 ObservePolicy OnCommit(content::NavigationHandle* navigation_handle) override;
154 154
155 ObservePolicy FlushMetricsOnAppEnterBackground( 155 ObservePolicy FlushMetricsOnAppEnterBackground(
156 const page_load_metrics::PageLoadTiming& timing, 156 const page_load_metrics::mojom::PageLoadTiming& timing,
157 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 157 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
158 158
159 void OnDomContentLoadedEventStart( 159 void OnDomContentLoadedEventStart(
160 const page_load_metrics::PageLoadTiming& timing, 160 const page_load_metrics::mojom::PageLoadTiming& timing,
161 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 161 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
162 void OnLoadEventStart( 162 void OnLoadEventStart(
163 const page_load_metrics::PageLoadTiming& timing, 163 const page_load_metrics::mojom::PageLoadTiming& timing,
164 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 164 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
165 void OnFirstPaintInPage( 165 void OnFirstPaintInPage(
166 const page_load_metrics::PageLoadTiming& timing, 166 const page_load_metrics::mojom::PageLoadTiming& timing,
167 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 167 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
168 void OnFirstTextPaintInPage( 168 void OnFirstTextPaintInPage(
169 const page_load_metrics::PageLoadTiming& timing, 169 const page_load_metrics::mojom::PageLoadTiming& timing,
170 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 170 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
171 void OnFirstImagePaintInPage( 171 void OnFirstImagePaintInPage(
172 const page_load_metrics::PageLoadTiming& timing, 172 const page_load_metrics::mojom::PageLoadTiming& timing,
173 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 173 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
174 void OnFirstContentfulPaintInPage( 174 void OnFirstContentfulPaintInPage(
175 const page_load_metrics::PageLoadTiming& timing, 175 const page_load_metrics::mojom::PageLoadTiming& timing,
176 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 176 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
177 void OnParseStart( 177 void OnParseStart(
178 const page_load_metrics::PageLoadTiming& timing, 178 const page_load_metrics::mojom::PageLoadTiming& timing,
179 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 179 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
180 void OnParseStop( 180 void OnParseStop(
181 const page_load_metrics::PageLoadTiming& timing, 181 const page_load_metrics::mojom::PageLoadTiming& timing,
182 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 182 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
183 183
184 void OnComplete( 184 void OnComplete(
185 const page_load_metrics::PageLoadTiming& timing, 185 const page_load_metrics::mojom::PageLoadTiming& timing,
186 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 186 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
187 void OnFailedProvisionalLoad( 187 void OnFailedProvisionalLoad(
188 const page_load_metrics::FailedProvisionalLoadInfo& failed_load_info, 188 const page_load_metrics::FailedProvisionalLoadInfo& failed_load_info,
189 const page_load_metrics::PageLoadExtraInfo& extra_info) override; 189 const page_load_metrics::PageLoadExtraInfo& extra_info) override;
190 190
191 void OnUserInput(const blink::WebInputEvent& event) override; 191 void OnUserInput(const blink::WebInputEvent& event) override;
192 192
193 private: 193 private:
194 FromGWSPageLoadMetricsLogger logger_; 194 FromGWSPageLoadMetricsLogger logger_;
195 195
196 DISALLOW_COPY_AND_ASSIGN(FromGWSPageLoadMetricsObserver); 196 DISALLOW_COPY_AND_ASSIGN(FromGWSPageLoadMetricsObserver);
197 }; 197 };
198 198
199 #endif // CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_FROM_GWS_PAGE_LOAD_METRICS _OBSERVER_H_ 199 #endif // CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_FROM_GWS_PAGE_LOAD_METRICS _OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698