Embed a mojo ApplicationManager in content/browser

content/public/browser/content_browser_client.h

Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index eff374d1c72f409e37044a2475f8b31bc7bdbca1..e64fa4e21d02044da42cbb963c6f5e61d5b5f864 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -81,6 +81,7 @@ namespace content {
 
 enum class PermissionType;
 class AccessTokenStore;
+class ApplicationRegistry;
 class BrowserChildProcessHost;
 class BrowserContext;
 class BrowserMainParts;
@@ -578,6 +579,10 @@ class CONTENT_EXPORT ContentBrowserClient {
       ServiceRegistry* registry,
       RenderFrameHost* render_frame_host) {}
 
+  // Registers in-process Mojo application loaders with the browser's global
+  // Mojo shell.
+  virtual void RegisterMojoApplications(ApplicationRegistry* registry) {}

[jam, 2015/05/29 01:48:54]

why call the embedder and give it another interface so that it can call it back?
i.e. why not just give it here a vector of GURL/StaticAppFactory that it can
push_back?

[Ken Rockot(use gerrit already), 2015/05/29 03:20:40]

I think there are a few decent reasons:

 - I experimented with passing in a map and it adds lots of code,
   mostly because:
 - There's no need to allocate extra containers in some cases;
   MojoShellContext for example doesn't hold a map of its own, it
   just forwards URL/Loader pairs to ApplicationManager.
 - We eventually need to register more than just static factories
 - Things which hold URL/app mappings may want to hand out a
   registration interface to more than just Content*Client.

[jam, 2015/05/29 15:11:50]

I'm not sure I understand what you mean here.

are you saying that it's inefficient to copy the stl container class?
why?
examples?

in general, for content api (per link in previous round of reviews) we try to be
as minimal as possible. what can be done with one method shouldn't be done with
a method and interface

+
   // Registers additional navigator.connect service factories available in a
   // particular NavigatorConnectContext.
   virtual void GetAdditionalNavigatorConnectServices(