Pass the final URL an app was loaded from to the app in Initialize().

mojo/public/interfaces/application/application.mojom

Index: mojo/public/interfaces/application/application.mojom
diff --git a/mojo/public/interfaces/application/application.mojom b/mojo/public/interfaces/application/application.mojom
index cffcb1fc7606aa30ea59360f86f8bd31dd4e7537..7e8c49362715e107cfb3d3e02a4ea8b644a391f6 100644
--- a/mojo/public/interfaces/application/application.mojom
+++ b/mojo/public/interfaces/application/application.mojom
@@ -13,8 +13,9 @@ import "mojo/public/interfaces/application/shell.mojom";
 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.
-  Initialize(Shell shell, array<string>? args);
+  // called once. The |url| parameter is the final url the application was found
+  // at, after all redirects and resolutions.
+  Initialize(Shell shell, string url, array<string>? args);

[sky, 2015/02/05 23:41:22]

Is there a reason we don't want to pass the url as a chain? What if there was
redirects or other things, might the urls along the path be interesting?

[Aaron Boodman, 2015/02/05 23:45:27]

I guess since we don't know the shape of the sandbox yet (origin-shaped?
url-shaped? something else?) we don't know what information makes sense to share
with the app.

And since the final URL seems like all that is needed right now, I thought it
made sense to be conservative.

[sky, 2015/02/05 23:49:51]

Doesn't the array<string> provide you with more flexibility for the future? It
can be one for now, but it's easy to make it contain more in the future.

As to the specific case, IMO we should share all and restrict that in the future
if necessary.

[Aaron Boodman, 2015/02/06 00:11:54]

AFAIK, after release, it would not be possible to restrict because interfaces
have to be backward compatible.

True, we are a long way away from that. But another reason that I prefer to just
pass the single URL is so that if there is a reason to add more of the URLs in
the future, we will find out about it and have a discussion, rather than just
silently growing dependencies on parts of an API that we weren't sure we should
have.

[Aaron Boodman, 2015/02/06 00:27:55]

With a moment more thought (and double checking with Yuzhu) I realized that
changing the type either way would not be possible without breaking backward
compat.

But it doesn't matter because we would use the ordinal system to rename the old
method to InitializeDeprecated and add a new method with the new params named
Initialize.

So moving the interface either direction -- to more permissive or less
permissive will work.

It's true that just having an array now would be less work in the future, but by
that logic we should just make all our types array<string>, or just JavaScript
:).

 
   // Called when another application (identified by |requestor_url|) attempts to
   // open a connection to this application.