Introduce ResourceRequesterInfo to abstract the requester of resource request

content/browser/loader/url_loader_factory_impl.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/loader/url_loader_factory_impl.h"
 
 #include "content/browser/loader/resource_dispatcher_host_impl.h"
 #include "content/browser/loader/resource_message_filter.h"

[yhirano, 2016/11/10 06:42:29]

Not needed.

[horo, 2016/11/10 14:46:33]

Done.

+#include "content/browser/loader/resource_requester_info.h"
 #include "content/common/resource_request.h"
 #include "content/common/url_loader.mojom.h"
 #include "content/public/browser/browser_thread.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace content {
 
 namespace {
 
 void DispatchSyncLoadResult(
     const URLLoaderFactoryImpl::SyncLoadCallback& callback,
     const SyncLoadResult* result) {
   // |result| can be null when a loading task is aborted unexpectedly. Reply
   // with a failure result on that case.
   // TODO(tzik): Test null-result case.
   if (!result) {
     SyncLoadResult failure;
     failure.error_code = net::ERR_FAILED;
     callback.Run(failure);
     return;
   }
 
   callback.Run(*result);
 }
 
 } // namespace
 
 URLLoaderFactoryImpl::URLLoaderFactoryImpl(
-    scoped_refptr<ResourceMessageFilter> resource_message_filter)
-    : resource_message_filter_(std::move(resource_message_filter)) {
-  DCHECK(resource_message_filter_);
+    std::unique_ptr<ResourceRequesterInfo> requester_info)
+    : requester_info_(std::move(requester_info)) {
+  DCHECK(requester_info_->IsRenderer());
+  DCHECK(requester_info_->filter());
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 }
 
 URLLoaderFactoryImpl::~URLLoaderFactoryImpl() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 }
 
 void URLLoaderFactoryImpl::CreateLoaderAndStart(
     mojom::URLLoaderAssociatedRequest request,
     int32_t routing_id,
     int32_t request_id,
     const ResourceRequest& url_request,
     mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info) {
   CreateLoaderAndStart(std::move(request), routing_id, request_id, url_request,
-                       std::move(client_ptr_info),
-                       resource_message_filter_.get());
+                       std::move(client_ptr_info), requester_info_->clone());
 }
 
 void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id,
                                     int32_t request_id,
                                     const ResourceRequest& url_request,
                                     const SyncLoadCallback& callback) {
   SyncLoad(routing_id, request_id, url_request, callback,
-           resource_message_filter_.get());
+           requester_info_->clone());
 }
 
 // static
 void URLLoaderFactoryImpl::CreateLoaderAndStart(
     mojom::URLLoaderAssociatedRequest request,
     int32_t routing_id,
     int32_t request_id,
     const ResourceRequest& url_request,
     mojom::URLLoaderClientAssociatedPtrInfo client_ptr_info,
-    ResourceMessageFilter* filter) {
+    std::unique_ptr<ResourceRequesterInfo> requester_info) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   mojom::URLLoaderClientAssociatedPtr client;
   client.Bind(std::move(client_ptr_info));
 
   ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get();
   rdh->OnRequestResourceWithMojo(routing_id, request_id, url_request,
-                                 std::move(request), std::move(client), filter);
+                                 std::move(request), std::move(client),
+                                 std::move(requester_info));
 }
 
 // static
-void URLLoaderFactoryImpl::SyncLoad(int32_t routing_id,
-                                    int32_t request_id,
-                                    const ResourceRequest& url_request,
-                                    const SyncLoadCallback& callback,
-                                    ResourceMessageFilter* filter) {
+void URLLoaderFactoryImpl::SyncLoad(
+    int32_t routing_id,
+    int32_t request_id,
+    const ResourceRequest& url_request,
+    const SyncLoadCallback& callback,
+    std::unique_ptr<ResourceRequesterInfo> requester_info) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   ResourceDispatcherHostImpl* rdh = ResourceDispatcherHostImpl::Get();
-  rdh->OnSyncLoadWithMojo(routing_id, request_id, url_request, filter,
+  rdh->OnSyncLoadWithMojo(routing_id, request_id, url_request,
+                          std::move(requester_info),
                           base::Bind(&DispatchSyncLoadResult, callback));
 }
 
 void URLLoaderFactoryImpl::Create(
-    scoped_refptr<ResourceMessageFilter> filter,
+    std::unique_ptr<ResourceRequesterInfo> requester_info,
     mojo::InterfaceRequest<mojom::URLLoaderFactory> request) {
   mojo::MakeStrongBinding(
-      base::WrapUnique(new URLLoaderFactoryImpl(std::move(filter))),
+      base::WrapUnique(new URLLoaderFactoryImpl(std::move(requester_info))),
       std::move(request));
 }
 
 }  // namespace content