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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 1408343007: QUIC - Delete Net.QuicClientHelloRejectReasons.Insecure histogram. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 | « no previous file | 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 af3692113337f2de718a8965740d20c9b847c540..1bca560b9c86defb6e689e5529a5a2cd91723abf 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -24141,10 +24141,12 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
-<histogram name="Net.QuicClientHelloRejectReasons" enum="QuicRejectReasons">
+<histogram name="Net.QuicClientHelloRejectReasons.Secure"
+ enum="QuicRejectReasons">
<owner>rtenneti@chromium.org</owner>
<summary>
- The reject reasons for QUIC's CHLO (client hello) message from server
+ The reject reasons for QUIC's CHLO (client hello) message from server for
+ secure QUIC.
</summary>
</histogram>
@@ -77692,14 +77694,6 @@ To add a new entry, add it with any value and run test to compute valid value.
<affected-histogram name="Navigation.TimeToURLJobStart"/>
</histogram_suffixes>
-<histogram_suffixes name="Net.QuicClientHelloRejectReasons.QuicIsSecureOrNot"
- separator=".">
- <owner>rtenneti@chromium.org</owner>
- <suffix name="Insecure" label="for insecure QUIC."/>
Alexei Svitkine (slow) 2015/10/29 20:30:04 Just mark this suffix with an inner <obsolete> tag
ramant (doing other things) 2015/10/29 20:45:45 Done.
- <suffix name="Secure" label="for secure QUIC."/>
- <affected-histogram name="Net.QuicClientHelloRejectReasons"/>
-</histogram_suffixes>
-
<histogram_suffixes name="Net.QuicSession.21CumulativePackets" separator="_">
<owner>rch@chromium.org</owner>
<suffix name="First21"
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698