Pass ServiceProvider and ServiceProvider& params in Connect

mojo/public/interfaces/application/application.mojom

Index: mojo/public/interfaces/application/application.mojom
diff --git a/mojo/public/interfaces/application/application.mojom b/mojo/public/interfaces/application/application.mojom
index 4cf21108d0d6cfe8e45d218efd5ea8d89c1eaabf..70cc7ca3233b977cdcfb08506f717a89bfeb8875 100644
--- a/mojo/public/interfaces/application/application.mojom
+++ b/mojo/public/interfaces/application/application.mojom
@@ -12,5 +12,8 @@ interface Application {
   // Initialize is guaranteed to be called before any AcceptConnection calls.
   Initialize(array<string>? args);
 
-  AcceptConnection(string? requestor_url, ServiceProvider? provider);
+  // Called in response to a call to mojo.Shell.ConnectToApplication().
+  AcceptConnection(string? requestor_url,

[Aaron Boodman, 2015/01/09 17:53:36]

We should really remove the question mark. requestor_url is not optional. I can
do it in a separate change though.

+                   ServiceProvider&? services,

[darin (slow to review), 2015/01/09 17:31:16]

I'm a little confused by the use of & here. Usually that is a way to get
something, and I would expect the thing the caller gets here to be the exported
services. It seems like the signature should be:

  AccceptConnection(string? requestor_url,
                    ServiceProvider? requestor_services,
                    ServiceProvider&? exported_services);

[Aaron Boodman, 2015/01/09 17:53:35]

These names are always confusing because they can be interpreted both ways:

'exported_services' can mean either services exported from the app receiving
this call, or services exported from the app making the call.

Perhaps we should say connector and connectee, or requestor and requestee, or
client and provider...

+                   ServiceProvider? exported_services);
 };