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

Side by Side 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: incorporated nhiroki and kinuko's comment 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 unified diff | Download patch
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Represents the browser side of the browser <--> renderer communication 5 // Represents the browser side of the browser <--> renderer communication
6 // channel. There will be one RenderProcessHost per renderer process. 6 // channel. There will be one RenderProcessHost per renderer process.
7 7
8 #include "content/browser/renderer_host/render_process_host_impl.h" 8 #include "content/browser/renderer_host/render_process_host_impl.h"
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 #include "content/common/child_process_messages.h" 134 #include "content/common/child_process_messages.h"
135 #include "content/common/content_switches_internal.h" 135 #include "content/common/content_switches_internal.h"
136 #include "content/common/frame_messages.h" 136 #include "content/common/frame_messages.h"
137 #include "content/common/in_process_child_thread_params.h" 137 #include "content/common/in_process_child_thread_params.h"
138 #include "content/common/render_process_messages.h" 138 #include "content/common/render_process_messages.h"
139 #include "content/common/resource_messages.h" 139 #include "content/common/resource_messages.h"
140 #include "content/common/service_manager/child_connection.h" 140 #include "content/common/service_manager/child_connection.h"
141 #include "content/common/service_manager/service_manager_connection_impl.h" 141 #include "content/common/service_manager/service_manager_connection_impl.h"
142 #include "content/common/site_isolation_policy.h" 142 #include "content/common/site_isolation_policy.h"
143 #include "content/common/view_messages.h" 143 #include "content/common/view_messages.h"
144 #include "content/common/worker_fetch_context_factory.mojom.h"
144 #include "content/public/browser/browser_context.h" 145 #include "content/public/browser/browser_context.h"
145 #include "content/public/browser/browser_thread.h" 146 #include "content/public/browser/browser_thread.h"
146 #include "content/public/browser/content_browser_client.h" 147 #include "content/public/browser/content_browser_client.h"
147 #include "content/public/browser/notification_service.h" 148 #include "content/public/browser/notification_service.h"
148 #include "content/public/browser/notification_types.h" 149 #include "content/public/browser/notification_types.h"
149 #include "content/public/browser/render_process_host_factory.h" 150 #include "content/public/browser/render_process_host_factory.h"
150 #include "content/public/browser/render_process_host_observer.h" 151 #include "content/public/browser/render_process_host_observer.h"
151 #include "content/public/browser/render_widget_host.h" 152 #include "content/public/browser/render_widget_host.h"
152 #include "content/public/browser/render_widget_host_iterator.h" 153 #include "content/public/browser/render_widget_host_iterator.h"
153 #include "content/public/browser/resource_context.h" 154 #include "content/public/browser/resource_context.h"
(...skipping 17 matching lines...) Expand all
171 #include "gpu/command_buffer/service/gpu_switches.h" 172 #include "gpu/command_buffer/service/gpu_switches.h"
172 #include "gpu/ipc/host/gpu_memory_buffer_support.h" 173 #include "gpu/ipc/host/gpu_memory_buffer_support.h"
173 #include "ipc/ipc.mojom.h" 174 #include "ipc/ipc.mojom.h"
174 #include "ipc/ipc_channel.h" 175 #include "ipc/ipc_channel.h"
175 #include "ipc/ipc_channel_mojo.h" 176 #include "ipc/ipc_channel_mojo.h"
176 #include "ipc/ipc_logging.h" 177 #include "ipc/ipc_logging.h"
177 #include "media/base/media_switches.h" 178 #include "media/base/media_switches.h"
178 #include "media/media_features.h" 179 #include "media/media_features.h"
179 #include "mojo/edk/embedder/embedder.h" 180 #include "mojo/edk/embedder/embedder.h"
180 #include "mojo/public/cpp/bindings/associated_interface_ptr.h" 181 #include "mojo/public/cpp/bindings/associated_interface_ptr.h"
182 #include "mojo/public/cpp/bindings/strong_binding.h"
181 #include "net/url_request/url_request_context_getter.h" 183 #include "net/url_request/url_request_context_getter.h"
182 #include "ppapi/features/features.h" 184 #include "ppapi/features/features.h"
183 #include "services/resource_coordinator/memory/coordinator/coordinator_impl.h" 185 #include "services/resource_coordinator/memory/coordinator/coordinator_impl.h"
184 #include "services/service_manager/public/cpp/binder_registry.h" 186 #include "services/service_manager/public/cpp/binder_registry.h"
185 #include "services/service_manager/public/cpp/connector.h" 187 #include "services/service_manager/public/cpp/connector.h"
186 #include "services/service_manager/public/cpp/interface_provider.h" 188 #include "services/service_manager/public/cpp/interface_provider.h"
187 #include "services/service_manager/runner/common/client_util.h" 189 #include "services/service_manager/runner/common/client_util.h"
188 #include "services/service_manager/runner/common/switches.h" 190 #include "services/service_manager/runner/common/switches.h"
189 #include "services/shape_detection/public/interfaces/barcodedetection.mojom.h" 191 #include "services/shape_detection/public/interfaces/barcodedetection.mojom.h"
190 #include "services/shape_detection/public/interfaces/constants.mojom.h" 192 #include "services/shape_detection/public/interfaces/constants.mojom.h"
(...skipping 273 matching lines...) Expand 10 before | Expand all | Expand 10 after
464 // Forwards service requests to Service Manager since the renderer cannot launch 466 // Forwards service requests to Service Manager since the renderer cannot launch
465 // out-of-process services on is own. 467 // out-of-process services on is own.
466 template <typename R> 468 template <typename R>
467 void ForwardShapeDetectionRequest(R request) { 469 void ForwardShapeDetectionRequest(R request) {
468 service_manager::Connector* connector = 470 service_manager::Connector* connector =
469 ServiceManagerConnection::GetForProcess()->GetConnector(); 471 ServiceManagerConnection::GetForProcess()->GetConnector();
470 connector->BindInterface(shape_detection::mojom::kServiceName, 472 connector->BindInterface(shape_detection::mojom::kServiceName,
471 std::move(request)); 473 std::move(request));
472 } 474 }
473 475
476 class WorkerFetchContextFactoryImpl : public mojom::WorkerFetchContextFactory {
477 public:
478 static void Create(
479 int render_process_id,
480 ResourceMessageFilter* resource_message_filter,
481 scoped_refptr<ServiceWorkerContextWrapper> service_worker_context,
482 mojom::WorkerFetchContextFactoryRequest request) {
483 DCHECK(base::FeatureList::IsEnabled(features::kOffMainThreadFetch));
484 mojo::MakeStrongBinding(
485 base::MakeUnique<WorkerFetchContextFactoryImpl>(
486 render_process_id, resource_message_filter->GetWeakPtr(),
487 std::move(service_worker_context)),
488 std::move(request));
489 }
490 WorkerFetchContextFactoryImpl(
491 int render_process_id,
492 base::WeakPtr<ResourceMessageFilter> resource_message_filter,
493 scoped_refptr<ServiceWorkerContextWrapper> service_worker_context)
494 : render_process_id_(render_process_id),
495 url_loader_factory_binding_(resource_message_filter.get()),
496 service_worker_context_(std::move(service_worker_context)) {}
497 ~WorkerFetchContextFactoryImpl() override {}
498
499 void CreateWorkerFetchContext(
500 mojom::URLLoaderFactoryAssociatedRequest loader_request,
501 mojom::ServiceWorkerClientAssociatedPtrInfo client_ptr_info,
502 int service_worker_provider_id) override {
503 url_loader_factory_binding_.Bind(std::move(loader_request));
504 service_worker_context_->BindWorkerFetchContext(render_process_id_,
505 service_worker_provider_id,
506 std::move(client_ptr_info));
507 };
508 void GetURLLoaderFactory(
509 mojom::URLLoaderFactoryAssociatedRequest loader_request) override {
510 url_loader_factory_binding_.Bind(std::move(loader_request));
511 }
512
513 private:
514 const int render_process_id_;
515 mojo::AssociatedBinding<mojom::URLLoaderFactory> url_loader_factory_binding_;
516
517 scoped_refptr<ServiceWorkerContextWrapper> service_worker_context_;
518 };
519
474 } // namespace 520 } // namespace
475 521
476 RendererMainThreadFactoryFunction g_renderer_main_thread_factory = NULL; 522 RendererMainThreadFactoryFunction g_renderer_main_thread_factory = NULL;
477 523
478 base::MessageLoop* g_in_process_thread; 524 base::MessageLoop* g_in_process_thread;
479 525
480 // Stores the maximum number of renderer processes the content module can 526 // Stores the maximum number of renderer processes the content module can
481 // create. 527 // create.
482 static size_t g_max_renderer_count_override = 0; 528 static size_t g_max_renderer_count_override = 0;
483 529
(...skipping 805 matching lines...) Expand 10 before | Expand all | Expand 10 after
1289 make_scoped_refptr( 1335 make_scoped_refptr(
1290 storage_partition_impl_->GetBackgroundFetchContext()))); 1336 storage_partition_impl_->GetBackgroundFetchContext())));
1291 1337
1292 registry->AddInterface(base::Bind(&RenderProcessHostImpl::CreateMusGpuRequest, 1338 registry->AddInterface(base::Bind(&RenderProcessHostImpl::CreateMusGpuRequest,
1293 base::Unretained(this))); 1339 base::Unretained(this)));
1294 1340
1295 registry->AddInterface( 1341 registry->AddInterface(
1296 base::Bind(&VideoCaptureHost::Create, 1342 base::Bind(&VideoCaptureHost::Create,
1297 BrowserMainLoop::GetInstance()->media_stream_manager())); 1343 BrowserMainLoop::GetInstance()->media_stream_manager()));
1298 1344
1345 if (base::FeatureList::IsEnabled(features::kOffMainThreadFetch)) {
1346 scoped_refptr<ServiceWorkerContextWrapper> service_worker_context(
1347 static_cast<ServiceWorkerContextWrapper*>(
1348 storage_partition_impl_->GetServiceWorkerContext()));
1349 registry->AddInterface(
1350 base::Bind(&WorkerFetchContextFactoryImpl::Create, GetID(),
1351 base::Unretained(resource_message_filter_.get()),
1352 service_worker_context));
kinuko 2017/04/14 14:00:26 nit: it feels a little strange why we pass RMF as
horo 2017/04/17 15:12:04 Done.
1353 }
1354
1299 // This is to support usage of WebSockets in cases in which there is no 1355 // This is to support usage of WebSockets in cases in which there is no
1300 // associated RenderFrame (e.g., Shared Workers). 1356 // associated RenderFrame (e.g., Shared Workers).
1301 AddUIThreadInterface( 1357 AddUIThreadInterface(
1302 registry.get(), base::Bind(&WebSocketManager::CreateWebSocket, GetID(), 1358 registry.get(), base::Bind(&WebSocketManager::CreateWebSocket, GetID(),
1303 MSG_ROUTING_NONE)); 1359 MSG_ROUTING_NONE));
1304 1360
1305 // Chrome browser process only provides DiscardableSharedMemory service when 1361 // Chrome browser process only provides DiscardableSharedMemory service when
1306 // Chrome is not running in mus+ash. 1362 // Chrome is not running in mus+ash.
1307 if (!service_manager::ServiceManagerIsRemote()) { 1363 if (!service_manager::ServiceManagerIsRemote()) {
1308 discardable_memory::DiscardableSharedMemoryManager* manager = 1364 discardable_memory::DiscardableSharedMemoryManager* manager =
(...skipping 1774 matching lines...) Expand 10 before | Expand all | Expand 10 after
3083 LOG(ERROR) << "Terminating render process for bad Mojo message: " << error; 3139 LOG(ERROR) << "Terminating render process for bad Mojo message: " << error;
3084 3140
3085 // The ReceivedBadMessage call below will trigger a DumpWithoutCrashing. 3141 // The ReceivedBadMessage call below will trigger a DumpWithoutCrashing.
3086 // Capture the error message in a crash key value. 3142 // Capture the error message in a crash key value.
3087 base::debug::ScopedCrashKey error_key_value("mojo-message-error", error); 3143 base::debug::ScopedCrashKey error_key_value("mojo-message-error", error);
3088 bad_message::ReceivedBadMessage(render_process_id, 3144 bad_message::ReceivedBadMessage(render_process_id,
3089 bad_message::RPH_MOJO_PROCESS_ERROR); 3145 bad_message::RPH_MOJO_PROCESS_ERROR);
3090 } 3146 }
3091 3147
3092 } // namespace content 3148 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698