ensure layout is initialized soon enough during page load
Categories
(Core :: Layout, enhancement, P3)
Tracking
()
People
(Reporter: smaug, Assigned: smaug)
References
(Blocks 1 open bug)
Details
Assignee | ||
Updated•6 years ago
|
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Assignee | ||
Comment 3•6 years ago
|
||
Comment 4•6 years ago
|
||
Needinfo'ing Henri so he's aware of this bug and can give any thoughts he may have.
Assignee | ||
Comment 5•6 years ago
|
||
I did assign this to myself and I've discussed about this with Henri (on IRC) :)
(In reply to Olli Pettay [:smaug] from comment #5)
I did assign this to myself and I've discussed about this with Henri (on IRC) :)
To recap for others:
- The point at which the parser emits the "start layout" operation relative to the structure of the HTML document comes from the old parser. While I didn't write the relevant code in the old parser, the reason is easy to infer: Typically
<head>
is hidden, so letting layout uselessly react to insertions into<head>
is avoided by starting layout once<body>
is seen. - The parser thread queues up operations and moves them to the main thread notably:
- On
</script>
end tag being parsed. - On
</script>
end tag after the script has finished executing on the main thread (speculatively parsed operations may have queued up on the parser thread). - On end-of-stream.
- On timer.
- On
From the above, we should expect the timer delay to affect the start of layout mainly on pages that don't have a bunch of scripts in head. If the timer delays seem to be affecting the layout start time pages that do have scripts in head, I don't have an immediate explanation.
Assignee | ||
Comment 7•6 years ago
|
||
Yeah, it is not just parser, but also things like bug 1454529 (paint suppression) and what not.
And given the current RefreshDriver setup, we need to get layout up and running very early so that
we don't need to wait for vsync too long when we actually do have some reasonable elements to paint.
Comment 8•6 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #6)
To recap for others:
Thanks, it's very valuable to have this sort of background and useful info captured in bugs rather than disappearing into IRC logs where it's unlikely to be discovered by anyone doing bug archeology.
Comment 9•6 years ago
|
||
(In reply to Olli Pettay [:smaug] from comment #7)
And given the current RefreshDriver setup, we need to get layout up and running very early so that
we don't need to wait for vsync too long when we actually do have some reasonable elements to paint.
I'm unclear on how to square what Henri said in comment 6 about us starting layout once <body> is seen, and your concern that layout is starting too late. Anyway, I hadn't noticed you'd assigned this to yourself. I guess at the point you're looking for layout folks input you'll needinfo us and we can hash out what it is that you're concerned about. :)
Comment 10•6 years ago
|
||
Oh, and can you set a priority for this bug to take it off the triage queue?
Updated•6 years ago
|
Assignee | ||
Comment 11•6 years ago
|
||
Need to get rid of paint suppression timer, bug 1454529.
Updated•2 years ago
|
Description
•