Remove shell::Connection* parameter to InterfaceFactory<T>::Create()

services/shell/public/cpp/lib/interface_registry.cc

Index: services/shell/public/cpp/lib/interface_registry.cc
diff --git a/services/shell/public/cpp/lib/interface_registry.cc b/services/shell/public/cpp/lib/interface_registry.cc
index b585eaee26654e0908cc5ed5ded2914e38909b1d..673ebdc4a0c9d20db526c3c0ad82e085f90c048f 100644
--- a/services/shell/public/cpp/lib/interface_registry.cc
+++ b/services/shell/public/cpp/lib/interface_registry.cc
@@ -64,7 +64,9 @@ void InterfaceRegistry::GetInterface(const mojo::String& interface_name,
 
   auto iter = name_to_binder_.find(interface_name);
   if (iter != name_to_binder_.end()) {
-    iter->second->BindInterface(connection_, interface_name, std::move(handle));
+    iter->second->BindInterface(connection_->GetRemoteIdentity(),

[Ken Rockot(use gerrit already), 2016/07/26 00:14:46]

None of the uses with a null connection need the remote identity, so can we just
pass a dummy Identity if connection_ is null?

+                                interface_name,
+                                std::move(handle));
   } else if (connection_ && !connection_->AllowsInterface(interface_name)) {
     LOG(ERROR) << "Connection CapabilityFilter prevented binding to "
                << "interface: " << interface_name << " connection_name:"
@@ -72,6 +74,8 @@ void InterfaceRegistry::GetInterface(const mojo::String& interface_name,
                << connection_->GetRemoteIdentity().name();
   } else if (!default_binder_.is_null()) {
     default_binder_.Run(interface_name, std::move(handle));
+  } else {
+    LOG(ERROR) << "Failed to locate a binder for interface: " << interface_name;
   }
 }