diff options
| author | simonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-07 22:02:39 +0000 |
|---|---|---|
| committer | simonhatch@chromium.org <simonhatch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-07 22:02:39 +0000 |
| commit | ed038bff0dc6d14b38f8c74d89d252b35f105881 (patch) | |
| tree | d2b182617900f6940848950f2cfb52743ab5cc10 /webkit/child | |
| parent | ca6e0a823730e79cf546491f7a548a684920480a (diff) | |
| download | chromium_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.h | 3 |
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 |