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

Unified Diff: net/spdy/spdy_session_spdy3_unittest.cc

Issue 9618002: SPDY - integration of spdy/3 code. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 9 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_session_spdy2_unittest.cc ('k') | net/spdy/spdy_settings_storage.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/spdy/spdy_session_spdy3_unittest.cc
===================================================================
--- net/spdy/spdy_session_spdy3_unittest.cc (revision 126086)
+++ net/spdy/spdy_session_spdy3_unittest.cc (working copy)
@@ -24,6 +24,10 @@
spdy::SpdyFramer::set_enable_compression_default(false);
}
protected:
+ virtual void SetUp() {
+ SpdySession::set_default_protocol(SSLClientSocket::kProtoSPDY3);
+ }
+
virtual void TearDown() {
// Wanted to be 100% sure PING is disabled.
SpdySession::set_enable_ping_based_connection_checking(false);
@@ -497,8 +501,7 @@
session_deps.host_resolver->set_synchronous_mode(true);
spdy::SpdySettings new_settings;
- spdy::SettingsFlagsAndId id(0);
- id.set_id(spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
+ spdy::SettingsFlagsAndId id(0, spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
const size_t max_concurrent_streams = 2;
new_settings.push_back(spdy::SpdySetting(id, max_concurrent_streams));
@@ -530,8 +533,8 @@
// Initialize the SpdySettingsStorage with 1 max concurrent streams.
SpdySessionPool* spdy_session_pool(http_session->spdy_session_pool());
spdy::SpdySettings old_settings;
- id.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
- old_settings.push_back(spdy::SpdySetting(id, 1));
+ spdy::SettingsFlagsAndId id1(spdy::SETTINGS_FLAG_PLEASE_PERSIST, id.id());
+ old_settings.push_back(spdy::SpdySetting(id1, 1));
spdy_session_pool->http_server_properties()->SetSpdySettings(
test_host_port_pair, old_settings);
@@ -612,9 +615,8 @@
// Initialize the SpdySettingsStorage with 1 max concurrent streams.
SpdySessionPool* spdy_session_pool(http_session->spdy_session_pool());
spdy::SpdySettings settings;
- spdy::SettingsFlagsAndId id(0);
- id.set_id(spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
- id.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
+ spdy::SettingsFlagsAndId id(spdy::SETTINGS_FLAG_PLEASE_PERSIST,
+ spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
settings.push_back(spdy::SpdySetting(id, 1));
spdy_session_pool->http_server_properties()->SetSpdySettings(
test_host_port_pair, settings);
@@ -685,9 +687,7 @@
spdy::SpdySettings settings;
const uint32 kBogusSettingId = 0xABAB;
const uint32 kBogusSettingValue = 0xCDCD;
- spdy::SettingsFlagsAndId id(0);
- id.set_id(kBogusSettingId);
- id.set_flags(spdy::SETTINGS_FLAG_PERSISTED);
+ spdy::SettingsFlagsAndId id(spdy::SETTINGS_FLAG_PERSISTED, kBogusSettingId);
settings.push_back(spdy::SpdySetting(id, kBogusSettingValue));
MockConnect connect_data(SYNCHRONOUS, OK);
scoped_ptr<spdy::SpdyFrame> settings_frame(
@@ -712,9 +712,9 @@
HostPortPair test_host_port_pair(kTestHost, kTestPort);
HostPortProxyPair pair(test_host_port_pair, ProxyServer::Direct());
- id.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
+ spdy::SettingsFlagsAndId id1(spdy::SETTINGS_FLAG_PLEASE_PERSIST, id.id());
settings.clear();
- settings.push_back(spdy::SpdySetting(id, kBogusSettingValue));
+ settings.push_back(spdy::SpdySetting(id1, kBogusSettingValue));
SpdySessionPool* spdy_session_pool(http_session->spdy_session_pool());
spdy_session_pool->http_server_properties()->SetSpdySettings(
test_host_port_pair, settings);
@@ -873,9 +873,8 @@
const int kTestPort = 80;
HostPortPair test_host_port_pair(kTestHost, kTestPort);
spdy::SpdySettings test_settings;
- spdy::SettingsFlagsAndId id(0);
- id.set_id(spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
- id.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
+ spdy::SettingsFlagsAndId id(spdy::SETTINGS_FLAG_PLEASE_PERSIST,
+ spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
const size_t max_concurrent_streams = 2;
test_settings.push_back(spdy::SpdySetting(id, max_concurrent_streams));
@@ -897,9 +896,8 @@
HttpServerProperties* test_http_server_properties =
spdy_session_pool->http_server_properties();
- spdy::SettingsFlagsAndId id(0);
- id.set_id(spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
- id.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
+ spdy::SettingsFlagsAndId id(spdy::SETTINGS_FLAG_PLEASE_PERSIST,
+ spdy::SETTINGS_MAX_CONCURRENT_STREAMS);
const size_t max_concurrent_streams = 2;
spdy::SpdySettings test_settings;
test_settings.push_back(spdy::SpdySetting(id, max_concurrent_streams));
« no previous file with comments | « net/spdy/spdy_session_spdy2_unittest.cc ('k') | net/spdy/spdy_settings_storage.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698