Update the default generic font family settings for Chinese/Japanese

Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available[1]. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better than Yu Gothic at this point.

Win Japanese serif
Changed to Yu Mincho if available[1].
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available[1]. These fonts are available
since 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are fonts available since 10.8/10.9.

[1] The first-available-font-list was added in crbug.com/658646

BUG=579842, 506837, 592199, 587072, 648256
Committed: https://crrev.com/0a4a11b1cfe0a7122fb0120672cce84e2bd78c6d
Cr-Commit-Position: refs/heads/master@{#429366}

[kojii, 2016-10-26 06:59:47]

Description was changed from

==========
WIP: generic-ja

BUG=
==========

to

==========
Update the default generic font family settings for Japanese Windows

This patch updates the default generic font family settings for
Japanese Windows using the first available font list introduced in
crbug.com/658646.

This patch also re-fix crbug.com/506837 in a way that the font falls
back to Yu Gothic only when Meiryo isn't available. This change allows
Japanese Win10 to use Meiryo, while other languages of Win10 (without
Japanese optional fonts installed) to use Yu Gothic.

BUG=579842, 506837
==========

[kojii, 2016-10-26 08:15:02]

The CQ bit was checked by kojii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 08:15:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-26 08:37:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-26 08:37:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kojii, 2016-10-26 13:47:07]

The CQ bit was checked by kojii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 13:47:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kojii, 2016-10-26 14:14:58]

Description was changed from

==========
Update the default generic font family settings for Japanese Windows

This patch updates the default generic font family settings for
Japanese Windows using the first available font list introduced in
crbug.com/658646.

This patch also re-fix crbug.com/506837 in a way that the font falls
back to Yu Gothic only when Meiryo isn't available. This change allows
Japanese Win10 to use Meiryo, while other languages of Win10 (without
Japanese optional fonts installed) to use Yu Gothic.

BUG=579842, 506837
==========

to

==========
Allow the default generic font family settings to find the first available font

This patch allows the default generic font family settings to have a
list of fonts that resolves to the first available font on the system.

Since only small part of the settings are lists, this patch introduces
a convention that the font name is a list only when it starts with ",".
This avoids scanning the name and checking the font availability when
it's not necessary.

MaybeGetLocalizedFontName is moved to FontSettingsUtilities to
eliminate the duplication in options and extensions.

The extension API chrome.fontSettings.getFont[1] is unchanged. It
resolves to the first available font on getting the value.

Resource changes in grd/xtb files will be in following patches (e.g.,
crrev.com/2454583002)

[1] https://developer.chrome.com/extensions/fontSettings#method-getFont

BUG=658646
==========

[kojii, 2016-10-26 14:44:37]

Description was changed from

==========
Allow the default generic font family settings to find the first available font

This patch allows the default generic font family settings to have a
list of fonts that resolves to the first available font on the system.

Since only small part of the settings are lists, this patch introduces
a convention that the font name is a list only when it starts with ",".
This avoids scanning the name and checking the font availability when
it's not necessary.

MaybeGetLocalizedFontName is moved to FontSettingsUtilities to
eliminate the duplication in options and extensions.

The extension API chrome.fontSettings.getFont[1] is unchanged. It
resolves to the first available font on getting the value.

Resource changes in grd/xtb files will be in following patches (e.g.,
crrev.com/2454583002)

[1] https://developer.chrome.com/extensions/fontSettings#method-getFont

BUG=658646
==========

to

==========
Update the default generic font family settings for Japanese Windows

This patch updates the default generic font family settings for
Japanese Windows using the first available font list introduced in
crbug.com/658646.

This patch also re-fix crbug.com/506837 in a way that the font falls
back to Yu Gothic only when Meiryo isn't available. This change allows
Japanese Win10 to use Meiryo, while other languages of Win10 (without
Japanese optional fonts installed) to use Yu Gothic.

BUG=579842, 506837
==========

[commit-bot: I haz the power, 2016-10-26 15:21:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-26 15:21:40]

Dry run: This issue passed the CQ dry run.

[kojii, 2016-10-26 23:15:18]

The CQ bit was checked by kojii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 23:15:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-27 01:53:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-27 01:53:12]

Dry run: This issue passed the CQ dry run.

[kojii, 2016-10-31 08:13:57]

The CQ bit was checked by kojii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-31 08:14:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-31 09:02:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-31 09:02:58]

Dry run: This issue passed the CQ dry run.

[kojii, 2016-10-31 13:55:20]

The CQ bit was checked by kojii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-31 13:55:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kojii, 2016-10-31 14:31:52]

The CQ bit was checked by kojii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-31 14:31:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kojii, 2016-10-31 15:04:36]

Description was changed from

==========
Update the default generic font family settings for Japanese Windows

This patch updates the default generic font family settings for
Japanese Windows using the first available font list introduced in
crbug.com/658646.

This patch also re-fix crbug.com/506837 in a way that the font falls
back to Yu Gothic only when Meiryo isn't available. This change allows
Japanese Win10 to use Meiryo, while other languages of Win10 (without
Japanese optional fonts installed) to use Yu Gothic.

BUG=579842, 506837
==========

to

==========
Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better at this point.

Win Japanese serif
Changed to Yu Mincho if available.
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available. These are new fonts in 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are available since 10.8/10.9.

BUG=579842, 506837, 592199, 587072, 648256
==========

[kojii, 2016-10-31 15:07:36]

Description was changed from

==========
Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better at this point.

Win Japanese serif
Changed to Yu Mincho if available.
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available. These are new fonts in 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are available since 10.8/10.9.

BUG=579842, 506837, 592199, 587072, 648256
==========

to

==========
Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available[1]. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better at this point.

Win Japanese serif
Changed to Yu Mincho if available[1].
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available[1]. These fonts are available
since 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are fonts available since 10.8/10.9.

[1] The first-available-font-list was added in crbug.com/658646

BUG=579842, 506837, 592199, 587072, 648256
==========

[commit-bot: I haz the power, 2016-10-31 15:12:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-31 15:12:24]

Dry run: This issue passed the CQ dry run.

[kojii, 2016-10-31 15:15:50]

Description was changed from

==========
Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available[1]. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better at this point.

Win Japanese serif
Changed to Yu Mincho if available[1].
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available[1]. These fonts are available
since 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are fonts available since 10.8/10.9.

[1] The first-available-font-list was added in crbug.com/658646

BUG=579842, 506837, 592199, 587072, 648256
==========

to

==========
Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available[1]. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better than Yu Gothic at this point.

Win Japanese serif
Changed to Yu Mincho if available[1].
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available[1]. These fonts are available
since 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are fonts available since 10.8/10.9.

[1] The first-available-font-list was added in crbug.com/658646

BUG=579842, 506837, 592199, 587072, 648256
==========

[kojii, 2016-10-31 15:20:13]

kojii@chromium.org changed reviewers:
+ battre@chromium.org, jshin@chromium.org

[kojii, 2016-10-31 15:20:14]

PTAL.

battre@: prefs_tab_helper.cc
jshin@: all

clang-format decided to re-format the whole array for prefs_tab_helper.cc, but
it's adding only two entries.

[jungshik at Google, 2016-11-01 00:12:02]

https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
File chrome/app/resources/locale_settings_win.grd (right):

https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
chrome/app/resources/locale_settings_win.grd:235: ,Yu Mincho,MS PMincho,Times
New Roman
What's the point of Times New Roman at the end? It does not cover Japanese so
that having it just delays the trigger of next font fallback process, no?

https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
chrome/app/resources/locale_settings_win.grd:308: Microsoft JhengHei
Does non-Trad-Chinese Windows 10 (out-of-box) have this font and other TC fonts?
If not, shouldn't we do the same for TC?

[kojii, 2016-11-01 02:35:23]

https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
File chrome/app/resources/locale_settings_win.grd (right):

https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
chrome/app/resources/locale_settings_win.grd:235: ,Yu Mincho,MS PMincho,Times
New Roman
On 2016/11/01 at 00:12:02, jungshik at google wrote:
> What's the point of Times New Roman at the end? It does not cover Japanese so
that having it just delays the trigger of next font fallback process, no?

Yes. English Win10 has 0 serif Japanese font, so doing this will:
* Japanese will fallback to next, then to system fallback that uses Meiryo/Yu
Gothic.
* Latin characters can use serif font.
Not really ideal, but at least Latin rendered in serif looks good to me. WDYT?

https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
chrome/app/resources/locale_settings_win.grd:308: Microsoft JhengHei
On 2016/11/01 at 00:12:01, jungshik at google wrote:
> Does non-Trad-Chinese Windows 10 (out-of-box) have this font and other TC
fonts? If not, shouldn't we do the same for TC?

It does, and this font is available since Vista, so we don't need fallback on
any supported platforms.

Japanese added Meiryo in Vista, but then Yu Gothic in 8.1, and Win10 defaulting
to Yu Gothic. Vista and Win10 using different default needs
first-available-list, but Chinese use the same default.

[jungshik at Google, 2016-11-01 18:50:38]

On 2016/11/01 02:35:23, kojii wrote:
>
https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
> File chrome/app/resources/locale_settings_win.grd (right):
> 
>
https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
> chrome/app/resources/locale_settings_win.grd:235: ,Yu Mincho,MS PMincho,Times
> New Roman
> On 2016/11/01 at 00:12:02, jungshik at google wrote:
> > What's the point of Times New Roman at the end? It does not cover Japanese
so
> that having it just delays the trigger of next font fallback process, no?
> 
> Yes. English Win10 has 0 serif Japanese font, so doing this will:
> * Japanese will fallback to next, then to system fallback that uses Meiryo/Yu
> Gothic.
> * Latin characters can use serif font.
> Not really ideal, but at least Latin rendered in serif looks good to me. WDYT?

Hmm... I'm not sure if it's worth listing TNR to get Latin to be serif. BTW, you
don't want Hans Serif fonts to be used for lang=ja/generic=serif, do you?  How
about just listing 'Meiryo, Yu Gothic' in place of TNR?  'generic=serif' is
completely ignored if we do that unless users of non-Japanese Win 10 takes an
extra step to install Japanese serif fonts. 

What would you say to that?  

>
https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
> chrome/app/resources/locale_settings_win.grd:308: Microsoft JhengHei
> On 2016/11/01 at 00:12:01, jungshik at google wrote:
> > Does non-Trad-Chinese Windows 10 (out-of-box) have this font and other TC
> fonts? If not, shouldn't we do the same for TC?
> 
> It does, and this font is available since Vista, so we don't need fallback on
> any supported platforms.

Thank you for the clarification on MS JH + Win 10. 

I know that MS JhengHei was added to Vista. My worry was that MS may have done
the same thing with MS JH as they did with Meiryo on Win 10. 
It's good that it didn't. As you know well, MS reversed its font availability
policy on Win 10 (going back to Win XP days, in a sense). At least on Windows 7,
I believe virtually all the fonts are available out of box (no need to take an
extra step to install CJK or complex script fonts). On Windows 10, users need to
take extra steps to get more fonts (e.g. Serif Japanese fonts or Meiryo). 



BTW, it's very tempting to suggest that 'Noto Sans CJK {JP,KR,TC,SC}, Noto Sans
{JP,KR,TC,SC}' be at the beginning of lists for ja, ko, zh-TW, zh-CN /
sans-serif preferences. :-) 

Thank you for making it possible to have an ordered-list.

[kojii, 2016-11-02 01:27:49]

On 2016/11/01 at 18:50:38, jshin wrote:
> On 2016/11/01 02:35:23, kojii wrote:
> >
https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
> > File chrome/app/resources/locale_settings_win.grd (right):
> > 
> >
https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
> > chrome/app/resources/locale_settings_win.grd:235: ,Yu Mincho,MS
PMincho,Times
> > New Roman
> > On 2016/11/01 at 00:12:02, jungshik at google wrote:
> > > What's the point of Times New Roman at the end? It does not cover Japanese
so
> > that having it just delays the trigger of next font fallback process, no?
> > 
> > Yes. English Win10 has 0 serif Japanese font, so doing this will:
> > * Japanese will fallback to next, then to system fallback that uses
Meiryo/Yu
> > Gothic.
> > * Latin characters can use serif font.
> > Not really ideal, but at least Latin rendered in serif looks good to me.
WDYT?
> 
> Hmm... I'm not sure if it's worth listing TNR to get Latin to be serif. BTW,
you don't want Hans Serif fonts to be used for lang=ja/generic=serif, do you? 
How about just listing 'Meiryo, Yu Gothic' in place of TNR?  'generic=serif' is
completely ignored if we do that unless users of non-Japanese Win 10 takes an
extra step to install Japanese serif fonts. 
> 
> What would you say to that?  

I agree it's unusual, Gecko has something like this and thought maybe good, but
you worrying is a good sign that this isn't a good idea.

I'll just remove it, we'll then likely to go system fallback which uses
Meiryo,Yu Gothic anyway. We're shipping MS PGothic and nobody claims about this.

> 
> >
https://codereview.chromium.org/2454583002/diff/120001/chrome/app/resources/l...
> > chrome/app/resources/locale_settings_win.grd:308: Microsoft JhengHei
> > On 2016/11/01 at 00:12:01, jungshik at google wrote:
> > > Does non-Trad-Chinese Windows 10 (out-of-box) have this font and other TC
> > fonts? If not, shouldn't we do the same for TC?
> > 
> > It does, and this font is available since Vista, so we don't need fallback
on
> > any supported platforms.
> 
> Thank you for the clarification on MS JH + Win 10. 
> 
> I know that MS JhengHei was added to Vista. My worry was that MS may have done
the same thing with MS JH as they did with Meiryo on Win 10. 
> It's good that it didn't. As you know well, MS reversed its font availability
policy on Win 10 (going back to Win XP days, in a sense). At least on Windows 7,
I believe virtually all the fonts are available out of box (no need to take an
extra step to install CJK or complex script fonts). On Windows 10, users need to
take extra steps to get more fonts (e.g. Serif Japanese fonts or Meiryo). 

Yeah, Win10 has at least one font for each script even in non-native versions,
so if Vista has Win10 default, and the font rendering quality is good,
first-available-list isn't needed. Meiryo,Yu Gothic hit this criteria. And serif
CJK too.

> BTW, it's very tempting to suggest that 'Noto Sans CJK {JP,KR,TC,SC}, Noto
Sans {JP,KR,TC,SC}' be at the beginning of lists for ja, ko, zh-TW, zh-CN /
sans-serif preferences. :-) 

Ah, good point. The beginning, I'm not sure though. Last I checked, Win font
renderer can't render Noto at better quality than built-in fonts (it's CFF,
which is good for Linux/Android though.) Can we do this separately, after I
checked the rendering quality? It should then be resource only, you and I can
land it.

> Thank you for making it possible to have an ordered-list.

;)

[kojii, 2016-11-02 01:31:56]

The CQ bit was checked by kojii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-02 01:32:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kojii, 2016-11-02 01:33:08]

jshin@ PTAL, removed Times New Roman.

battre@ PTAL for prefs_tab_helper.cc. I'll not land until jshin@ signed off.

[commit-bot: I haz the power, 2016-11-02 02:28:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-02 02:28:10]

Dry run: This issue passed the CQ dry run.

[battre, 2016-11-02 09:50:39]

lgtm

[jungshik at Google, 2016-11-02 18:10:11]

LGTM

https://codereview.chromium.org/2454583002/diff/140001/chrome/app/resources/p...
File chrome/app/resources/platform_locale_settings/locale_settings_win_ja.xtb
(right):

https://codereview.chromium.org/2454583002/diff/140001/chrome/app/resources/p...
chrome/app/resources/platform_locale_settings/locale_settings_win_ja.xtb:6:
<translation id="IDS_SERIF_FONT_FAMILY">,Yu Mincho,MS PMincho</translation>
I'd append "Meiryo,Yu Gothic" to the end to prevent a font fallback from picking
up a non-Japanese font (e.g. Hans or Korean font). This has to be done to other
generic family for which the presence of the font listed is not guaranteed.  
Aha...  I think our font fallback already takes into account UI language to
prioritize the font suitable for the UI language. In that case, this won't be
necessary.

[kojii, 2016-11-02 19:27:11]

The CQ bit was checked by kojii@chromium.org

[kojii, 2016-11-02 19:27:12]

Thank you for your review.

https://codereview.chromium.org/2454583002/diff/140001/chrome/app/resources/p...
File chrome/app/resources/platform_locale_settings/locale_settings_win_ja.xtb
(right):

https://codereview.chromium.org/2454583002/diff/140001/chrome/app/resources/p...
chrome/app/resources/platform_locale_settings/locale_settings_win_ja.xtb:6:
<translation id="IDS_SERIF_FONT_FAMILY">,Yu Mincho,MS PMincho</translation>
On 2016/11/02 at 18:10:11, jungshik at google wrote:
> I'd append "Meiryo,Yu Gothic" to the end to prevent a font fallback from
picking up a non-Japanese font (e.g. Hans or Korean font). This has to be done
to other generic family for which the presence of the font listed is not
guaranteed.  
> Aha...  I think our font fallback already takes into account UI language to
prioritize the font suitable for the UI language. In that case, this won't be
necessary.

Right, we do. We use first-CJK lang of lang, accept-lang, UI lang to
disambiguate unified Han.

[commit-bot: I haz the power, 2016-11-02 19:28:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-02 19:36:57]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-11-02 19:55:23]

Description was changed from

==========
Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available[1]. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better than Yu Gothic at this point.

Win Japanese serif
Changed to Yu Mincho if available[1].
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available[1]. These fonts are available
since 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are fonts available since 10.8/10.9.

[1] The first-available-font-list was added in crbug.com/658646

BUG=579842, 506837, 592199, 587072, 648256
==========

to

==========
Update the default generic font family settings for Chinese/Japanese

This patch updates the default generic font family settings for
Chinese and Japanese.

Win Japanese standard/sans-serif
Changed to Meiryo if available[1]. It was once Meiryo, but
crbug.com/506837 changed to Yu Gothic because Meiryo is not available
on non-Japanese versions of Win10.
Meiryo is preferred when available because its rendering quality is
better than Yu Gothic at this point.

Win Japanese serif
Changed to Yu Mincho if available[1].
Also added Times New Roman as fallback, since neither Yu Mincho nor MS
PMincho are available on non-Japanese versions of Win10.

Win Chinese cursive
Changed to Kaiti/DFKai-SB (SC/TC.)

Mac Chinese standard/sans-serif
Changed to PingFang SC/TC if available[1]. These fonts are available
since 10.11.

Mac Chinese cursive
Changed to Kaiti SC/TC. These are fonts available since 10.8/10.9.

[1] The first-available-font-list was added in crbug.com/658646

BUG=579842, 506837, 592199, 587072, 648256

Committed: https://crrev.com/0a4a11b1cfe0a7122fb0120672cce84e2bd78c6d
Cr-Commit-Position: refs/heads/master@{#429366}
==========

[commit-bot: I haz the power, 2016-11-02 19:55:24]

Patchset 8 (id:??) landed as
https://crrev.com/0a4a11b1cfe0a7122fb0120672cce84e2bd78c6d
Cr-Commit-Position: refs/heads/master@{#429366}