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

Unified Diff: net/spdy/spdy_session.h

Issue 2600973002: Implement HTTP/2 settings field trial parameters. (Closed)
Patch Set: Re: #7. Created 4 years 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_network_transaction_unittest.cc ('k') | net/spdy/spdy_session.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_session.h
diff --git a/net/spdy/spdy_session.h b/net/spdy/spdy_session.h
index aef7606135349635e251938d6bc1eda6039307ac..61f2a9d14840448dbd3dddeec8b8e570d4b07f85 100644
--- a/net/spdy/spdy_session.h
+++ b/net/spdy/spdy_session.h
@@ -75,12 +75,6 @@ const int32_t kDefaultInitialWindowSize = 65535;
// sends a SETTINGS frame with a different value.
const size_t kInitialMaxConcurrentStreams = 100;
-// Specifies the maximum HPACK dynamic table size the server is allowed to set.
-const int kMaxHeaderTableSize = 64 * 1024;
-
-// Specifies the maximum concurrent streams server could send (via push).
-const int kMaxConcurrentPushedStreams = 1000;
-
// If more than this many bytes have been read or more than that many
// milliseconds have passed, return ERR_IO_PENDING from ReadLoop.
const int kYieldAfterBytesRead = 32 * 1024;
@@ -307,7 +301,7 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
bool enable_sending_initial_data,
bool enable_ping_based_connection_checking,
size_t session_max_recv_window_size,
- size_t stream_max_recv_window_size,
+ const SettingsMap& initial_settings,
TimeFunc time_func,
ServerPushDelegate* push_delegate,
ProxyDelegate* proxy_delegate,
@@ -1094,6 +1088,12 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
// Initialized to OK.
Error error_on_close_;
+ // Settings that are sent in the initial SETTINGS frame
+ // (if |enable_sending_initial_data_| is true),
+ // and also control SpdySession parameters like initial receive window size
+ // and maximum HPACK dynamic table size.
+ const SettingsMap initial_settings_;
+
// Limits
size_t max_concurrent_streams_;
size_t max_concurrent_pushed_streams_;
@@ -1151,6 +1151,9 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
// this value for the initial send window size.
int32_t stream_initial_send_window_size_;
+ // The maximum HPACK dynamic table size the server is allowed to set.
+ uint32_t max_header_table_size_;
+
// Initial receive window size for this session's streams. There are
// plans to add a command line switch that would cause a SETTINGS
// frame with window size announcement to be sent on startup. Newly
« no previous file with comments | « net/spdy/spdy_network_transaction_unittest.cc ('k') | net/spdy/spdy_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698