Add support for UA spoofing, and spoof Safari's UA string when loading URLs...

net/url_request/url_request_context.h

 // Copyright (c) 2006-2008 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 represents contextual information (cookies, cache, etc.)
 // that's useful when processing resource requests.
 // The class is reference-counted so that it can be cleaned up after any
 // requests that are using it have been completed.
 
 #ifndef NET_URL_REQUEST_URL_REQUEST_CONTEXT_H_
 #define NET_URL_REQUEST_URL_REQUEST_CONTEXT_H_
 
-#include <string>
-
-#include "base/basictypes.h"
 #include "base/ref_counted.h"
 #include "base/scoped_ptr.h"
+#include "base/string_util.h"
 #include "net/base/cookie_policy.h"
 #include "net/ftp/ftp_auth_cache.h"
 #include "net/http/http_transaction_factory.h"
 
 namespace net {
 class CookieMonster;
 class ProxyService;
 }
 
 // Subclass to provide application-specific context for URLRequest instances.
@@ skipping 18 matching lines @@
 
   // Gets the cookie store for this context.
   net::CookieMonster* cookie_store() { return cookie_store_; }
 
   // Gets the cookie policy for this context.
   net::CookiePolicy* cookie_policy() { return &cookie_policy_; }
 
   // Gets the FTP authentication cache for this context.
   net::FtpAuthCache* ftp_auth_cache() { return &ftp_auth_cache_; }
 
-  // Gets the UA string to use for this context.
-  const std::string& user_agent() const { return user_agent_; }
-
   // Gets the value of 'Accept-Charset' header field.
   const std::string& accept_charset() const { return accept_charset_; }
 
   // Gets the value of 'Accept-Language' header field.
   const std::string& accept_language() const { return accept_language_; }
 
-  // Do not call this directly.  TODO(darin): extending from RefCounted* should
-  // not require a public destructor!
+  // Gets the UA string to use for the given URL.  If the given URL is not a
+  // valid URL, then the default UA string should be returned.  Subclasses

[wtc, 2009/01/27 19:19:24]

It's not clear what you meant by the "default UA string".
Is it the empty string that the base class returns?

Update: after seeing how you use this method in
url_request_inet_job.cc, I understand what you meant.
You may want to rephrase the comment like this:
    Pass an invalid URL (such as GURL()) to get the
    default UA string.
This stresses the fact that it is the proper way to
get the default UA string, rather than an error
handling mode.

[darin (slow to review), 2009/01/27 19:52:06]

Nice suggestion, thanks.

+  // should override this method to provide a UA string.
+  virtual const std::string& GetUserAgent(const GURL& url) const {
+    return EmptyString();
+  }
+
+ protected:
+  friend class base::RefCountedThreadSafe<URLRequestContext>;
+
   virtual ~URLRequestContext() {}

[wtc, 2009/01/27 19:19:24]

Just curious: can we make the destructor private?

[darin (slow to review), 2009/01/27 19:52:06]

yes!

[darin (slow to review), 2009/01/27 20:00:11]

actually, nevermind.  a private destructor prevents inheritance.

 
- protected:
   // The following members are expected to be initialized and owned by
   // subclasses.
   net::ProxyService* proxy_service_;
   net::HttpTransactionFactory* http_transaction_factory_;
   net::CookieMonster* cookie_store_;
   net::CookiePolicy cookie_policy_;
   net::FtpAuthCache ftp_auth_cache_;
-  std::string user_agent_;
   std::string accept_language_;
   std::string accept_charset_;
 
  private:
-  DISALLOW_EVIL_CONSTRUCTORS(URLRequestContext);
+  DISALLOW_COPY_AND_ASSIGN(URLRequestContext);
 };
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_CONTEXT_H_
-