SPDY4 shouldn't emit RST_STREAM payloads.

SpdyFramer will accept them, and frame them if present. We may want to leave this for later SPDY versions, but certainly don't emit them in SPDY4.

BUG=345769

Review URL: https://codereview.chromium.org/316983006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275512 0039d316-1c4b-4281-b951-d872f2087c98

net/spdy/buffered_spdy_framer.cc

@@ -298,7 +298,9 @@ SpdyFrame* BufferedSpdyFramer::CreateSynReply(
 SpdyFrame* BufferedSpdyFramer::CreateRstStream(
     SpdyStreamId stream_id,
     SpdyRstStreamStatus status) const {
-  SpdyRstStreamIR rst_ir(stream_id, status, "RST");
+  // RST_STREAM payloads are not part of any SPDY spec.
+  // SpdyFramer will accept them, but don't create them.
+  SpdyRstStreamIR rst_ir(stream_id, status, "");
   return spdy_framer_.SerializeRstStream(rst_ir);
 }
 

net/spdy/spdy_test_util_common.cc

@@ -921,7 +921,7 @@ SpdyFrame* SpdyTestUtil::ConstructSpdyWindowUpdate(
 SpdyFrame* SpdyTestUtil::ConstructSpdyRstStream(
     SpdyStreamId stream_id,
     SpdyRstStreamStatus status) const {
-  SpdyRstStreamIR rst_ir(stream_id, status, "RST");
+  SpdyRstStreamIR rst_ir(stream_id, status, "");
   return CreateFramer(false)->SerializeRstStream(rst_ir);
 }