Initial support for partitioning cookies for isolated apps.

chrome/browser/net/chrome_url_request_context.cc

 // 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.
 
 #include "chrome/browser/net/chrome_url_request_context.h"
 
 #include "base/message_loop.h"
 #include "base/message_loop_proxy.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_thread.h"
@@ skipping 59 matching lines @@
       : profile_io_data_(profile_io_data) {}
 
   virtual scoped_refptr<ChromeURLRequestContext> Create() {
     return profile_io_data_->GetExtensionsRequestContext();
   }
 
  private:
   const scoped_refptr<const ProfileIOData> profile_io_data_;
 };
 
+// Factory that creates the ChromeURLRequestContext for a given isolated app.
+class FactoryForIsolatedApp : public ChromeURLRequestContextFactory {
+ public:
+  explicit FactoryForIsolatedApp(const ProfileIOData* profile_io_data,
+                                 const Extension* installed_app)
+      : profile_io_data_(profile_io_data),
+        installed_app_(installed_app) {}
+
+  virtual scoped_refptr<ChromeURLRequestContext> Create() {
+    return profile_io_data_->GetIsolatedAppRequestContext(installed_app_);
+  }
+
+ private:
+  const scoped_refptr<const ProfileIOData> profile_io_data_;
+  const scoped_refptr<const Extension> installed_app_;
+};
+
 // Factory that creates the ChromeURLRequestContext for media.
 class FactoryForMedia : public ChromeURLRequestContextFactory {
  public:
   explicit FactoryForMedia(const ProfileIOData* profile_io_data)
       : profile_io_data_(profile_io_data) {
   }
 
   virtual scoped_refptr<ChromeURLRequestContext> Create() {
     return profile_io_data_->GetMediaRequestContext();
   }
@@ skipping 118 matching lines @@
 ChromeURLRequestContextGetter::CreateOriginalForExtensions(
     Profile* profile, const ProfileIOData* profile_io_data) {
   DCHECK(!profile->IsOffTheRecord());
   return new ChromeURLRequestContextGetter(
       profile,
       new FactoryForExtensions(profile_io_data));
 }
 
 // static
 ChromeURLRequestContextGetter*
+ChromeURLRequestContextGetter::CreateOriginalForIsolatedApp(Profile* profile,

[willchan no longer on Chromium, 2011/03/01 23:39:24]

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Functi...
says:
"All parameters should be aligned if possible."

Add this in addition to the Chromium style guide rule from earlier, and it'll
probably make sense to move Profile* to the next line, and do one per line, like
you do for CreateOffTheRecordForIsolatedApp().

[Charlie Reis, 2011/03/03 01:08:05]

Done.

+    const ProfileIOData* profile_io_data, const Extension* installed_app) {
+  DCHECK(!profile->IsOffTheRecord());
+  return new ChromeURLRequestContextGetter(
+      profile,
+      new FactoryForIsolatedApp(profile_io_data, installed_app));
+}
+
+// static
+ChromeURLRequestContextGetter*
 ChromeURLRequestContextGetter::CreateOffTheRecord(
     Profile* profile, const ProfileIOData* profile_io_data) {
   DCHECK(profile->IsOffTheRecord());
   return new ChromeURLRequestContextGetter(
       profile, new FactoryForMain(profile_io_data));
 }
 
 // static
 ChromeURLRequestContextGetter*
 ChromeURLRequestContextGetter::CreateOffTheRecordForExtensions(
     Profile* profile, const ProfileIOData* profile_io_data) {
   DCHECK(profile->IsOffTheRecord());
   return new ChromeURLRequestContextGetter(
       profile, new FactoryForExtensions(profile_io_data));
 }
 
+// static
+ChromeURLRequestContextGetter*
+ChromeURLRequestContextGetter::CreateOffTheRecordForIsolatedApp(
+    Profile* profile,
+    const ProfileIOData* profile_io_data,
+    const Extension* installed_app) {
+  DCHECK(profile->IsOffTheRecord());
+  return new ChromeURLRequestContextGetter(
+      profile, new FactoryForIsolatedApp(profile_io_data, installed_app));
+}
+
 void ChromeURLRequestContextGetter::CleanupOnUIThread() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   // Unregister for pref notifications.
   DCHECK(!registrar_.IsEmpty()) << "Called more than once!";
   registrar_.RemoveAll();
 }
 
 // NotificationObserver implementation.
 void ChromeURLRequestContextGetter::Observe(
     NotificationType type,
@@ skipping 133 matching lines @@
       net::HttpUtil::GenerateAcceptLanguageHeader(accept_language));
 }
 
 void ChromeURLRequestContext::OnDefaultCharsetChange(
     const std::string& default_charset) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   set_referrer_charset(default_charset);
   set_accept_charset(
       net::HttpUtil::GenerateAcceptCharsetHeader(default_charset));
 }