Mojo: Use InterfaceFactory<Interface> for service registration

mojo/public/cpp/application/default_interface_provider.h

Index: mojo/public/cpp/application/default_interface_provider.h
diff --git a/mojo/public/cpp/application/default_interface_provider.h b/mojo/public/cpp/application/default_interface_provider.h
new file mode 100644
index 0000000000000000000000000000000000000000..60d52fd64590f23b55a5b0cf4b999aae6eb757a0
--- /dev/null
+++ b/mojo/public/cpp/application/default_interface_provider.h
@@ -0,0 +1,28 @@
+// 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.
+
+#ifndef MOJO_PUBLIC_APPLICATION_DEFAULT_INTERFACE_FACTORY_H_
+#define MOJO_PUBLIC_APPLICATION_DEFAULT_INTERFACE_FACTORY_H_
+
+#include "mojo/public/cpp/application/interface_provider.h"
+
+namespace mojo {
+
+// Use this class to allocate and bind default-constructed instances of Impl in
+// response to requests for interface Interface.
+template <typename Impl, typename Interface = typename Impl::Implements>
+class DefaultInterfaceProvider : public InterfaceProvider<Interface> {

[DaveMoore, 2014/07/15 00:08:29]

Default is still wrong, unless it's used as a default by some higher level code.
SimpleInterfaceProvider maybe?

[jamesr, 2014/07/15 00:33:10]

There's only one use of this class, so it's not really worth having this
boilerplate.  I merged the implementation into the only user

+ public:
+  DefaultInterfaceProvider() {}
+  virtual ~DefaultInterfaceProvider() {}
+  virtual void BindToRequest(ApplicationConnection* connection,
+                             InterfaceRequest<Interface> request)
+      MOJO_OVERRIDE {
+    mojo::BindToRequest(new Impl, &request);
+  }
+};
+
+}  // namespace mojo
+
+#endif  // MOJO_PUBLIC_APPLICATION_DEFAULT_INTERFACE_FACTORY_H_