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

Unified Diff: net/websockets/websocket_stream_test.cc

Issue 1324113002: Introduce WebSocketDeflateParameters. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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/websockets/websocket_extension.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/websockets/websocket_stream_test.cc
diff --git a/net/websockets/websocket_stream_test.cc b/net/websockets/websocket_stream_test.cc
index 195002aa0b800754c89bcb171c83759cd0b3328f..439304963b52ebe9b9fb2fc76532ffee2779aa07 100644
--- a/net/websockets/websocket_stream_test.cc
+++ b/net/websockets/websocket_stream_test.cc
@@ -563,58 +563,7 @@ TEST_F(WebSocketStreamCreateExtensionTest, OnlyOnePerMessageDeflateAllowed) {
failure_message());
}
-// permessage-deflate parameters may not be duplicated.
-TEST_F(WebSocketStreamCreateExtensionTest, NoDuplicateParameters) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; client_no_context_takeover; "
- "client_no_context_takeover");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received duplicate permessage-deflate extension parameter "
- "client_no_context_takeover",
- failure_message());
-}
-
-// permessage-deflate parameters must start with "client_" or "server_"
-TEST_F(WebSocketStreamCreateExtensionTest, BadParameterPrefix) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; absurd_no_context_takeover");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received an unexpected permessage-deflate extension parameter",
- failure_message());
-}
-
-// permessage-deflate parameters must be either *_no_context_takeover or
-// *_max_window_bits
-TEST_F(WebSocketStreamCreateExtensionTest, BadParameterSuffix) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; client_max_content_bits=5");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received an unexpected permessage-deflate extension parameter",
- failure_message());
-}
-
-// *_no_context_takeover parameters must not have an argument
-TEST_F(WebSocketStreamCreateExtensionTest, BadParameterValue) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; client_no_context_takeover=true");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received invalid client_no_context_takeover parameter",
- failure_message());
-}
-
-// *_max_window_bits must have an argument
+// client_max_window_bits must have an argument
TEST_F(WebSocketStreamCreateExtensionTest, NoMaxWindowBitsArgument) {
CreateAndConnectWithExtensions("permessage-deflate; client_max_window_bits");
EXPECT_FALSE(stream_);
@@ -625,65 +574,8 @@ TEST_F(WebSocketStreamCreateExtensionTest, NoMaxWindowBitsArgument) {
failure_message());
}
-// *_max_window_bits must be an integer
-TEST_F(WebSocketStreamCreateExtensionTest, MaxWindowBitsValueInteger) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; server_max_window_bits=banana");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received invalid server_max_window_bits parameter",
- failure_message());
-}
-
-// *_max_window_bits must be >= 8
-TEST_F(WebSocketStreamCreateExtensionTest, MaxWindowBitsValueTooSmall) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; server_max_window_bits=7");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received invalid server_max_window_bits parameter",
- failure_message());
-}
-
-// *_max_window_bits must be <= 15
-TEST_F(WebSocketStreamCreateExtensionTest, MaxWindowBitsValueTooBig) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; client_max_window_bits=16");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received invalid client_max_window_bits parameter",
- failure_message());
-}
-
-// *_max_window_bits must not start with 0
-TEST_F(WebSocketStreamCreateExtensionTest, MaxWindowBitsValueStartsWithZero) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; client_max_window_bits=08");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received invalid client_max_window_bits parameter",
- failure_message());
-}
-
-// *_max_window_bits must not start with +
-TEST_F(WebSocketStreamCreateExtensionTest, MaxWindowBitsValueStartsWithPlus) {
- CreateAndConnectWithExtensions(
- "permessage-deflate; server_max_window_bits=+9");
- EXPECT_FALSE(stream_);
- EXPECT_TRUE(has_failed());
- EXPECT_EQ(
- "Error during WebSocket handshake: Error in permessage-deflate: "
- "Received invalid server_max_window_bits parameter",
- failure_message());
-}
+// Other cases for permessage-deflate parameters are tested in
+// websocket_deflate_parameters_test.cc.
// TODO(ricea): Check that WebSocketDeflateStream is initialised with the
// arguments from the server. This is difficult because the data written to the
« no previous file with comments | « net/websockets/websocket_extension.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698