Factor DocumentSubresourceFilter out of SubresourceFilterAgent.

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index ec4c0d63c5aebcf8c33f79a300ff42577c472938..6b808741731215381389a0bf31629d8e69e7061c 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -512,8 +512,11 @@ void ChromeContentRendererClient::RenderFrameCreated(
   new AutofillAgent(render_frame, password_autofill_agent,
                     password_generation_agent);
 
-  new subresource_filter::SubresourceFilterAgent(
-      render_frame, subresource_filter_ruleset_dealer_.get());
+  // There is no render thread, thus no RulesetDealer in ChromeRenderViewTests.
+  if (subresource_filter_ruleset_dealer_) {
+    new subresource_filter::SubresourceFilterAgent(

[pkalinnikov, 2016/07/19 11:57:32]

How does this object not leak?

[engedy, 2016/07/19 12:57:28]

As a RenderFrameObserver it is owned by the |render_frame|, which takes care of
destruction.

[pkalinnikov, 2016/07/19 13:45:18]

Acknowledged.

+        render_frame, subresource_filter_ruleset_dealer_.get());
+  }
 }
 
 void ChromeContentRendererClient::RenderViewCreated(