[OS X] Remove support for composite fonts
Created attachment 258795 [details]
Created attachment 258796 [details]
Created attachment 259213 [details]
Comment on attachment 259213 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=259213&action=review
> + CTRunGetStringIndices() to map from the code points to the glyphs. This approach is
> + fundamentally incorrect, as it will combine consecutive code points together in the
> + string if possible to produce a glyph.
“Fundamentally incorrect” is not a good characterization of the approach. A more accurate statement would be that the implementation of this approach is currently buggy in that it relies on getCFStringAttributes() to return attributes that turn off essential ligatures, but it has become possible for it to be used in cases where getCFStringAttributes() return attributes that do not turn off essential ligatures.
Anyway, I think this preamble is unnecessary.
> + The only way WebKit will ever trigger this code path is if we are inspecting a
Committed r188566: <http://trac.webkit.org/changeset/188566>
Follow up patch http://trac.webkit.org/changeset/188569
*** Bug 147859 has been marked as a duplicate of this bug. ***