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

Unified Diff: net/websockets/websocket_extension_parser_test.cc

Issue 266243004: Clang format slam. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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
Index: net/websockets/websocket_extension_parser_test.cc
diff --git a/net/websockets/websocket_extension_parser_test.cc b/net/websockets/websocket_extension_parser_test.cc
index dc7dc859d832b2f492f3a325cd87fe3183b6b93d..7bbb7ebf3284bcfd04a248bd6f3987f131a04381 100644
--- a/net/websockets/websocket_extension_parser_test.cc
+++ b/net/websockets/websocket_extension_parser_test.cc
@@ -66,37 +66,37 @@ TEST(WebSocketExtensionParserTest, ParseOneExtensionWithParams) {
TEST(WebSocketExtensionParserTest, InvalidPatterns) {
const char* patterns[] = {
- "fo\ao", // control in extension name
- "fo\x01o", // control in extension name
- "fo<o", // separator in extension name
- "foo/", // separator in extension name
- ";bar", // empty extension name
- "foo bar", // missing ';'
- "foo;", // extension parameter without name and value
- "foo; b\ar", // control in parameter name
- "foo; b\x7fr", // control in parameter name
- "foo; b[r", // separator in parameter name
- "foo; ba:", // separator in parameter name
- "foo; =baz", // empty parameter name
- "foo; bar=", // empty parameter value
- "foo; =", // empty parameter name and value
- "foo; bar=b\x02z", // control in parameter value
- "foo; bar=b@z", // separator in parameter value
- "foo; bar=b\\z", // separator in parameter value
- "foo; bar=b?z", // separator in parameter value
- "\"foo\"", // quoted extension name
- "foo; \"bar\"", // quoted parameter name
- "foo; bar=\"\a2\"", // control in quoted parameter value
- "foo; bar=\"b@z\"", // separator in quoted parameter value
- "foo; bar=\"b\\\\z\"", // separator in quoted parameter value
- "foo; bar=\"\"", // quoted empty parameter value
- "foo; bar=\"baz", // unterminated quoted string
- "foo; bar=\"baz \"", // space in quoted string
- "foo; bar baz", // mising '='
- "foo; bar - baz", // '-' instead of '=' (note: "foo; bar-baz" is valid).
- "foo; bar=\r\nbaz", // CRNL not followed by a space
- "foo; bar=\r\n baz", // CRNL followed by a space
- "foo, bar" // multiple extensions
+ "fo\ao", // control in extension name
+ "fo\x01o", // control in extension name
+ "fo<o", // separator in extension name
+ "foo/", // separator in extension name
+ ";bar", // empty extension name
+ "foo bar", // missing ';'
+ "foo;", // extension parameter without name and value
+ "foo; b\ar", // control in parameter name
+ "foo; b\x7fr", // control in parameter name
+ "foo; b[r", // separator in parameter name
+ "foo; ba:", // separator in parameter name
+ "foo; =baz", // empty parameter name
+ "foo; bar=", // empty parameter value
+ "foo; =", // empty parameter name and value
+ "foo; bar=b\x02z", // control in parameter value
+ "foo; bar=b@z", // separator in parameter value
+ "foo; bar=b\\z", // separator in parameter value
+ "foo; bar=b?z", // separator in parameter value
+ "\"foo\"", // quoted extension name
+ "foo; \"bar\"", // quoted parameter name
+ "foo; bar=\"\a2\"", // control in quoted parameter value
+ "foo; bar=\"b@z\"", // separator in quoted parameter value
+ "foo; bar=\"b\\\\z\"", // separator in quoted parameter value
+ "foo; bar=\"\"", // quoted empty parameter value
+ "foo; bar=\"baz", // unterminated quoted string
+ "foo; bar=\"baz \"", // space in quoted string
+ "foo; bar baz", // mising '='
+ "foo; bar - baz", // '-' instead of '=' (note: "foo; bar-baz" is valid).
+ "foo; bar=\r\nbaz", // CRNL not followed by a space
+ "foo; bar=\r\n baz", // CRNL followed by a space
+ "foo, bar" // multiple extensions
};
for (size_t i = 0; i < arraysize(patterns); ++i) {

Powered by Google App Engine
This is Rietveld 408576698