Fix WebSocketServer extension parser.

net/websockets/websocket_extension.cc

Index: net/websockets/websocket_extension.cc
diff --git a/net/websockets/websocket_extension.cc b/net/websockets/websocket_extension.cc
index bf7a54bdbf3e17b79f4da23476525ea19d11a39e..f1e6a24090ccf8831125ba916f043ec7780365f5 100644
--- a/net/websockets/websocket_extension.cc
+++ b/net/websockets/websocket_extension.cc
@@ -45,4 +45,35 @@ bool WebSocketExtension::Equals(const WebSocketExtension& other) const {
   return this_parameters == other_parameters;
 }
 
+std::string WebSocketExtension::ToString() const {
+  if (name_.empty()) {
+    return std::string();
+  }
+
+  std::string result = name_;
+
+  for (const auto& param : parameters_) {
+    result += "; " + param.name();
+    if (!param.HasValue())
+      continue;
+
+    result += "=";
+    const auto& value = param.value();
+    if (value.find('"') == std::string::npos) {
+      result += value;
+    } else {
+      result += '"';

[tyoshino (SeeGerritForStatus), 2015/09/16 09:33:10]

per HTTP rfc, we need to scan all non-token characters.

In WebSocket extension header, any non token character is disallowed even in the
quoted-string. So, this is unnecessary and should be DCHECK or something. It's
ok to address this later. Leave TODO?

[yhirano, 2015/09/16 10:47:46]

Done.

+      for (const auto& c : value) {
+        if (c == '"') {
+          result += "\\\"";
+        } else {
+          result += c;
+        }
+      }
+      result += '"';
+    }
+  }
+  return result;
+}
+
 }  // namespace net