diff options
| author | simonjam@chromium.org <simonjam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-05 19:45:32 +0000 |
|---|---|---|
| committer | simonjam@chromium.org <simonjam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-05 19:45:32 +0000 |
| commit | 102ab68eeb932e56276abd727b2b850e29abcd0b (patch) | |
| tree | 85c8a8726e363cff02c86cec70bd00fa6b12b040 /webkit | |
| parent | 185c0345eceb16301eac0b2a129382517f6acf04 (diff) | |
| download | chromium_src-102ab68eeb932e56276abd727b2b850e29abcd0b.zip chromium_src-102ab68eeb932e56276abd727b2b850e29abcd0b.tar.gz chromium_src-102ab68eeb932e56276abd727b2b850e29abcd0b.tar.bz2 | |
Use WebKit's resource priorities instead of computing our own.
This has no effect on PLT, Speed Index, TTFB, or first paint.
This simply allows us to benefit from improvements to WebKit's prioritization. For instance, deferred scripts should be lower priority than blocking scripts. We'll likely use the priority to deprioritize preloads too.
In practical terms, the only significant change is that WebKit prioritizes CSS above JS and fonts. That didn't affect the metrics.
BUG=None
Review URL: https://chromiumcodereview.appspot.com/12045105
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180759 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit')
| -rw-r--r-- | webkit/glue/resource_loader_bridge.cc | 1 | ||||
| -rw-r--r-- | webkit/glue/resource_loader_bridge.h | 3 | ||||
| -rw-r--r-- | webkit/glue/weburlloader_impl.cc | 1 |
3 files changed, 5 insertions, 0 deletions
diff --git a/webkit/glue/resource_loader_bridge.cc b/webkit/glue/resource_loader_bridge.cc index 6c20f12..dad32d7 100644 --- a/webkit/glue/resource_loader_bridge.cc +++ b/webkit/glue/resource_loader_bridge.cc @@ -51,6 +51,7 @@ ResourceLoaderBridge::RequestInfo::RequestInfo() load_flags(0), requestor_pid(0), request_type(ResourceType::MAIN_FRAME), + priority(WebKit::WebURLRequest::PriorityUnresolved), request_context(0), appcache_host_id(0), routing_id(0), diff --git a/webkit/glue/resource_loader_bridge.h b/webkit/glue/resource_loader_bridge.h index d20040f..840c802 100644 --- a/webkit/glue/resource_loader_bridge.h +++ b/webkit/glue/resource_loader_bridge.h @@ -246,6 +246,9 @@ class ResourceLoaderBridge { // load, or a sub objects load. ResourceType::Type request_type; + // Indicates the priority of this request, as determined by WebKit. + WebKit::WebURLRequest::Priority priority; + // Used for plugin to browser requests. uint32 request_context; diff --git a/webkit/glue/weburlloader_impl.cc b/webkit/glue/weburlloader_impl.cc index 127dbba..b64c6a9 100644 --- a/webkit/glue/weburlloader_impl.cc +++ b/webkit/glue/weburlloader_impl.cc @@ -432,6 +432,7 @@ 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.appcache_host_id = request.appCacheHostID(); request_info.routing_id = request.requestorID(); request_info.download_to_file = request.downloadToFile(); |