diff options
| author | wolfbeast <mcwerewolf@gmail.com> | 2018-05-09 17:52:17 +0200 |
|---|---|---|
| committer | wolfbeast <mcwerewolf@gmail.com> | 2018-05-09 17:52:17 +0200 |
| commit | 2619698a2eb7363940d1e6a2d71a9ce7b366d3f5 (patch) | |
| tree | a0e493460026b72f243d932f401409f26ee9a89e /gfx/thebes/gfxPlatformGtk.cpp | |
| parent | 361748956bfcd31ca0edb5e2c845fb8c043945ba (diff) | |
| download | UXP-2619698a2eb7363940d1e6a2d71a9ce7b366d3f5.tar UXP-2619698a2eb7363940d1e6a2d71a9ce7b366d3f5.tar.gz UXP-2619698a2eb7363940d1e6a2d71a9ce7b366d3f5.tar.lz UXP-2619698a2eb7363940d1e6a2d71a9ce7b366d3f5.tar.xz UXP-2619698a2eb7363940d1e6a2d71a9ce7b366d3f5.zip | |
Font fallback should detect emoji range for color emoji font.
Diffstat (limited to 'gfx/thebes/gfxPlatformGtk.cpp')
| -rw-r--r-- | gfx/thebes/gfxPlatformGtk.cpp | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/gfx/thebes/gfxPlatformGtk.cpp b/gfx/thebes/gfxPlatformGtk.cpp index 1fb3bc4fd..be75332d6 100644 --- a/gfx/thebes/gfxPlatformGtk.cpp +++ b/gfx/thebes/gfxPlatformGtk.cpp @@ -240,9 +240,14 @@ gfxPlatformGtk::GetCommonFallbackFonts(uint32_t aCh, uint32_t aNextCh, Script aRunScript, nsTArray<const char*>& aFontList) { - if (aNextCh == 0xfe0fu) { - // if char is followed by VS16, try for a color emoji glyph - aFontList.AppendElement(kFontTwemojiMozilla); + EmojiPresentation emoji = GetEmojiPresentation(aCh); + if (emoji != EmojiPresentation::TextOnly) { + if (aNextCh == kVariationSelector16 || + (aNextCh != kVariationSelector15 && + emoji == EmojiPresentation::EmojiDefault)) { + // if char is followed by VS16, try for a color emoji glyph + aFontList.AppendElement(kFontTwemojiMozilla); + } } aFontList.AppendElement(kFontDejaVuSerif); @@ -250,15 +255,6 @@ gfxPlatformGtk::GetCommonFallbackFonts(uint32_t aCh, uint32_t aNextCh, aFontList.AppendElement(kFontDejaVuSans); aFontList.AppendElement(kFontFreeSans); - if (!IS_IN_BMP(aCh)) { - uint32_t p = aCh >> 16; - if (p == 1) { // try color emoji font, unless VS15 (text style) present - if (aNextCh != 0xfe0fu && aNextCh != 0xfe0eu) { - aFontList.AppendElement(kFontTwemojiMozilla); - } - } - } - // add fonts for CJK ranges // xxx - this isn't really correct, should use the same CJK font ordering // as the pref font code |