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

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

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 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/css_scanning_page_load_metr ics_observer.h" 5 #include "chrome/browser/page_load_metrics/observers/css_scanning_page_load_metr ics_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 "third_party/WebKit/public/platform/WebLoadingBehaviorFlag.h" 8 #include "third_party/WebKit/public/platform/WebLoadingBehaviorFlag.h"
9 9
10 CssScanningMetricsObserver::CssScanningMetricsObserver() {} 10 CssScanningMetricsObserver::CssScanningMetricsObserver() {}
(...skipping 12 matching lines...) Expand all
23 page_load_metrics::PageLoadMetricsObserver::ObservePolicy 23 page_load_metrics::PageLoadMetricsObserver::ObservePolicy
24 CssScanningMetricsObserver::OnStart( 24 CssScanningMetricsObserver::OnStart(
25 content::NavigationHandle* navigation_handle, 25 content::NavigationHandle* navigation_handle,
26 const GURL& currently_committed_url, 26 const GURL& currently_committed_url,
27 bool started_in_foreground) { 27 bool started_in_foreground) {
28 return started_in_foreground ? CONTINUE_OBSERVING : STOP_OBSERVING; 28 return started_in_foreground ? CONTINUE_OBSERVING : STOP_OBSERVING;
29 } 29 }
30 30
31 page_load_metrics::PageLoadMetricsObserver::ObservePolicy 31 page_load_metrics::PageLoadMetricsObserver::ObservePolicy
32 CssScanningMetricsObserver::OnHidden( 32 CssScanningMetricsObserver::OnHidden(
33 const page_load_metrics::PageLoadTiming&, 33 const page_load_metrics::mojom::PageLoadTiming&,
34 const page_load_metrics::PageLoadExtraInfo&) { 34 const page_load_metrics::PageLoadExtraInfo&) {
35 return STOP_OBSERVING; 35 return STOP_OBSERVING;
36 } 36 }
37 37
38 void CssScanningMetricsObserver::OnFirstContentfulPaintInPage( 38 void CssScanningMetricsObserver::OnFirstContentfulPaintInPage(
39 const page_load_metrics::PageLoadTiming& timing, 39 const page_load_metrics::mojom::PageLoadTiming& timing,
40 const page_load_metrics::PageLoadExtraInfo& info) { 40 const page_load_metrics::PageLoadExtraInfo& info) {
41 if (!css_preload_found_) 41 if (!css_preload_found_)
42 return; 42 return;
43 43
44 PAGE_LOAD_HISTOGRAM( 44 PAGE_LOAD_HISTOGRAM(
45 "PageLoad.Clients.CssScanner.PaintTiming." 45 "PageLoad.Clients.CssScanner.PaintTiming."
46 "ParseStartToFirstContentfulPaint", 46 "ParseStartToFirstContentfulPaint",
47 timing.paint_timing.first_contentful_paint.value() - 47 timing.paint_timing->first_contentful_paint.value() -
48 timing.parse_timing.parse_start.value()); 48 timing.parse_timing->parse_start.value());
49 } 49 }
50 50
51 void CssScanningMetricsObserver::OnFirstMeaningfulPaintInMainFrameDocument( 51 void CssScanningMetricsObserver::OnFirstMeaningfulPaintInMainFrameDocument(
52 const page_load_metrics::PageLoadTiming& timing, 52 const page_load_metrics::mojom::PageLoadTiming& timing,
53 const page_load_metrics::PageLoadExtraInfo& info) { 53 const page_load_metrics::PageLoadExtraInfo& info) {
54 if (!css_preload_found_) 54 if (!css_preload_found_)
55 return; 55 return;
56 56
57 PAGE_LOAD_HISTOGRAM( 57 PAGE_LOAD_HISTOGRAM(
58 "PageLoad.Clients.CssScanner.Experimental.PaintTiming." 58 "PageLoad.Clients.CssScanner.Experimental.PaintTiming."
59 "ParseStartToFirstMeaningfulPaint", 59 "ParseStartToFirstMeaningfulPaint",
60 timing.paint_timing.first_meaningful_paint.value() - 60 timing.paint_timing->first_meaningful_paint.value() -
61 timing.parse_timing.parse_start.value()); 61 timing.parse_timing->parse_start.value());
62 } 62 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698