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

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

Issue 2481093003: Introduce ResourceRequesterInfo to abstract the requester of resource request (Closed)
Patch Set: fix unittests 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/url_loader_factory_impl.h
diff --git a/content/browser/loader/url_loader_factory_impl.h b/content/browser/loader/url_loader_factory_impl.h
index e989c3d872b7cb1d63d84ce875eb72dfb61212b3..8027f2065d7b5b7ec9c41933c12b864853b761f4 100644
--- a/content/browser/loader/url_loader_factory_impl.h
+++ b/content/browser/loader/url_loader_factory_impl.h
@@ -12,7 +12,7 @@
namespace content {
-class ResourceMessageFilter;
+class ResourceRequesterInfo;
// This class is an implementation of mojom::URLLoaderFactory that creates
// a mojom::URLLoader.
@@ -32,29 +32,29 @@ class URLLoaderFactoryImpl final : public mojom::URLLoaderFactory {
const SyncLoadCallback& callback) override;
static void CreateLoaderAndStart(
+ const ResourceRequesterInfo* requester_info,
mojom::URLLoaderAssociatedRequest request,
int32_t routing_id,
int32_t request_id,
const ResourceRequest& url_request,
- mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info,
- ResourceMessageFilter* filter);
- static void SyncLoad(int32_t routing_id,
+ mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info);
+ static void SyncLoad(const ResourceRequesterInfo* requester_info,
+ int32_t routing_id,
int32_t request_id,
const ResourceRequest& request,
- const SyncLoadCallback& callback,
- ResourceMessageFilter* filter);
+ const SyncLoadCallback& callback);
// Creates a URLLoaderFactoryImpl instance. The instance is held by the
// StrongBinding in it, so this function doesn't return the instance.
CONTENT_EXPORT static void Create(
- scoped_refptr<ResourceMessageFilter> resource_message_filter,
+ std::unique_ptr<ResourceRequesterInfo> requester_info,
mojom::URLLoaderFactoryRequest request);
private:
explicit URLLoaderFactoryImpl(
- scoped_refptr<ResourceMessageFilter> resource_message_filter);
+ std::unique_ptr<ResourceRequesterInfo> requester_info);
- scoped_refptr<ResourceMessageFilter> resource_message_filter_;
+ std::unique_ptr<ResourceRequesterInfo> requester_info_;
DISALLOW_COPY_AND_ASSIGN(URLLoaderFactoryImpl);
};

Powered by Google App Engine
This is Rietveld 408576698