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

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

Issue 2859393002: Report page load timing information for child frames. (Closed)
Patch Set: rebase 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 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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/subresource_filter_metrics_ observer.h" 5 #include "chrome/browser/page_load_metrics/observers/subresource_filter_metrics_ observer.h"
6 6
7 #include "chrome/browser/page_load_metrics/page_load_metrics_util.h" 7 #include "chrome/browser/page_load_metrics/page_load_metrics_util.h"
8 #include "components/subresource_filter/content/browser/content_subresource_filt er_driver_factory.h" 8 #include "components/subresource_filter/content/browser/content_subresource_filt er_driver_factory.h"
9 #include "third_party/WebKit/public/platform/WebLoadingBehaviorFlag.h" 9 #include "third_party/WebKit/public/platform/WebLoadingBehaviorFlag.h"
10 10
(...skipping 194 matching lines...) Expand 10 before | Expand all | Expand 10 after
205 internal::kHistogramSubresourceFilterParseBlockedOnScriptExecution, 205 internal::kHistogramSubresourceFilterParseBlockedOnScriptExecution,
206 timing.parse_timing.parse_blocked_on_script_execution_duration.value()); 206 timing.parse_timing.parse_blocked_on_script_execution_duration.value());
207 PAGE_LOAD_HISTOGRAM( 207 PAGE_LOAD_HISTOGRAM(
208 internal:: 208 internal::
209 kHistogramSubresourceFilterParseBlockedOnScriptExecutionDocumentWrite, 209 kHistogramSubresourceFilterParseBlockedOnScriptExecutionDocumentWrite,
210 timing.parse_timing 210 timing.parse_timing
211 .parse_blocked_on_script_execution_from_document_write_duration 211 .parse_blocked_on_script_execution_from_document_write_duration
212 .value()); 212 .value());
213 } 213 }
214 214
215 void SubresourceFilterMetricsObserver::OnFirstContentfulPaint( 215 void SubresourceFilterMetricsObserver::OnFirstContentfulPaintInPage(
216 const page_load_metrics::PageLoadTiming& timing, 216 const page_load_metrics::PageLoadTiming& timing,
217 const page_load_metrics::PageLoadExtraInfo& info) { 217 const page_load_metrics::PageLoadExtraInfo& info) {
218 if (!subresource_filter_observed_) 218 if (!subresource_filter_observed_)
219 return; 219 return;
220 220
221 if (WasStartedInForegroundOptionalEventInForeground( 221 if (WasStartedInForegroundOptionalEventInForeground(
222 timing.paint_timing.first_contentful_paint, info)) { 222 timing.paint_timing.first_contentful_paint, info)) {
223 PAGE_LOAD_HISTOGRAM( 223 PAGE_LOAD_HISTOGRAM(
224 internal::kHistogramSubresourceFilterFirstContentfulPaint, 224 internal::kHistogramSubresourceFilterFirstContentfulPaint,
225 timing.paint_timing.first_contentful_paint.value()); 225 timing.paint_timing.first_contentful_paint.value());
226 PAGE_LOAD_HISTOGRAM( 226 PAGE_LOAD_HISTOGRAM(
227 internal::kHistogramSubresourceFilterParseStartToFirstContentfulPaint, 227 internal::kHistogramSubresourceFilterParseStartToFirstContentfulPaint,
228 timing.paint_timing.first_contentful_paint.value() - 228 timing.paint_timing.first_contentful_paint.value() -
229 timing.parse_timing.parse_start.value()); 229 timing.parse_timing.parse_start.value());
230 } 230 }
231 } 231 }
232 232
233 void SubresourceFilterMetricsObserver::OnFirstMeaningfulPaint( 233 void SubresourceFilterMetricsObserver::
234 const page_load_metrics::PageLoadTiming& timing, 234 OnFirstMeaningfulPaintInMainFrameDocument(
235 const page_load_metrics::PageLoadExtraInfo& info) { 235 const page_load_metrics::PageLoadTiming& timing,
236 const page_load_metrics::PageLoadExtraInfo& info) {
236 if (!subresource_filter_observed_) 237 if (!subresource_filter_observed_)
237 return; 238 return;
238 239
239 if (WasStartedInForegroundOptionalEventInForeground( 240 if (WasStartedInForegroundOptionalEventInForeground(
240 timing.paint_timing.first_meaningful_paint, info)) { 241 timing.paint_timing.first_meaningful_paint, info)) {
241 PAGE_LOAD_HISTOGRAM( 242 PAGE_LOAD_HISTOGRAM(
242 internal::kHistogramSubresourceFilterFirstMeaningfulPaint, 243 internal::kHistogramSubresourceFilterFirstMeaningfulPaint,
243 timing.paint_timing.first_meaningful_paint.value()); 244 timing.paint_timing.first_meaningful_paint.value());
244 PAGE_LOAD_HISTOGRAM( 245 PAGE_LOAD_HISTOGRAM(
245 internal::kHistogramSubresourceFilterParseStartToFirstMeaningfulPaint, 246 internal::kHistogramSubresourceFilterParseStartToFirstMeaningfulPaint,
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
365 foreground_duration.value()); 366 foreground_duration.value());
366 if (timing.paint_timing.first_paint && 367 if (timing.paint_timing.first_paint &&
367 timing.paint_timing.first_paint < foreground_duration) { 368 timing.paint_timing.first_paint < foreground_duration) {
368 PAGE_LOAD_LONG_HISTOGRAM( 369 PAGE_LOAD_LONG_HISTOGRAM(
369 internal::kHistogramSubresourceFilterForegroundDurationAfterPaint, 370 internal::kHistogramSubresourceFilterForegroundDurationAfterPaint,
370 foreground_duration.value() - 371 foreground_duration.value() -
371 timing.paint_timing.first_paint.value()); 372 timing.paint_timing.first_paint.value());
372 } 373 }
373 } 374 }
374 } 375 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698