Pass script-based locale names to Skia

Pass script-based locale names to Skia

Skia on Android (required by /system/etc/fallback_fonts.xml) on L
requires script-based locale names (e.g. zh-Hans and zh-Hant), but
chromium uses region-based locale names (e.g. zh-CN and zh-TW).

Convert region-based locale names to scirpt-based locale names before
passing locale names to Skia.

For now this is for "zh" only.

BUG=433074
TEST=ManualTests/zh-font-fallback-android.html

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185464

[Xianzhu, 2014-11-14 19:35:58]

wangxianzhu@chromium.org changed reviewers:
+ dominik.rottsches@intel.com, eae@chromium.org

[eae, 2014-11-14 19:42:39]

On 2014/11/14 19:35:59, Xianzhu wrote:

Could you add a test for this please?

[Xianzhu, 2014-11-14 20:04:24]

On 2014/11/14 19:42:39, eae wrote:
> On 2014/11/14 19:35:59, Xianzhu wrote:
> 
> Could you add a test for this please?

This seems not feasible for now because the result depends on font
configurations of the system. This change affects only Android L release on
which zh-Hant and zh-Hans are used in /etc/system/fallback_fonts.xml. This
doesn't affect Android K and early releases because there is no zh-specific
rules in their /etc/system/fallback_fonts.xml.

A pixel test might work if our layout test system supported baselines for
different Android platforms, but not feasible for now.

Would a ManualTest look good?

[Xianzhu, 2014-11-17 17:54:29]

On 2014/11/14 20:04:24, Xianzhu wrote:
> On 2014/11/14 19:42:39, eae wrote:
> > On 2014/11/14 19:35:59, Xianzhu wrote:
> > 
> > Could you add a test for this please?
> 
> This seems not feasible for now because the result depends on font
> configurations of the system. This change affects only Android L release on
> which zh-Hant and zh-Hans are used in /etc/system/fallback_fonts.xml. This
> doesn't affect Android K and early releases because there is no zh-specific
> rules in their /etc/system/fallback_fonts.xml.
> 
> A pixel test might work if our layout test system supported baselines for
> different Android platforms, but not feasible for now.
> 
> Would a ManualTest look good?

Added a manual test. PTAL.

[eae, 2014-11-17 19:17:04]

LGTM

[Xianzhu, 2014-11-17 19:20:56]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-11-17 19:21:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/729573003/20001

[commit-bot: I haz the power, 2014-11-17 20:26:56]

Committed patchset #2 (id:20001) as 185464