Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

mojo/shell/public/cpp/connection.h

Index: mojo/shell/public/cpp/connection.h
diff --git a/mojo/shell/public/cpp/connection.h b/mojo/shell/public/cpp/connection.h
index a2f6b772be90e7a022e6c38d32d036bbb8fba9ed..e05243fcb1a3a1b64dd21fe9a1f65796c3ec622b 100644
--- a/mojo/shell/public/cpp/connection.h
+++ b/mojo/shell/public/cpp/connection.h
@@ -95,14 +95,6 @@ class Connection {
   // been established.
   virtual bool GetRemoteApplicationID(uint32_t* remote_id) const = 0;
 
-  // Returns the id of the deepest shell client factory used in connecting to
-  // the application. See GetRemoteApplicationID() for details about the return
-  // value. A |shell_client_factory_id| value of Shell::kInvalidApplicationID
-  // indicates no shell client factory was used in connecting to the
-  // application.
-  virtual bool GetRemoteShellClientFactoryID(
-      uint32_t* shell_client_factory_id) const = 0;
-
   // See description in GetRemoteApplicationID()/
   // GetRemoteShellClientFactoryID(). If the ids are available, |callback| is
   // run immediately.