Add two interfaces for ServiceWorkerProviderContext/ProviderHost

content/browser/service_worker/service_worker_test_utils.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/service_worker/service_worker_test_utils.h"
 
 #include <utility>
 
 #include "content/browser/service_worker/service_worker_provider_host.h"
 
 namespace content {
 
+ServiceWorkerRemoteProviderEndpoint::ServiceWorkerRemoteProviderEndpoint() {}
+ServiceWorkerRemoteProviderEndpoint::ServiceWorkerRemoteProviderEndpoint(
+    ServiceWorkerRemoteProviderEndpoint&& other)
+    : host_ptr(std::move(other.host_ptr)),
+      client_request(std::move(other.client_request)) {}
+
+ServiceWorkerRemoteProviderEndpoint::~ServiceWorkerRemoteProviderEndpoint() {}
+
 std::unique_ptr<ServiceWorkerProviderHost> CreateProviderHostForWindow(
     int process_id,
     int provider_id,
     bool is_parent_frame_secure,
-    base::WeakPtr<ServiceWorkerContextCore> context) {
+    base::WeakPtr<ServiceWorkerContextCore> context,
+    ServiceWorkerRemoteProviderEndpoint* output_endpoint) {
   ServiceWorkerProviderHostInfo info(provider_id, MSG_ROUTING_NONE,
                                      SERVICE_WORKER_PROVIDER_FOR_WINDOW,
                                      is_parent_frame_secure);
+  mojom::ServiceWorkerProviderAssociatedPtr client_ptr;
+  output_endpoint->client_request = mojo::MakeIsolatedRequest(&client_ptr);
+  info.client_ptr_info = client_ptr.PassInterface();
+  info.host_request = mojo::MakeIsolatedRequest(&output_endpoint->host_ptr);

[falken, 2017/05/22 08:28:12]

these four lines seem to be repeated throughout this patch,. is it possible to
factor them out to a function?

[shimazu, 2017/05/23 06:29:34]

Done.

   return ServiceWorkerProviderHost::Create(process_id, std::move(info),
                                            std::move(context), nullptr);
 }
 
 std::unique_ptr<ServiceWorkerProviderHost>
 CreateProviderHostForServiceWorkerContext(
     int process_id,
     int provider_id,
     bool is_parent_frame_secure,
-    base::WeakPtr<ServiceWorkerContextCore> context) {
+    base::WeakPtr<ServiceWorkerContextCore> context,
+    ServiceWorkerRemoteProviderEndpoint* output_endpoint) {
   ServiceWorkerProviderHostInfo info(provider_id, MSG_ROUTING_NONE,
                                      SERVICE_WORKER_PROVIDER_FOR_CONTROLLER,
                                      is_parent_frame_secure);
+  mojom::ServiceWorkerProviderAssociatedPtr client_ptr;
+  output_endpoint->client_request = mojo::MakeIsolatedRequest(&client_ptr);
+  info.client_ptr_info = client_ptr.PassInterface();
+  info.host_request = mojo::MakeIsolatedRequest(&output_endpoint->host_ptr);
   return ServiceWorkerProviderHost::Create(process_id, std::move(info),
                                            std::move(context), nullptr);
 }
 
 std::unique_ptr<ServiceWorkerProviderHost> CreateProviderHostWithDispatcherHost(
     int process_id,
     int provider_id,
     base::WeakPtr<ServiceWorkerContextCore> context,
     int route_id,
-    ServiceWorkerDispatcherHost* dispatcher_host) {
+    ServiceWorkerDispatcherHost* dispatcher_host,
+    ServiceWorkerRemoteProviderEndpoint* output_endpoint) {
   ServiceWorkerProviderHostInfo info(provider_id, route_id,
                                      SERVICE_WORKER_PROVIDER_FOR_WINDOW, true);
+  mojom::ServiceWorkerProviderAssociatedPtr client_ptr;
+  output_endpoint->client_request = mojo::MakeIsolatedRequest(&client_ptr);
+  info.client_ptr_info = client_ptr.PassInterface();
+  info.host_request = mojo::MakeIsolatedRequest(&output_endpoint->host_ptr);
   return ServiceWorkerProviderHost::Create(process_id, std::move(info),
                                            std::move(context), dispatcher_host);
 }
 
 }  // namespace content