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

Unified Diff: mojo/public/interfaces/application/shell.mojom

Issue 1965563002: Neuter the wrong-way ServiceProvider (a.k.a. exposed_services). (Closed) Base URL: https://github.com/domokit/mojo.git@work795_js_wrong_way
Patch Set: rebased 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/interfaces/application/shell.mojom
diff --git a/mojo/public/interfaces/application/shell.mojom b/mojo/public/interfaces/application/shell.mojom
index dc210c74461f6781eacf1e2375ca7721aa68fe6b..d00ac01d9ad99324d9dd45823da32ebd2b49ac12 100644
--- a/mojo/public/interfaces/application/shell.mojom
+++ b/mojo/public/interfaces/application/shell.mojom
@@ -12,7 +12,13 @@ import "mojo/public/interfaces/application/service_provider.mojom";
// system and request connections to other applications.
interface Shell {
// See |ApplicationConnector.ConnectToApplication()|.
- // TODO(vtl): Maybe this should be deprecated/removed.
+ //
+ // WARNING: |exposed_services| is deprecated and non-functional, and will be
+ // removed. Note that this also means that |services| will become required.
+ // https://github.com/domokit/mojo/issues/762
+ //
+ // TODO(vtl): Maybe this should be deprecated/removed (in favor of just using
+ // |ApplicationConnector|).
ConnectToApplication(string application_url,
ServiceProvider&? services,
ServiceProvider? exposed_services);
« no previous file with comments | « mojo/public/interfaces/application/application_connector.mojom ('k') | shell/application_manager/application_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698