bug 797405 - use harfbuzz on Windows for all text shaping except Hangul. r=jdaggett

modules/libpref/src/init/all.js

@@ -349,13 +349,17 @@ pref("gfx.font_rendering.graphite.enabled", true);
 //  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 -1 to have all text shaped via the harfbuzz backend.
+// Set the pref to 255 to have all text shaped via the harfbuzz backend.
 #ifdef XP_WIN
-// use harfbuzz for default (0x01) + arabic (0x02) + hebrew (0x04)
-pref("gfx.font_rendering.harfbuzz.scripts", 7);
+// 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)
-pref("gfx.font_rendering.harfbuzz.scripts", -1);
+// 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.
+pref("gfx.font_rendering.harfbuzz.scripts", 255);
 #endif
 
 #ifdef XP_WIN