Network service: Safe browsing check for sub-resources from renderer.

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 48 matching lines @@
 
 namespace media {
 class KeySystemProperties;
 }
 
 namespace content {
 class BrowserPluginDelegate;
 class MediaStreamRendererFactory;
 class RenderFrame;
 class RenderView;
+class URLLoaderThrottle;
 
 // Embedder API for participating in renderer logic.
 class CONTENT_EXPORT ContentRendererClient {
  public:
   virtual ~ContentRendererClient() {}
 
   // Notifies us that the RenderThread has been created.
   virtual void RenderThreadStarted() {}
 
   // Notifies that a new RenderFrame has been created.
@@ skipping 137 matching lines @@
   // header will be send for the navigation. Otherwise, the referrer header is
   // set according to the frame's referrer policy.
   virtual bool ShouldFork(blink::WebLocalFrame* frame,
                           const GURL& url,
                           const std::string& http_method,
                           bool is_initial_navigation,
                           bool is_server_redirect,
                           bool* send_referrer);
 
   // Notifies the embedder that the given frame is requesting the resource at
-  // |url|.  If the function returns true, the url is changed to |new_url|.
-  virtual bool WillSendRequest(blink::WebLocalFrame* frame,
-                               ui::PageTransition transition_type,
-                               const blink::WebURL& url,
-                               GURL* new_url);
+  // |url|.
+  // |throttles| is appended with URLLoaderThrottle instances that should be

[vakh (use Gerrit instead), 2017/05/31 23:53:09]

Nit: These three comments could be in one paragraph.

[yzshen1, 2017/06/01 17:45:04]

Done.

+  // applied to the resource loading. It is only used when network service is
+  // enabled.
+  // If the function returns true, the url is changed to |new_url|.
+  virtual bool WillSendRequest(
+      blink::WebLocalFrame* frame,
+      ui::PageTransition transition_type,
+      const blink::WebURL& url,
+      std::vector<std::unique_ptr<URLLoaderThrottle>>* throttles,
+      GURL* new_url);
 
   // Returns true if the request is associated with a document that is in
   // ""prefetch only" mode, and will not be rendered.
   virtual bool IsPrefetchOnly(RenderFrame* render_frame,
                               const blink::WebURLRequest& request);
 
   // See blink::Platform.
   virtual unsigned long long VisitedLinkHash(const char* canonical_url,
                                              size_t length);
   virtual bool IsLinkVisited(unsigned long long link_hash);
@@ skipping 119 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_