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

Unified Diff: content/browser/loader/resource_message_filter.cc

Issue 2481093003: Introduce ResourceRequesterInfo to abstract the requester of resource request (Closed)
Patch Set: Created 4 years, 1 month 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_message_filter.cc
diff --git a/content/browser/loader/resource_message_filter.cc b/content/browser/loader/resource_message_filter.cc
index 7fd049eb4034192ded7c7a8ff033e9b92617fe9f..55c6777295eaa9eeca4f594b8bcf7e037d10d077 100644
--- a/content/browser/loader/resource_message_filter.cc
+++ b/content/browser/loader/resource_message_filter.cc
@@ -7,6 +7,7 @@
#include "content/browser/appcache/chrome_appcache_service.h"
#include "content/browser/blob_storage/chrome_blob_storage_context.h"
#include "content/browser/loader/resource_dispatcher_host_impl.h"
+#include "content/browser/loader/resource_requester_info.h"
#include "content/browser/loader/url_loader_factory_impl.h"
#include "content/browser/service_worker/service_worker_context_wrapper.h"
#include "content/common/resource_messages.h"
@@ -17,7 +18,6 @@ namespace content {
ResourceMessageFilter::ResourceMessageFilter(
int child_id,
- int process_type,
ChromeAppCacheService* appcache_service,
ChromeBlobStorageContext* blob_storage_context,
storage::FileSystemContext* file_system_context,
@@ -26,7 +26,6 @@ ResourceMessageFilter::ResourceMessageFilter(
: BrowserMessageFilter(ResourceMsgStart),
BrowserAssociatedInterface<mojom::URLLoaderFactory>(this, this),
child_id_(child_id),
- process_type_(process_type),
appcache_service_(appcache_service),
blob_storage_context_(blob_storage_context),
file_system_context_(file_system_context),
@@ -71,17 +70,19 @@ void ResourceMessageFilter::CreateLoaderAndStart(
int32_t request_id,
const ResourceRequest& url_request,
mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info) {
- URLLoaderFactoryImpl::CreateLoaderAndStart(std::move(request), routing_id,
- request_id, url_request,
- std::move(client_ptr_info), this);
+ URLLoaderFactoryImpl::CreateLoaderAndStart(
+ std::move(request), routing_id, request_id, url_request,
+ std::move(client_ptr_info),
+ ResourceRequesterInfo::CreateForRenderer(GetWeakPtr()));
}
void ResourceMessageFilter::SyncLoad(int32_t routing_id,
int32_t request_id,
const ResourceRequest& url_request,
const SyncLoadCallback& callback) {
- URLLoaderFactoryImpl::SyncLoad(routing_id, request_id, url_request, callback,
- this);
+ URLLoaderFactoryImpl::SyncLoad(
+ routing_id, request_id, url_request, callback,
+ ResourceRequesterInfo::CreateForRenderer(GetWeakPtr()));
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698