| Index: net/http/http_server_properties_impl_unittest.cc
|
| ===================================================================
|
| --- net/http/http_server_properties_impl_unittest.cc (revision 126086)
|
| +++ net/http/http_server_properties_impl_unittest.cc (working copy)
|
| @@ -291,9 +291,7 @@
|
|
|
| // Check by initializing with www.google.com:443 spdy server settings.
|
| spdy::SpdySettings spdy_settings;
|
| - spdy::SettingsFlagsAndId spdy_setting(0);
|
| - spdy_setting.set_flags(spdy::SETTINGS_FLAG_PERSISTED);
|
| - spdy_setting.set_id(1234);
|
| + spdy::SettingsFlagsAndId spdy_setting(spdy::SETTINGS_FLAG_PERSISTED, 1234);
|
| spdy_settings.push_back(std::make_pair(spdy_setting, 31337));
|
| spdy_settings_map[spdy_server_google] = spdy_settings;
|
| impl_.InitializeSpdySettingsServers(&spdy_settings_map);
|
| @@ -306,7 +304,7 @@
|
| EXPECT_EQ(31337U, spdy_settings2.front().second);
|
| }
|
|
|
| -TEST_F(SpdySettingsServerPropertiesTest, SpdySettingsTest) {
|
| +TEST_F(SpdySettingsServerPropertiesTest, SetSpdySettings) {
|
| HostPortPair spdy_server_empty("", 443);
|
| spdy::SpdySettings spdy_settings0 = impl_.GetSpdySettings(spdy_server_empty);
|
| EXPECT_EQ(0U, spdy_settings0.size()); // Returns kEmptySpdySettings
|
| @@ -314,9 +312,7 @@
|
| // Add www.google.com:443 as persisting.
|
| HostPortPair spdy_server_google("www.google.com", 443);
|
| spdy::SpdySettings spdy_settings1;
|
| - spdy::SettingsFlagsAndId id1(0);
|
| - id1.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
|
| - id1.set_id(1234);
|
| + spdy::SettingsFlagsAndId id1(spdy::SETTINGS_FLAG_PLEASE_PERSIST, 1234);
|
| spdy_settings1.push_back(std::make_pair(id1, 31337));
|
| EXPECT_TRUE(impl_.SetSpdySettings(spdy_server_google, spdy_settings1));
|
| spdy::SpdySettings spdy_settings1_ret =
|
| @@ -331,8 +327,7 @@
|
| // Add mail.google.com:443 as not persisting.
|
| HostPortPair spdy_server_mail("mail.google.com", 443);
|
| spdy::SpdySettings spdy_settings2;
|
| - spdy::SettingsFlagsAndId id2(0);
|
| - id2.set_id(5678);
|
| + spdy::SettingsFlagsAndId id2(0, 5678);
|
| spdy_settings2.push_back(std::make_pair(id2, 62667));
|
| EXPECT_FALSE(impl_.SetSpdySettings(spdy_server_mail, spdy_settings2));
|
| spdy::SpdySettings spdy_settings2_ret =
|
| @@ -342,9 +337,7 @@
|
| // Add docs.google.com:443 as persisting
|
| HostPortPair spdy_server_docs("docs.google.com", 443);
|
| spdy::SpdySettings spdy_settings3;
|
| - spdy::SettingsFlagsAndId id3(0);
|
| - id3.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
|
| - id3.set_id(9012);
|
| + spdy::SettingsFlagsAndId id3(spdy::SETTINGS_FLAG_PLEASE_PERSIST, 9012);
|
| spdy_settings3.push_back(std::make_pair(id3, 93997));
|
| EXPECT_TRUE(impl_.SetSpdySettings(spdy_server_docs, spdy_settings3));
|
| spdy::SpdySettings spdy_settings3_ret =
|
| @@ -357,13 +350,63 @@
|
| EXPECT_EQ(93997U, spdy_setting3_ret.second);
|
| }
|
|
|
| +TEST_F(SpdySettingsServerPropertiesTest, SetSpdySetting) {
|
| + HostPortPair spdy_server_empty("", 443);
|
| + spdy::SpdySettings spdy_settings0 = impl_.GetSpdySettings(spdy_server_empty);
|
| + EXPECT_EQ(0U, spdy_settings0.size()); // Returns kEmptySpdySettings
|
| +
|
| + // Add www.google.com:443 as persisting.
|
| + HostPortPair spdy_server_google("www.google.com", 443);
|
| + spdy::SettingsFlagsAndId id1(spdy::SETTINGS_FLAG_PLEASE_PERSIST, 1234);
|
| + EXPECT_TRUE(impl_.SetSpdySetting(
|
| + spdy_server_google, std::make_pair(id1, 31337)));
|
| + spdy::SpdySettings spdy_settings1_ret =
|
| + impl_.GetSpdySettings(spdy_server_google);
|
| + ASSERT_EQ(1U, spdy_settings1_ret.size());
|
| + spdy::SpdySetting spdy_setting1_ret = spdy_settings1_ret.front();
|
| + spdy::SettingsFlagsAndId id1_ret(spdy_setting1_ret.first);
|
| + EXPECT_EQ(1234U, id1_ret.id());
|
| + EXPECT_EQ(spdy::SETTINGS_FLAG_PERSISTED, id1_ret.flags());
|
| + EXPECT_EQ(31337U, spdy_setting1_ret.second);
|
| +
|
| + // Add mail.google.com:443 as not persisting.
|
| + HostPortPair spdy_server_mail("mail.google.com", 443);
|
| + spdy::SettingsFlagsAndId id2(0, 5678);
|
| + EXPECT_FALSE(impl_.SetSpdySetting(
|
| + spdy_server_mail, std::make_pair(id2, 62667)));
|
| + spdy::SpdySettings spdy_settings2_ret =
|
| + impl_.GetSpdySettings(spdy_server_mail);
|
| + EXPECT_EQ(0U, spdy_settings2_ret.size()); // Returns kEmptySpdySettings
|
| +
|
| + // Add docs.google.com:443 as persisting
|
| + HostPortPair spdy_server_docs("docs.google.com", 443);
|
| + spdy::SettingsFlagsAndId id3(spdy::SETTINGS_FLAG_PLEASE_PERSIST, 9012);
|
| + EXPECT_TRUE(impl_.SetSpdySetting(
|
| + spdy_server_docs, std::make_pair(id3, 93997)));
|
| + spdy::SpdySettings spdy_settings3_ret =
|
| + impl_.GetSpdySettings(spdy_server_docs);
|
| + ASSERT_EQ(1U, spdy_settings3_ret.size());
|
| + spdy::SpdySetting spdy_setting3_ret = spdy_settings3_ret.front();
|
| + spdy::SettingsFlagsAndId id3_ret(spdy_setting3_ret.first);
|
| + EXPECT_EQ(9012U, id3_ret.id());
|
| + EXPECT_EQ(spdy::SETTINGS_FLAG_PERSISTED, id3_ret.flags());
|
| + EXPECT_EQ(93997U, spdy_setting3_ret.second);
|
| + // Check data for www.google.com:443.
|
| + spdy::SpdySettings spdy_settings4_ret =
|
| + impl_.GetSpdySettings(spdy_server_google);
|
| + ASSERT_EQ(1U, spdy_settings4_ret.size());
|
| + spdy::SpdySetting spdy_setting4_ret = spdy_settings4_ret.front();
|
| + spdy::SettingsFlagsAndId id4_ret(spdy_setting4_ret.first);
|
| + EXPECT_EQ(1234U, id4_ret.id());
|
| + EXPECT_EQ(spdy::SETTINGS_FLAG_PERSISTED, id4_ret.flags());
|
| + EXPECT_EQ(31337U, spdy_setting4_ret.second);
|
| +}
|
| +
|
| TEST_F(SpdySettingsServerPropertiesTest, Clear) {
|
| // Add www.google.com:443 as persisting.
|
| HostPortPair spdy_server_google("www.google.com", 443);
|
| spdy::SpdySettings spdy_settings1;
|
| - spdy::SettingsFlagsAndId id1(0);
|
| - id1.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
|
| - id1.set_id(1234);
|
| + spdy::SettingsFlagsAndId id1(spdy::SETTINGS_FLAG_PLEASE_PERSIST, 1234);
|
| spdy_settings1.push_back(std::make_pair(id1, 31337));
|
| EXPECT_TRUE(impl_.SetSpdySettings(spdy_server_google, spdy_settings1));
|
| spdy::SpdySettings spdy_settings1_ret =
|
| @@ -378,9 +421,7 @@
|
| // Add docs.google.com:443 as persisting
|
| HostPortPair spdy_server_docs("docs.google.com", 443);
|
| spdy::SpdySettings spdy_settings3;
|
| - spdy::SettingsFlagsAndId id3(0);
|
| - id3.set_flags(spdy::SETTINGS_FLAG_PLEASE_PERSIST);
|
| - id3.set_id(9012);
|
| + spdy::SettingsFlagsAndId id3(spdy::SETTINGS_FLAG_PLEASE_PERSIST, 9012);
|
| spdy_settings3.push_back(std::make_pair(id3, 93997));
|
| EXPECT_TRUE(impl_.SetSpdySettings(spdy_server_docs, spdy_settings3));
|
| spdy::SpdySettings spdy_settings3_ret =
|
|
|