SafeBrowsing support for WebSocket (post-network-servicification)

content/public/renderer/content_renderer_client.h

Index: content/public/renderer/content_renderer_client.h
diff --git a/content/public/renderer/content_renderer_client.h b/content/public/renderer/content_renderer_client.h
index 989fd178d0a606a43cda0b8b0c7c7f605c43ecfc..03cd2c6ae07e7fdcac91722e5ce345117e87efd4 100644
--- a/content/public/renderer/content_renderer_client.h
+++ b/content/public/renderer/content_renderer_client.h
@@ -47,12 +47,13 @@ class WebPlugin;
 class WebPrescientNetworking;
 class WebRTCPeerConnectionHandler;
 class WebRTCPeerConnectionHandlerClient;
+class WebSocketHandshakeThrottle;
 class WebSpeechSynthesizer;
 class WebSpeechSynthesizerClient;
 class WebThemeEngine;
 class WebURL;
-class WebURLResponse;
 class WebURLRequest;
+class WebURLResponse;
 struct WebPluginParams;
 struct WebURLError;
 }  // namespace blink
@@ -175,6 +176,11 @@ class CONTENT_EXPORT ContentRendererClient {
   // the content layer will provide an engine.
   virtual blink::WebThemeEngine* OverrideThemeEngine();
 
+  // Allows the embedder to provide a WebSocketHandshakeThrottle. If it returns
+  // NULL then none will be used.
+  virtual std::unique_ptr<blink::WebSocketHandshakeThrottle>

[yzshen1, 2017/06/20 19:52:19]

Out of curiosity: why inserting the method in the middle instead of appending?

[Adam Rice, 2017/06/21 06:08:36]

I'm attempting to imitate the order from blink::Profile. Whether this is
actually a good idea or not I don't know.

blink::Profile methods are grouped by the thing they pertain to and then ordered
by the name of that thing. So WebS_o_cket comes before WebS_p_eechSynthesizer.

+  CreateWebSocketHandshakeThrottle();
+
   // Allows the embedder to override the WebSpeechSynthesizer used.
   // If it returns NULL the content layer will provide an engine.
   virtual std::unique_ptr<blink::WebSpeechSynthesizer>