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

Unified Diff: mojo/public/cpp/application/service_connector.h

Issue 1984463002: More cleanup: Get rid of ServiceConnectorRegistry. (Closed) Base URL: https://github.com/domokit/mojo.git@work791_app_conn_devo_4
Patch Set: Created 4 years, 7 months 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/public/cpp/application/service_connector.h
diff --git a/mojo/public/cpp/application/service_connector.h b/mojo/public/cpp/application/service_connector.h
index 378be9e25028079edd8e3ed87d6f36f24c7eda98..8b919b3e3d36bf37a455973e2b98e11bd773fa49 100644
--- a/mojo/public/cpp/application/service_connector.h
+++ b/mojo/public/cpp/application/service_connector.h
@@ -13,15 +13,16 @@ namespace mojo {
struct ConnectionContext;
+// |ServiceConnector| is the generic, type-unsafe interface for objects used by
+// |ServiceProviderImpl| to connect generic "interface requests" (i.e., just
+// message pipes) specified by service name to service implementations.
class ServiceConnector {
public:
virtual ~ServiceConnector() {}
- // Asks the ServiceConnector to connect to the specified service. If the
- // ServiceConnector connects to the service it should take ownership of
- // the handle in |handle|.
+ // Connects to the |ServiceConnector|'s service (if the |ServiceConnector|
+ // wishes to refuse the connection, it should simply drop |handle|).
virtual void ConnectToService(const ConnectionContext& connection_context,
- const std::string& interface_name,
ScopedMessagePipeHandle handle) = 0;
};
« no previous file with comments | « mojo/public/cpp/application/lib/service_provider_impl.cc ('k') | mojo/public/cpp/application/service_provider_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698