Use enum BinaryType for WebSocket.binaryType, instead of DOMString

Source/modules/websockets/DOMWebSocketTest.cpp

Index: Source/modules/websockets/DOMWebSocketTest.cpp
diff --git a/Source/modules/websockets/DOMWebSocketTest.cpp b/Source/modules/websockets/DOMWebSocketTest.cpp
index e21e5a823cd8b34ab90b2f0b4b415790664b1af0..83be5bc0a9c9e8fdddf7d175055f71906d599ce1 100644
--- a/Source/modules/websockets/DOMWebSocketTest.cpp
+++ b/Source/modules/websockets/DOMWebSocketTest.cpp
@@ -666,21 +666,28 @@ TEST_F(DOMWebSocketTest, binaryType)
 {
     EXPECT_EQ("blob", m_websocket->binaryType());
 

[Nils Barth (inactive), 2015/04/02 14:56:35]

BTW, could you remove these blank lines?
They're useless and nowhere else in this test uses them.

[Paritosh Kumar, 2015/04/02 15:41:32]

I think, this space is between one operation call and EXPECT_* macro.

[Nils Barth (inactive), 2015/04/02 15:54:55]

Ok, point taken.
(It's a bit loose spacing for my taste, but ok to leave as is.)

[Paritosh Kumar, 2015/04/02 16:14:04]

Thanks. Leaving this space.

-    m_websocket->setBinaryType("hoge");
+    m_websocket->setBinaryType("arraybuffer");
+
+    EXPECT_EQ("arraybuffer", m_websocket->binaryType());
+
+    m_websocket->setBinaryType("blob");
 
     EXPECT_EQ("blob", m_websocket->binaryType());
+}
+
+class DOMWebSocketDeathTest : public DOMWebSocketTestBase, public ::testing::Test {
+public:
+};
 
+TEST_F(DOMWebSocketDeathTest, binaryType)
+{
     m_websocket->setBinaryType("arraybuffer");
 

[Nils Barth (inactive), 2015/04/02 14:56:35]

Same for these lines.

     EXPECT_EQ("arraybuffer", m_websocket->binaryType());
 
-    m_websocket->setBinaryType("fuga");
+    EXPECT_DEATH(m_websocket->setBinaryType("hoge"), "");
 

[Paritosh Kumar, 2015/04/02 15:41:32]

We can remove this space.

[Nils Barth (inactive), 2015/04/02 15:54:55]

Good point; could you? Thanks!

[Paritosh Kumar, 2015/04/02 16:14:04]

Thanks Removed.

     EXPECT_EQ("arraybuffer", m_websocket->binaryType());
-
-    m_websocket->setBinaryType("blob");
-
-    EXPECT_EQ("blob", m_websocket->binaryType());
 }
 
 // FIXME: We should add tests for suspend / resume.