ChangeLog

2018-11-18  Zan Dobersek  <zdobersek@igalia.com>

        HarfBuzzFace::CacheEntry should use 32-bit values in its HashMap

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

        Reviewed by NOBODY (OOPS!).

        The HashMap-based glyph cache contained in HarfBuzzFace::CacheEntry

        objects is used to map given Unicode codepoints to corresponding

        glyph indices (which occurs in the harfBuzzGetGlyph() function that's

        invoked by HarfBuzz).

        The 16-bit unsigned integer as the map's value type is not enough here

        when the glyph index mapping is done through Freetype -- its API returns

        32-bit glyph indices, and Cairo assigns this value to the 64-bit

        unsigned index variable in the cairo_glyph_t struct. The value type is

        thus bumped to 32 bits to match the unsigned type size of the index's

        origin.

        * platform/graphics/harfbuzz/HarfBuzzFace.h:

        * platform/graphics/harfbuzz/HarfBuzzFaceCairo.cpp:

2018-11-17  Wenson Hsieh  <wenson_hsieh@apple.com>

        [iOS] Remove all usages of UIItemProvider, UIItemProviderReading, and related classes

Source/WebCore/platform/graphics/harfbuzz/HarfBuzzFace.h

@@public:

private:

    class CacheEntry : public RefCounted<CacheEntry> {

    public:

63 using GlyphCache = HashMap<uint32_t, uint16_t>;

63 using GlyphCache = HashMap<uint32_t, uint32_t>;

        static Ref<CacheEntry> create(hb_face_t* face)

        {

Source/WebCore/platform/graphics/harfbuzz/HarfBuzzFaceCairo.cpp

namespace WebCore {

struct HarfBuzzFontData {

53 WTF::HashMap<uint32_t, uint16_t>& glyphCacheForFaceCacheEntry;

53 WTF::HashMap<uint32_t, uint32_t>& glyphCacheForFaceCacheEntry;

    RefPtr<cairo_scaled_font_t> cairoScaledFont;

};