Hook embedded shell up to MojoShellConnection

mojo/shell/application_manager.h

Index: mojo/shell/application_manager.h
diff --git a/mojo/shell/application_manager.h b/mojo/shell/application_manager.h
index b6e657c426b8c8a1b4aa53579dc41f6cc2d1fa8d..b5c664c6bc71c4cb910139d494d574b69f491e6f 100644
--- a/mojo/shell/application_manager.h
+++ b/mojo/shell/application_manager.h
@@ -80,6 +80,13 @@ class ApplicationManager : public ShellClient,
   // instance of the target application is running, one will be loaded.
   void Connect(scoped_ptr<ConnectParams> params);
 
+  // Creates a new ApplicationInstance for the target application identified by
+  // |params|. No resolution is done on the target URL and there must not be any
+  // existing instances for the given URL. Returns a pending ShellClientRequest
+  // for the instance.

[Ben Goodger (Google), 2016/02/25 03:02:58]

I'm wondering if we can make this method sound less general purpose and more
specific to this specific use case. i.e.

you are getting a ShellClientRequest for an initialization case where the params
describes the embedder.

InitInstanceForEmbedder()?

you should also ensure this only gets called once.

[Ken Rockot(use gerrit already), 2016/02/25 04:42:05]

sgtm and done

+  mojom::ShellClientRequest CreateInstanceForURL(
+      scoped_ptr<ConnectParams> params);
+
   // Sets the default Loader to be used if not overridden by SetLoaderForURL().
   void set_default_loader(scoped_ptr<ApplicationLoader> loader) {
     default_loader_ = std::move(loader);