Bug 99413 - REGRESSION (r131365): WidthIterator::advance() is needlessly passed a GlyphBuffer in many cases
Summary: REGRESSION (r131365): WidthIterator::advance() is needlessly passed a GlyphBu...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: mitz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-15 21:56 PDT by mitz
Modified: 2012-10-15 22:08 PDT (History)
0 users

See Also:

Attachments
Remove the GlyphBuffer* parameter to floadWidthForSimpleText() and make it pass a GlyphBuffer* to advance() only when necessary (6.07 KB, patch)
2012-10-15 22:03 PDT, mitz 		adele: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2012-10-15 21:56:33 PDT 
REGRESSION (r131365): WidthIterator::advance() is needlessly passed a GlyphBuffer in many cases
Comment 1 mitz 2012-10-15 22:03:01 PDT 
Created attachment 168850 [details]
Remove the GlyphBuffer* parameter to floadWidthForSimpleText() and make it pass a GlyphBuffer* to advance() only when necessary
Comment 2 mitz 2012-10-15 22:08:40 PDT 
Fixed in <http://trac.webkit.org/r131410>.