[Net] Propagate priority changes from URLRequest to HttpTransaction

net/http/http_network_transaction.cc

Index: net/http/http_network_transaction.cc
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index 5b120f765ae7da7fdf9e368e4639df27b9a8f175..643244b66eafb2699be330f97b3d274f77d13417 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -385,6 +385,27 @@ UploadProgress HttpNetworkTransaction::GetUploadProgress() const {
   return static_cast<HttpStream*>(stream_.get())->GetUploadProgress();
 }
 
+bool HttpNetworkTransaction::GetLoadTimingInfo(

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

Thanks for fixing my misplacement of this function!

+    LoadTimingInfo* load_timing_info) const {
+  if (!stream_ || !stream_->GetLoadTimingInfo(load_timing_info))
+    return false;
+
+  load_timing_info->proxy_resolve_start =
+      proxy_info_.proxy_resolve_start_time();
+  load_timing_info->proxy_resolve_end = proxy_info_.proxy_resolve_end_time();
+  load_timing_info->send_start = send_start_time_;
+  load_timing_info->send_end = send_end_time_;
+  load_timing_info->receive_headers_end = receive_headers_end_;
+  return true;
+}
+
+void HttpNetworkTransaction::SetPriority(RequestPriority priority) {
+  net_log_.AddEvent(NetLog::TYPE_HTTP_TRANSACTION_SET_PRIORITY,
+                    NetLog::IntegerCallback("priority", priority));

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

I don't think this is an event worth logging.

HttpNetworkTransactions, HttpCache::Transactions, and URLRequests all share the
same BoundNetLog object.  The structure of the log itself makes it clear when
there's a transaction going on and when there isn't.

So we'd basically always see 3 of these events in a row, except when the
priority is set very early on in the process.

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

Okay. I think we should still keep the new one URLRequest, since having the
priority updated is a distinct event overall. Ideally, the only other things
that would log the new priority would be the actual priority queues themselves
(or the owner of the queues), but I haven't plumbed it through yet.  How about I
keep the one in HttpNetworkTransaction (the lowest-layer one) and change the
TODO to remove the log once it's plumbed through?

Also, I think having the priority set early on is the common case.

[mmenke, 2013/03/12 22:58:26]

I agree that we should keep the one in URLRequest.  I don't see the point of
having a temporary one here, unless we have a temporary unit test that checks it
or something.

+  // TODO(akalin): Plumb this through to |stream_request_| and
+  // |stream_|.
+}
+
 void HttpNetworkTransaction::OnStreamReady(const SSLConfig& used_ssl_config,
                                            const ProxyInfo& used_proxy_info,
                                            HttpStreamBase* stream) {
@@ -482,20 +503,6 @@ void HttpNetworkTransaction::OnHttpsProxyTunnelResponse(
   OnIOComplete(ERR_HTTPS_PROXY_TUNNEL_RESPONSE);
 }
 
-bool HttpNetworkTransaction::GetLoadTimingInfo(
-    LoadTimingInfo* load_timing_info) const {
-  if (!stream_ || !stream_->GetLoadTimingInfo(load_timing_info))
-    return false;
-
-  load_timing_info->proxy_resolve_start =
-      proxy_info_.proxy_resolve_start_time();
-  load_timing_info->proxy_resolve_end = proxy_info_.proxy_resolve_end_time();
-  load_timing_info->send_start = send_start_time_;
-  load_timing_info->send_end = send_end_time_;
-  load_timing_info->receive_headers_end = receive_headers_end_;
-  return true;
-}
-
 bool HttpNetworkTransaction::is_https_request() const {
   return request_->url.SchemeIs("https");
 }