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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tools/metrics/histograms/histograms.xml

Side-by-side diff isn't available for this file because of its large size.
Issue 1384213002: Page Abort Events for relevant navigations (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Bryan review. Postponing BG/Background decision for now Created 5 years, 2 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:
Download patch
« components/page_load_metrics/browser/metrics_web_contents_observer.h ('K') | « components/page_load_metrics/browser/metrics_web_contents_observer_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 9035cd749e1768bbfebaf56ee3a670b958908aca..8b4985339cf9c3f7f06a1d171fccae50051f1e96 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -30096,6 +30096,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="PageLoad.EventCounts" enum="PageLoadEvent">
<owner>csharrison@chromium.org</owner>
<owner>bmcquade@chromium.org</owner>
+ <obsolete>
+ deprecated in favor of PageLoad.Events.*
+ </obsolete>
<summary>
Captures counts of various page load events. These are enumerated in the
enum page_load_metrics::PageLoadEvent, and include events like 'page load
@@ -30103,6 +30106,37 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="PageLoad.Events.Committed" enum="CommittedLoadEvent">
+ <owner>csharrison@chromium.org</owner>
+ <owner>bmcquade@chromium.org</owner>
+ <summary>
+ Captures counts of load events post-commit, on relevant page loads (i.e.
+ http/https, not same-page, not an error page). These events include aborts
+ before first layout and successful first layouts. Note that the event
+ 'Committed load started' when segmented by background/foreground specifies
+ whether the provisional load started in the background or foreground, not
+ the actual commit event. This is because we don't have all the data to
Randy Smith (Not in Mondays) 2015/10/15 18:39:09 The rest aren't segmented by actual commit event,
+ filter relevant page loads on provisional loads.
+ </summary>
+</histogram>
+
+<histogram name="PageLoad.Events.InternalError" enum="InternalErrorLoadEvent">
+ <owner>csharrison@chromium.org</owner>
+ <owner>bmcquade@chromium.org</owner>
+ <summary>
+ Counts of various internal error conditions in the page_load_metrics system.
+ </summary>
+</histogram>
+
+<histogram name="PageLoad.Events.Provisional" enum="ProvisionalLoadEvent">
+ <owner>csharrison@chromium.org</owner>
+ <owner>bmcquade@chromium.org</owner>
+ <summary>
+ Captures counts of provisional load events. These include aborted, failed,
+ and successful (committed) provisional loads.
+ </summary>
+</histogram>
+
<histogram name="PageLoad.Timing.NavigationToDOMContentLoadedEventFired"
units="milliseconds">
<owner>bmcquade@chromium.org</owner>
@@ -30151,6 +30185,28 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="PageLoad.Timing2.NavigationToFirstBackground"
+ units="milliseconds">
+ <owner>bmcquade@chromium.org</owner>
+ <owner>csharrison@chromium.org</owner>
+ <summary>
+ Measures the time from navigation timing's navigation start to the time the
+ user first backgrounds the tab. Only measures navigations that started in
+ the foreground.
+ </summary>
+</histogram>
+
+<histogram name="PageLoad.Timing2.NavigationToFirstForeground"
+ units="milliseconds">
+ <owner>bmcquade@chromium.org</owner>
+ <owner>csharrison@chromium.org</owner>
+ <summary>
+ Measures the time from navigation timing's navigation start to the time the
+ user first foregrounds an initially backgrounded tab. Only measures
+ navigations that started in the background.
+ </summary>
+</histogram>
+
<histogram name="PageLoad.Timing2.NavigationToFirstLayout" units="milliseconds">
<owner>bmcquade@chromium.org</owner>
<owner>csharrison@chromium.org</owner>
@@ -54406,6 +54462,12 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<int value="505" label="505: HTTP Version Not Supported"/>
</enum>
+<enum name="CommittedLoadEvent" type="int">
+ <int value="0" label="Committed load started"/>
+ <int value="1" label="Committed load failed before first layout"/>
+ <int value="2" label="Successful first layout"/>
+</enum>
+
<enum name="CompositedScrolling" type="int">
<int value="0" label="Is scrollable area"/>
<int value="1" label="Needs to be stacking container"/>
@@ -63105,6 +63167,14 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<int value="8" label="AVX2"/>
</enum>
+<enum name="InternalErrorLoadEvent" type="int">
+ <summary>Internal Errors in the page_load_metrics system</summary>
+ <int value="0" label="Invalid timing IPC sent from renderer"/>
+ <int value="1" label="IPC received while not tracking a committed load"/>
+ <int value="2" label="IPC received from a frame we navigated away from"/>
+ <int value="3" label="IPC received from a bad URL scheme"/>
+</enum>
+
<enum name="InterruptReason" type="int">
<int value="0" label="NONE"/>
<int value="1" label="FILE_FAILED"/>
@@ -67362,6 +67432,9 @@ To add a new entry, add it with any value and run test to compute valid value.
</enum>
<enum name="PageLoadEvent" type="int">
+ <obsolete>
+ Deprecated in favor of {Committed|Provisional|InternalError}LoadEvent
+ </obsolete>
<int value="0" label="Started main frame load"/>
<int value="1" label="Failed load before commit"/>
<int value="2" label="Aborted load before commit"/>
@@ -69471,6 +69544,20 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="5" label="SPDY 4.0"/>
</enum>
+<enum name="ProvisionalLoadEvent" type="int">
+ <int value="0"
+ label="Provisional load stopped by user with no error. Should not occur
+ pre-PlzNavigate"/>
+ <int value="1"
+ label="Provisional load failed with net::ERR_ABORTED (includes
+ downloads, 204s, and other non user-initiated abort conditions
+ (crbug.com/542369)"/>
+ <int value="2"
+ label="Provisional load failed with an error other than
+ net::ERR_ABORTED"/>
+ <int value="3" label="Provisional load successfully committed"/>
+</enum>
+
<enum name="ProvisionalSaveFailure" type="int">
<int value="0" label="SAVING_DISABLED"/>
<int value="1" label="EMPTY_PASSWORD"/>
@@ -78383,6 +78470,14 @@ To add a new entry, add it with any value and run test to compute valid value.
<affected-histogram name="PageLoad.Timing2.NavigationToLoadEventFired"/>
</histogram_suffixes>
+<histogram_suffixes name="PageLoadBackgrounded2" separator=".">
+ <suffix name="Background"
+ label="Event occurred at least partially in the background. Replaces
+ PageLoadBackgrounded."/>
+ <affected-histogram name="PageLoad.Events.Committed"/>
+ <affected-histogram name="PageLoad.Events.Provisional"/>
+</histogram_suffixes>
+
<histogram_suffixes name="PageLoadType">
<suffix name="HistoryLoad"
label="but only for user pressing back or forward"/>
« components/page_load_metrics/browser/metrics_web_contents_observer.h ('K') | « components/page_load_metrics/browser/metrics_web_contents_observer_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698