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:

Platform

Assignee:

Darin Adler <darin>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

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+

Description Darin Adler 2007-10-16 16:56:39 PDT

Right now the cache can't handle caching the missing glyph.

Comment 1 Darin Adler 2007-10-16 16:59:54 PDT

Created attachment 16689 [details]
patch to cache missing glyphs too

Comment 2 mitz 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.

Comment 3 Darin Adler 2007-10-16 18:01:02 PDT

Created attachment 16690 [details]
patch to cache missing glyphs too (Windows fixed now)

Comment 4 mitz 2007-10-16 18:11:34 PDT

Comment on attachment 16690 [details]
patch to cache missing glyphs too (Windows fixed now)

r=me

Comment 5 Darin Adler 2007-10-16 22:02:29 PDT

Committed revision 26696.