RESOLVED FIXED 50539
[Qt] Report zero width for zero size fonts 
https://bugs.webkit.org/show_bug.cgi?id=50539
Summary [Qt] Report zero width for zero size fonts
Robert Hogan
Reported Sunday, December 5, 2010 5:45:15 PM UTC
.
Attachments
Patch (1.80 KB, patch)
2010-12-05 09:55 PST, Robert Hogan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Robert Hogan
Comment 1 Sunday, December 5, 2010 5:55:25 PM UTC
Created attachment 75632 [details] Patch
Kenneth Rohde Christiansen
Comment 2 Sunday, December 5, 2010 10:03:34 PM UTC
Comment on attachment 75632 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=75632&action=review pls fix before landing > WebCore/platform/graphics/qt/FontQt.cpp:266 > + return 0.0f; this should be 0 according to the style guide > WebCore/platform/graphics/qt/FontQt.cpp:291 > + return 0.0f; here as well
Csaba Osztrogonác
Comment 3 Monday, December 6, 2010 10:26:28 AM UTC
Comment on attachment 75632 [details] Patch Fixed patch landed in http://trac.webkit.org/changeset/73352
Robert Hogan
Comment 4 Monday, December 6, 2010 7:26:36 PM UTC
Andreas/Kenneth, Is there a bit of a disconnect between Font::drawTextCommon and the actual size of the font (according to webkit)? It uses FontMetrics which may not be appropriate in the case of zero-sized fonts. I suspect QtWebKit is rendering the font full-sized rather than 'no-sized' because of this. What do you think?
Note You need to log in before you can comment on or make changes to this bug.