Bug 15536

Summary: need to cache missing glyph so we're not slow on pages that show missing glyphs
Product: WebKit Reporter: Darin Adler <darin>
Component: PlatformAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
patch to cache missing glyphs too
none
patch to cache missing glyphs too (Windows fixed now) mitz: review+

Darin Adler
Reported 2007-10-16 16:56:39 PDT
Right now the cache can't handle caching the missing glyph.
Attachments
patch to cache missing glyphs too (7.27 KB, patch)
2007-10-16 16:59 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
patch to cache missing glyphs too (Windows fixed now) (7.16 KB, patch)
2007-10-16 18:01 PDT, Darin Adler 		mitz: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2007-10-16 16:59:54 PDT
Created attachment 16689 [details] patch to cache missing glyphs too
mitz
Comment 2 2007-10-16 17:32:27 PDT
Comment on attachment 16689 [details] patch to cache missing glyphs too + glyphRecord = (ATSLayoutRecord *)((char *)glyphRecord + wkGetGlyphVectorRecordSize(glyphVector)); Looks like this was copied from Mac by mistake.
Darin Adler
Comment 3 2007-10-16 18:01:02 PDT
Created attachment 16690 [details] patch to cache missing glyphs too (Windows fixed now)
mitz
Comment 4 2007-10-16 18:11:34 PDT
Comment on attachment 16690 [details] patch to cache missing glyphs too (Windows fixed now) r=me
Darin Adler
Comment 5 2007-10-16 22:02:29 PDT
Committed revision 26696.
Note You need to log in before you can comment on or make changes to this bug.