Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(471)

Unified Diff: net/http/http_network_transaction.cc

Issue 12701011: [Net] Propagate priority changes from URLRequest to HttpTransaction (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix leaks Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/http/http_network_transaction.h ('k') | net/http/http_stream_factory_impl_job.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_network_transaction.cc
diff --git a/net/http/http_network_transaction.cc b/net/http/http_network_transaction.cc
index 4393c57d20f9307f032b2cbe5b409c3cc244b62c..3c2596dce67c18d982118bbaec68f536393518f5 100644
--- a/net/http/http_network_transaction.cc
+++ b/net/http/http_network_transaction.cc
@@ -387,6 +387,26 @@ UploadProgress HttpNetworkTransaction::GetUploadProgress() const {
return static_cast<HttpStream*>(stream_.get())->GetUploadProgress();
}
+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;
+}
+
+void HttpNetworkTransaction::SetPriority(RequestPriority priority) {
+ priority_ = priority;
+ // 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) {
@@ -484,20 +504,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");
}
« no previous file with comments | « net/http/http_network_transaction.h ('k') | net/http/http_stream_factory_impl_job.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698