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 <memory>
+
+#include "base/callback.h"
+#include "base/memory/ref_counted.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 or page save request.
+class CONTENT_EXPORT ResourceRequesterInfo {
+ public:
+  typedef base::Callback<void(ResourceType resource_type,
+                              ResourceContext**,
+                              net::URLRequestContext**)>
+      GetContextsCallback;
+
+  // Creates a ResourceRequesterInfo for renderer process initiated request.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForRenderer(
+      int child_id,
+      ChromeAppCacheService* appcache_service,
+      ChromeBlobStorageContext* blob_storage_context,
+      storage::FileSystemContext* file_system_context,
+      ServiceWorkerContextWrapper* service_worker_context,
+      const GetContextsCallback& get_contexts_callback);
+
+  // Creates a ResourceRequesterInfo for a renderer process initiated request
+  // for testing.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForRendererTesting(
+      int child_id);
+
+  // Creates a ResourceRequesterInfo for a browser side navigation request which
+  // is initiated by the browser process.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForBrowserSideNavigation(
+      scoped_refptr<ServiceWorkerContextWrapper> service_worker_context);
+
+  // Creates a ResourceRequesterInfo for a download or page save request.
+  static std::unique_ptr<ResourceRequesterInfo> CreateForDownloadOrPageSave(
+      int child_id);
+
+  ~ResourceRequesterInfo();
+
+  std::unique_ptr<ResourceRequesterInfo> clone() const;
+
+  bool IsRenderer() const { return type_ == RequesterType::RENDERER; }
+  bool IsBrowserSideNavigation() const {
+    return type_ == RequesterType::BROWSER_SIDE_NAVIGATION;
+  }
+
+  // Returns the renderer process ID associated with the request. Returns -1 for
+  // browser side navigation requests. Even if the ResourceMessageFilter has
+  // been destroyed, this method of renderer type request info returns the
+  // valid process ID which was assigned to the renderer process of the filter.
+  int child_id() const { return child_id_; }
+
+  // Sets the ResourceMessageFilter of the renderer type requester info.
+  void set_filter(base::WeakPtr<ResourceMessageFilter> filter);
+
+  // 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

[mmenke, 2016/11/17 16:27:20]

exited

[horo, 2016/11/17 17:50:27]

Done.

+  // renderer requests.
+  ResourceMessageFilter* filter() const { return filter_.get(); }
+
+  // Returns the ResourceContext and URLRequestContext associated to the
+  // request. Currently this method is available only for renderer requests.
+  void GetContexts(ResourceType resource_type,
+                   ResourceContext** resource_context,
+                   net::URLRequestContext** request_context) const;
+
+  // Returns the ChromeAppCacheService associated with the request. Currently
+  // this method is available only for renderer requests.
+  ChromeAppCacheService* appcache_service() const {
+    return appcache_service_.get();
+  }
+
+  // Returns the ChromeBlobStorageContext associated with the request. Currently
+  // this method is available only for renderer requests.
+  ChromeBlobStorageContext* blob_storage_context() const {
+    return blob_storage_context_.get();
+  }
+
+  // Returns the FileSystemContext associated with the request. Currently this
+  // method is available only for renderer requests.
+  storage::FileSystemContext* file_system_context() const {
+    return file_system_context_.get();
+  }
+
+  // Returns the ServiceWorkerContext associated with the request. Currently
+  // this method is available for renderer requests and browser side navigation
+  // requests.
+  ServiceWorkerContextWrapper* service_worker_context() const {
+    return service_worker_context_.get();
+  }
+
+ private:
+  enum class RequesterType {
+    RENDERER,
+    BROWSER_SIDE_NAVIGATION,
+    DOWNLOAD_OR_PAGE_SAVE
+  };
+
+  ResourceRequesterInfo(RequesterType type,
+                        int child_id,
+                        ChromeAppCacheService* appcache_service,
+                        ChromeBlobStorageContext* blob_storage_context,
+                        storage::FileSystemContext* file_system_context,
+                        ServiceWorkerContextWrapper* service_worker_context,
+                        const GetContextsCallback& get_contexts_callback);
+
+  const RequesterType type_;
+  // The filter might be deleted if the process exited.
+  base::WeakPtr<ResourceMessageFilter> filter_;
+  const int child_id_;
+  scoped_refptr<ChromeAppCacheService> appcache_service_;
+  scoped_refptr<ChromeBlobStorageContext> blob_storage_context_;
+  scoped_refptr<storage::FileSystemContext> file_system_context_;
+  scoped_refptr<ServiceWorkerContextWrapper> service_worker_context_;
+  GetContextsCallback get_contexts_callback_;

[mmenke, 2016/11/17 16:27:20]

I think all of these be const, except filter?  (That is, const scoped_refptr,
not scoped_refptr<const...>)

[horo, 2016/11/17 17:50:27]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(ResourceRequesterInfo);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_LOADER_RESOURCE_REQUESTER_INFO_H_