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

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

Issue 2715423003: [Mojo-Loading] Use independent URLLoaderClient (Closed)
Patch Set: fix Created 3 years, 10 months 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 4d9bd1276363857b107087b7ea8559c471ab0f29..9ee17d52a0e434735047c6afa84798635a2a8c8e 100644
--- a/content/browser/loader/resource_dispatcher_host_impl.h
+++ b/content/browser/loader/resource_dispatcher_host_impl.h
@@ -298,13 +298,12 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
void OnRenderFrameDeleted(const GlobalFrameRoutingId& global_routing_id);
// Called when loading a request with mojo.
- void OnRequestResourceWithMojo(
- ResourceRequesterInfo* requester_info,
- int routing_id,
- int request_id,
- const ResourceRequest& request,
- mojom::URLLoaderAssociatedRequest mojo_request,
- mojom::URLLoaderClientAssociatedPtr url_loader_client);
+ void OnRequestResourceWithMojo(ResourceRequesterInfo* requester_info,
+ int routing_id,
+ int request_id,
+ const ResourceRequest& request,
+ mojom::URLLoaderAssociatedRequest mojo_request,
+ mojom::URLLoaderClientPtr url_loader_client);
void OnSyncLoadWithMojo(ResourceRequesterInfo* requester_info,
int routing_id,
@@ -535,13 +534,12 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
int request_id,
const ResourceRequest& request_data);
- void OnRequestResourceInternal(
- ResourceRequesterInfo* requester_info,
- int routing_id,
- int request_id,
- const ResourceRequest& request_data,
- mojom::URLLoaderAssociatedRequest mojo_request,
- mojom::URLLoaderClientAssociatedPtr url_loader_client);
+ void OnRequestResourceInternal(ResourceRequesterInfo* requester_info,
+ int routing_id,
+ int request_id,
+ const ResourceRequest& request_data,
+ mojom::URLLoaderAssociatedRequest mojo_request,
+ mojom::URLLoaderClientPtr url_loader_client);
void OnSyncLoad(ResourceRequesterInfo* requester_info,
int request_id,
@@ -552,14 +550,13 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
// Update the ResourceRequestInfo and internal maps when a request is
// transferred from one process to another.
- void UpdateRequestForTransfer(
- ResourceRequesterInfo* requester_info,
- int route_id,
- int request_id,
- const ResourceRequest& request_data,
- LoaderMap::iterator iter,
- mojom::URLLoaderAssociatedRequest mojo_request,
- mojom::URLLoaderClientAssociatedPtr url_loader_client);
+ void UpdateRequestForTransfer(ResourceRequesterInfo* requester_info,
+ int route_id,
+ int request_id,
+ const ResourceRequest& request_data,
+ LoaderMap::iterator iter,
+ mojom::URLLoaderAssociatedRequest mojo_request,
+ mojom::URLLoaderClientPtr url_loader_client);
// If |request_data| is for a request being transferred from another process,
// then CompleteTransfer method can be used to complete the transfer.
@@ -568,7 +565,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
const ResourceRequest& request_data,
int route_id,
mojom::URLLoaderAssociatedRequest mojo_request,
- mojom::URLLoaderClientAssociatedPtr url_loader_client);
+ mojom::URLLoaderClientPtr url_loader_client);
void BeginRequest(
ResourceRequesterInfo* requester_info,
@@ -577,7 +574,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
const SyncLoadResultCallback& sync_result_handler, // only valid for sync
int route_id,
mojom::URLLoaderAssociatedRequest mojo_request,
- mojom::URLLoaderClientAssociatedPtr url_loader_client);
+ mojom::URLLoaderClientPtr url_loader_client);
// There are requests which need decisions to be made like the following:
// Whether the presence of certain HTTP headers like the Origin header are
@@ -596,7 +593,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
int route_id,
const net::HttpRequestHeaders& headers,
mojom::URLLoaderAssociatedRequest mojo_request,
- mojom::URLLoaderClientAssociatedPtr url_loader_client,
+ mojom::URLLoaderClientPtr url_loader_client,
bool continue_request,
int error_code);
@@ -611,7 +608,7 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
int child_id,
ResourceContext* resource_context,
mojom::URLLoaderAssociatedRequest mojo_request,
- mojom::URLLoaderClientAssociatedPtr url_loader_client);
+ mojom::URLLoaderClientPtr url_loader_client);
// Wraps |handler| in the standard resource handlers for normal resource
// loading and navigation requests. This adds MimeTypeResourceHandler and

Powered by Google App Engine
This is Rietveld 408576698