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

Issue 2411663002: prerender: Don't set the priority to net::IDLE on Android. (Closed)

Created:
4 years, 2 months ago by Benoit L
Modified:
4 years, 2 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2883
Project:
chromium
Visibility:
Public.

Description

prerender: Don't set the priority to net::IDLE on Android. Resource requests issued by prerendering renderers have their priority set to net::IDLE. This causes issues with content::ResourceScheduler, leading to very poor loading performance for prerendered content. This simply disables the prioritization on Android, as this is where the priority lowering is the most problematic, and where potential downsides from this patch are the most limited. This is a temporary fix. BUG=652746 Review-Url: https://codereview.chromium.org/2399973003 Cr-Commit-Position: refs/heads/master@{#424131} (cherry picked from commit 915cfe1d23cf249cff0b51b6f2a7434035d5e6c5) Committed: https://chromium.googlesource.com/chromium/src/+/f49bbc55974f5e09484a38917d37877c3ef71251

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -7 lines) Patch
M chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc View 4 chunks +18 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Benoit L
4 years, 2 months ago (2016-10-11 11:39:01 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
f49bbc55974f5e09484a38917d37877c3ef71251.

Powered by Google App Engine
This is Rietveld 408576698