Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1279)

Unified Diff: mojo/services/test_service/test_request_tracker_application.cc

Issue 718473003: Add mojo::Binding<Interface> for more flexible pipe<->impl binding (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Rename Binding::router() -> Binding::internal_router() Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..a6a026106ddf5613aa4c2bb8d833c44ff6269917 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,15 +26,23 @@ 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;
}
void TestRequestTrackerApplication::Create(
ApplicationConnection* connection,
InterfaceRequest<TestTimeService> request) {
- BindToRequest(new TestTimeServiceImpl(connection), &request);
+ new TestTimeServiceImpl(connection, request.Pass());
+}
+
+void TestRequestTrackerApplication::Create(
+ ApplicationConnection* connection,
+ InterfaceRequest<TestRequestTracker> request) {
+ TestRequestTrackerImpl* impl =
+ BindToRequest(new TestRequestTrackerImpl(&context_), &request);
+ impl->OnConnectionEstablished();
}
} // namespace test

Powered by Google App Engine
This is Rietveld 408576698