1790789 - Remove "integral" property from the MathML operator dictionary

Status: RESOLVED FIXED

(Core :: MathML, task)

Description

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

This is a legacy hack to help selection of larger variants for integrals, for fonts that don't properly present DisplayOperatorMinHeight.

The code points of operators having the integral property are located in two contiguous blocks, so it can be tested by a direct check:

operator.\u222B.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # ∫
operator.\u222C.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # double integral
operator.\u222D.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # triple integral
operator.\u222E.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # ∮
operator.\u222F.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # ∯
operator.\u2230.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # volume integral
operator.\u2231.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # clockwise integral
operator.\u2232.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # ∲
operator.\u2233.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # ∳

operator.\u2A0B.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # summation with integral
operator.\u2A0C.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # quadruple integral operator
operator.\u2A0D.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # finite part integral
operator.\u2A0E.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral with double stroke
operator.\u2A0F.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral average with slash
operator.\u2A10.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # circulation function
operator.\u2A11.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # anticlockwise integration
operator.\u2A12.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # line integration with rectangular path around pole
operator.\u2A13.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # line integration with semicircular path around pole
operator.\u2A14.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # line integration not including the pole
operator.\u2A15.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral around a point operator
operator.\u2A16.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # quaternion integral operator
operator.\u2A17.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral with leftwards arrow with hook
operator.\u2A18.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral with times sign
operator.\u2A19.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral with intersection
operator.\u2A1A.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral with union
operator.\u2A1B.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral with overbar
operator.\u2A1C.prefix = lspace:3 rspace:3 largeop symmetric direction:vertical integral # integral with underbar

Comment 1

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

Attachment: Bug 1790789 - Remove "integral" property from the MathML operator dictionary. r=emilio

"integral" is an internal operator property that Gecko uses when
selecting larger variants for integrals, for fonts that don't provide
a good value for DisplayOperatorMinHeight. The code points of operators
having the integral property are located in two contiguous blocks, so
this commit replaces existing implementation with a direct check. There
is no behavior change.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/68182638a22b
Remove "integral" property from the MathML operator dictionary. r=emilio

Comment 3

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/68182638a22b