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

Unified Diff: content/browser/renderer_host/render_process_host_impl.cc

Issue 2804843005: Implement the infrastructure of creating WorkerFetchContext in worker global scope. (Closed)
Patch Set: s/WebScheduler.h/web_scheduler.h/ Created 3 years, 8 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
« no previous file with comments | « no previous file | content/browser/service_worker/service_worker_context_core.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index ed09e523ed9f716db9d43a9582ce8add78ac52bb..d9686c7626fa07151f6d1486a8513ee52be6f3b4 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -141,6 +141,7 @@
#include "content/common/service_manager/service_manager_connection_impl.h"
#include "content/common/site_isolation_policy.h"
#include "content/common/view_messages.h"
+#include "content/common/worker_url_loader_factory_provider.mojom.h"
#include "content/public/browser/browser_context.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/content_browser_client.h"
@@ -178,6 +179,7 @@
#include "media/media_features.h"
#include "mojo/edk/embedder/embedder.h"
#include "mojo/public/cpp/bindings/associated_interface_ptr.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
#include "net/url_request/url_request_context_getter.h"
#include "ppapi/features/features.h"
#include "services/resource_coordinator/memory/coordinator/coordinator_impl.h"
@@ -471,6 +473,52 @@ void ForwardShapeDetectionRequest(R request) {
std::move(request));
}
+class WorkerURLLoaderFactoryProviderImpl
+ : public mojom::WorkerURLLoaderFactoryProvider {
+ public:
+ static void Create(
+ int render_process_id,
+ scoped_refptr<ResourceMessageFilter> resource_message_filter,
+ scoped_refptr<ServiceWorkerContextWrapper> service_worker_context,
+ mojom::WorkerURLLoaderFactoryProviderRequest request) {
+ DCHECK(base::FeatureList::IsEnabled(features::kOffMainThreadFetch));
+ mojo::MakeStrongBinding(
+ base::MakeUnique<WorkerURLLoaderFactoryProviderImpl>(
+ render_process_id, resource_message_filter->GetWeakPtr(),
+ std::move(service_worker_context)),
+ std::move(request));
+ }
+ WorkerURLLoaderFactoryProviderImpl(
+ int render_process_id,
+ base::WeakPtr<ResourceMessageFilter> resource_message_filter,
+ scoped_refptr<ServiceWorkerContextWrapper> service_worker_context)
+ : render_process_id_(render_process_id),
+ url_loader_factory_binding_(resource_message_filter.get()),
+ service_worker_context_(std::move(service_worker_context)) {}
+ ~WorkerURLLoaderFactoryProviderImpl() override {}
+
+ void GetURLLoaderFactoryAndRegisterClient(
+ mojom::URLLoaderFactoryAssociatedRequest loader_request,
+ mojom::ServiceWorkerWorkerClientAssociatedPtrInfo client_ptr_info,
+ int service_worker_provider_id) override {
+ url_loader_factory_binding_.Bind(std::move(loader_request));
+ service_worker_context_->BindWorkerFetchContext(render_process_id_,
+ service_worker_provider_id,
+ std::move(client_ptr_info));
+ }
+
+ void GetURLLoaderFactory(
+ mojom::URLLoaderFactoryAssociatedRequest loader_request) override {
+ url_loader_factory_binding_.Bind(std::move(loader_request));
+ }
+
+ private:
+ const int render_process_id_;
+ mojo::AssociatedBinding<mojom::URLLoaderFactory> url_loader_factory_binding_;
+
+ scoped_refptr<ServiceWorkerContextWrapper> service_worker_context_;
+};
+
} // namespace
RendererMainThreadFactoryFunction g_renderer_main_thread_factory = NULL;
@@ -1294,6 +1342,15 @@ void RenderProcessHostImpl::RegisterMojoInterfaces() {
base::Bind(&VideoCaptureHost::Create,
BrowserMainLoop::GetInstance()->media_stream_manager()));
+ if (base::FeatureList::IsEnabled(features::kOffMainThreadFetch)) {
+ scoped_refptr<ServiceWorkerContextWrapper> service_worker_context(
+ static_cast<ServiceWorkerContextWrapper*>(
+ storage_partition_impl_->GetServiceWorkerContext()));
+ registry->AddInterface(
+ base::Bind(&WorkerURLLoaderFactoryProviderImpl::Create, GetID(),
+ resource_message_filter_, service_worker_context));
+ }
+
// This is to support usage of WebSockets in cases in which there is no
// associated RenderFrame (e.g., Shared Workers).
AddUIThreadInterface(
« no previous file with comments | « no previous file | content/browser/service_worker/service_worker_context_core.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698