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

Unified Diff: net/spdy/spdy_session.h

Issue 11415219: Move a number of static variables SPDY to HttpNetworkSession::Params. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix flip_in_mem_edsm_server Created 8 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_proxy_client_socket_spdy3_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 22cd6123abd729e54ebdbec0dac5d814b8ea3f1d..a51daae507bbe2b9b40de3108b154e242db6ed85 100644
--- a/net/spdy/spdy_session.h
+++ b/net/spdy/spdy_session.h
@@ -94,6 +94,8 @@ COMPILE_ASSERT(PROTOCOL_ERROR_UNEXPECTED_PING ==
class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
public BufferedSpdyFramerVisitorInterface {
public:
+ typedef base::TimeTicks (*TimeFunc)(void);
+
// Defines an interface for producing SpdyIOBuffers.
class NET_EXPORT_PRIVATE SpdyIOBufferProducer {
public:
@@ -132,6 +134,14 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
HttpServerProperties* http_server_properties,
bool verify_domain_authentication,
bool enable_sending_initial_settings,
+ bool enable_credential_frames,
+ bool enable_compression,
+ bool enable_ping_based_connection_checking,
+ NextProto default_protocol_,
+ size_t initial_recv_window_size,
+ size_t initial_max_concurrent_streams,
+ size_t max_concurrent_streams_limit,
+ TimeFunc time_func,
const HostPortPair& trusted_spdy_proxy,
NetLog* net_log);
@@ -249,30 +259,6 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
// if server bound certs are not supported in this session.
ServerBoundCertService* GetServerBoundCertService() const;
- // Reset all static settings to initialized values. Used to init test suite.
- static void ResetStaticSettingsToInit();
-
- // Specify the SPDY protocol to be used for SPDY session which do not use NPN
- // to negotiate a particular protocol.
- static void set_default_protocol(NextProto default_protocol);
-
- // Sets the max concurrent streams per session, as a ceiling on any server
- // specific SETTINGS value.
- static void set_max_concurrent_streams(size_t value);
-
- // Enable sending of PING frame with each request.
- static void set_enable_ping_based_connection_checking(bool enable);
-
- // Enable the sending of CREDENTIAL frames.
- static void set_enable_credential_frames(bool enable);
-
- // The initial max concurrent streams per session, can be overridden by the
- // server via SETTINGS.
- static void set_init_max_concurrent_streams(size_t value);
-
- // Sets the initial receive window size for newly created sessions.
- static void set_default_initial_recv_window_size(size_t value);
-
// Send WINDOW_UPDATE frame, called by a stream whenever receive window
// size is increased.
void SendWindowUpdate(SpdyStreamId stream_id, int32 delta_window_size);
@@ -425,8 +411,6 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
CLOSED
};
- typedef base::TimeTicks (*TimeFunc)(void);
-
virtual ~SpdySession();
void ProcessPendingCreateStreams();
@@ -560,8 +544,6 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
// Helper methods for testing
// --------------------------
- static TimeFunc set_time_func(TimeFunc new_time_func);
-
void set_connection_at_risk_of_loss_time(base::TimeDelta duration) {
connection_at_risk_of_loss_time_ = duration;
}
@@ -666,6 +648,7 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
// Limits
size_t max_concurrent_streams_; // 0 if no limit
+ size_t max_concurrent_streams_limit_;
// Some statistics counters for the session.
int streams_initiated_count_;
@@ -717,6 +700,10 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
// Outside of tests, these should always be true.
bool verify_domain_authentication_;
bool enable_sending_initial_settings_;
+ bool enable_credential_frames_;
+ bool enable_compression_;
+ bool enable_ping_based_connection_checking_;
+ NextProto default_protocol_;
SpdyCredentialState credential_state_;
@@ -745,6 +732,8 @@ class NET_EXPORT SpdySession : public base::RefCounted<SpdySession>,
// 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
« no previous file with comments | « net/spdy/spdy_proxy_client_socket_spdy3_unittest.cc ('k') | net/spdy/spdy_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698