Skip to content

Commit

Permalink
Bug 1876684 - Convert operator-1.xhtml to internal WPT test. r=emilio
Browse files Browse the repository at this point in the history
operator-1.xhtml verifies negative values molspace/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] w3c/mathml-core#132

Differential Revision: https://phabricator.services.mozilla.com/D200430

UltraBlame original commit: 919bd2893087e81f586cfc58c6d67a9629739551
  • Loading branch information
marco-c committed Feb 8, 2024
1 parent 4a82c9c commit 25ddc8c
Show file tree
Hide file tree
Showing 7 changed files with 23 additions and 11 deletions.
9 changes: 0 additions & 9 deletions layout/reftests/mathml/operator-1.xhtml

This file was deleted.

1 change: 0 additions & 1 deletion layout/reftests/mathml/reftest.list
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ random-if(gtkWidget) == mpadded-9.html mpadded-9-ref.html # bug 1309430
== munder-mover-align-accent-false.html munder-mover-align-accent-false-ref.html
== munderover-empty-scripts.html munderover-empty-scripts-ref.html
== mo-glyph-size.html mo-glyph-size-ref.html
== operator-1.xhtml operator-1-ref.xhtml
fuzzy(0-1,0-80) fuzzy-if(Android,0-255,0-105) fuzzy-if(gtkWidget,0-255,0-136) skip-if(winWidget) == multiscripts-1.html multiscripts-1-ref.html # Windows: bug 1314684; Android: bug 1392254; Linux: bug 1599638
== mathml-mmultiscript-base.html mathml-mmultiscript-base-ref.html
== mathml-mmultiscript-mprescript.html mathml-mmultiscript-mprescript-ref.html
Expand Down
4 changes: 4 additions & 0 deletions testing/web-platform/mozilla/tests/mathml/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,10 @@ any specification:
See [bug 1790548](https://bugzilla.mozilla.org/1790548)
and [bug 1636428](https://bugzilla.mozilla.org/1636428).

- `negative-lengths`: Tests for negative lengths, for which support or
interpretation is unclear in current version of MathML Core.
See [issue 132](https://github.com/w3c/mathml-core/issues/132).

- `tables`: Tests for
[table features](https://www.w3.org/TR/MathML3/chapter3.html#presm.tabmat)
that are in the initial version of MathML Core.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,6 @@
<mn>3</mn>
<mo>+</mo>
<mn>2</mn>
</math>
</math>
</body>
</html>
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>negative lspace/rspace</title>
<link rel="match" href="negative-lspace-rspace-001-ref.xhtml"/>
<link rel="assert" href="Verify that negative lspace/rspace are ignored."/>
<link rel="help" href="https://bugzilla.mozilla.org/show_bug.cgi?id=411227"/>
<link rel="help" href="https://github.com/w3c/mathml-core/issues/132"/>
</head>
<body>
<math xmlns="http://www.w3.org/1998/Math/MathML">
<mn>3</mn>
<mo lspace="-5pt" rspace="-4pt">+</mo>
<mn>2</mn>
</math>
</body>
</html>
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@
<title>negative mspace</title>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
<link rel="match" href="negative-mspace-1-ref.html"/>
<link rel="assert" href="Verify that negative width on an mspace element causes overlap of previous and next siblings."/>
<link rel="help" href="https://github.com/w3c/mathml-core/issues/132"/>
</head>
<body>
<p>
Expand Down

0 comments on commit 25ddc8c

Please sign in to comment.