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

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

Issue 2034383003: Removed exposed_services from mojom definitions. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 6 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 d00ac01d9ad99324d9dd45823da32ebd2b49ac12..134ee366b674ba4daf26b52d237245a697ea98a2 100644
--- a/mojo/public/interfaces/application/shell.mojom
+++ b/mojo/public/interfaces/application/shell.mojom
@@ -13,15 +13,9 @@ import "mojo/public/interfaces/application/service_provider.mojom";
interface Shell {
// See |ApplicationConnector.ConnectToApplication()|.
//
- // 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);
+ ConnectToApplication(string application_url, ServiceProvider& services);
CreateApplicationConnector(
ApplicationConnector& application_connector_request);

Powered by Google App Engine
This is Rietveld 408576698