Add mojo::Binding<Interface> for more flexible pipe<->impl binding

mojo/services/test_service/test_request_tracker_application.cc

Index: mojo/services/test_service/test_request_tracker_application.cc
diff --git a/mojo/services/test_service/test_request_tracker_application.cc b/mojo/services/test_service/test_request_tracker_application.cc
index b533377547afc8257c8d3c5046f829f40020e0c6..d25c0fd4fd5fd9d7dc8aae7a2f96cd8606dae031 100644
--- a/mojo/services/test_service/test_request_tracker_application.cc
+++ b/mojo/services/test_service/test_request_tracker_application.cc
@@ -15,8 +15,7 @@ namespace mojo {
 namespace test {
 
 TestRequestTrackerApplication::TestRequestTrackerApplication()
-    : test_tracked_request_factory_(&context_),
-      test_request_tracker_factory_(&context_) {
+    : test_tracked_request_factory_(&context_) {
 }
 
 TestRequestTrackerApplication::~TestRequestTrackerApplication() {
@@ -27,8 +26,8 @@ bool TestRequestTrackerApplication::ConfigureIncomingConnection(
   // Every instance of the service and recorder shares the context.
   // Note, this app is single-threaded, so this is thread safe.
   connection->AddService(&test_tracked_request_factory_);
-  connection->AddService(&test_request_tracker_factory_);
-  connection->AddService(this);
+  connection->AddService<TestTimeService>(this);
+  connection->AddService<TestRequestTracker>(this);
   return true;
 }
 
@@ -38,6 +37,14 @@ void TestRequestTrackerApplication::Create(
   BindToRequest(new TestTimeServiceImpl(connection), &request);
 }
 
+void TestRequestTrackerApplication::Create(
+    ApplicationConnection* connection,
+    InterfaceRequest<TestRequestTracker> request) {
+  TestRequestTrackerImpl* impl =
+      BindToRequest(new TestRequestTrackerImpl(&context_), &request);

[DaveMoore, 2014/11/11 17:18:45]

Removing OnConnectionEstablished() means we can no longer use the provided
factory object here. That's sad.

[jamesr, 2014/11/11 17:47:49]

The fact that the OnConnectionEstablished() exists at all is the bug (see
previous comment).  The constructor should take care of this.

+  impl->OnConnectionEstablished();
+}
+
 }  // namespace test
 }  // namespace mojo