[SPDY] Close SPDY sessions on session flow control errors

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 fe3584829c5dcfb7b2f0b75f99e40779b77a58c0..eb944062650daf7141a5edeb6fa893ff1b0c69d3 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -4761,6 +4761,7 @@ other types of suffix sets.
   <int value="803" label="DNS_TIMED_OUT"/>
   <int value="804" label="DNS_CACHE_MISS"/>
   <int value="805" label="DNS_SEARCH_EMPTY"/>
+  <int value="806" label="DNS_SORT_ERROR"/>
 </enum>
 
 <enum name="NetPreconnectUtilization" type="int">
@@ -5070,6 +5071,8 @@ other types of suffix sets.
 </enum>
 
 <enum name="SpdyProtocolErrorDetails" type="int">
+<!-- SpdyFramer::SpdyErrors -->

[jar (doing other things), 2013/04/19 06:16:11]

nit: I *think* these comments will be removed during the pretty-print run, which
will also alphabetize etc.

[akalin, 2013/04/19 08:46:24]

Nope, they were kept.

+
   <int value="0" label="No error"/>
   <int value="1" label="Invalid Control Frame"/>
   <int value="2" label="Control Frame Payload Too Large"/>
@@ -5079,24 +5082,31 @@ other types of suffix sets.
   <int value="6" label="Compress Failure"/>
   <int value="7" label="Credential Frame Corrupt"/>
   <int value="8" label="Invalid Data Frame Flags"/>
-  <int value="9" label="Invalid Status Code"/>
-  <int value="10" label="Protocol Error"/>
-  <int value="11" label="Invalid Stream"/>
-  <int value="12" label="Refused Stream"/>
-  <int value="13" label="Unsupported Version"/>
-  <int value="14" label="Cancel"/>
-  <int value="15" label="Internal Error"/>
-  <int value="16" label="Flow Control Error"/>
-  <int value="17" label="Stream In Use"/>
-  <int value="18" label="Stream Already Closed"/>
-  <int value="19" label="Invalid Credentials"/>
-  <int value="20" label="Frame Too Large"/>
-  <int value="21" label="Unexpected Ping"/>
-  <int value="22" label="Rst Stream For Non Active Stream"/>
-  <int value="23" label="Spdy Compression Failure"/>
-  <int value="24" label="Request For Secure Content Over Insecure Session"/>
-  <int value="25" label="Protocol Error Syn Reply Not Received"/>
-  <int value="26" label="Num Spdy Protocol Error Details"/>
+  <int value="9" label="Invalid Control Frame Flags"/>
+<!-- SpdyRstStreamStatus -->
+
+  <int value="10" label="(Unused)"/>
+  <int value="11" label="Protocol Error"/>

[jar (doing other things), 2013/04/19 06:16:11]

Usually, when the enums change, best practice is to define a new enum, and at
most, obsolete the old enums (and perhasp the histograms that use the old
enums).

...unless of course the enums were already/always wrong.

If you just change the enums, then all the old histogram data will appear
corrupt and confused :-(.  Since there are generally lots of recent/older
versions sending in UMA data, this tends to lead to a lot of confusion.

[akalin, 2013/04/19 08:46:24]

Done. Looks like I inadvertently changed the enums in a previous CL. :( So I
defined a new enum and new histograms and added various comments.

+  <int value="12" label="Invalid Stream"/>
+  <int value="13" label="Refused Stream"/>
+  <int value="14" label="Unsupported Version"/>
+  <int value="15" label="Cancel"/>
+  <int value="16" label="Internal Error"/>
+  <int value="17" label="Flow Control Error"/>
+  <int value="18" label="Stream In Use"/>
+  <int value="19" label="Stream Already Closed"/>
+  <int value="20" label="Invalid Credentials"/>
+  <int value="21" label="Frame Too Large"/>
+<!-- SpdySession errors -->
+
+  <int value="22" label="Unexpected Ping"/>
+  <int value="23" label="Rst Stream For Non Active Stream"/>
+  <int value="24" label="Spdy Compression Failure"/>
+  <int value="25" label="Request For Secure Content Over Insecure Session"/>
+  <int value="26" label="Syn Reply Not Received"/>
+  <int value="27" label="Invalid Window Update Size"/>
+  <int value="28" label="Receive Window Size Violation"/>
+  <int value="29" label="(Unused)"/>
 </enum>
 
 <enum name="SpdySessionGet" type="int">