Introduce ResourceRequesterInfo to abstract the requester of resource request

content/browser/loader/resource_requester_info.h

+// 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.
+
+#ifndef CONTENT_BROWSER_LOADER_RESOURCE_REQUESTER_INFO_H_
+#define CONTENT_BROWSER_LOADER_RESOURCE_REQUESTER_INFO_H_
+
+#include "base/memory/weak_ptr.h"
+#include "content/common/content_export.h"
+#include "content/public/common/resource_type.h"
+
+namespace net {
+class URLRequestContext;
+}  // namespace net
+
+namespace storage {
+class FileSystemContext;
+}  // namespace storage
+
+namespace content {
+class BrowserMessageFilter;
+class ChromeAppCacheService;
+class ChromeBlobStorageContext;
+class ResourceContext;
+class ResourceMessageFilter;
+class ServiceWorkerContextWrapper;
+
+// This class represents the type of resource request.
+// Currently there are three types:
+// - Resource request from renderer processes.
+// - Resource request for browser side navigation. (PlzNavigate).
+// - Download request of page save request.
+class CONTENT_EXPORT ResourceRequesterInfo {

[kinuko, 2016/11/10 08:17:57]

Currently this one and RMF have somewhat overlapping fields / methods, which
makes the purpose of this class a little unclear.  Could we more clearly split
the role between them?

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

ResourceRequesterInfoForRenderer can get the contexts from RMF.
But ForBrowserSideNavigation and ForDownloadOrPageSave can't.

ResourceRequesterInfoForBrowserSideNavigation has
scoped_refptr<ServiceWorkerContextWrapper> which was acquired from 
ServiceWorkerNavigationHandle on UI thread.

I think to support ServiceWorker for download and SavePage
(https://crbug.com/468227), I have to find a way to pass ServiceWorkerContext to
ResourceRequesterInfoForDownloadOrPageSave.

+ public:
+  // Creates a ResourceRequesterInfo for renderer process initiated request.
+  // |filter| can't be null.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForRenderer(
+      base::WeakPtr<ResourceMessageFilter> filter);
+
+  // Creates a ResourceRequesterInfo for renderer process initiated request
+  // without ResourceMessageFilter for unittests.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForRendererTesting(
+      int child_id);
+
+  // Creates a ResourceRequesterInfo for browser side navigation request which
+  // is initiated by browser process.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForBrowserSideNavigation(
+      scoped_refptr<ServiceWorkerContextWrapper> service_worker_context);
+
+  // Creates a ResourceRequesterInfo for download or page save request.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForDownloadOrPageSave(
+      int child_id);
+
+  virtual ~ResourceRequesterInfo() {}
+
+  virtual std::unique_ptr<ResourceRequesterInfo> clone() const = 0;
+
+  virtual bool IsRenderer() const = 0;
+  virtual bool IsBrowserSideNavigation() const = 0;
+
+  // Returns the renderer process associated with the request. Returns -1 for
+  // browser side navigation requests.
+  virtual int child_id() const = 0;
+
+  // Returns the filter for sending IPC messages to the renderer process. This
+  // may return null if the process eixted. This method is available only for
+  // renderer requests.
+  virtual BrowserMessageFilter* filter() const = 0;
+
+  // Returns the ResourceContext and URLRequestContext associated to the
+  // request. Currently this method is available only for renderer requests.
+  virtual void GetContexts(ResourceType resource_type,
+                           ResourceContext** resource_context,
+                           net::URLRequestContext** request_context) const = 0;
+
+  // Returns the ChromeAppCacheService associated with the request. Currently
+  // this method is available only for renderer requests.
+  virtual ChromeAppCacheService* appcache_service() const = 0;
+
+  // Returns the ChromeBlobStorageContext associated with the request. Currently
+  // this method is available only for renderer requests.
+  virtual ChromeBlobStorageContext* blob_storage_context() const = 0;
+
+  // Returns the FileSystemContext associated with the request. Currently this
+  // method is available only for renderer requests.
+  virtual storage::FileSystemContext* file_system_context() const = 0;
+
+  // Returns the ServiceWorkerContext associated with the request. Currently
+  // this method is available for renderer requests and browser side navigation
+  // requests.
+  virtual ServiceWorkerContextWrapper* service_worker_context() const = 0;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_LOADER_RESOURCE_REQUESTER_INFO_H_