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>
Status: RESOLVED FIXED    
Severity: Normal CC: kling, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
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]
Patch
Comment 2 Andreas Kling 2011-10-09 01:55:51 PDT 
Comment on attachment 110290 [details]
Patch

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

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.