Move the functions which block, resume and cancel requests for a frame route id out of ResourceDisp…

content/public/browser/render_frame_host.h

Index: content/public/browser/render_frame_host.h
diff --git a/content/public/browser/render_frame_host.h b/content/public/browser/render_frame_host.h
index 4bb87a6e1661799620b606dca4faa1db5fc0aba2..c6ffa80c8e6e4a91860292dc387ef9ee3631e186 100644
--- a/content/public/browser/render_frame_host.h
+++ b/content/public/browser/render_frame_host.h
@@ -59,6 +59,17 @@ class CONTENT_EXPORT RenderFrameHost : public IPC::Listener,
   // Returns a RenderFrameHost given its accessibility tree ID.
   static RenderFrameHost* FromAXTreeID(int ax_tree_id);
 
+  // Causes all new requests for the root RenderFrameHost and its children to be
+  // blocked (not being started) until ResumeBlockedRequestsForFrame is called.
+  static void BlockRequestsForFrame(RenderFrameHost* root_frame_host);

[jam, 2017/03/20 21:56:09]

why are these static instead of just methods?

[ananta, 2017/03/20 23:03:14]

Thanks done

+
+  // Resumes any blocked request for the specified root RenderFrameHost and
+  // child frame hosts.
+  static void ResumeBlockedRequestsForFrame(RenderFrameHost* root_frame_host);
+
+  // Cancels any blocked request for the frame and its subframes.
+  static void CancelBlockedRequestsForFrame(RenderFrameHost* root_frame_host);

[jam, 2017/03/20 21:56:09]

this is only called from inside content, so just put it on RenderFrameHostImpl
per https://www.chromium.org/developers/content-module/content-api

[ananta, 2017/03/20 23:03:14]

Done.

+
   ~RenderFrameHost() override {}
 
   // Returns the route id for this frame.