Bug 28288 – Qt: Font Sizes are wrong for font families including monospace

• Status: RESOLVED
• Resolution: FIXED
• Component: Text

[Eric Seidel (no email)]

Qt: Font Sizes are wrong for font families including monospace

fast/css/getComputedStyle/computed-style-font-family-monospace.html	expected	actual	diff	pretty diff
fast/css/getComputedStyle/computed-style-font-family.html	expected	actual	diff	pretty diff

I expect the way qt fonts work is just different from other ports and the fix that I made in bug 19161 just doesn't work for Qt.

I'll skip these tests for now so that the Qt bot is green.  Someone in Qt-land will need to look at this.

[Eric Seidel (no email)]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog
	M	LayoutTests/platform/qt/Skipped
Committed r47254
	M	LayoutTests/platform/qt/Skipped
	M	LayoutTests/ChangeLog
r47254 = 890728473e778c7105daeecb9d385b028ef68d65 (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk
http://trac.webkit.org/changeset/47254

[Andras Becsi]

This issue seems to be related to default font size settings in qwebsettings.cpp and DumpRenderTree.cpp. There also seems to be a little fray because of the different default values given in these source files. 
QtWebKit uses 14px font size for both regular and monospace fonts in qwebsettings.cpp but DumpRenderTree.cpp uses 13px for both, whereas the gtk and mac ports use 16px for regular fonts and 13px for monospace, so the test obviously fails on Qt.

[Andras Becsi]

Created attachment 41498 [details]
Enable previously disabled passing fast/css tests after r48873 landed.

[Csaba Osztrogonác]

(In reply to comment #3)
> Created an attachment (id=41498) [details]
> Enable previously disabled passing fast/css tests after r48873 landed.

Sending        LayoutTests/ChangeLog
Sending        LayoutTests/platform/qt/Skipped
..
Committed revision 49849.

[Zoltan Horvath]

Patch landed. Closing bug.