1876684 - Export more MathML reftests to WPT

Status: RESOLVED FIXED

(Core :: MathML, task)

Comment 1

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Export mfrac-* reftests to WPT. r=emilio

mfrac-A-5.html: Removed, This is testing linethickness="100%" but is
already covered by frac-linethickness-001.html.

mfrac-A-4.html/mfrac-A-8.html: Removed, these are testing horizontal
alignment of the numerator/denominator but this is already covered
by frac-numalign-denomalign-001.html or frac-1.html.

mfrac-B-2.html/mfrac-B-3.html/mfrac-C-2.html/mfrac-D-2.html: These
are testing invalidation after dynamic updates of the linethickness
attributes. They are already covered by frac-linethickness-002.html but
it's a testharness and does not cover values above/below 100%. So
export them to frac-linethickness-*.html.

mfrac-E-1.html: Moved to frac-created-dynamically-4.html ; as that's
similar but slightly different to other existing tests. The whitespace
around variables does not seem important for this test so remove it.

Comment 2

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Export mspace-1.html reftests to WPT. r=emilio

This is just a basic reftest for the mspace element that does not seem
to have equivalent in WPT.

Comment 3

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Export mo-invisibleoperators-*.html reftests to WPT. r=emilio

These tests are testing Gecko's special handling of invisible operators
introduced in bug 969867 and bug 522393. mo-invisibleoperators-2.html
is just dynamically replacing the invisible operators into ASCII text
and is compatible with MathML-Core implementations.
mo-invisibleoperators.html verifies that invisible operators are really
invisible (no size, background, foreground) which is also compatible
with MathML-Core implementations, as long as the default math fonts
really have empty glyphs for these operators, which should be the case
for modern font sets.

Comment 4

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Export MathML dir-* tests to WPT. r=emilio

• dir-3.html/dir-5.html: These are approximative verification of overall
  RTL directionality using "covering rectangles". Export them, forcing
  the Ahem font to make them a bit more robust.

• dir-4-ref.html: Removed (the test was removed in bug 1783995).

Comment 5

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Convert mo@accent tests to internal WPT tests. r=emilio

This should be removed (see bug 1790548 and bug 1636428) but convert
them as internal WPT ones for now.

Comment 6

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Export some dynamic reftests to WPT. r=emilio

• mo-movablelimits-dynamic.html: This is the same as the WPT test
  mathml/presentation-markup/operators/mo-movablelimits-dynamic.html

• munderover-accentunder-dynamic/munderover-accent-dynamic: There
  does not seem to be any equivalent tests. It's possible the tests
  need some refinement to make sure the attributes have an effect on
  this testcase, but for now it's safe to just export them as is.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/871aedcfa184
Export mfrac-* reftests to WPT. r=emilio
https://hg.mozilla.org/integration/autoland/rev/09b93cf63a53
Export mspace-1.html reftests to WPT. r=emilio
https://hg.mozilla.org/integration/autoland/rev/3225b8ea16e4
Export mo-invisibleoperators-*.html reftests to WPT. r=emilio

Comment 8

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

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/44312 for changes under testing/web-platform/tests

Comment 9

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/871aedcfa184
https://hg.mozilla.org/mozilla-central/rev/09b93cf63a53
https://hg.mozilla.org/mozilla-central/rev/3225b8ea16e4

Comment 10

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

Upstream PR merged by moz-wptsync-bot

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/eb00222e5577
Export MathML dir-* tests to WPT. r=emilio
https://hg.mozilla.org/integration/autoland/rev/6c31dd56c2b2
Convert mo@accent tests to internal WPT tests. r=emilio
https://hg.mozilla.org/integration/autoland/rev/607f0b698ed2
Export some dynamic reftests to WPT. r=emilio

Comment 12

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

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/44351 for changes under testing/web-platform/tests

Comment 13

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/eb00222e5577
https://hg.mozilla.org/mozilla-central/rev/6c31dd56c2b2
https://hg.mozilla.org/mozilla-central/rev/607f0b698ed2

Comment 14

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

Upstream PR was closed without merging

Comment 15

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Convert operator-1.xhtml to internal WPT test. r=emilio

operator-1.xhtml verifies negative values mo@lspace/rspace but support
and interpretation in MathML Core is unclear at this point [1] so keep
it as internal for now. This also introduces a new negative-lengths
subdirectory for handling similar tests.

[1] https://github.com/w3c/mathml-core/issues/132

Comment 16

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

Upstream PR merged by moz-wptsync-bot

Comment 17

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

Upstream PR merged by moz-wptsync-bot

Comment 18

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Export link-001.html as a tentative WPT. r=emilio

This tests color of <mtext href> links. Although links were mentioned
in a previous draft of MathML Core, they have been removed from the
current version and adding them back is pending consensus. However,
upstream WPT still contains tests for MathML links so it makes sense
to have all of these tentative tests.

This commit exports link-001.html as a tentative mismatch reftest
with proper renaming (note that the <link rel="mismatch"> refers to a
non-existent link-color-001-ref.xhtml file).

Similarly, all the existing tests assuming support for href are
renamed as 'tentative' tests.

Finally, we also remove a few annotations from bug 1798871.

There is no behavior change.

Comment 19

[Frédéric Wang (:fredw)]

Attachment: Bug 1876684 - Export some MathML mpadded reftests to WPT. r=emilio

• mpadded-1/2/3.html, mpadded-1-2.html: These are testing visual
  rendering of two nested mpadded elements with basic attribute values
  that are accepted by MathML Core.

• mpadded-4.html: This is testing mpadded attribute with pseudo
  units, which are not supported by MathML Core, so convert to
  internal WPT test.

• mpadded-5/6: These are testing negative values of some mpadded
  attributes whose interpretation or support is unclear in MathML
  Core. So convert to internal WPT tests for now.

Comment 20

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/919bd2893087
Convert operator-1.xhtml to internal WPT test. r=emilio
https://hg.mozilla.org/integration/autoland/rev/5ec8a1a09553
Export link-001.html as a tentative WPT. r=emilio
https://hg.mozilla.org/integration/autoland/rev/a55c77e16ea8
Export some MathML mpadded reftests to WPT. r=emilio

Comment 21

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

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/44391 for changes under testing/web-platform/tests

Comment 22

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/919bd2893087
https://hg.mozilla.org/mozilla-central/rev/5ec8a1a09553
https://hg.mozilla.org/mozilla-central/rev/a55c77e16ea8

Comment 23

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

Upstream PR merged by moz-wptsync-bot

Comment 24

[Frédéric Wang (:fredw)]

Opened bug 1882817 as a follow-up.