1724001 - [wpt-sync] Sync PR 29893 - [mathml] Use embellished operator properties for underover gaps

Status: RESOLVED FIXED

(Core :: MathML, task, P4)

Description

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

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

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

b'Fr\xc3\xa9d\xc3\xa9ric Wang <[e-mail]>' wrote:

[mathml] Use embellished operator properties for underover gaps

The MathML Core spec [1] [2] contains the following cases for
underover layout:

1. The base is an embellished operator with the largeop property.
2. The base is an embellished operator with the stretchy property
   and stretch axis inline.

Currently, the underover algorithm only checks if the base is
an \<mo> element. This CL modifies to instead check whether it is
an embellished operators and if that's the case retrieve these
properties from the core operator.

New WPT tests are added to cover the changes from this CL:

• underover-parameters-and-embellished-operator-1.html is a copy
  of underover-parameters-and-operator-1.html with \<mo>
  embellished with an \<mrow>. It checks (1) above.
• underover-parameters-and-embellished-operator-2.html is a copy
  of underover-parameters-and-operator-2.html with \<mo>
  embellished with an \<mrow>. It checks (2) above.

[1] https://w3c.github.io/mathml-core/#base-with-underscript
[2] https://w3c.github.io/mathml-core/#base-with-overscript

Bug: 6606, 1124298
Change-Id: I2f30ed19cceacb7e2fbd70f83e2e81af9b0537c8
Reviewed-on: https://chromium-review.googlesource.com/3071438
WPT-Export-Revision: 774799f1312373625f721211765ca9bc2a854af3

Comment 1

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=aec369ca29f2d04471f12c764c4566c3a8c56e6b

Comment 2

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

CI Results

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

Total 2 tests and 4 subtests

Status Summary

Firefox

OK : 2
PASS: 4
FAIL: 4

Chrome

OK : 2
FAIL: 8

Safari

OK : 2
FAIL: 8

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/mathml/presentation-markup/scripts/underover-parameters-and-embellished-operator-2.html
StretchStackBottomShiftDown: FAIL (Chrome: FAIL, Safari: FAIL)
StretchStackGapBelowMin: FAIL (Chrome: FAIL, Safari: FAIL)
StretchStackTopShiftUp: FAIL (Chrome: FAIL, Safari: FAIL)
StretchStackGapAboveMin: FAIL (Chrome: FAIL, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5fe144b21c82
[wpt PR 29893] - [mathml] Use embellished operator properties for underover gaps, a=testonly
https://hg.mozilla.org/integration/autoland/rev/31bf96933b2d
[wpt PR 29893] - Update wpt metadata, a=testonly

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/5fe144b21c82
https://hg.mozilla.org/mozilla-central/rev/31bf96933b2d