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

Unified Diff: webkit/glue/weburlloader_impl.cc

Issue 12243002: Pass WebKit priority changes and parsing messages up to the browser process. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rename IPC Created 7 years, 10 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
Index: webkit/glue/weburlloader_impl.cc
diff --git a/webkit/glue/weburlloader_impl.cc b/webkit/glue/weburlloader_impl.cc
index b64c6a979cf4986583062ac3a2b01341b04d714f..862d819b5921762e7421e8a90d83c03819710fc5 100644
--- a/webkit/glue/weburlloader_impl.cc
+++ b/webkit/glue/weburlloader_impl.cc
@@ -268,6 +268,31 @@ void PopulateURLResponse(
}
}
+net::RequestPriority ConvertWebKitPriorityToNetPriority(
+ const WebURLRequest::Priority& priority) {
+ switch (priority) {
+ case WebURLRequest::PriorityVeryHigh:
+ return net::HIGHEST;
+
+ case WebURLRequest::PriorityHigh:
+ return net::MEDIUM;
+
+ case WebURLRequest::PriorityMedium:
+ return net::LOW;
+
+ case WebURLRequest::PriorityLow:
+ return net::LOWEST;
+
+ case WebURLRequest::PriorityVeryLow:
+ return net::IDLE;
+
+ case WebURLRequest::PriorityUnresolved:
+ default:
+ NOTREACHED();
+ return net::LOW;
+ }
+}
+
} // namespace
// WebURLLoaderImpl::Context --------------------------------------------------
@@ -285,6 +310,7 @@ class WebURLLoaderImpl::Context : public base::RefCounted<Context>,
void Cancel();
void SetDefersLoading(bool value);
+ void DidChangePriority(WebURLRequest::Priority new_priority);
void Start(
const WebURLRequest& request,
ResourceLoaderBridge::SyncLoadResponse* sync_load_response,
@@ -353,6 +379,13 @@ void WebURLLoaderImpl::Context::SetDefersLoading(bool value) {
bridge_->SetDefersLoading(value);
}
+void WebURLLoaderImpl::Context::DidChangePriority(
+ WebURLRequest::Priority new_priority) {
+ if (bridge_.get())
+ bridge_->DidChangePriority(
+ ConvertWebKitPriorityToNetPriority(new_priority));
+}
+
void WebURLLoaderImpl::Context::Start(
const WebURLRequest& request,
ResourceLoaderBridge::SyncLoadResponse* sync_load_response,
@@ -432,7 +465,8 @@ void WebURLLoaderImpl::Context::Start(
request_info.requestor_pid = request.requestorProcessID();
request_info.request_type =
ResourceType::FromTargetType(request.targetType());
- request_info.priority = request.priority();
+ request_info.priority =
+ ConvertWebKitPriorityToNetPriority(request.priority());
request_info.appcache_host_id = request.appCacheHostID();
request_info.routing_id = request.requestorID();
request_info.download_to_file = request.downloadToFile();
@@ -783,4 +817,8 @@ void WebURLLoaderImpl::setDefersLoading(bool value) {
context_->SetDefersLoading(value);
}
+void WebURLLoaderImpl::didChangePriority(WebURLRequest::Priority new_priority) {
+ context_->DidChangePriority(new_priority);
+}
+
} // namespace webkit_glue

Powered by Google App Engine
This is Rietveld 408576698