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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 560573002: Introduce new SPDY Version UMA histogram. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use sparse histogram. Created 6 years, 3 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
« no previous file with comments | « net/spdy/spdy_test_util_common.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 96a571ee1b69bd6287b396cdd7b79b8fc4abaef4..1b4e3aa08f5b3df4dbb3fb41066716f384b67d32 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -17367,12 +17367,24 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</histogram>
<histogram name="Net.SpdyVersion" enum="ProtocolVersion">
+ <obsolete>
+ Deprecated on 2014-09-11, because the uploaded values were changing as
+ protocols were removed, therefore statistics couldn't be combined accross
+ different builds. Replaced by Net.SpdyVersion2.
+ </obsolete>
<owner>rch@chromium.org</owner>
<summary>
The SPDY protocol version that is used to talk to SPDY servers.
</summary>
</histogram>
+<histogram name="Net.SpdyVersion2" enum="SpdyProtocolVersion">
+ <owner>bnc@chromium.org</owner>
+ <summary>
+ The SPDY protocol version that is used to talk to SPDY servers.
Alexei Svitkine (slow) 2014/09/12 21:14:30 Nit: While you're here, please expand this descrip
Bence 2014/09/15 15:39:27 Done.
+ </summary>
+</histogram>
+
<histogram name="Net.SSL_CipherSuite" enum="SSLCipherSuite">
<owner>agl@chromium.org</owner>
<owner>rsleevi@chromium.org</owner>
@@ -48493,10 +48505,10 @@ To add a new entry, add it with any value and run test to compute valid value.
<enum name="ProtocolVersion" type="int">
<int value="0" label="UNKNOWN"/>
<int value="1" label="HTTP 1.1"/>
- <int value="2" label="SPDY 1.0"/>
- <int value="3" label="SPDY 2.0"/>
- <int value="4" label="SPDY 2.1"/>
- <int value="5" label="SPDY 3.0"/>
+ <int value="2" label="SPDY 2.0"/>
+ <int value="3" label="SPDY 3.0"/>
+ <int value="4" label="SPDY 3.1"/>
+ <int value="5" label="SPDY 4.0"/>
</enum>
<enum name="ProvisionalSaveFailure" type="int">
@@ -49878,6 +49890,13 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="34" label="Peer exhibiting suspect behavior."/>
</enum>
+<enum name="SpdyProtocolVersion" type="int">
+ <int value="100" label="SPDY 2.0"/>
+ <int value="101" label="SPDY 3.0"/>
+ <int value="102" label="SPDY 3.1"/>
+ <int value="103" label="HTTP/2 draft-14"/>
+</enum>
+
<enum name="SpdySessionGet" type="int">
<int value="0" label="created new"/>
<int value="1" label="found existing"/>
« no previous file with comments | « net/spdy/spdy_test_util_common.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698