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

Unified Diff: net/spdy/spdy_protocol.cc

Issue 286143011: Accept SETTINGS_COMPRESS_DATA frames for HTTP2. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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
« no previous file with comments | « net/spdy/spdy_protocol.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_protocol.cc
diff --git a/net/spdy/spdy_protocol.cc b/net/spdy/spdy_protocol.cc
index 94c8144a67643dbc749053f0d662b9aeb521ea54..3fe90c2e3faee65b0d72051f49ac2be9f0c9a193 100644
--- a/net/spdy/spdy_protocol.cc
+++ b/net/spdy/spdy_protocol.cc
@@ -215,9 +215,9 @@ bool SpdyConstants::IsValidSettingId(SpdyMajorVersion version,
return false;
}
- // INITIAL_WINDOW_SIZE is the last valid setting id.
+ // COMPRESS_DATA is the last valid setting id.
if (setting_id_field >
- SerializeSettingId(version, SETTINGS_INITIAL_WINDOW_SIZE)) {
+ SerializeSettingId(version, SETTINGS_COMPRESS_DATA)) {
return false;
}
@@ -261,6 +261,8 @@ SpdySettingsIds SpdyConstants::ParseSettingId(SpdyMajorVersion version,
return SETTINGS_MAX_CONCURRENT_STREAMS;
case 4:
return SETTINGS_INITIAL_WINDOW_SIZE;
+ case 5:
+ return SETTINGS_COMPRESS_DATA;
}
break;
}
@@ -304,6 +306,8 @@ int SpdyConstants::SerializeSettingId(SpdyMajorVersion version,
return 3;
case SETTINGS_INITIAL_WINDOW_SIZE:
return 4;
+ case SETTINGS_COMPRESS_DATA:
+ return 5;
default:
LOG(DFATAL) << "Serializing unhandled setting id " << id;
return -1;
« no previous file with comments | « net/spdy/spdy_protocol.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698