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

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

Issue 1426673009: Log UMA for navigation start timestamp skew for browser-initiated loads (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@navigation_start_renderer
Patch Set: use UMA_HISTOGRAM_TIMES Created 5 years, 1 month 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
« no previous file with comments | « content/renderer/render_frame_impl.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 4b5e451151f0128dfe31aca6f5273003d5970495..950408d4069c5bf1ab722c7911ff0f9491acf17c 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -20819,6 +20819,32 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Navigation.Start.RendererBrowserDifference.Negative"
+ units="milliseconds">
+ <owner>csharrison@chromium.org</owner>
+ <summary>
+ The difference between Now() in the renderer and browser_navigation_start
+ for browser-initiated navigations if the difference is negative. This value
+ should ideally only reflect the IPC time between the browser process and
+ renderer process, but in practice could be skewed by inter-process timing
+ errors. If the clocks used for TimeTicks are monotonic across processes, all
+ samples should fall into the Positive bucket.
+ </summary>
+</histogram>
+
+<histogram name="Navigation.Start.RendererBrowserDifference.Positive"
+ units="milliseconds">
+ <owner>csharrison@chromium.org</owner>
+ <summary>
+ The difference between Now() in the renderer and browser_navigation_start
+ for browser-initiated navigations if the difference is positive. This value
+ should ideally only reflect the IPC time between the browser process and
+ renderer process, but in practice could be skewed by inter-process timing
+ errors. If the clocks used for TimeTicks are monotonic across processes, all
+ samples should fall into the Positive bucket.
+ </summary>
+</histogram>
+
<histogram name="Navigation.TimeToCommit" units="milliseconds">
<owner>carlosk@chromium.org</owner>
<summary>
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698