Extract a Connector interface from Shell that can be cloned & passed to other threads

mojo/shell/public/interfaces/shell.mojom

Index: mojo/shell/public/interfaces/shell.mojom
diff --git a/mojo/shell/public/interfaces/shell.mojom b/mojo/shell/public/interfaces/shell.mojom
index d7160ea2f6f8d3d75b02bf4b31c8f64bdbe2b5e2..2233fab9a2ae6e3e74dec599a394055465a9e9d5 100644
--- a/mojo/shell/public/interfaces/shell.mojom
+++ b/mojo/shell/public/interfaces/shell.mojom
@@ -20,9 +20,8 @@ struct CapabilityFilter {
   map<string, array<string>> filter;
 };
 
-// An interface through which a Mojo application may communicate with the Mojo
-// system and request connections to other applications.
-interface Shell {
+// Encapsulates establishing connections with other Mojo applications.
+interface Connector {
   const uint32 kInvalidApplicationID = 0;
   const uint32 kUserRoot = 0;
   const uint32 kUserInherit = 1;
@@ -83,6 +82,16 @@ interface Shell {
           InterfaceProvider? local_interfaces,
           CapabilityFilter filter) => (uint32 application_id);
 
+  // Clones this Connector so it can be passed to another thread.
+  Clone(Connector& request);
+};
+
+// Wraps functionality exposed by the Shell to a Mojo application instance.
+interface Shell {
+  // Obtain a Connector that can be used to create connections with other
+  // applications.
+  GetConnector(Connector& connector);

[sky, 2016/02/24 17:08:07]

One thing not readily obvious from this api is that the implementation of
connector is tied to the life of the applicationinstance. I think you should say
that.

+
   // When there are no more instantiated services in an application, it should
   // start its shutdown process by calling this method. Additionally, it should
   // keep track of any new service requests that come in. The shell will then