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

Side by Side Diff: chrome/browser/page_load_metrics/observers/data_reduction_proxy_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 2016 The Chromium Authors. All rights reserved. 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 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/data_reduction_proxy_metric s_observer.h" 5 #include "chrome/browser/page_load_metrics/observers/data_reduction_proxy_metric s_observer.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/metrics/histogram_macros.h" 9 #include "base/metrics/histogram_macros.h"
10 #include "base/optional.h" 10 #include "base/optional.h"
(...skipping 336 matching lines...) Expand 10 before | Expand all | Expand 10 after
347 } 347 }
348 348
349 void DataReductionProxyMetricsObserver::OnFirstLayout( 349 void DataReductionProxyMetricsObserver::OnFirstLayout(
350 const page_load_metrics::PageLoadTiming& timing, 350 const page_load_metrics::PageLoadTiming& timing,
351 const page_load_metrics::PageLoadExtraInfo& info) { 351 const page_load_metrics::PageLoadExtraInfo& info) {
352 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX( 352 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(
353 info, data_, timing.document_timing.first_layout, 353 info, data_, timing.document_timing.first_layout,
354 internal::kHistogramFirstLayoutSuffix); 354 internal::kHistogramFirstLayoutSuffix);
355 } 355 }
356 356
357 void DataReductionProxyMetricsObserver::OnFirstPaint( 357 void DataReductionProxyMetricsObserver::OnFirstPaintInPage(
358 const page_load_metrics::PageLoadTiming& timing, 358 const page_load_metrics::PageLoadTiming& timing,
359 const page_load_metrics::PageLoadExtraInfo& info) { 359 const page_load_metrics::PageLoadExtraInfo& info) {
360 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(info, data_, 360 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(info, data_,
361 timing.paint_timing.first_paint, 361 timing.paint_timing.first_paint,
362 internal::kHistogramFirstPaintSuffix); 362 internal::kHistogramFirstPaintSuffix);
363 } 363 }
364 364
365 void DataReductionProxyMetricsObserver::OnFirstTextPaint( 365 void DataReductionProxyMetricsObserver::OnFirstTextPaintInPage(
366 const page_load_metrics::PageLoadTiming& timing, 366 const page_load_metrics::PageLoadTiming& timing,
367 const page_load_metrics::PageLoadExtraInfo& info) { 367 const page_load_metrics::PageLoadExtraInfo& info) {
368 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX( 368 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(
369 info, data_, timing.paint_timing.first_text_paint, 369 info, data_, timing.paint_timing.first_text_paint,
370 internal::kHistogramFirstTextPaintSuffix); 370 internal::kHistogramFirstTextPaintSuffix);
371 } 371 }
372 372
373 void DataReductionProxyMetricsObserver::OnFirstImagePaint( 373 void DataReductionProxyMetricsObserver::OnFirstImagePaintInPage(
374 const page_load_metrics::PageLoadTiming& timing, 374 const page_load_metrics::PageLoadTiming& timing,
375 const page_load_metrics::PageLoadExtraInfo& info) { 375 const page_load_metrics::PageLoadExtraInfo& info) {
376 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX( 376 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(
377 info, data_, timing.paint_timing.first_image_paint, 377 info, data_, timing.paint_timing.first_image_paint,
378 internal::kHistogramFirstImagePaintSuffix); 378 internal::kHistogramFirstImagePaintSuffix);
379 } 379 }
380 380
381 void DataReductionProxyMetricsObserver::OnFirstContentfulPaint( 381 void DataReductionProxyMetricsObserver::OnFirstContentfulPaintInPage(
382 const page_load_metrics::PageLoadTiming& timing, 382 const page_load_metrics::PageLoadTiming& timing,
383 const page_load_metrics::PageLoadExtraInfo& info) { 383 const page_load_metrics::PageLoadExtraInfo& info) {
384 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX( 384 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(
385 info, data_, timing.paint_timing.first_contentful_paint, 385 info, data_, timing.paint_timing.first_contentful_paint,
386 internal::kHistogramFirstContentfulPaintSuffix); 386 internal::kHistogramFirstContentfulPaintSuffix);
387 } 387 }
388 388
389 void DataReductionProxyMetricsObserver::OnFirstMeaningfulPaint( 389 void DataReductionProxyMetricsObserver::
390 const page_load_metrics::PageLoadTiming& timing, 390 OnFirstMeaningfulPaintInMainFrameDocument(
391 const page_load_metrics::PageLoadExtraInfo& info) { 391 const page_load_metrics::PageLoadTiming& timing,
392 const page_load_metrics::PageLoadExtraInfo& info) {
392 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX( 393 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(
393 info, data_, timing.paint_timing.first_meaningful_paint, 394 info, data_, timing.paint_timing.first_meaningful_paint,
394 internal::kHistogramFirstMeaningfulPaintSuffix); 395 internal::kHistogramFirstMeaningfulPaintSuffix);
395 } 396 }
396 397
397 void DataReductionProxyMetricsObserver::OnParseStart( 398 void DataReductionProxyMetricsObserver::OnParseStart(
398 const page_load_metrics::PageLoadTiming& timing, 399 const page_load_metrics::PageLoadTiming& timing,
399 const page_load_metrics::PageLoadExtraInfo& info) { 400 const page_load_metrics::PageLoadExtraInfo& info) {
400 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(info, data_, 401 RECORD_FOREGROUND_HISTOGRAMS_FOR_SUFFIX(info, data_,
401 timing.parse_timing.parse_start, 402 timing.parse_timing.parse_start,
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 443
443 DataReductionProxyPingbackClient* 444 DataReductionProxyPingbackClient*
444 DataReductionProxyMetricsObserver::GetPingbackClient() const { 445 DataReductionProxyMetricsObserver::GetPingbackClient() const {
445 return DataReductionProxyChromeSettingsFactory::GetForBrowserContext( 446 return DataReductionProxyChromeSettingsFactory::GetForBrowserContext(
446 browser_context_) 447 browser_context_)
447 ->data_reduction_proxy_service() 448 ->data_reduction_proxy_service()
448 ->pingback_client(); 449 ->pingback_client();
449 } 450 }
450 451
451 } // namespace data_reduction_proxy 452 } // namespace data_reduction_proxy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698