RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v4]

[Prasanta Sadhukhan]

On Wed, 16 Mar 2022 06:02:28 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts
>
> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   korean fallback added

As I mentioned initially,
> tested with sanserif, serif, monospaced, dialog, dialoginput with PLAIN, BOLD, ITALIC, BOLD-ITALIC style **in that order**

so 1st 4 are sansserif, next 4 are serif etc.

As I see, we have this font support in the fontconfig properties 

Serif = Batang				 batang.ttc
sansserif = Malgun Gothic	malgun.ttf
monospaced = Gulimche		gulim.ttc
dialog = Malgun Gothic		malgun.ttf
dialoginput = Gulim			gulim.ttf

Now in my WIndows 10, I have these font files irrespective of Korean font pack installed or not, 
so I guess that is why there is **no difference** in the font image with the fix with/without korean font pack



Now, without the fix, [comparing with/without the fix with Korean font pack installed https://github.com/openjdk/jdk/pull/7643#issuecomment-1070307750 and https://github.com/openjdk/jdk/pull/7643#issuecomment-1070348395 ] there is minor difference (AFICS) in bold, italic corresponds to sansserif (2-3 position), monospaced (10-11), dialog (14-15), dialoginput (18-19) which used Gulim before fix but used Malgun Gothic Bold with fix **which i guess is expected**..

Also, there is clear difference in 6-8 th position for Bold, Italic, Bold-Italic which corresponds to **serif font** which uses same Batang font before and after the fix which I guess is because of the "first form", "second form" fallback issue discusses in this thread . I think with the fix `allfont.korean-fallback` is in effect so it uses malgun (which is why 1-4 position looks same as 5.-8) and without fix, it uses Batang

-------------

PR: https://git.openjdk.java.net/jdk/pull/7643