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

chrome/browser/profiles/profile.h

Index: chrome/browser/profiles/profile.h
===================================================================
--- chrome/browser/profiles/profile.h	(revision 82344)
+++ chrome/browser/profiles/profile.h	(working copy)
@@ -333,13 +333,14 @@
   // happen on the UI thread.
   virtual net::URLRequestContextGetter* GetRequestContext() = 0;
 
-  // Returns the request context appropriate for the given app. If installed_app
-  // is null or installed_app->is_storage_isolated() returns false, this is
-  // equivalent to calling GetRequestContext().
+  // Returns the request context appropriate for the given app. If the renderer

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

"for the given app" -> "for the given renderer process"?

I'm wondering if we should just rename this GetRequestContextForRenderProcess. 
The fact that it bases the decision on whether the process has an app isn't
critical.

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

Done
sure, that sounds like a more appropriate name now.  Done.

+  // process doesn't have an assosicated installed app, or if the installed
+  // app's is_storage_isolated() returns false, this is equivalent to calling
+  // GetRequestContext().
   // TODO(creis): After isolated app storage is no longer an experimental
   // feature, consider making this the default contract for GetRequestContext.
   virtual net::URLRequestContextGetter* GetRequestContextForPossibleApp(
-      const Extension* installed_app) = 0;
+      int renderer_child_id) = 0;
 
   // Returns the request context for media resources asociated with this
   // profile.