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

Unified Diff: net/spdy/spdy_protocol.cc

Issue 2445113002: Add HTTP/2 error code NO_ERROR. (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/spdy/spdy_protocol.cc
diff --git a/net/spdy/spdy_protocol.cc b/net/spdy/spdy_protocol.cc
index e8dd952fcbca8075e5354f734db086788f71728f..1cf988114244e0aebd82f08a5ca4d9f01b11cce3 100644
--- a/net/spdy/spdy_protocol.cc
+++ b/net/spdy/spdy_protocol.cc
@@ -390,7 +390,7 @@ bool SpdyConstants::IsValidRstStreamStatus(SpdyMajorVersion version,
case HTTP2:
// NO_ERROR is the first valid status code.
if (rst_stream_status_field <
- SerializeRstStreamStatus(version, RST_STREAM_PROTOCOL_ERROR)) {
+ SerializeRstStreamStatus(version, RST_STREAM_NO_ERROR)) {
return false;
}
@@ -447,6 +447,8 @@ SpdyRstStreamStatus SpdyConstants::ParseRstStreamStatus(
break;
case HTTP2:
switch (rst_stream_status_field) {
+ case 0:
+ return RST_STREAM_NO_ERROR;
case 1:
return RST_STREAM_PROTOCOL_ERROR;
case 2:
@@ -509,6 +511,8 @@ int SpdyConstants::SerializeRstStreamStatus(
}
case HTTP2:
switch (rst_stream_status) {
+ case RST_STREAM_NO_ERROR:
+ return 0;
case RST_STREAM_PROTOCOL_ERROR:
return 1;
case RST_STREAM_INTERNAL_ERROR:

Powered by Google App Engine
This is Rietveld 408576698