Add a port of hb_icu_script_to_script to HarfbuzzShaper

Add a port of hb_icu_script_to_script to HarfbuzzShaper

Harfbuzz on CrOS does not have hb-icu leading to a compile failure
when system_harfbuzz is turned on to avoid a linker warning.
(see http://crbug.com/353127).

HarfbuzzShaper uses just one function (hb_icu_script_to_script) from
hb-icu and it's a simple wrapper. It's ported to HarfbuzzShaper to avoid
the dependency on hb-icu.

BUG=356929
TEST=content_shell gets built on Linux and no change in layout tests.

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

[jochen (gone - plz use gerrit), 2014-03-27 08:09:45]

lgtm

[eae, 2014-03-27 16:12:45]

LGTM w/nit

https://codereview.chromium.org/214413002/diff/1/Source/platform/fonts/harfbu...
File Source/platform/fonts/harfbuzz/HarfBuzzShaper.cpp (right):

https://codereview.chromium.org/214413002/diff/1/Source/platform/fonts/harfbu...
Source/platform/fonts/harfbuzz/HarfBuzzShaper.cpp:761: // without hb-icu.
Could you include a link to the bug in question here please?

[jungshik at Google, 2014-03-27 17:36:50]

The CQ bit was checked by jshin@chromium.org

[commit-bot: I haz the power, 2014-03-27 17:36:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jshin@chromium.org/214413002/20001

[jungshik at Google, 2014-03-27 17:37:34]

Thanks for the review. Adding to CQ

https://codereview.chromium.org/214413002/diff/1/Source/platform/fonts/harfbu...
File Source/platform/fonts/harfbuzz/HarfBuzzShaper.cpp (right):

https://codereview.chromium.org/214413002/diff/1/Source/platform/fonts/harfbu...
Source/platform/fonts/harfbuzz/HarfBuzzShaper.cpp:761: // without hb-icu.
On 2014/03/27 16:12:45, eae wrote:
> Could you include a link to the bug in question here please?

Done.

[behdad, 2014-03-27 18:38:57]

Grumpy lgtm :)

[commit-bot: I haz the power, 2014-03-27 19:35:32]

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

[commit-bot: I haz the power, 2014-03-27 19:35:32]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[jungshik at Google, 2014-03-28 04:51:01]

The CQ bit was checked by jshin@chromium.org

[commit-bot: I haz the power, 2014-03-28 04:51:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jshin@chromium.org/214413002/20001

[commit-bot: I haz the power, 2014-03-28 06:01:44]

Change committed as 170276