Bug 134153

Summary: [EFL] [GTK] Install Latin Modern Math on the bots and update the MathML test references
Product: WebKit Reporter: Frédéric Wang (:fredw) <fred.wang>
Component: Tools / TestsAssignee: Frédéric Wang (:fredw) <fred.wang>
Status: RESOLVED FIXED    
Severity: Normal CC: bunhere, cdumez, clopez, commit-queue, dbarton, gyuyoung.kim, mrobinson, sergio
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 133604    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch gyuyoung.kim: review+

Description Frédéric Wang (:fredw) 2014-06-20 23:46:15 PDT
Follow-up of bug 133604.

https://github.com/mrobinson/webkitgtk-test-fonts/pull/5
Comment 1 Frédéric Wang (:fredw) 2014-06-22 01:31:16 PDT
Created attachment 233560 [details]
Patch

Just a quick run with the Latin Modern Math font:

- The mathml/opentype/ tests pass again ;-)

- mathml/presentation/menclose-notation-default-longdiv-expected.txt just has a 1px size diff in the text reference

- the pixel tests mathml/presentation/roots-expected.txt should be updated to use the new style.

- same for mathml/presentation/mo-stretch.html. It seems that Latin Modern Math is not able to stretch the simple integral at arbitrary size, so this is different from what is tested here. I don't think that's too important for MathML though, since by default integrals are just "largeop" not "stretchy".

- mathml/presentation/bug95015.html fails. I think the test is a bit fragile and does not pass on all platforms.

- same for mathml/presentation/mo-stretch.html. Here the difference is even not visible at the naked eye.
Comment 2 Frédéric Wang (:fredw) 2014-06-24 08:33:03 PDT
Created attachment 233707 [details]
Patch
Comment 3 Frédéric Wang (:fredw) 2014-06-24 08:41:52 PDT
@Martin: can you review the patch and take care of the GTK bots?

@Gyuyoung Kim: I 've finally been able to build the EFL port again, but not to execute the tests. However, I've verified that the MathML pages look ok with Latin Modern Math. I'll let you rebaseline the tests and I expect it will be more or less the same as what is done for GTK. The version with Latin Modern Math is https://github.com/mrobinson/webkitgtk-test-fonts/archive/master.zip (perhaps Martin can suggest a better link).

(In reply to comment #1)
> Created an attachment (id=233560) [details]
> - same for mathml/presentation/mo-stretch.html. Here the difference is even not visible at the naked eye.

I meant mo-stacked-glyphs.html. This is already covered by mo-stretch, though.
Comment 4 Martin Robinson 2014-06-24 09:06:13 PDT
Comment on attachment 233707 [details]
Patch

You can actually take care of the bots in this patch. I pushed the 0.0.5 tag to the webkitgtk-test-fonts repository on GitHub. You simply need to update the tag reference in Tools/gtk/jhbuild.modules.
Comment 5 Frédéric Wang (:fredw) 2014-06-24 10:21:44 PDT
Created attachment 233716 [details]
Patch
Comment 6 Martin Robinson 2014-06-24 10:28:04 PDT
Comment on attachment 233716 [details]
Patch

Is the appropriate font now selected in mathml.css?
Comment 7 Frédéric Wang (:fredw) 2014-06-24 10:41:03 PDT
(In reply to comment #6)
> (From update of attachment 233716 [details])
> Is the appropriate font now selected in mathml.css?

This was done in bug 133604. It's just that the jhbuild envs didn't have Latin Modern Math installed.
Comment 8 Frédéric Wang (:fredw) 2014-06-24 23:28:28 PDT
Created attachment 233792 [details]
Patch
Comment 9 Gyuyoung Kim 2014-06-24 23:30:32 PDT
Comment on attachment 233792 [details]
Patch

LGTM.
Comment 10 Frédéric Wang (:fredw) 2014-06-24 23:31:52 PDT
Committed r170419: <http://trac.webkit.org/changeset/170419>
Comment 11 Carlos Alberto Lopez Perez 2014-06-27 10:17:30 PDT
This caused failures on the following tests:

    fast/dynamic/text-combine.html
    fast/regions/repaint/japanese-rl-selection-repaint-in-regions.html
    fast/repaint/japanese-rl-selection-clear.html
    fast/repaint/japanese-rl-selection-repaint.html
    fast/text/decorations-with-text-combine.html
    fast/text/emphasis-vertical.html
    fast/text/international/002.html
    fast/text/international/text-combine-image-test.html
    fast/text/international/text-spliced-font.html
    fast/text/international/wrap-CJK-001.html
    fast/writing-mode/border-vertical-lr.html
    fast/writing-mode/japanese-lr-selection.html
    fast/writing-mode/japanese-lr-text.html
    fast/writing-mode/japanese-rl-selection.html
    fast/writing-mode/japanese-rl-text.html


I have rebaselined the tests on https://bugs.webkit.org/show_bug.cgi?id=134400