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

Unified Diff: third_party/mojo/src/mojo/public/interfaces/application/application.mojom

Issue 975973002: Update mojo sdk to rev f68e697e389943cd9bf9652397312280e96b127a (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: shake fist at msvc Created 5 years, 10 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: third_party/mojo/src/mojo/public/interfaces/application/application.mojom
diff --git a/third_party/mojo/src/mojo/public/interfaces/application/application.mojom b/third_party/mojo/src/mojo/public/interfaces/application/application.mojom
index 03fe94369dcfd5ec6ff19571fff240e142bc3038..eaa742d0651b74a2ceb7c05e383bb5531695f4dd 100644
--- a/third_party/mojo/src/mojo/public/interfaces/application/application.mojom
+++ b/third_party/mojo/src/mojo/public/interfaces/application/application.mojom
@@ -10,11 +10,19 @@ import "mojo/public/interfaces/application/shell.mojom";
// This is the primary interface implemented by every Mojo application. It
// allows the application to receive its startup arguments from the shell, and
// to be notified of events that occur during its execution.
+//
+// TODO(aa): It would be good to reorder the parameters once we have interface
+// versioning.
interface Application {
// Initializes the application with the specified arguments. This method is
// guaranteed to be called before any other method is called, and will only be
- // called once. The |url| parameter is the final url the application was found
- // at, after all redirects and resolutions.
+ // called once.
+ //
+ // The |url| parameter is the identity of the application as far as the shell
+ // is concerned. This will be the URL the application was found at, after all
+ // mappings, resolution, and redirects. And it will not include the
+ // querystring, since the querystring is not part of an application's
+ // identity.
Initialize(Shell shell, array<string>? args, string url);
// Called when another application (identified by |requestor_url|) attempts to
@@ -37,9 +45,14 @@ interface Application {
//
// This application is free to ignore the |services| or |exposed_services|
// parameters if it does not wish to offer or request services.
+ //
+ // resolved_url is the URL that was requested to create this connection, after
+ // all mappings, resolutions, and redirects. This will include any querystring
+ // that was part of the request.
AcceptConnection(string requestor_url,
ServiceProvider&? services,
- ServiceProvider? exposed_services);
+ ServiceProvider? exposed_services,
+ string resolved_url);
// Called to request the application shut itself down gracefully.
RequestQuit();

Powered by Google App Engine
This is Rietveld 408576698