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

chrome/browser/profiles/profile.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.
 
 // This class gathers state related to a single user profile.
 
 #ifndef CHROME_BROWSER_PROFILES_PROFILE_H_
 #define CHROME_BROWSER_PROFILES_PROFILE_H_
 #pragma once
 
@@ skipping 315 matching lines @@
   virtual fileapi::FileSystemContext* GetFileSystemContext() = 0;
 
   // Returns the BrowserSignin object assigned to this profile.
   virtual BrowserSignin* GetBrowserSignin() = 0;
 
   // Returns the request context information associated with this profile.  Call
   // this only on the UI thread, since it can send notifications that should
   // 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.
   virtual net::URLRequestContextGetter* GetRequestContextForMedia() = 0;
 
   // Returns the request context used for extension-related requests.  This
   // is only used for a separate cookie store currently.
   virtual net::URLRequestContextGetter* GetRequestContextForExtensions() = 0;
 
   // Returns the request context used within an installed app that has
@@ skipping 241 matching lines @@
   bool restored_last_session_;
 
   // Accessibility events will only be propagated when the pause
   // level is zero.  PauseAccessibilityEvents and ResumeAccessibilityEvents
   // increment and decrement the level, respectively, rather than set it to
   // true or false, so that calls can be nested.
   int accessibility_pause_level_;
 };
 
 #endif  // CHROME_BROWSER_PROFILES_PROFILE_H_