Bug 1548522 - Remove support for the menclose's "radical" notation. r…

…=emilio

See https://groups.google.com/forum/#!topic/mozilla.dev.platform/vwAkuZIEhnY

* Introduce a new preference option to disable menclose's "radical" notation.
* Disable the notation in Nightly and when running WPT tests.
* Enable the notation in other channels together with a counter and
  deprecation warning.
* Update WPT test legacy-menclose-radical-notation.html
  - Fix test: "radical" should be equivalent to "", which is not the same as
    the default value "longdiv".
    See w3c/mathml#144
  - Add a test "box radical" which should be equivalent to "box".
  - Remove failure expectation.
* Enable the radical notation for MathML reftests testing it.

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

UltraBlame original commit: d2613eaa13690e3771e84ef6ff5080a8fa348a66

dom/base/nsDeprecatedOperationList.h

@@ -172,5 +172,9 @@ MathML_DeprecatedMathSpaceValue
 )
 DEPRECATED_OPERATION
 (
+MathML_DeprecatedMencloseNotationRadical
+)
+DEPRECATED_OPERATION
+(
 MathML_DeprecatedStyleAttribute
 )

dom/locales/en-US/chrome/dom/dom.properties

@@ -8140,6 +8140,44 @@ NOTE
 Do
 not
 translate
+radical
+notation
+and
+menclose
+.
+MathML_DeprecatedMencloseNotationRadical
+=
+The
+radical
+value
+is
+deprecated
+for
+the
+notation
+attribute
+of
+the
+<
+menclose
+>
+element
+and
+will
+be
+removed
+at
+a
+future
+date
+.
+#
+LOCALIZATION
+NOTE
+:
+Do
+not
+translate
 MathML
 background
 color

layout/mathml/nsMathMLmencloseFrame.cpp

@@ -33,6 +33,15 @@ include
 "
 mozilla
 /
+StaticPrefs_mathml
+.
+h
+"
+#
+include
+"
+mozilla
+/
 gfx
 /
 2D
@@ -442,6 +451,36 @@ radical
 )
 )
 {
+if
+(
+!
+StaticPrefs
+:
+:
+mathml_deprecated_menclose_notation_radical_disabled
+(
+)
+)
+{
+mContent
+-
+>
+OwnerDoc
+(
+)
+-
+>
+WarnOnceAbout
+(
+dom
+:
+:
+Document
+:
+:
+eMathML_DeprecatedMencloseNotationRadical
+)
+;
 rv
 =
 AllocateMathMLChar
@@ -461,6 +500,7 @@ mNotationsToDraw
 NOTATION_RADICAL
 ;
 }
+}
 else
 if
 (

layout/reftests/mathml/reftest.list

@@ -490,6 +490,15 @@ html
 #
 bug
 1309426
+pref
+(
+mathml
+.
+deprecated_menclose_notation_radical
+.
+disabled
+false
+)
 =
 =
 dir
@@ -4888,6 +4897,15 @@ menclose
 ref
 .
 html
+pref
+(
+mathml
+.
+deprecated_menclose_notation_radical
+.
+disabled
+false
+)
 !
 =
 menclose
@@ -5497,6 +5515,15 @@ madruwb
 ref
 .
 html
+pref
+(
+mathml
+.
+deprecated_menclose_notation_radical
+.
+disabled
+false
+)
 fails
 -
 if
@@ -5808,6 +5835,15 @@ madruwb
 ref
 .
 html
+pref
+(
+mathml
+.
+deprecated_menclose_notation_radical
+.
+disabled
+false
+)
 =
 =
 menclose
@@ -6156,6 +6192,15 @@ menclose
 ref
 .
 html
+pref
+(
+mathml
+.
+deprecated_menclose_notation_radical
+.
+disabled
+false
+)
 =
 =
 menclose

modules/libpref/init/StaticPrefList.yaml

@@ -33225,6 +33225,37 @@ always
 Whether
 to
 disable
+deprecated
+"
+radical
+"
+notation
+for
+the
+menclose
+element
+.
+-
+name
+:
+mathml
+.
+deprecated_menclose_notation_radical
+.
+disabled
+type
+:
+bool
+value
+:
+IS_NIGHTLY_BUILD
+mirror
+:
+always
+#
+Whether
+to
+disable
 legacy
 MathML
 number

testing/web-platform/meta/mathml/__dir__.ini

@@ -10,6 +10,13 @@ disabled
 true
 mathml
 .
+deprecated_menclose_notation_radical
+.
+disabled
+:
+true
+mathml
+.
 legacy_number_syntax
 .
 disabled

testing/web-platform/tests/mathml/presentation-markup/menclose/legacy-menclose-radical-notation-ref.html

@@ -103,6 +103,26 @@
 menclose
 >
 <
+menclose
+notation
+=
+"
+box
+"
+>
+<
+mn
+>
+123
+<
+/
+mn
+>
+<
+/
+menclose
+>
+<
 /
 math
 >

testing/web-platform/tests/mathml/presentation-markup/menclose/legacy-menclose-radical-notation.html

@@ -120,8 +120,7 @@
 radical
 notation
 is
-not
-supported
+ignored
 .
 "
 >
@@ -222,6 +221,27 @@
 menclose
 >
 <
+menclose
+notation
+=
+"
+box
+radical
+"
+>
+<
+mn
+>
+123
+<
+/
+mn
+>
+<
+/
+menclose
+>
+<
 /
 math
 >