Roll harfbuzz-ng 3b8fd9c48f4bde368bf2d465c148b9743a9216ee

Roll harfbuzz-ng 3b8fd9c48f4bde368bf2d465c148b9743a9216ee

The revision provides HB_NO_MT macro that disables multi-threading
support. We don't need multi-threading support for harfbuzz-ng.

BUG=none
TEST=compiled ("make harfbuzz-ng")


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140446

[bashi, 2012-06-04 04:32:01]

Hi Tony, Behdad,

Could you take a look? The current revision fails to build on mac because the
build system treats warnings as errors and the revision generates warnings when
there is no multi-threading support on the system. The latest revision provides
a way to disable multi-threading support.

Files to review:
- README.chromium
- harfbuzz.gyp

[behdad, 2012-06-04 12:26:39]

On 2012/06/04 04:32:01, bashik wrote:
> Hi Tony, Behdad,
> 
> Could you take a look? The current revision fails to build on mac because the
> build system treats warnings as errors and the revision generates warnings
when
> there is no multi-threading support on the system. The latest revision
provides
> a way to disable multi-threading support.
> 
> Files to review:
> - README.chromium
> - harfbuzz.gyp

LGTM.

[tony, 2012-06-04 16:40:49]

LGTM, does this change the results of any layout tests?

[bashi, 2012-06-04 22:56:55]

On 2012/06/04 16:40:49, tony wrote:
> LGTM, does this change the results of any layout tests?

This CL doesn't change any layout tests yet.
css3/font-feature-settings-rendering.html will be affected on mac after
https://bugs.webkit.org/show_bug.cgi?id=69826 is landed.

[commit-bot: I haz the power, 2012-06-04 22:57:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bashi@chromium.org/10510004/1

[commit-bot: I haz the power, 2012-06-05 00:33:22]

Change committed as 140446