[Webkit-unassigned] [Bug 25770] [chromium] Crash in FontFallbackList::determinePitch(const Font* font)

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=25770





--- Comment #19 from David Levin <levin at chromium.org>  2011-01-13 01:00:34 PST ---
(In reply to comment #18)
> (In reply to comment #17)
> > (In reply to comment #16)
> > > Where is the patch to clean up the file?
> > 
> > Thank you for your notification. I have filed Bug 50611 <http://webkit.org/b/50611> and uploaded a refactoring change there.
> > 
> > Regards,
> > 
> > Hironori Bono
> 
> Unfortunately, this crash still happens after Chrome merged this change and I would like to post another change, which just adds a list of fallback fonts in FontCache::getLastResortFallbackFont(). Is it better to file another bug for this change, or re-open this bug?
> 
> Regards,
> 
> Hironori Bono



Hmm good question. I would go with a new bug since bugs in webkit ~= code reviews in chromium. You wouldn't try to repro an old code review and start putting in new patches The old comments would make it more confusing. (Feel free to reference this bug in the new one.)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.