[Net] Propagate priority changes from URLRequest to HttpTransaction

net/url_request/url_request_http_job.h

Index: net/url_request/url_request_http_job.h
diff --git a/net/url_request/url_request_http_job.h b/net/url_request/url_request_http_job.h
index 8020625e14c67b4d3ebae22b28ae63a5ab40d04b..cc2ea4d364ced1c1de3ad2a8475d272618f763dc 100644
--- a/net/url_request/url_request_http_job.h
+++ b/net/url_request/url_request_http_job.h
@@ -14,6 +14,7 @@
 #include "base/time.h"
 #include "net/base/auth.h"
 #include "net/base/completion_callback.h"
+#include "net/base/net_export.h"
 #include "net/cookies/cookie_store.h"
 #include "net/http/http_request_info.h"
 #include "net/url_request/url_request_job.h"
@@ -30,7 +31,7 @@ class URLRequestContext;
 
 // A URLRequestJob subclass that is built on top of HttpTransaction.  It
 // provides an implementation for both HTTP and HTTPS.
-class URLRequestHttpJob : public URLRequestJob {
+class NET_EXPORT_PRIVATE URLRequestHttpJob : public URLRequestJob {
  public:
   static URLRequestJob* Factory(URLRequest* request,
                                 NetworkDelegate* network_delegate,
@@ -41,6 +42,8 @@ class URLRequestHttpJob : public URLRequestJob {
                     NetworkDelegate* network_delegate,
                     const HttpUserAgentSettings* http_user_agent_settings);
 
+  virtual ~URLRequestHttpJob();
+
   // Shadows URLRequestJob's version of this method so we can grab cookies.
   void NotifyHeadersComplete();
 
@@ -73,6 +76,7 @@ class URLRequestHttpJob : public URLRequestJob {
   virtual void SetUpload(UploadDataStream* upload) OVERRIDE;
   virtual void SetExtraRequestHeaders(
       const HttpRequestHeaders& headers) OVERRIDE;
+  virtual void SetPriority(RequestPriority priority) OVERRIDE;
   virtual void Start() OVERRIDE;
   virtual void Kill() OVERRIDE;
   virtual LoadState GetLoadState() const OVERRIDE;
@@ -99,6 +103,8 @@ class URLRequestHttpJob : public URLRequestJob {
   virtual HostPortPair GetSocketAddress() const OVERRIDE;
   virtual void NotifyURLRequestDestroyed() OVERRIDE;
 
+  RequestPriority priority_;

[mmenke, 2013/03/20 17:12:59]

Same comment as in the URLRequestFtpJob case.  Ship has kinda sailed here
already on all those other fields being protected, though it actually looks to
me like they can all be private, since nothing seems to inherit from
URLRequestHttpJob.

[akalin, 2013/03/21 01:30:33]

Done, privatized everything I could.

+
   HttpRequestInfo request_info_;
   const HttpResponseInfo* response_info_;
 
@@ -151,8 +157,6 @@ class URLRequestHttpJob : public URLRequestJob {
   class HttpFilterContext;
   class HttpTransactionDelegateImpl;
 
-  virtual ~URLRequestHttpJob();
-
   void RecordTimer();
   void ResetTimer();