[Net] Propagate priority changes from URLRequest to HttpTransaction

net/url_request/url_request_http_job.cc

Index: net/url_request/url_request_http_job.cc
diff --git a/net/url_request/url_request_http_job.cc b/net/url_request/url_request_http_job.cc
index caf344c325363afafb7dd6ed1fa8c42306d93e41..f2e272eee8341a87995b63fc1881c33ba1360f66 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -1578,6 +1578,11 @@ void URLRequestHttpJob::NotifyURLRequestDestroyed() {
   awaiting_callback_ = false;
 }
 
+void URLRequestHttpJob::SetPriority(RequestPriority priority) {

[mmenke, 2013/03/12 18:40:22]

request_info_.priority = priority;?

We may want to separate priority out from request_info_ in a future CL.  I
believe request_info_ is currently constant for the duration of a transaction,
and seems a little unexpected to be changing something that's constant out from
under lower level objects that have access to it, even if we notify them of the
change.

Downside of that is that we'd need to track the priority directly at a couple
levels that we currently just rely on a pointer to the HttpRequestInfo for.

[akalin, 2013/03/12 22:08:28]

Done, added TODO to separate out priority.

+  if (transaction_.get())

[mmenke, 2013/03/12 18:40:22]

nit:  "transaction_" is now preferred.  This file uses .get() a lot
unnecessarily, for legacy reasons, but figure we should stick with the preferred
style for new code.

[akalin, 2013/03/12 22:08:28]

Done.

+    transaction_->SetPriority(priority);
+}
+
 void URLRequestHttpJob::OnDetachRequest() {
   http_transaction_delegate_->OnDetachRequest();
 }