Bug 183446 - Do not rely on Latin Modern Math to test fraction or radical bars
Summary: Do not rely on Latin Modern Math to test fraction or radical bars
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Frédéric Wang (:fredw)
URL:
Keywords:
Depends on:
Blocks: 160161
  Show dependency treegraph
 
Reported: 2018-03-08 04:03 PST by Frédéric Wang (:fredw)
Modified: 2018-03-12 07:14 PDT (History)
9 users (show)

See Also:


Attachments
Patch (9.44 KB, patch)
2018-03-08 04:14 PST, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff
Patch (10.83 KB, patch)
2018-03-08 04:47 PST, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff
Patch (10.59 KB, patch)
2018-03-09 03:04 PST, Frédéric Wang (:fredw)
rego: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang (:fredw) 2018-03-08 04:03:00 PST
The following tests rely on Latin Modern Math to test fraction linethickness. They are skipped on iOS/macOS/win platforms:

* mathml/opentype/fraction-line.html was added in bug 155639 to test OpenType MATH parameters for fraction, but these are now tested by LayoutTests/imported/mathml-in-html5/mathml/presentation-markup/fractions/frac-parameters-*.html
* mathml/presentation/fractions-linethickness.html could use the same WOFF font as frac-parameters-*.html too.
Comment 1 Frédéric Wang (:fredw) 2018-03-08 04:14:15 PST
Created attachment 335289 [details]
Patch
Comment 2 Frédéric Wang (:fredw) 2018-03-08 04:47:17 PST
Created attachment 335291 [details]
Patch
Comment 3 Frédéric Wang (:fredw) 2018-03-09 03:04:29 PST
Created attachment 335412 [details]
Patch
Comment 4 Manuel Rego Casasnovas 2018-03-12 06:58:04 PDT
Comment on attachment 335412 [details]
Patch

Great work!
Comment 5 Frédéric Wang (:fredw) 2018-03-12 07:13:09 PDT
Committed r229533: <https://trac.webkit.org/changeset/229533>