Don't hold the installed app in BrowserProcessRenderHost, since that's in content layer now. Ext...

chrome/browser/extensions/extension_service.h

 // Copyright (c) 2011 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 CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 139 matching lines @@
   // Determine if the downloaded extension came from the theme mini-gallery,
   // Used to test if we need to show the "Loading" dialog for themes.
   static bool IsDownloadFromMiniGallery(const GURL& download_url);
 
   // Returns the Extension of hosted or packaged apps, NULL otherwise.
   const Extension* GetInstalledApp(const GURL& url);
 
   // Returns whether the URL is from either a hosted or packaged app.
   bool IsInstalledApp(const GURL& url);
 
+  // Associates a renderer process with the given installed app.
+  void SetInstalledAppForRenderer(const Extension* app, int renderer_child_id);

[Charlie Reis, 2011/04/21 00:25:12]

Let's swap these arguments so it's more like a traditional put(key, value) call.

[jam, 2011/04/21 06:52:19]

Done.

+
+  // If the renderer is hosting an installed app, returns it, otherwise returns
+  // NULL.
+  const Extension* GetInstalledAppForRenderer(int renderer_child_id);
+
   // Attempts to uninstall an extension from a given ExtensionService. Returns
   // true iff the target extension exists.
   static bool UninstallExtensionHelper(ExtensionService* extensions_service,
                                        const std::string& extension_id);
 
   // Constructor stores pointers to |profile| and |extension_prefs| but
   // ownership remains at caller.
   ExtensionService(Profile* profile,
                    const CommandLine* command_line,
                    const FilePath& install_directory,
@@ skipping 385 matching lines @@
 
   // Used to quickly check if an extension was terminated.
   std::set<std::string> terminated_extension_ids_;
 
   // Hold the set of pending extensions.
   PendingExtensionManager pending_extension_manager_;
 
   // The map of extension IDs to their runtime data.
   ExtensionRuntimeDataMap extension_runtime_data_;
 
+  // Holds a map between renderer process IDs that are associated with an
+  // installed app and their app.
+  typedef std::map<int, scoped_refptr<const Extension> > InstalledAppMap;
+  InstalledAppMap installed_app_hosts_;
+
   // The full path to the directory where extensions are installed.
   FilePath install_directory_;
 
   // Whether or not extensions are enabled.
   bool extensions_enabled_;
 
   // Whether to notify users when they attempt to install an extension.
   bool show_extensions_prompts_;
 
   // The backend that will do IO on behalf of this instance.
@@ skipping 66 matching lines @@
   NaClModuleInfoList nacl_module_list_;
 
   FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
                            InstallAppsWithUnlimtedStorage);
   FRIEND_TEST_ALL_PREFIXES(ExtensionServiceTest,
                            InstallAppsAndCheckStorageProtection);
   DISALLOW_COPY_AND_ASSIGN(ExtensionService);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_H_