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

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

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/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..6ab07d5765af9d72946c04208afea24f1f19e832 100644
--- a/content/browser/loader/url_loader_factory_impl.h
+++ b/content/browser/loader/url_loader_factory_impl.h
@@ -13,6 +13,7 @@
namespace content {
class ResourceMessageFilter;
yhirano 2016/11/10 06:42:29 Not needed.
horo 2016/11/10 14:46:33 Done.
+class ResourceRequesterInfo;
// This class is an implementation of mojom::URLLoaderFactory that creates
// a mojom::URLLoader.
@@ -37,24 +38,24 @@ class URLLoaderFactoryImpl final : public mojom::URLLoaderFactory {
int32_t request_id,
const ResourceRequest& url_request,
mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info,
- ResourceMessageFilter* filter);
+ std::unique_ptr<ResourceRequesterInfo> requester_info);
static void SyncLoad(int32_t routing_id,
int32_t request_id,
const ResourceRequest& request,
const SyncLoadCallback& callback,
- ResourceMessageFilter* filter);
+ std::unique_ptr<ResourceRequesterInfo> requester_info);
// 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