Bug 87501

Summary: [Qt] Acid3 test reveals that WTR does not pick up monospace fonts for testing
Product: WebKit Reporter: Andras Becsi <abecsi>
Component: Tools / TestsAssignee: Bruno Abinader (history only) <bruno.abinader>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarcelo, ossy, pierre.rossi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: Linux   
Bug Depends on: 85203, 92119    
Bug Blocks:    
Attachments:
Description Flags
Unskip/update layout tests results for Qt patch none

Andras Becsi
Reported 2012-05-25 07:27:18 PDT
Qt unfortunately skips the http/tests/misc/acid3.html test, although it passes in MiniBrowser. Comparing the result to the platform independent result suggests that the text in the test should be displayed using monospace fonts, but instead it is displayed using serif fonts. The Qt test infrastructure does not seem to pick up the correct font from the testfonts repository, this needs more investigation.
Attachments
Unskip/update layout tests results for Qt patch (46.15 KB, patch)
2012-07-24 06:50 PDT, Bruno Abinader (history only) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Caio Marcelo de Oliveira Filho
Comment 1 2012-05-25 07:43:37 PDT
We have the problem in DRT as well. Actually even our pixel expected results (used for manual checking from what I've understood) show non-monospace fonts in places they should be monospace :-(
Bruno Abinader (history only)
Comment 2 2012-07-24 06:50:46 PDT
Created attachment 154047 [details] Unskip/update layout tests results for Qt patch This patch unskips and updates the Qt layout test results after monospace fix landed in r121971. The update was necessary due to font changes affecting both DRT and pixel test results.
WebKit Review Bot
Comment 3 2012-07-24 07:34:00 PDT
Comment on attachment 154047 [details] Unskip/update layout tests results for Qt patch Clearing flags on attachment: 154047 Committed r123474: <http://trac.webkit.org/changeset/123474>
WebKit Review Bot
Comment 4 2012-07-24 07:34:04 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.