Some mpadded reftests assume that all monospace characters are from the same font
Categories
(Core :: MathML, defect, P5)
Tracking
()
People
(Reporter: karlt, Assigned: fredw)
References
Details
Attachments
(1 file)
See layout/reftests/mathml/mpadded-7.html layout/reftests/mathml/mpadded-8.html layout/reftests/mathml/mpadded-9.html in https://treeherder.mozilla.org/logviewer.html#?job_id=28924854&repo=try#L22912 triggered by changes for bug 1309086.
|
||
Comment 1•7 years ago
|
||
Too late for firefox 52, mass-wontfix.
|
||
Updated•7 years ago
|
|
||
Updated•2 years ago
|
|
Assignee | |
Comment 2•3 months ago
|
||
These tests were initially added in bug 685628, and were relying
on <mtext mathvariant="monospace"> so we can assume that all
glyphs have the same logical width/height even if they may have
different ink width/height.
At that time mathvariant="monospace" was probably always using
font-family: monospace but after bug 114365, these were generally
directly mapped to Mathematical Alphanumeric Symbols and sometimes
they were falling back to the old font-family: monospace. Tests
were added to continue checking the latter too. After bug 1846482,
the one using the legacy mathvariant="monospace" were just
replaced with direct use of Mathematical Alphanumeric Symbols.
To keep the spirit of the original tests, we just need a font
with two glyphs with same logical width/height but different ink
width/height.
Probably the easiest solution is to only keep the tests with
monospace Mathematical Alphanumeric Symbols. We should have fonts
for that in modern systems, except maybe on Android. But even if Tofu
glyphs are used, the tests will probably pass. If not, we could still
create a custom WOFF font for the sake of these tests.
|
|
Assignee | |
Comment 3•3 months ago
|
||
These tests were initially added in bug 685628, and were relying
on <mtext mathvariant="monospace"> so we can assume that all
glyphs have the same logical width/height even if they may have
different ink width/height.
At that time mathvariant="monospace" was probably always using
font-family: monospace but after bug 114365, these were generally
directly mapped to Mathematical Alphanumeric Symbols and sometimes
they were falling back to the old font-family: monospace. Tests
were added to continue checking the latter too. After bug 1846482,
the one using the legacy mathvariant="monospace" were just
replaced with direct use of Mathematical Alphanumeric Symbols.
|
||
Updated•3 months ago
|
Pushed by fwang@igalia.com: https://hg.mozilla.org/integration/autoland/rev/d748f63d4f05 Remove font-family: monospace from some mpadded reftests. r=emilio
|
||
Comment 5•3 months ago
|
||
| bugherder | ||
Description
•