[webkit-reviews] review requested: [Bug 52422] [chromium] More crash in FontFallbackList::determinePitch(const Font* font) : [Attachment 78889] A blind and speculative fix

[bugzilla-daemon at webkit.org]

Hironori Bono <hbono at chromium.org> has asked  for review:
Bug 52422: [chromium] More crash in FontFallbackList::determinePitch(const
Font* font)
https://bugs.webkit.org/show_bug.cgi?id=52422

Attachment 78889: A blind and speculative fix
https://bugs.webkit.org/attachment.cgi?id=78889&action=review

------- Additional Comments from Hironori Bono <hbono at chromium.org>
Greetings,

This is just a blind and speculative fix (inspired by "FontCacheWin.cpp") for
this crash. Even though I do not have any clear ideas for writing layout tests
for this issue, it may be a good idea to add a function (such as
layoutTestController.disableFont("Arial")) that prevents DumpRenderTree from
loading the given fonts?

Regards,

Hironori Bono