Create a new URLRequestJobFactory for isolated request contexts.

chrome/browser/profiles/profile_io_data.h

Index: chrome/browser/profiles/profile_io_data.h
diff --git a/chrome/browser/profiles/profile_io_data.h b/chrome/browser/profiles/profile_io_data.h
index 22e381f1a9f61c20bbe2118c0061be7f8702e8f2..36d6f1b3621ff9233caf7819a58aeac2b308157e 100644
--- a/chrome/browser/profiles/profile_io_data.h
+++ b/chrome/browser/profiles/profile_io_data.h
@@ -146,7 +146,8 @@ class ProfileIOData {
     explicit MediaRequestContext(
         chrome_browser_net::LoadTimeStats* load_time_stats);
 
-    void SetHttpTransactionFactory(net::HttpTransactionFactory* http_factory);
+    void SetHttpTransactionFactory(
+        scoped_ptr<net::HttpTransactionFactory> http_factory);

[Charlie Reis, 2012/09/20 17:47:03]

Why pass a scoped_ptr?  I haven't seen that.  Was this broken before?

[awong, 2012/09/20 22:11:00]

It's new functionality as of January. This way the type system expresses
ownership transfer.

 
    private:
     virtual ~MediaRequestContext();
@@ -162,13 +163,16 @@ class ProfileIOData {
         chrome_browser_net::LoadTimeStats* load_time_stats);
 
     void SetCookieStore(net::CookieStore* cookie_store);
-    void SetHttpTransactionFactory(net::HttpTransactionFactory* http_factory);
+    void SetHttpTransactionFactory(
+        scoped_ptr<net::HttpTransactionFactory> http_factory);
+    void SetJobFactory(scoped_ptr<net::URLRequestJobFactory> job_factory);
 
    private:
     virtual ~AppRequestContext();
 
     scoped_refptr<net::CookieStore> cookie_store_;
     scoped_ptr<net::HttpTransactionFactory> http_factory_;
+    scoped_ptr<net::URLRequestJobFactory> job_factory_;
   };
 
   // Created on the UI thread, read on the IO thread during ProfileIOData lazy