QUIC - disable QUIC if packet loss rate is bad for a connection.

tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index db3356fd21c8b3f2c97f14929ce5fdd4acc12b81..edb2715f0052520e5899d117849835272570c188 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -19894,6 +19894,15 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   </summary>
 </histogram>
 
+<histogram name="Net.QuicStreamFactory.QuicIsDisabled" enum="Ports">
+  <owner>rtenneti@chromium.org</owner>
+  <summary>
+    Recorded whenever QUIC is disabled for a port due to high packet loss rate
+    and when number of consecutive lossy connections goes over a threshold. QUIC

[Mark P, 2015/03/26 21:23:05]

"It is an and. We need to have high packet loss for 5 consecutive QUIC
handshakes. Then only we will disable."
Then perhaps write:
... is disabled for a port due to repeated lossy connections (high packet loss
on multiple consecutive connections).

(Feel free to edit.  Or drop "on multiple consecutive connections" if you think
it's too redundant.)

[ramant (doing other things), 2015/03/26 22:10:31]

Done.

+    is disabled until next reboot of Chrome. Recorded once per chrome session.

[Mark P, 2015/03/26 21:23:05]

either omit this last sentence (it's obvious given the two sentence before) or
clarify it (is this on a per-profile basis? is that why you're using the term
session?).

[ramant (doing other things), 2015/03/26 22:10:31]

omitted "Recorded once per chrome session."

Done.

+  </summary>
+</histogram>
+
 <histogram name="Net.QuicVerifyProofFailed.HandshakeConfirmed"
     enum="BooleanHandshakeConfirmed">
   <owner>rtenneti@chromium.org</owner>
@@ -57083,6 +57092,11 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="11" label="Bad Key Validation Signature"/>
 </enum>
 
+<enum name="Ports" type="int">
+  <int value="80" label="Port 80"/>
+  <int value="443" label="Port 443"/>
+</enum>
+
 <enum name="PostMergeVerificationOutcome" type="int">
   <int value="0" label="Undefined"/>
   <int value="1" label="Succeeded"/>