Closed
Bug 1480059
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 12253 - Don't re-attach ::first-letter during style recalc.
Categories
(Core :: CSS Parsing and Computation, enhancement, P4)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 12253 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/12253 Details from upstream follow. Rune Lillesveen <futhark@chromium.org> wrote: > Don't re-attach ::first-letter during style recalc. > > Update (create or destroy) the FirstLetterPseudoElement and its style > at the point where the block level element's layout sub-tree is up-to- > date. This means we will always attach the layout tree for the > ::first-letter element in Element::AttachLayoutTree, but the pseudo > element and its style may be created/updated at style recalc time, > layout tree rebuild time, or layout tree attachment time depending on > when we know what will be the LayoutText from which we will format the > first letter if any. > > UpdateFirstLetterPseudoElement is split out from UpdatePseudoElement to > make the code easier to read as the former case has some exceptional > cases. > > We no longer use the pseudo style cache for ::first-letter as we will > now compute the style only once per pass with the correct inheritance > parent. > > Bug: 847218 > Change-Id: I7a1e2a60122891fa38998ff85e566bec0a38b513 > > Reviewed-on: https://chromium-review.googlesource.com/1155591 > WPT-Export-Revision: 6cbbd052941276e659b5aedaa162a904984f014e
Assignee | ||
Updated•6 years ago
|
Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Assignee | ||
Comment 1•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=f52fb48b92c32734429282ab705c5de9a95e76e8
Assignee | ||
Comment 2•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=faa962f963665649c329c01b369e0135da9a1b7a
Assignee | ||
Comment 3•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=705e6bb1c826f0e119f0708a059fd793640ba7da
Assignee | ||
Comment 4•6 years ago
|
||
Ran 1 tests PASS : 1
Assignee | ||
Comment 5•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=4f6e0987025e758ba8092019ed2e34000bf9748c
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/fb7b1227df5b [wpt PR 12253] - Don't re-attach ::first-letter during style recalc., a=testonly
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/fb7b1227df5b
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•