Initial support for partitioning cookies for isolated apps.

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 328 matching lines @@
   virtual const Extension* GetTheme() = 0;
 
   // Returns or creates the ThemeProvider associated with this profile
   virtual BrowserThemeProvider* GetThemeProvider() = 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 URLRequestContextGetter* GetRequestContext() = 0;
 
+  // Returns the request context appropriate for the given app. If installed_app

[willchan no longer on Chromium, 2011/03/08 22:29:14]

Do we really need two new member functions? Is this one temporary while this
feature is experimental, and will go away in the future? I think that if
possible, we should keep it to only one, since this class is large enough
already. But up to you.

[Charlie Reis, 2011/03/09 00:10:55]

Yes, I'd rather get some experience with this experiment before converting all
the existing GetRequestContext calls over.  I think it's a good idea to settle
on this as the default contract in the future.

I've added a comment to that effect.

+  // is null or installed_app->is_storage_isolated() returns false, this is
+  // equivalent to calling GetRequestContext().
+  virtual URLRequestContextGetter* GetRequestContextForPossibleApp(
+      const Extension* installed_app) = 0;
+
   // Returns the request context for media resources asociated with this
   // profile.
   virtual URLRequestContextGetter* GetRequestContextForMedia() = 0;
 
   // Returns the request context used for extension-related requests.  This
   // is only used for a separate cookie store currently.
   virtual URLRequestContextGetter* GetRequestContextForExtensions() = 0;
 
+  // Returns the request context used within an installed app that has
+  // requested isolated storage.
+  virtual URLRequestContextGetter* GetRequestContextForIsolatedApp(
+      const Extension* installed_app) = 0;
+
   // Called by the ExtensionService that lives in this profile. Gives the
   // profile a chance to react to the load event before the EXTENSION_LOADED
   // notification has fired. The purpose for handling this event first is to
   // avoid race conditions by making sure URLRequestContexts learn about new
   // extensions before anything else needs them to know.
   virtual void RegisterExtensionWithRequestContexts(
       const Extension* extension) {}
 
   // Called by the ExtensionService that lives in this profile. Lets the
   // profile clean up its RequestContexts once all the listeners to the
@@ skipping 233 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_