 Chromium Code Reviews
 Chromium Code Reviews Issue 2900563002:
  Network service: Safe browsing check for sub-resources from renderer.  (Closed)
    
  
    Issue 2900563002:
  Network service: Safe browsing check for sub-resources from renderer.  (Closed) 
  | 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 0e554ba8467af157948ca32d042f7dbac426df8f..d9fb4de35675703a3f1b5cfbd7d37d9ef1102ab7 100644 | 
| --- a/content/public/renderer/content_renderer_client.h | 
| +++ b/content/public/renderer/content_renderer_client.h | 
| @@ -66,6 +66,7 @@ class BrowserPluginDelegate; | 
| class MediaStreamRendererFactory; | 
| class RenderFrame; | 
| class RenderView; | 
| +class URLLoaderThrottle; | 
| // Embedder API for participating in renderer logic. | 
| class CONTENT_EXPORT ContentRendererClient { | 
| @@ -223,11 +224,17 @@ class CONTENT_EXPORT ContentRendererClient { | 
| 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 paragrap
 
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. |