[Webkit-unassigned] [Bug 51106] [Qt] Implement the fast font path for Qt

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=51106





--- Comment #14 from Andreas Kling <kling at webkit.org>  2011-03-27 06:40:56 PST ---
(In reply to comment #13)
> (From update of attachment 86213 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=86213&action=review
> 
> > Source/WebCore/platform/graphics/qt/GlyphPageTreeNodeQt.cpp:67
> > +    for (unsigned i = 0; i < length; ++i) {
> > +        Glyph glyph = getGlyphForCharacterAndFont(qstring.at(i), font);
> 
> Wouldn't it be nice to get the glyphs for the page in one shot, instead of creating a QTextLayout (that'll heap alloc in the impl) for each character? :)

Most definitely.

For anyone passing by, this bug is currently pending new API in Qt before deciding how to continue. :)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.