Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

mojo/shell/BUILD.gn

 # 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.
 
 import("//mojo/public/mojo_application.gni")
 import("//mojo/public/mojo_application_manifest.gni")
 import("//mojo/public/tools/bindings/mojom.gni")
 import("//testing/test.gni")
 
 group("all") {
@@ skipping 18 matching lines @@
     "connect_to_application_params.h",
     "connect_util.cc",
     "connect_util.h",
     "identity.cc",
     "identity.h",
     "native_runner.h",
     "shell_application_delegate.cc",
     "shell_application_delegate.h",
     "shell_application_loader.cc",
     "shell_application_loader.h",
-    "shell_client_factory_connection.cc",
-    "shell_client_factory_connection.h",
     "static_application_loader.cc",
     "static_application_loader.h",
     "switches.cc",
     "switches.h",
   ]
 
   public_deps = [
     "//base",
     "//mojo/common",
     "//mojo/public/cpp/bindings",
@@ skipping 180 matching lines @@
   deps = [
     ":test_bindings",
     "//base",
     "//build/config/sanitizers:deps",
     "//mojo/common:common_base",
     "//mojo/converters/network",
     "//mojo/shell/public/cpp",
     "//mojo/shell/runner/child:test_native_main",
   ]
 }