summaryrefslogtreecommitdiffstats
path: root/webkit/child
diff options
context:
space:
mode:
authorsimonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-04-07 22:02:39 +0000
committersimonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-04-07 22:02:39 +0000
commited038bff0dc6d14b38f8c74d89d252b35f105881 (patch)
treed2b182617900f6940848950f2cfb52743ab5cc10 /webkit/child
parentca6e0a823730e79cf546491f7a548a684920480a (diff)
downloadchromium_src-ed038bff0dc6d14b38f8c74d89d252b35f105881.zip
chromium_src-ed038bff0dc6d14b38f8c74d89d252b35f105881.tar.gz
chromium_src-ed038bff0dc6d14b38f8c74d89d252b35f105881.tar.bz2
This CL adds an intra-priority level sorting value for finer grained control of resource loading. The blink side of this patch sorts visible images by estimated on screen real-estate, preferring to download images with more visual impact first. Ie. On a page with a large banner, and dozens of small icons, the large banner will most likely be scheduled ahead of the others.
In the future, we can also use this to experiment with other signals, ie. prioritize by distance to viewport. This needs https://codereview.chromium.org/153983004/ to land first. In our tests, this was a nice win on speed-index for desktop, with a gain of about ~2-3%. BUG= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260665 Review URL: https://codereview.chromium.org/146333004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@262204 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/child')
-rw-r--r--webkit/child/resource_loader_bridge.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/webkit/child/resource_loader_bridge.h b/webkit/child/resource_loader_bridge.h
index 0b877b4..51cdf5f 100644
--- a/webkit/child/resource_loader_bridge.h
+++ b/webkit/child/resource_loader_bridge.h
@@ -58,7 +58,8 @@ class ResourceLoaderBridge {
// Call this method when the priority of the requested resource changes after
// Start() has been called. This method may only be called after a successful
// call to the Start method.
- virtual void DidChangePriority(net::RequestPriority new_priority) = 0;
+ virtual void DidChangePriority(net::RequestPriority new_priority,
+ int intra_priority_value) = 0;
// Call this method to load the resource synchronously (i.e., in one shot).
// This is an alternative to the Start method. Be warned that this method