Refactor the two URLRequestContext subclasses in profile.cc into a new shared

ChromeRequestContext class. This will allow us to put browser-specific request
context here rather than in URLRequestContext and eliminates a lot of
duplicate code.

I looked at having two different classes using either inheritance (as proposed
by the existing TODO) or composition, but it seems like there isn't enough
difference between these two classes to justify it.

Removed is_off_the_record() because it wasn't being used anywhere and did a
few other minor code cleanup things.

Review URL: http://codereview.chromium.org/16408

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@7448 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 1 insertions, 6 deletions

diff --git a/net/url_request/url_request_context.h b/net/url_request/url_request_context.h
index 5ea25db..ee05c9d 100644
--- a/net/url_request/url_request_context.h
+++ b/net/url_request/url_request_context.h
@@ -31,8 +31,7 @@ class URLRequestContext :
   URLRequestContext()
       : proxy_service_(NULL),
         http_transaction_factory_(NULL),
-        cookie_store_(NULL),
-        is_off_the_record_(false) {
+        cookie_store_(NULL) {
   }
 
   // Get the proxy service for this context.
@@ -63,9 +62,6 @@ class URLRequestContext :
   // Gets the value of 'Accept-Language' header field.
   const std::string& accept_language() const { return accept_language_; }
 
-  // Returns true if this context is off the record.
-  bool is_off_the_record() { return is_off_the_record_; }
-
   // Do not call this directly.  TODO(darin): extending from RefCounted* should
   // not require a public destructor!
   virtual ~URLRequestContext() {}
@@ -79,7 +75,6 @@ class URLRequestContext :
   net::CookiePolicy cookie_policy_;
   net::AuthCache ftp_auth_cache_;
   std::string user_agent_;
-  bool is_off_the_record_;
   std::string accept_language_;
   std::string accept_charset_;