Open Bug 1663886 Opened 5 years ago Updated 5 years ago

[wpt-sync] Sync PR 25447 - Make DefinitelyNewFormattingContext return true for SVG foreignObject

Categories

(Core :: MathML, task, P4)

task

Tracking

()

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 25447 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/25447
Details from upstream follow.

b'Fr\xc3\xa9d\xc3\xa9ric Wang <fwang@igalia.com>' wrote:

Make DefinitelyNewFormattingContext return true for SVG foreignObject

An SVG foreignObject element currently always forces legacy layout [1].
Additionally, when such an element has visible overflow,
DefinitelyNewFormattingContext returns a false negative, causing all
the siblings of the SVG root to also fallback to legacy layout.
However, SVG foreignObject elements always establish a new formatting
context since they are treated as absolutely-positioned [2]. This CL
fixes the bug by making DefinitelyNewFormattingContext always return
true for an SVG foreignObject element.

The fix is verified using a MathML test copied from integration-4.html,
the one used for MathML-in-foreignObject support [1], with the
assumption that MathML will never be implemented for legacy layout.

Bug: 989920, 6606

[1] crbug.com/989916
[2] https://svgwg.org/svg2-draft/embedded.html#Placement

Change-Id: I7f7cde90da728f1b1959728d39fed2e2070888f9
Reviewed-on: https://chromium-review.googlesource.com/2400378
WPT-Export-Revision: 462cf65945eeafa3c758a4f2682e49a038dcec54

Component: web-platform-tests → MathML
Product: Testing → Core

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 2 subtests

Status Summary

Firefox

OK : 1
PASS: 2

Chrome

OK : 1
FAIL: 2

Safari

OK : 1
PASS: 2

Links

GitHub PR Head
GitHub PR Base

You need to log in before you can comment on or make changes to this bug.