diff options
| author | Matt Brubeck <mbrubeck@limpet.net> | 2016-04-28 07:59:42 -0700 |
|---|---|---|
| committer | Matt Brubeck <mbrubeck@limpet.net> | 2016-04-28 07:59:42 -0700 |
| commit | 97a58e96898a67b585419b73fa411fb4c12be54a (patch) | |
| tree | f80141d4bb1b06aaf72440be2dddca293218774c /components/script/dom | |
| parent | 4d05bf23b82b7e36210aaa2e911ec4c79a1ed2d0 (diff) | |
| download | servo-97a58e96898a67b585419b73fa411fb4c12be54a.tar.gz servo-97a58e96898a67b585419b73fa411fb4c12be54a.zip | |
Don't assume the first glyph is part of the first char
Shaper::save_glyph_results incorrectly starts its loop by setting glyph_span
to a length of 1. This means that the `if glyph_span.len() == 0` test in the
inner loop will never succeed.
Instead the glyph span should start out empty, and a glyph should be added only
as the corresponding char is found. For comparison, see the Gecko code this
was ported from:
https://hg.mozilla.org/mozilla-central/file/ab0044bf/gfx/thebes/gfxHarfBuzzShaper.cpp#l1682
Diffstat (limited to 'components/script/dom')
0 files changed, 0 insertions, 0 deletions