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] w3c/mathml-core#132 Differential Revision: https://phabricator.services.mozilla.com/D200430
erichdongubler-mozilla · Feb 3, 2024 · 4c16f3d · 4c16f3d
1 parent 57e9ce5
commit 4c16f3d
Show file tree

Hide file tree

Showing 7 changed files with 23 additions and 11 deletions.
diff --git a/layout/reftests/mathml/operator-1.xhtml b/layout/reftests/mathml/operator-1.xhtml
diff --git a/layout/reftests/mathml/reftest.list b/layout/reftests/mathml/reftest.list
@@ -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

diff --git a/testing/web-platform/mozilla/tests/mathml/README.md b/testing/web-platform/mozilla/tests/mathml/README.md
@@ -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.

diff --git a/layout/reftests/mathml/operator-1-ref.xhtml → ...gths/negative-lspace-rspace-001-ref.xhtml b/layout/reftests/mathml/operator-1-ref.xhtml → ...gths/negative-lspace-rspace-001-ref.xhtml
@@ -4,6 +4,6 @@
     <mn>3</mn>
     <mo>+</mo>
     <mn>2</mn>
-  </math> 
+  </math>
 </body>
 </html>
diff --git a/testing/web-platform/mozilla/tests/mathml/negative-lengths/negative-lspace-rspace-001.xhtml b/testing/web-platform/mozilla/tests/mathml/negative-lengths/negative-lspace-rspace-001.xhtml
@@ -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>
diff --git a/...a/tests/mathml/negative-mspace-1-ref.html → ...gative-lengths/negative-mspace-1-ref.html b/...a/tests/mathml/negative-mspace-1-ref.html → ...gative-lengths/negative-mspace-1-ref.html
diff --git a/...zilla/tests/mathml/negative-mspace-1.html → ...l/negative-lengths/negative-mspace-1.html b/...zilla/tests/mathml/negative-mspace-1.html → ...l/negative-lengths/negative-mspace-1.html
@@ -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>