Add RegisterService, split out of Connect().

services/service_manager/public/interfaces/connector.mojom

 // Copyright 2014 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.
 
 module service_manager.mojom;
 
 import "services/service_manager/public/interfaces/interface_provider.mojom";
 
 const string kRootUserID = "505C0EE9-3013-43C0-82B0-A84F50CF8D84";
 const string kInheritUserID = "D26290E4-4485-4EAE-81A2-66D1EEB40A9D";
@@ skipping 69 matching lines @@
   handle<message_pipe> service;
 
   // Allows the client process launcher to tell the service manager the PID of
   // the process it created (the pid isn't supplied directly here as the process
   // may not have been launched by the time Connect() is called.)
   handle<message_pipe> pid_receiver_request;
 };
 
 // Encapsulates establishing connections with other Services.
 interface Connector {
+  // Registers a Service in a process started by someone other than the Service
+  // Manager.
+  RegisterService(Identity name,

[Ken Rockot(use gerrit already), 2017/01/05 20:31:48]

nit: I think RegisterInstance might be a more accurate name

+                  ClientProcessConnection client_process_connection);

[Ken Rockot(use gerrit already), 2017/01/05 20:31:48]

nit: Maybe just flatten service and pid_receiver_request into the args here?
Also you could use a PIDReceiver& instead of a raw pipe handle.

+
   // Requests a connection with another service. The service originating the
   // request is referred to as the "source" and the one receiving the "target".
   //
   // The connection is embodied by a pair of message pipes binding the
   // InterfaceProvider interface, which allows both the source and target
   // services to export interfaces to one another. The interfaces bound via
   // these InterfaceProviders are brokered by the service manager according to
   // the security policy defined by each service in its manifest.
   //
   // If the target service is not running, the service manager will run it,
@@ skipping 26 matching lines @@
   //    this callback.
   //
   Connect(Identity target,
           InterfaceProvider&? remote_interfaces,
           ClientProcessConnection? client_process_connection) =>
       (ConnectResult result, string user_id);
 
   // Clones this Connector so it can be passed to another thread.
   Clone(Connector& request);
 };