Add a Windows layout test for validating symbol CMAP encoding support

Add a Windows layout test for validating symbol CMAP encoding support

In version 1.3.0 HarfBuzz adds support [1] for symbol encoding
CMAP tables (platform ID = 3, encoding ID = 0, glyphs offset by
0xF000), for which support in Blink regressed when we moved to
the OpenType functions from HarfBuzz. This layout test adds a
custom SymbolCMAPTest font which uses the Microsoft specific
symbol CMAP encoding. It also adds a reference font which has the
same glyph for comparison at Private Use area codepoint 0xE000.

The test font's CMAP table for platformID="3" looks as follows,
when dumped using ttx:

    <cmap_format_4 platformID="3" platEncID="0" language="0">
      <map code="0xf020" name="space"/>
      <map code="0xf041" name="A"/>
    </cmap_format_4>

So, it was verified to follow the Symbol CMAP encoding pattern.

[1] https://github.com/behdad/harfbuzz/commit/34f9aa582c3a03b578c7eae3d2e8860a0bd5cb00

BUG=627953
R=behdad,kojii,eae

Committed: https://crrev.com/b550f7d3b713058b9a719fdfd79cd863638d6327
Cr-Commit-Position: refs/heads/master@{#407794}

[drott, 2016-07-26 13:06:19]

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

[commit-bot: I haz the power, 2016-07-26 13:06:33]

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-07-26 13:06:34]

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

[commit-bot: I haz the power, 2016-07-26 13:06:35]

Dry run: CLs for remote refs other than refs/pending/heads/master must contain
NOTRY=true and NOPRESUBMIT=true in order for the CQ to process them

[drott, 2016-07-26 13:25:18]

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

[commit-bot: I haz the power, 2016-07-26 13:25:38]

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-07-26 13:25:39]

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

[commit-bot: I haz the power, 2016-07-26 13:25:40]

Dry run: CLs for remote refs other than refs/pending/heads/master must contain
NOTRY=true and NOPRESUBMIT=true in order for the CQ to process them

[drott, 2016-07-26 13:35:02]

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

[commit-bot: I haz the power, 2016-07-26 13:35: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-07-26 13:35:20]

lgtm, you're amazing.

[drott, 2016-07-26 13:36:55]

Description was changed from

==========
Add a Windows layout test for validating symbol CMAP encoding support

In version 1.3.0 HarfBuzz adds support [1] for symbol encoding
CMAP tables (platform ID = 3, encoding ID = 0, glyphs offset by
0xFF00), for which support in Blink regressed when we moved to
the OpenType functions from HarfBuzz. This layout test adds a
custom SymbolCMAPTest font which uses the Microsoft specific
symbol CMAP encoding. It also adds a reference font which has the
same glyph for comparison at Private Use are codepoint 0xE000.

The test font's CMAP table for platformID="3" looks as follows,
when dumped using ttx:

    <cmap_format_4 platformID="3" platEncID="0" language="0">
      <map code="0xf020" name="space"/>
      <map code="0xf041" name="A"/>
    </cmap_format_4>

So, it was verified to follow the Symbol CMAP encoding pattern.

[1]
https://github.com/behdad/harfbuzz/commit/34f9aa582c3a03b578c7eae3d2e8860a0bd...

BUG=627953
R=behdad,kojii,eae
==========

to

==========
Add a Windows layout test for validating symbol CMAP encoding support

In version 1.3.0 HarfBuzz adds support [1] for symbol encoding
CMAP tables (platform ID = 3, encoding ID = 0, glyphs offset by
0xFF00), for which support in Blink regressed when we moved to
the OpenType functions from HarfBuzz. This layout test adds a
custom SymbolCMAPTest font which uses the Microsoft specific
symbol CMAP encoding. It also adds a reference font which has the
same glyph for comparison at Private Use area codepoint 0xE000.

The test font's CMAP table for platformID="3" looks as follows,
when dumped using ttx:

    <cmap_format_4 platformID="3" platEncID="0" language="0">
      <map code="0xf020" name="space"/>
      <map code="0xf041" name="A"/>
    </cmap_format_4>

So, it was verified to follow the Symbol CMAP encoding pattern.

[1]
https://github.com/behdad/harfbuzz/commit/34f9aa582c3a03b578c7eae3d2e8860a0bd...

BUG=627953
R=behdad,kojii,eae
==========

[drott, 2016-07-26 13:39:33]

On 2016/07/26 at 13:35:20, kojii wrote:
> lgtm, you're amazing.

Very kind, Koji, thank you (for the quick review as well)!

[behdad, 2016-07-26 14:19:12]

lgtm

[commit-bot: I haz the power, 2016-07-26 14:44:25]

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

[commit-bot: I haz the power, 2016-07-26 14:44:26]

Dry run: This issue passed the CQ dry run.

[drott, 2016-07-26 14:50:58]

The CQ bit was checked by drott@chromium.org

[commit-bot: I haz the power, 2016-07-26 14:51:15]

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

[drott, 2016-07-26 14:51:41]

Description was changed from

==========
Add a Windows layout test for validating symbol CMAP encoding support

In version 1.3.0 HarfBuzz adds support [1] for symbol encoding
CMAP tables (platform ID = 3, encoding ID = 0, glyphs offset by
0xFF00), for which support in Blink regressed when we moved to
the OpenType functions from HarfBuzz. This layout test adds a
custom SymbolCMAPTest font which uses the Microsoft specific
symbol CMAP encoding. It also adds a reference font which has the
same glyph for comparison at Private Use area codepoint 0xE000.

The test font's CMAP table for platformID="3" looks as follows,
when dumped using ttx:

    <cmap_format_4 platformID="3" platEncID="0" language="0">
      <map code="0xf020" name="space"/>
      <map code="0xf041" name="A"/>
    </cmap_format_4>

So, it was verified to follow the Symbol CMAP encoding pattern.

[1]
https://github.com/behdad/harfbuzz/commit/34f9aa582c3a03b578c7eae3d2e8860a0bd...

BUG=627953
R=behdad,kojii,eae
==========

to

==========
Add a Windows layout test for validating symbol CMAP encoding support

In version 1.3.0 HarfBuzz adds support [1] for symbol encoding
CMAP tables (platform ID = 3, encoding ID = 0, glyphs offset by
0xF000), for which support in Blink regressed when we moved to
the OpenType functions from HarfBuzz. This layout test adds a
custom SymbolCMAPTest font which uses the Microsoft specific
symbol CMAP encoding. It also adds a reference font which has the
same glyph for comparison at Private Use area codepoint 0xE000.

The test font's CMAP table for platformID="3" looks as follows,
when dumped using ttx:

    <cmap_format_4 platformID="3" platEncID="0" language="0">
      <map code="0xf020" name="space"/>
      <map code="0xf041" name="A"/>
    </cmap_format_4>

So, it was verified to follow the Symbol CMAP encoding pattern.

[1]
https://github.com/behdad/harfbuzz/commit/34f9aa582c3a03b578c7eae3d2e8860a0bd...

BUG=627953
R=behdad,kojii,eae
==========

[commit-bot: I haz the power, 2016-07-26 14:54:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-26 14:55:55]

Description was changed from

==========
Add a Windows layout test for validating symbol CMAP encoding support

In version 1.3.0 HarfBuzz adds support [1] for symbol encoding
CMAP tables (platform ID = 3, encoding ID = 0, glyphs offset by
0xF000), for which support in Blink regressed when we moved to
the OpenType functions from HarfBuzz. This layout test adds a
custom SymbolCMAPTest font which uses the Microsoft specific
symbol CMAP encoding. It also adds a reference font which has the
same glyph for comparison at Private Use area codepoint 0xE000.

The test font's CMAP table for platformID="3" looks as follows,
when dumped using ttx:

    <cmap_format_4 platformID="3" platEncID="0" language="0">
      <map code="0xf020" name="space"/>
      <map code="0xf041" name="A"/>
    </cmap_format_4>

So, it was verified to follow the Symbol CMAP encoding pattern.

[1]
https://github.com/behdad/harfbuzz/commit/34f9aa582c3a03b578c7eae3d2e8860a0bd...

BUG=627953
R=behdad,kojii,eae
==========

to

==========
Add a Windows layout test for validating symbol CMAP encoding support

In version 1.3.0 HarfBuzz adds support [1] for symbol encoding
CMAP tables (platform ID = 3, encoding ID = 0, glyphs offset by
0xF000), for which support in Blink regressed when we moved to
the OpenType functions from HarfBuzz. This layout test adds a
custom SymbolCMAPTest font which uses the Microsoft specific
symbol CMAP encoding. It also adds a reference font which has the
same glyph for comparison at Private Use area codepoint 0xE000.

The test font's CMAP table for platformID="3" looks as follows,
when dumped using ttx:

    <cmap_format_4 platformID="3" platEncID="0" language="0">
      <map code="0xf020" name="space"/>
      <map code="0xf041" name="A"/>
    </cmap_format_4>

So, it was verified to follow the Symbol CMAP encoding pattern.

[1]
https://github.com/behdad/harfbuzz/commit/34f9aa582c3a03b578c7eae3d2e8860a0bd...

BUG=627953
R=behdad,kojii,eae

Committed: https://crrev.com/b550f7d3b713058b9a719fdfd79cd863638d6327
Cr-Commit-Position: refs/heads/master@{#407794}
==========

[commit-bot: I haz the power, 2016-07-26 14:55:56]

Patchset 2 (id:??) landed as
https://crrev.com/b550f7d3b713058b9a719fdfd79cd863638d6327
Cr-Commit-Position: refs/heads/master@{#407794}

[eae, 2016-07-26 17:09:52]

LGTM