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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 1376473003: Notify NQE of TCP RTT values (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed rsleevi comments Created 4 years, 9 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
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index e909a2df2da07cd6a703971b40bf6290abf9e165..9c035b80b971eb74076fe873567760c118f48391 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -47323,6 +47323,18 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="SPW.TCPRTT.NotifyDelay" units="ms">
+ <owner>bengr@chromium.org</owner>
+ <owner>tbansal@chromium.org</owner>
+ <summary>
+ Time taken to read tcp_info struct through a syscall on POSIX platforms, and
+ notifying the RTT from the struct to the socket performance watcher.
+
+ Logged only on POSIX platforms when Chromium successfully read the RTT from
+ the tcp_info struct.
Ryan Sleevi 2016/02/25 22:48:47 This entire histogram description seems wrong - be
tbansal1 2016/02/26 23:43:34 Thanks,.removed the UMA. Now, I am using the Scope
+ </summary>
+</histogram>
+
<histogram name="Sqlite.AppCache.Error" enum="SqliteErrorCode">
<obsolete>
Moved to Sqlite.Error.AppCache in M-27.

Powered by Google App Engine
This is Rietveld 408576698