bug 964313 - use harfbuzz for all text shaping (now including Hangul) on Windows. r=jdaggett
authorJonathan Kew <jkew@mozilla.com>
Thu, 30 Jan 2014 09:48:37 +0000
changeset 181977 e4a97d6b2d408b8063096bc32474e4f8b24c5667
parent 181976 4bf651486f133ba4ff1067de994d8882139c1387
child 181978 6b93f7f23e507b52f046c16e7639eb51a10ab4a7
push id3343
push userffxbld
push dateMon, 17 Mar 2014 21:55:32 +0000
treeherdermozilla-beta@2f7d3415f79f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjdaggett
bugs964313
milestone29.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
bug 964313 - use harfbuzz for all text shaping (now including Hangul) on Windows. r=jdaggett
modules/libpref/src/init/all.js
--- a/modules/libpref/src/init/all.js
+++ b/modules/libpref/src/init/all.js
@@ -360,27 +360,19 @@ pref("gfx.font_rendering.graphite.enable
 //  SHAPING_HEBREW    = 0x0004,
 //  SHAPING_HANGUL    = 0x0008,
 //  SHAPING_MONGOLIAN = 0x0010,
 //  SHAPING_INDIC     = 0x0020,
 //  SHAPING_THAI      = 0x0040
 // (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);
 pref("gfx.font_rendering.directwrite.use_gdi_table_loading", true);
 #endif
 
 pref("gfx.font_rendering.opentype_svg.enabled", true);