From 244e290b69eac8cb3ca0703e4c7c9a494eb03316 Mon Sep 17 00:00:00 2001 From: Jonathan Kew Date: Thu, 30 Jan 2014 09:48:37 +0000 Subject: [PATCH] bug 964313 - use harfbuzz for all text shaping (now including Hangul) on Windows. r=jdaggett --- modules/libpref/src/init/all.js | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/modules/libpref/src/init/all.js b/modules/libpref/src/init/all.js index 3893584eb82..32256cdbeb3 100644 --- a/modules/libpref/src/init/all.js +++ b/modules/libpref/src/init/all.js @@ -365,17 +365,9 @@ pref("gfx.font_rendering.graphite.enabled", true); // (see http://mxr.mozilla.org/mozilla-central/ident?i=ShapingType) // Scripts not listed are grouped in the default category. // Set the pref to 255 to have all text shaped via the harfbuzz backend. -#ifdef XP_WIN -// Use harfbuzz for everything except Hangul (0x08). Harfbuzz doesn't yet -// have a Hangul shaper, which means that the marks U+302E/302F would not -// reorder properly in Malgun Gothic or similar fonts. -pref("gfx.font_rendering.harfbuzz.scripts", 247); -#else -// Use harfbuzz for all scripts (except when using AAT fonts on OS X). -// AFAICT, Core Text doesn't support full OpenType Hangul shaping anyway, -// so there's no benefit to excluding it here. +// Default setting: +// We use harfbuzz for all scripts (except when using AAT fonts on OS X). pref("gfx.font_rendering.harfbuzz.scripts", 255); -#endif #ifdef XP_WIN pref("gfx.font_rendering.directwrite.enabled", false);