WebSocket Pepper API: Implement extensions

ppapi/tests/test_websocket.cc

Index: ppapi/tests/test_websocket.cc
diff --git a/ppapi/tests/test_websocket.cc b/ppapi/tests/test_websocket.cc
index 2eb7af465f65d37b727626ca7afb41bb5f53ab8f..5dca18bc4e917402428eb09725cae8b61bfb197f 100644
--- a/ppapi/tests/test_websocket.cc
+++ b/ppapi/tests/test_websocket.cc
@@ -748,7 +748,8 @@ std::string TestWebSocket::TestCcInterfaces() {
   ASSERT_EQ(PP_WEBSOCKETSTATUSCODE_NORMAL_CLOSURE, ws.GetCloseCode());
   ASSERT_TRUE(AreEqualWithString(ws.GetCloseReason().pp_var(), reason.c_str()));
   ASSERT_EQ(true, ws.GetCloseWasClean());
-  ASSERT_TRUE(AreEqualWithString(ws.GetExtensions().pp_var(), ""));
+  ASSERT_TRUE(AreEqualWithString(ws.GetExtensions().pp_var(),
+      "x-webkit-deflate-frame"));

[dmichael (off chromium), 2012/03/09 18:27:16]

What's the extension do? Is there a way for a PPAPI user to use the extension?

[Takashi Toyoshima, 2012/03/09 18:46:15]

Extensions is defined in RFC for WebSocket.
This mechanism is used to extend base WebSocket protocol, E.g., adding zlib
compression, multiplexing, etc.

Then, this extensions property just show which extensions are applied between
the client and the server. The client could not control which one should be
applied. This is same with JavaScript version of API. But some client want to
know which one is applied. E.g, some client may want to restrict connection
numbers when multiplexing is not applied. Thus, this interface is useful for a
PPAPI user, too.

BTW, I think this expectation depends on WebKit side implementation. This
"x-webkit-deflate-frame" will be changed to "deflate-frame" near the future. And
other extensions might be added with delimiter like "deflate-frame,mux", again.

So, yesterday I land another layout test where all extensions is disabled by
default. This WebKit change will be merged soon, and there, I keep old
expectation "" for this line. In that case, the change for this unit test is
just changing server URL like s/echo/echo-with-no-extension/.

   ASSERT_TRUE(AreEqualWithString(ws.GetProtocol().pp_var(), ""));
   ASSERT_EQ(PP_WEBSOCKETREADYSTATE_CLOSED, ws.GetReadyState());
   ASSERT_TRUE(AreEqualWithString(ws.GetURL().pp_var(), kCloseServerURL));