[wpt-sync] Sync PR 16996 - [LayoutNG] OBJECTs with fallback content may not create a new FC.
Categories
(Core :: DOM: Core & HTML, defect, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync upstream])
Sync web-platform-tests PR 16996 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/16996
Details from upstream follow.
Morten Stenshorne <mstensho@chromium.org> wrote:
[LayoutNG] OBJECTs with fallback content may not create a new FC.
Avoid false positives from DefinitelyNewFormattingContext().
If an OBJECT element ends up using its fallback content, we cannot claim
that it's going to establish a new formatting context, just by looking
at the element type. Then the OBJECT behaves just like any generic
element.Always reporting that an OBJECT establishes a new formatting context,
allowed the parent to use LayoutNG, although the OBJECT had children
(e.g. a RUBY element) that require legacy layout. Letting NG and legacy
cooperate in the same block formatting context is something we don't
support.Bug: 966363
Change-Id: Ica9247e9936eb1dbdfc8dabdc96cdb2afdad5bac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1628711
Commit-Queue: Morten Stenshorne \<mstensho@chromium.org>
Commit-Queue: Emil A Eklund \<eae@chromium.org>
Auto-Submit: Morten Stenshorne \<mstensho@chromium.org>
Reviewed-by: Ian Kilpatrick \<ikilpatrick@chromium.org>
Reviewed-by: Emil A Eklund \<eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#663053}
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Assignee | ||
Comment 3•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Comment 5•6 years ago
|
||
bugherder |
Comment 6•6 years ago
|
||
bugherder |
Assignee | ||
Updated•6 years ago
|
Description
•