Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

mojo/shell/connect_util.cc

Index: mojo/shell/connect_util.cc
diff --git a/mojo/shell/connect_util.cc b/mojo/shell/connect_util.cc
index d33fad831330424111836dee56ad992b1c102226..6ad3f6ac51eb54b55e4e28823e3ff785e5910564 100644
--- a/mojo/shell/connect_util.cc
+++ b/mojo/shell/connect_util.cc
@@ -15,12 +15,12 @@ namespace shell {
 
 ScopedMessagePipeHandle ConnectToInterfaceByName(
     ApplicationManager* application_manager,
+    const Identity& source,
     const GURL& application_url,
     const std::string& interface_name) {
   shell::mojom::InterfaceProviderPtr remote_interfaces;
   scoped_ptr<ConnectToApplicationParams> params(new ConnectToApplicationParams);
-  params->set_source(Identity(GURL("mojo://shell/"), std::string(),
-                              GetPermissiveCapabilityFilter()));
+  params->set_source(source);
   params->SetTarget(Identity(application_url, std::string(),
                              GetPermissiveCapabilityFilter()));
   params->set_remote_interfaces(GetProxy(&remote_interfaces));