Eliminate ShellClientFactoryConnection & just have the ApplicationManager do it.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 5f9d0fd8e75474af8cc1b074f981337ee9123bef..cf88e5cf98f557f2c251fbd6e19b33b6661ea737 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -614,7 +614,7 @@ bool IsReload(FrameMsg_Navigate_Type::Value navigation_type) {
 RenderFrameImpl::CreateRenderFrameImplFunction g_create_render_frame_impl =
     nullptr;
 
-void OnGotRemoteIDs(uint32_t remote_id, uint32_t content_handler_id) {}
+void OnGotInstanceID(uint32_t instance_id) {}
 
 WebString ConvertRelativePathToHtmlAttribute(const base::FilePath& path) {
   DCHECK(!path.IsAbsolute());
@@ -6058,7 +6058,7 @@ mojo::shell::mojom::InterfaceProviderPtr RenderFrameImpl::ConnectToApplication(
   filter->filter.insert("*", std::move(all_interfaces));
   mojo_shell_->ConnectToApplication(
       std::move(request), GetProxy(&interface_provider), nullptr,
-      std::move(filter), base::Bind(&OnGotRemoteIDs));
+      std::move(filter), base::Bind(&OnGotInstanceID));
   return interface_provider;
 }