Bug 1548529 - Remove values "thin", "thick", "medium" values of mfraclinethickness. r=emilio

See w3c/mathml#4
and https://groups.google.com/forum/#!topic/mozilla.dev.platform/G91-vBeC3Rw

This commit introduces a new preference option
mathml.mfrac_linethickness_names.disabled to disable linethickness names. For
now, these names are only disabled in Nightly builds. Announcements and actual
disabling of this and other MathML features will be considered later.

* test_bug553917.html is updated to check that these values now cause an error
  message to be logged into the console.
* mstyle-1.xhtml is updated to use a numeric linethickness since the point of
  the test is just to check that the attribute is not supported on mstyle, not
  about the actual attribute value.
* Other fractions tests relying on linethickness names are executed with the
  proper preference adjustment.
* mathml/presentation-markup/fractions/frac-linethickness-001.html is now
  closer to its expectation ; however the test still fails because nonzero
  unitless values are not removed yet. See
  w3c/mathml#24

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

UltraBlame original commit: 0692436a2b33c134bf50654ce466e4bd5ddb05cf

Author: marco-c

layout/mathml/nsMathMLmfracFrame.cpp

@@ -160,6 +160,15 @@ h
 #
 include
 "
+mozilla
+/
+StaticPrefs_mathml
+.
+h
+"
+#
+include
+"
 nsLayoutUtils
 .
 h
@@ -712,6 +721,41 @@ IsEmpty
 {
 if
 (
+StaticPrefs
+:
+:
+mathml_mfrac_linethickness_names_disabled
+(
+)
+)
+{
+/
+/
+length
+value
+lineThickness
+=
+defaultThickness
+;
+ParseNumericValue
+(
+aThicknessAttribute
+&
+lineThickness
+nsMathMLElement
+:
+:
+PARSE_ALLOW_UNITLESS
+aPresContext
+aComputedStyle
+aFontSizeInflation
+)
+;
+}
+else
+{
+if
+(
 aThicknessAttribute
 .
 EqualsLiteral
@@ -768,13 +812,15 @@ defaultThickness
 -
 onePixel
 )
+{
 lineThickness
 =
 defaultThickness
 -
 onePixel
 ;
 }
+}
 else
 if
 (
@@ -840,13 +886,15 @@ defaultThickness
 +
 onePixel
 )
+{
 lineThickness
 =
 defaultThickness
 +
 onePixel
 ;
 }
+}
 else
 {
 /
@@ -873,6 +921,7 @@ aFontSizeInflation
 ;
 }
 }
+}
 /
 /
 use

layout/mathml/tests/test_bug553917.html

@@ -740,6 +740,111 @@
 /
 math
 >
+<
+math
+>
+<
+mfrac
+linethickness
+=
+"
+thin
+"
+>
+<
+mn
+>
+1
+<
+/
+mn
+>
+<
+mn
+>
+2
+<
+/
+mn
+>
+<
+/
+mfrac
+>
+<
+/
+math
+>
+<
+math
+>
+<
+mfrac
+linethickness
+=
+"
+medium
+"
+>
+<
+mn
+>
+1
+<
+/
+mn
+>
+<
+mn
+>
+2
+<
+/
+mn
+>
+<
+/
+mfrac
+>
+<
+/
+math
+>
+<
+math
+>
+<
+mfrac
+linethickness
+=
+"
+thick
+"
+>
+<
+mn
+>
+1
+<
+/
+mn
+>
+<
+mn
+>
+2
+<
+/
+mn
+>
+<
+/
+mfrac
+>
+<
+/
+math
+>
 *
 /
 LengthParsingError
@@ -754,6 +859,9 @@
 false
 false
 false
+false
+false
+false
 ]
 args
 :
@@ -793,6 +901,21 @@
 BADdepth
 "
 ]
+[
+"
+thin
+"
+]
+[
+"
+medium
+"
+]
+[
+"
+thick
+"
+]
 ]
 }
 /
@@ -1767,6 +1890,111 @@
 math
 >
 <
+math
+>
+<
+mfrac
+linethickness
+=
+"
+thin
+"
+>
+<
+mn
+>
+1
+<
+/
+mn
+>
+<
+mn
+>
+2
+<
+/
+mn
+>
+<
+/
+mfrac
+>
+<
+/
+math
+>
+<
+math
+>
+<
+mfrac
+linethickness
+=
+"
+medium
+"
+>
+<
+mn
+>
+1
+<
+/
+mn
+>
+<
+mn
+>
+2
+<
+/
+mn
+>
+<
+/
+mfrac
+>
+<
+/
+math
+>
+<
+math
+>
+<
+mfrac
+linethickness
+=
+"
+thick
+"
+>
+<
+mn
+>
+1
+<
+/
+mn
+>
+<
+mn
+>
+2
+<
+/
+mn
+>
+<
+/
+mfrac
+>
+<
+/
+math
+>
+<
 !
 -
 -

layout/reftests/mathml/mstyle-1.xhtml

@@ -350,7 +350,7 @@ true
 linethickness
 =
 "
-thick
+10px
 "
 >
 <