Bug 69715

Summary: Do not compare the first glyph to itself in Font::drawGlyphBuffer()
Product: WebKit Reporter: Benjamin Poulain <benjamin>
Component: Layout and RenderingAssignee: Benjamin Poulain <benjamin>
Severity: Normal CC: kling, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Benjamin Poulain 2011-10-08 22:28:36 PDT
In Font::drawGlyphBuffer():
1) the information of the first glyph is extracted
2) it is compared to itself, which never fails
3) then to the following glyphs are handled

We can just skip the first iteration which make the code a bit easier to read and hopefully a tiny-little-bit faster :)
Comment 1 Benjamin Poulain 2011-10-08 22:38:13 PDT
Created attachment 110290 [details]
Comment 2 Andreas Kling 2011-10-09 01:55:51 PDT
Comment on attachment 110290 [details]

Comment 3 WebKit Review Bot 2011-10-09 02:59:17 PDT
Comment on attachment 110290 [details]

Clearing flags on attachment: 110290

Committed r97023: <http://trac.webkit.org/changeset/97023>
Comment 4 WebKit Review Bot 2011-10-09 02:59:21 PDT
All reviewed patches have been landed.  Closing bug.