[SPDY] Address comments from review of r195309

net/spdy/spdy_session.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_SPDY_SPDY_SESSION_H_
 #define NET_SPDY_SPDY_SESSION_H_
 
 #include <deque>
 #include <map>
 #include <set>
@@ skipping 44 matching lines @@
 // The initial receive window size for both streams and sessions.
 const int32 kDefaultInitialRecvWindowSize = 10 * 1024 * 1024;  // 10MB
 
 class BoundNetLog;
 struct LoadTimingInfo;
 class SpdyStream;
 class SSLInfo;
 
 // NOTE: There's an enum of the same name (also with numeric suffixes)
 // in histograms.xml.
+//
+// WARNING: DO NOT INSERT ENUMS INTO THIS LIST! Add only to the end.
 enum SpdyProtocolErrorDetails {
   // SpdyFramer::SpdyErrors
   SPDY_ERROR_NO_ERROR,
   SPDY_ERROR_INVALID_CONTROL_FRAME,
   SPDY_ERROR_CONTROL_PAYLOAD_TOO_LARGE,
   SPDY_ERROR_ZLIB_INIT_FAILURE,
   SPDY_ERROR_UNSUPPORTED_VERSION,
   SPDY_ERROR_DECOMPRESS_FAILURE,
   SPDY_ERROR_COMPRESS_FAILURE,
   SPDY_ERROR_CREDENTIAL_FRAME_CORRUPT,
   SPDY_ERROR_INVALID_DATA_FRAME_FLAGS,
   SPDY_ERROR_INVALID_CONTROL_FRAME_FLAGS,
-
   // SpdyRstStreamStatus
   STATUS_CODE_INVALID,
   STATUS_CODE_PROTOCOL_ERROR,
   STATUS_CODE_INVALID_STREAM,
   STATUS_CODE_REFUSED_STREAM,
   STATUS_CODE_UNSUPPORTED_VERSION,
   STATUS_CODE_CANCEL,
   STATUS_CODE_INTERNAL_ERROR,
   STATUS_CODE_FLOW_CONTROL_ERROR,
   STATUS_CODE_STREAM_IN_USE,
   STATUS_CODE_STREAM_ALREADY_CLOSED,
   STATUS_CODE_INVALID_CREDENTIALS,
   STATUS_CODE_FRAME_TOO_LARGE,
-
   // SpdySession errors
   PROTOCOL_ERROR_UNEXPECTED_PING,
   PROTOCOL_ERROR_RST_STREAM_FOR_NON_ACTIVE_STREAM,
   PROTOCOL_ERROR_SPDY_COMPRESSION_FAILURE,
   PROTOCOL_ERROR_REQUEST_FOR_SECURE_CONTENT_OVER_INSECURE_SESSION,
   PROTOCOL_ERROR_SYN_REPLY_NOT_RECEIVED,
   PROTOCOL_ERROR_INVALID_WINDOW_UPDATE_SIZE,
   PROTOCOL_ERROR_RECEIVE_WINDOW_VIOLATION,
   NUM_SPDY_PROTOCOL_ERROR_DETAILS
 };
@@ skipping 843 matching lines @@
   // This SPDY proxy is allowed to push resources from origins that are
   // different from those of their associated streams.
   HostPortPair trusted_spdy_proxy_;
 
   TimeFunc time_func_;
 };
 
 }  // namespace net
 
 #endif  // NET_SPDY_SPDY_SESSION_H_