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

Unified Diff: content/browser/service_worker/service_worker_request_handler.h

Issue 2843043002: network service: Create URLLoader for service worker navigation case
Patch Set: fix rebase Created 3 years, 7 months 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/service_worker/service_worker_request_handler.h
diff --git a/content/browser/service_worker/service_worker_request_handler.h b/content/browser/service_worker/service_worker_request_handler.h
index 61dd8b5f17ef99c5f8a574e5d530b04bf09df406..ad1e83688caf05b71e36e5bb1e3cea22a14d218f 100644
--- a/content/browser/service_worker/service_worker_request_handler.h
+++ b/content/browser/service_worker/service_worker_request_handler.h
@@ -15,6 +15,7 @@
#include "content/common/content_export.h"
#include "content/common/service_worker/service_worker_status_code.h"
#include "content/common/service_worker/service_worker_types.h"
+#include "content/common/url_loader_factory.mojom.h"
#include "content/public/common/request_context_frame_type.h"
#include "content/public/common/request_context_type.h"
#include "content/public/common/resource_type.h"
@@ -33,6 +34,7 @@ class BlobStorageContext;
namespace content {
class ResourceContext;
+struct ResourceRequest;
class ResourceRequestBodyImpl;
class ServiceWorkerContextCore;
class ServiceWorkerContextWrapper;
@@ -49,7 +51,7 @@ class CONTENT_EXPORT ServiceWorkerRequestHandler
// Attaches a newly created handler if the given |request| needs to be handled
// by ServiceWorker.
static void InitializeForNavigation(
- net::URLRequest* request,
+ net::URLRequest* url_request,
ServiceWorkerNavigationHandleCore* navigation_handle_core,
storage::BlobStorageContext* blob_storage_context,
bool skip_service_worker,
@@ -60,6 +62,23 @@ class CONTENT_EXPORT ServiceWorkerRequestHandler
scoped_refptr<ResourceRequestBodyImpl> body,
const base::Callback<WebContents*(void)>& web_contents_getter);
+ // PlzNavigate and --enable-network-service
+ // Same as InitializeForNavigation(), but instead of attaching to a request,
+ // returns a URLLoaderFactoryPtr if the request needs to be handled.
+ static mojom::URLLoaderFactoryPtrInfo InitializeForNavigationNetworkService(
+ const ResourceRequest& resource_request,
+ ResourceContext* resource_context,
+ ServiceWorkerNavigationHandleCore* navigation_handle_core,
+ storage::BlobStorageContext* blob_storage_context,
+ bool skip_service_worker,
+ ResourceType resource_type,
+ RequestContextType request_context_type,
+ RequestContextFrameType frame_type,
+ bool is_parent_frame_secure,
+ scoped_refptr<ResourceRequestBodyImpl> body,
+ const base::Callback<WebContents*(void)>& web_contents_getter,
+ NetworkFallbackCallback network_fallback_callback);
+
// Attaches a newly created handler if the given |request| needs to
// be handled by ServiceWorker.
// TODO(kinuko): While utilizing UserData to attach data to URLRequest
@@ -109,6 +128,13 @@ class CONTENT_EXPORT ServiceWorkerRequestHandler
net::NetworkDelegate* network_delegate,
ResourceContext* context) = 0;
+ virtual mojom::URLLoaderFactoryPtrInfo MaybeGetURLLoaderFactory(
+ const ResourceRequest& resource_request,
+ ResourceContext* resource_context,
+ std::unique_ptr<ServiceWorkerRequestHandler> request_handler) {
+ return mojom::URLLoaderFactoryPtrInfo();
+ }
+
// Methods to support cross site navigations.
void PrepareForCrossSiteTransfer(int old_process_id);
void CompleteCrossSiteTransfer(int new_process_id,
@@ -133,6 +159,20 @@ class CONTENT_EXPORT ServiceWorkerRequestHandler
ResourceType resource_type_;
private:
+ static std::unique_ptr<ServiceWorkerRequestHandler>
+ InitializeForNavigationInternal(
+ const GURL& url,
+ ServiceWorkerNavigationHandleCore* navigation_handle_core,
+ storage::BlobStorageContext* blob_storage_context,
+ bool skip_service_worker,
+ ResourceType resource_type,
+ RequestContextType request_context_type,
+ RequestContextFrameType frame_type,
+ bool is_parent_frame_secure,
+ scoped_refptr<ResourceRequestBodyImpl> body,
+ const base::Callback<WebContents*(void)>& web_contents_getter,
+ NetworkFallbackCallback network_fallback_callback);
+
std::unique_ptr<ServiceWorkerProviderHost> host_for_cross_site_transfer_;
int old_process_id_;
int old_provider_id_;

Powered by Google App Engine
This is Rietveld 408576698