Move WebSocketHandleImpl into Blink

third_party/WebKit/Source/modules/modules.gypi

Index: third_party/WebKit/Source/modules/modules.gypi
diff --git a/third_party/WebKit/Source/modules/modules.gypi b/third_party/WebKit/Source/modules/modules.gypi
index e9dd040987d11e5ad49cc6554fdc86dab7d974a8..89b1edc45e6039676f7119d958fc64b92f4b866c 100644
--- a/third_party/WebKit/Source/modules/modules.gypi
+++ b/third_party/WebKit/Source/modules/modules.gypi
@@ -1971,6 +1971,9 @@
       'websockets/WebSocketChannelClient.h',
       'websockets/WebSocketFrame.cpp',
       'websockets/WebSocketFrame.h',
+      'websockets/WebSocketHandle.cpp',
+      'websockets/WebSocketHandle.h',
+      'websockets/WebSocketHandleClient.h',
       'websockets/WorkerWebSocketChannel.cpp',
       'websockets/WorkerWebSocketChannel.h',
       'webusb/NavigatorUSB.cpp',
@@ -2090,6 +2093,7 @@
       'webdatabase/QuotaTrackerTest.cpp',
       'websockets/DOMWebSocketTest.cpp',
       'websockets/DocumentWebSocketChannelTest.cpp',
+      'websockets/WebSocketHandle.cpp',

[darin (slow to review), 2016/08/22 17:28:33]

This is here because DocumentWebSocketChannelTest now has a dependency on
WebSocketHandle. That used to be an interface w/ just a bunch of pure virtual
methods. Now, instead, it is the real concrete class with virtual methods that
can be overridden for the sake of testing. I did that because it seemed better
to have less code (i.e., no WebSocketHandleImpl class), but it seems like an
uncommon approach, so flagging here for consideration.

     ],
   },
 }