Summary: | Preload scanner unhelpful on yahoo.com | ||
---|---|---|---|
Product: | WebKit | Reporter: | James Simonsen <simonjam> |
Component: | Page Loading | Assignee: | Nobody <webkit-unassigned> |
Status: | NEW --- | ||
Severity: | Normal | CC: | ap, gavinp, koivisto, nduca, pmeenan, skyul, tonyg |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Description
James Simonsen
2012-03-20 16:17:08 PDT
> 1. The preload scanner will only preload CSS and JS before first paint. It could've loaded 30 other resources 30 - 80 ms earlier without that restriction. IIRC, the restriction was added to prevent responses from interfering with first paint, which regressed some benchmark. At one point the PreloadScanner would not load any resources in the body while parsing the head. I investigated that in bug 45072 and relaxed that restriction so that the PreloadScanner would always load scripts and stylesheets, but left the restriction in place for images. Details in the bug. We should definitely revisit this as I was only testing with a limited subset of sets and it is certainly possible I missed something. > Requesting all 30 resources took 750 us, so it's not a significant delay. That suggests this logic is intended to prevent us from getting responses too quickly. I propose we implement the throttling on the response side, instead of the request side. Getting the requests out on the wire (or at least to the network stack) sooner is better. Network is often the weakest link. I like that idea a lot. Yeah, it is a good idea to revisit these assumptions. |