Chromium Code Reviews| 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 d9d08837604403d2782db63eee0e53d20a7a2c5d..3fb185fad4391f78bae668c7f523d1862b9eaa57 100644 |
| --- a/content/browser/renderer_host/render_process_host_impl.cc |
| +++ b/content/browser/renderer_host/render_process_host_impl.cc |
| @@ -1156,7 +1156,9 @@ void RenderProcessHostImpl::CreateMessageFilters() { |
| scoped_refptr<ServiceWorkerDispatcherHost> service_worker_filter = |
| new ServiceWorkerDispatcherHost( |
| - GetID(), message_port_message_filter_.get(), resource_context); |
| + GetID(), message_port_message_filter_.get(), resource_context, |
| + base::Bind(&RenderProcessHostImpl::CreateURLLoaderFactory, |
| + weak_factory_.GetWeakPtr())); |
| service_worker_filter->Init( |
| storage_partition_impl_->GetServiceWorkerContext()); |
| AddFilter(service_worker_filter.get()); |
| @@ -3008,4 +3010,15 @@ void RenderProcessHostImpl::OnMojoError(int render_process_id, |
| bad_message::RPH_MOJO_PROCESS_ERROR); |
| } |
| +// static |
| +bool RenderProcessHostImpl::CreateURLLoaderFactory( |
| + base::WeakPtr<RenderProcessHostImpl> weak_host, |
| + mojo::InterfaceRequest<mojom::URLLoaderFactory> request) { |
|
sadrul
2016/10/19 14:42:43
This doesn't need to be a static, right?
horo
2016/10/19 14:48:41
I want to return false if RenderProcessHostImpl wa
sadrul
2016/10/19 15:13:31
I think the return type can be void, and you can c
horo
2016/10/19 16:08:02
Ah, sounds good.
Done.
|
| + if (!weak_host) |
| + return false; |
| + URLLoaderFactoryImpl::Create(weak_host->resource_message_filter_, |
| + std::move(request)); |
| + return true; |
| +} |
| + |
| } // namespace content |