[webkit-reviews] review requested: [Bug 33876] [Qt] Shortcut for Font::floatWidthForComplexText : [Attachment 47605] First patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 28 04:29:01 PST 2010


Holger Freyther <zecke at selfish.org> has asked  for review:
Bug 33876: [Qt] Shortcut for Font::floatWidthForComplexText
https://bugs.webkit.org/show_bug.cgi?id=33876

Attachment 47605: First patch
https://bugs.webkit.org/attachment.cgi?id=47605&action=review

------- Additional Comments from Holger Freyther <zecke at selfish.org>
Only ' ', '\t' has the benefit of being fast with QFontMetrics::width. Any
other charachter might or might not have different rounding and about the same
speed as harfbuzz. This is a significant improvement on the pageloading test.


More information about the webkit-reviews mailing list