Make ServiceRegistry (which implements) ApplicationConnection just wrap a ServiceProviderImpl.

mojo/public/cpp/application/tests/service_provider_impl_unittest.cc

Index: mojo/public/cpp/application/tests/service_provider_impl_unittest.cc
diff --git a/mojo/public/cpp/application/tests/service_provider_impl_unittest.cc b/mojo/public/cpp/application/tests/service_provider_impl_unittest.cc
index 2483fc3ce152b1540d09e070640bd62654c1fdd5..f6767672d28ac97239ff6430fbc2833dccf715e0 100644
--- a/mojo/public/cpp/application/tests/service_provider_impl_unittest.cc
+++ b/mojo/public/cpp/application/tests/service_provider_impl_unittest.cc
@@ -338,6 +338,16 @@ TEST_F(ServiceProviderImplTest, FallbackServiceProvider) {
   EXPECT_FALSE(was_run);
 }
 
+TEST_F(ServiceProviderImplTest, ConstructRequestNotPending) {
+  ServiceProviderImpl impl(ConnectionContext(ConnectionContext::Type::INCOMING,
+                                             "https://example.com/remote.mojo",
+                                             "https://example.com/me.mojo"),
+                           InterfaceRequest<ServiceProvider>());
+  EXPECT_EQ(ConnectionContext::Type::UNKNOWN, impl.connection_context().type);
+  EXPECT_EQ(std::string(), impl.connection_context().remote_url);
+  EXPECT_EQ(std::string(), impl.connection_context().connection_url);
+}
+
 // TODO(vtl): Explicitly test |AddServiceForName()|?
 
 }  // namespace