Summary: | Crash when measuring a glyphs from a fallback SVG font | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Myles C. Maxfield <mmaxfield> | ||||
Component: | New Bugs | Assignee: | Myles C. Maxfield <mmaxfield> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | dino, jonlee, simon.fraser, thorton, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Myles C. Maxfield
2014-07-24 16:02:24 PDT
Created attachment 235476 [details]
Patch
Comment on attachment 235476 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=235476&action=review > Source/WebCore/platform/graphics/WidthIterator.cpp:127 > + if (iterator.run().renderingContext() && typesettingFeatures & Kerning) { Minor preference for parens around (typesettingFeatures & Kerning) > Source/WebCore/platform/graphics/WidthIterator.cpp:129 > + // However, doing the above FIXME seems like questionable design. Perhaps a larger refactoring of SVG fonts is necessary. Odd comment in this context. Just reword the FIXME. |