Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(511)

Unified Diff: content/browser/loader/resource_dispatcher_host_impl.h

Issue 1542743002: [RDHI] Refactored blocked_loaders_map_ to key by render frame route id (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: just comments / minor restructuring (trybots previous) Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/loader/resource_dispatcher_host_impl.h
diff --git a/content/browser/loader/resource_dispatcher_host_impl.h b/content/browser/loader/resource_dispatcher_host_impl.h
index cb2e89ae618ac86cdbc17b5936113f61297059ed..3d17d4f80b507414b64df52394c2a8c7d1e1a09c 100644
--- a/content/browser/loader/resource_dispatcher_host_impl.h
+++ b/content/browser/loader/resource_dispatcher_host_impl.h
@@ -36,6 +36,7 @@
#include "content/public/browser/download_url_parameters.h"
#include "content/public/browser/global_request_id.h"
#include "content/public/browser/notification_types.h"
+#include "content/public/browser/render_view_host.h"
#include "content/public/browser/resource_dispatcher_host.h"
#include "content/public/common/resource_type.h"
#include "ipc/ipc_message.h"
@@ -61,6 +62,7 @@ class ShareableFileReference;
namespace content {
class AppCacheService;
class AsyncRevalidationManager;
+class FrameTree;
class NavigationURLLoaderImplCore;
class ResourceContext;
class ResourceDispatcherHostDelegate;
@@ -103,8 +105,8 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
uint32 download_id,
const DownloadStartedCallback& started_callback) override;
void ClearLoginDelegateForRequest(net::URLRequest* request) override;
- void BlockRequestsForRoute(int child_id, int route_id) override;
- void ResumeBlockedRequestsForRoute(int child_id, int route_id) override;
+ void BlockRequestsForFrame(int child_id, int route_id) override;
+ void ResumeBlockedRequestsForFrame(int child_id, int route_id) override;
// Puts the resource dispatcher host in an inactive state (unable to begin
// new requests). Cancels all pending requests.
@@ -174,6 +176,8 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
return save_file_manager_.get();
}
+ void OnRenderFrameHostDeleted(int child_id, int route_id);
+
// Called when a RenderViewHost is created.
void OnRenderViewHostCreated(int child_id,
int route_id,
@@ -210,7 +214,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
void RemovePendingRequest(int child_id, int request_id);
// Cancels any blocked request for the specified route id.
- void CancelBlockedRequestsForRoute(int child_id, int route_id);
+ void CancelBlockedRequestsForFrame(int child_id, int route_id);
// Maintains a collection of temp files created in support of
// the download_to_file capability. Used to grant access to the
@@ -377,7 +381,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
// Force cancels any pending requests for the given route id. This method
// acts like CancelRequestsForProcess when route_id is -1.
- void CancelRequestsForRoute(int child_id, int route_id);
+ void CancelRequestsForFrame(int child_id, int route_id);
// The list of all requests that we have pending. This list is not really
// optimized, and assumes that we have relatively few requests pending at once
@@ -422,7 +426,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
void UpdateLoadInfo();
// Resumes or cancels (if |cancel_requests| is true) any blocked requests.
- void ProcessBlockedRequestsForRoute(int child_id,
+ void ProcessBlockedRequestsForFrame(int child_id,
int route_id,
bool cancel_requests);
@@ -549,7 +553,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
bool is_shutdown_;
typedef std::vector<linked_ptr<ResourceLoader> > BlockedLoadersList;
- typedef std::map<GlobalRoutingID, BlockedLoadersList*> BlockedLoadersMap;
+ typedef std::map<GlobalFrameRoutingID, BlockedLoadersList*> BlockedLoadersMap;
BlockedLoadersMap blocked_loaders_map_;
// Maps the child_ids to the approximate number of bytes

Powered by Google App Engine
This is Rietveld 408576698