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

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

Issue 2701753003: [WIP] off-main-thread loading
Patch Set: small fix 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 | « chrome/browser/flag_descriptions.cc ('k') | 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 b9f0d3a60af187b13faaef4751778f0f7eba4cf5..f58ca6248c279511dcf548941aa8a1a7bc19eb70 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -85,6 +85,7 @@
#include "content/browser/indexed_db/indexed_db_context_impl.h"
#include "content/browser/indexed_db/indexed_db_dispatcher_host.h"
#include "content/browser/loader/resource_message_filter.h"
+#include "content/browser/loader/resource_requester_info.h"
#include "content/browser/loader/resource_scheduler_filter.h"
#include "content/browser/loader/url_loader_factory_impl.h"
#include "content/browser/media/capture/audio_mirroring_manager.h"
@@ -142,6 +143,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_fetch_context_factory.mojom.h"
#include "content/public/browser/browser_context.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/content_browser_client.h"
@@ -473,6 +475,50 @@ void ForwardShapeDetectionRequest(R request) {
std::move(request));
}
+class WorkerFetchContextFactoryImpl : public mojom::WorkerFetchContextFactory {
+ public:
+ static void Create(
+ int render_process_id,
+ ResourceMessageFilter* resource_message_filter,
+ scoped_refptr<ServiceWorkerContextWrapper> service_worker_context,
+ mojom::WorkerFetchContextFactoryRequest request) {
+ DCHECK(base::FeatureList::IsEnabled(features::kOffMainThreadFetch));
+ mojo::MakeStrongBinding(
+ base::MakeUnique<WorkerFetchContextFactoryImpl>(
+ render_process_id, resource_message_filter->GetWeakPtr(),
+ std::move(service_worker_context)),
+ std::move(request));
+ }
+ WorkerFetchContextFactoryImpl(
+ 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)) {}
+ ~WorkerFetchContextFactoryImpl() override {}
+
+ void CreateWorkerFetchContext(
+ mojom::URLLoaderFactoryAssociatedRequest loader_request,
+ mojom::ServiceWorkerClientAssociatedPtrInfo 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 +1340,16 @@ 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(&WorkerFetchContextFactoryImpl::Create, GetID(),
+ base::Unretained(resource_message_filter_.get()),
+ 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 | « chrome/browser/flag_descriptions.cc ('k') | content/browser/service_worker/service_worker_context_core.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698