Move WebSocketHandleImpl into Blink

third_party/WebKit/Source/modules/modules.gypi

 {
   'includes': [
     '../core/core_generated.gypi',
     'modules_generated.gypi',
   ],
   'variables': {
     # Experimental hooks for embedder to provide extra IDL and source files.
     #
     # Note: this is not a supported API. If you rely on this, you will be broken
     # from time to time as the code generator changes in backward incompatible
@@ skipping 1953 matching lines @@
       'websockets/DOMWebSocket.h',
       'websockets/DocumentWebSocketChannel.cpp',
       'websockets/DocumentWebSocketChannel.h',
       'websockets/InspectorWebSocketEvents.cpp',
       'websockets/InspectorWebSocketEvents.h',
       'websockets/WebSocketChannel.cpp',
       'websockets/WebSocketChannel.h',
       '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',
       'webusb/NavigatorUSB.h',
       'webusb/USB.cpp',
       'webusb/USB.h',
       'webusb/USBAlternateInterface.cpp',
       'webusb/USBAlternateInterface.h',
       'webusb/USBConfiguration.cpp',
       'webusb/USBConfiguration.h',
@@ skipping 99 matching lines @@
       'push_messaging/PushMessageDataTest.cpp',
       'serviceworkers/ServiceWorkerContainerTest.cpp',
       'webaudio/AudioBasicProcessorHandlerTest.cpp',
       'webaudio/ConvolverNodeTest.cpp',
       'webaudio/DynamicsCompressorNodeTest.cpp',
       'webaudio/ScriptProcessorNodeTest.cpp',
       'webaudio/StereoPannerNodeTest.cpp',
       '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.

     ],
   },
 }