[RDHI] Refactored blocked_loaders_map_ to key by render frame route id

content/public/browser/resource_dispatcher_host.h

Index: content/public/browser/resource_dispatcher_host.h
diff --git a/content/public/browser/resource_dispatcher_host.h b/content/public/browser/resource_dispatcher_host.h
index d8b2495f22c91cad91df9daf3784fa42c598cd59..686a7620b17c726c3194d497dfefa322854a81f9 100644
--- a/content/public/browser/resource_dispatcher_host.h
+++ b/content/public/browser/resource_dispatcher_host.h
@@ -22,6 +22,7 @@ class ResourceContext;
 class ResourceDispatcherHostDelegate;
 struct DownloadSaveInfo;
 struct Referrer;
+class RenderFrameHost;
 
 class CONTENT_EXPORT ResourceDispatcherHost {
  public:
@@ -31,6 +32,14 @@ class CONTENT_EXPORT ResourceDispatcherHost {
   // Returns the singleton instance of the ResourceDispatcherHost.
   static ResourceDispatcherHost* Get();
 
+  // Causes all new requests for the route identified by the RenderFrameHost to

[nasko, 2016/01/16 00:15:58]

Does the "for the route identified by" bring any new meaning to this comment?
Callers care that the RFH is blocked from making requests and they are resumed
later.

[Charlie Harrison, 2016/01/21 18:52:55]

Simplified the comment.

+  // be blocked (not being started) until ResumeBlockedRequestsForRoute is
+  // called.
+  static void BlockRequestsForFrame(RenderFrameHost* frame_host);
+
+  // Resumes any blocked request for the specified RenderFrameHost.
+  static void ResumeBlockedRequestsForFrame(RenderFrameHost* frame_host);
+
   // This does not take ownership of the delegate. It is expected that the
   // delegate have a longer lifetime than the ResourceDispatcherHost.
   virtual void SetDelegate(ResourceDispatcherHostDelegate* delegate) = 0;
@@ -66,14 +75,6 @@ class CONTENT_EXPORT ResourceDispatcherHost {
   // Clears the ResourceDispatcherHostLoginDelegate associated with the request.
   virtual void ClearLoginDelegateForRequest(net::URLRequest* request) = 0;
 
-  // Causes all new requests for the route identified by |child_id| and
-  // |route_id| to be blocked (not being started) until
-  // ResumeBlockedRequestsForRoute is called.
-  virtual void BlockRequestsForRoute(int child_id, int route_id) = 0;
-
-  // Resumes any blocked request for the specified route id.
-  virtual void ResumeBlockedRequestsForRoute(int child_id, int route_id) = 0;
-
  protected:
   virtual ~ResourceDispatcherHost() {}
 };