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

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index bf90b0ae15404e326ea89b1e84085293b60788ed..23b753cdc6d3cf5e49b1eee6292eb893d48295c8 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -71,6 +71,7 @@
 #include "content/common/swapped_out_messages.h"
 #include "content/common/view_messages.h"
 #include "content/common/worker_url_loader_factory_provider.mojom.h"
+#include "content/public/child/url_loader_throttle.h"
 #include "content/public/common/appcache_info.h"
 #include "content/public/common/associated_interface_provider.h"
 #include "content/public/common/bindings_policy.h"
@@ -4365,8 +4366,9 @@ void RenderFrameImpl::WillSendRequest(blink::WebURLRequest& request) {
   }
 
   GURL new_url;
+  std::vector<std::unique_ptr<URLLoaderThrottle>> throttles;
   if (GetContentClient()->renderer()->WillSendRequest(
-          frame_, transition_type, request.Url(), &new_url)) {
+          frame_, transition_type, request.Url(), &new_url, &throttles)) {
     request.SetURL(WebURL(new_url));
   }
 
@@ -4462,6 +4464,7 @@ void RenderFrameImpl::WillSendRequest(blink::WebURLRequest& request) {
   }
 
   extra_data->set_url_loader_factory_override(url_loader_factory_.get());
+  extra_data->set_url_loader_throttles(std::move(throttles));

[kinuko, 2017/05/29 13:36:50]

Can you add a todo comment to note that we need to set up throttles for some
worker cases that don't go through here? (Could have my ldap there)

[yzshen1, 2017/05/31 00:30:06]

Done.

 
   request.SetExtraData(extra_data);