Closed
Bug 1438911
Opened 6 years ago
Closed 6 years ago
Consider not doing charset changes off a runnable.
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(1 file)
So I was looking at the full-doc restyles that stylo-chrome does, and the first one is the one posted from the charset runnable. This runnable was added in bug 703133. Gecko used to do a sync style reconstruct here, and this used to run out of an observer topic and what not. Now style reconstructs, at least for stylo, are async. Annoying bits here are multiple, but in chrome at least when the charset changes we don't have any frame or any style, so a PostRebuildAllStyleDataEvent would boil down to nothing, as it should. I think we can try to make it use other mechanism that doesn't defer this to the next event loop bit, where we'd actually have style data and we'd need to restyle the whole thing.
Assignee | ||
Comment 1•6 years ago
|
||
So this is probably harmless for stylo-chrome, I guess, because the document is resource://gre-resources/hiddenWindow.html, which is basically empty. But it may be worth doing for normal content docs.
No longer blocks: 1420423
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•6 years ago
|
||
Green: https://treeherder.mozilla.org/#/jobs?repo=try&revision=bf5204928819857b26f3bb1536c7ba537e4b5b8f
Assignee: nobody → emilio
Comment 4•6 years ago
|
||
mozreview-review |
Comment on attachment 8951706 [details] Bug 1438911: Avoid doing changeset changes off a runnable for stylo. https://reviewboard.mozilla.org/r/220982/#review227260
Attachment #8951706 -
Flags: review?(bzbarsky) → review+
Pushed by ecoal95@gmail.com: https://hg.mozilla.org/integration/autoland/rev/f21e0c72bebf Avoid doing changeset changes off a runnable for stylo. r=bz
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f21e0c72bebf
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Assignee | ||
Updated•5 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•