[wpt-sync] Sync PR 16817 - [LayoutNG] Don't force legacy layout on ::first-letter pseudos.
Categories
(Core :: DOM: Editor, defect, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync upstream])
Sync web-platform-tests PR 16817 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/16817
Details from upstream follow.
Morten Stenshorne <mstensho@chromium.org> wrote:
[LayoutNG] Don't force legacy layout on ::first-letter pseudos.
Due to the complicated behavior of ::first-letter elements, where the
placement of the ::first-letter DOM node in the DOM tree doesn't
correspond with where we place the LayoutObject in the layout tree, it's
tricky to figure out whether it's part of a subtree that requires us to
force legacy layout or not. So just skip checking. The ::first-letter
element itself can never force legacy layout on its own, so we'll just
do whatever the parent layout object does.Solution developed together with futhark@chromium.org
Bug: 962090
Change-Id: Ie112561bbd58c0c6f822f500247dbbaa1c8a8118
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1611602
Commit-Queue: Morten Stenshorne \<mstensho@chromium.org>
Reviewed-by: Rune Lillesveen \<futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#659529}
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f7f3360b3f24c00331d0b8349571918da974110
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=15ed7839d3600554f2df18f1a2ec5e23c9995f03
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=760db63d1f87a6705b4af143a45ab1b7d2f911cc
Assignee | ||
Comment 4•5 years ago
|
||
Failed to get results from try push
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/e36b71d063ed [wpt PR 16817] - [LayoutNG] Don't force legacy layout on ::first-letter pseudos., a=testonly
Comment 6•5 years ago
|
||
bugherder |
Comment 7•5 years ago
|
||
bugherder |
Assignee | ||
Updated•5 years ago
|
Description
•