SafeBrowsing support for WebSocket (post-network-servicification)

content/public/renderer/content_renderer_client.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 #define CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_
 
 #include <stddef.h>
 
 #include <map>
@@ skipping 29 matching lines @@
 class WebFrame;
 class WebLocalFrame;
 class WebMIDIAccessor;
 class WebMIDIAccessorClient;
 class WebMediaStreamCenter;
 class WebMediaStreamCenterClient;
 class WebPlugin;
 class WebPrescientNetworking;
 class WebRTCPeerConnectionHandler;
 class WebRTCPeerConnectionHandlerClient;
+class WebSocketHandshakeThrottle;
 class WebSpeechSynthesizer;
 class WebSpeechSynthesizerClient;
 class WebThemeEngine;
 class WebURL;
+class WebURLRequest;
 class WebURLResponse;
-class WebURLRequest;
 struct WebPluginParams;
 struct WebURLError;
 }  // namespace blink
 
 namespace media {
 class KeySystemProperties;
 }
 
 namespace content {
 class BrowserPluginDelegate;
@@ skipping 102 matching lines @@
       const blink::WebAudioLatencyHint& latency_hint);
 
   // Allows the embedder to override the blink::WebClipboard used. If it
   // returns NULL the content layer will handle clipboard interactions.
   virtual blink::WebClipboard* OverrideWebClipboard();
 
   // Allows the embedder to override the WebThemeEngine used. If it returns NULL
   // 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>
   OverrideSpeechSynthesizer(blink::WebSpeechSynthesizerClient* client);
 
   // Returns true if the renderer process should schedule the idle handler when
   // all widgets are hidden.
   virtual bool RunIdleHandlerWhenWidgetsHidden();
 
   // Returns true if the renderer process should allow shared timer suspension
@@ skipping 180 matching lines @@
   virtual std::unique_ptr<base::TaskScheduler::InitParams>
   GetTaskSchedulerInitParams();
 
   // Returns true if the media pipeline can be suspended, or false otherwise.
   virtual bool AllowMediaSuspend();
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_CONTENT_RENDERER_CLIENT_H_