Closed
Bug 1521334
Opened 6 years ago
Closed 6 years ago
Ensure parser starts layout soon enough
Categories
(Core :: DOM: HTML Parser, enhancement)
Core
DOM: HTML Parser
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: smaug, Assigned: smaug)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
hsivonen
:
review+
|
Details | Diff | Splinter Review |
In practice, just make the very long timers to fire sooner.
This bug doesn't end up changing anything too often, since in common case is that the layout is anyhow initialized.
https://bugzilla.mozilla.org/show_bug.cgi?id=1517850#c6 has some reasoning, and there are others too.
Attachment #9037811 -
Flags: review?(hsivonen)
Comment on attachment 9037811 [details] [diff] [review]
shorter_parser_timeouts.diff
I trust you've measured the effect.
Attachment #9037811 -
Flags: review?(hsivonen) → review+
Assignee | ||
Comment 2•6 years ago
|
||
I've pushed to try and didn't see regressions. If this improves something, it is metric like first contentful paint, which was just added to raptor.
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d9ae82723499
make parser to flush its tree operation queue sooner, r=hsivonen
Comment 4•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Updated•5 years ago
|
Type: defect → enhancement
You need to log in
before you can comment on or make changes to this bug.
Description
•