Cache ShellImpl by resolved URL, not initial URL

mojo/application_manager/application_manager.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MOJO_APPLICATION_MANAGER_APPLICATION_MANAGER_H_
 #define MOJO_APPLICATION_MANAGER_APPLICATION_MANAGER_H_
 
 #include <map>
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "mojo/application_manager/application_loader.h"
 #include "mojo/application_manager/application_manager_export.h"
 #include "mojo/public/interfaces/application/service_provider.mojom.h"
 #include "url/gurl.h"
 
 namespace mojo {
 
 class MOJO_APPLICATION_MANAGER_EXPORT ApplicationManager {
  public:
   class MOJO_APPLICATION_MANAGER_EXPORT Delegate {
    public:
     virtual ~Delegate();
     // Send when the Application holding the handle on the other end of the
     // Shell pipe goes away.
-    virtual void OnApplicationError(const GURL& url) = 0;
+    virtual void OnApplicationError(const GURL& url);
+    virtual GURL ResolveURL(const GURL& url);
   };
 
   // API for testing.
   class MOJO_APPLICATION_MANAGER_EXPORT TestAPI {
    public:
     explicit TestAPI(ApplicationManager* manager);
     ~TestAPI();
 
     // Returns true if the shared instance has been created.
     static bool HasCreatedInstance();
     // Returns true if there is a ShellImpl for this URL.
     bool HasFactoryForURL(const GURL& url) const;
 
    private:
     ApplicationManager* manager_;
 
     DISALLOW_COPY_AND_ASSIGN(TestAPI);
   };
 
   // Interface class for debugging only.
   class Interceptor {
    public:
     virtual ~Interceptor() {}
     // Called when ApplicationManager::Connect is called.
     virtual ServiceProviderPtr OnConnectToClient(
         const GURL& url,
         ServiceProviderPtr service_provider) = 0;
   };
 
-  ApplicationManager();
+  ApplicationManager(Delegate* delegate);
   ~ApplicationManager();
 
-  // Returns a shared instance, creating it if necessary.

[Aaron Boodman, 2014/10/31 05:26:17]

Neither of these were called from anywhere.

-  static ApplicationManager* GetInstance();
-
-  void SetDelegate(Delegate* delegate) { delegate_ = delegate; }
-
   // Loads a service if necessary and establishes a new client connection.
   void ConnectToApplication(const GURL& application_url,
                             const GURL& requestor_url,
                             ServiceProviderPtr service_provider);
 
   template <typename Interface>
   inline void ConnectToService(const GURL& application_url,
                                InterfacePtr<Interface>* ptr) {
     ScopedMessagePipeHandle service_handle =
         ConnectToServiceByName(application_url, Interface::Name_);
@@ skipping 40 matching lines @@
   typedef std::map<GURL, ApplicationLoader*> URLToLoaderMap;
   typedef std::map<GURL, ShellImpl*> URLToShellImplMap;
   typedef std::map<GURL, ContentHandlerConnection*> URLToContentHandlerMap;
   typedef std::map<GURL, std::vector<std::string> > URLToArgsMap;
 
   void ConnectToClient(ShellImpl* shell_impl,
                        const GURL& url,
                        const GURL& requestor_url,
                        ServiceProviderPtr service_provider);
 
-  void RegisterLoadedApplication(const GURL& service_url,
+  void RegisterLoadedApplication(const GURL& requested_url,
+                                 const GURL& resolved_url,
                                  const GURL& requestor_url,
                                  ServiceProviderPtr service_provider,
                                  ScopedMessagePipeHandle* shell_handle);
 
   void LoadWithContentHandler(const GURL& content_url,
                               const GURL& requestor_url,
                               const GURL& content_handler_url,
                               URLResponsePtr url_response,
                               ServiceProviderPtr service_provider);
 
@@ skipping 17 matching lines @@
   URLToArgsMap url_to_args_;
 
   base::WeakPtrFactory<ApplicationManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ApplicationManager);
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_APPLICATION_MANAGER_APPLICATION_MANAGER_H_