Allowing post area names to be >32767.

Allowing post area names to be >32767.

Some fonts that cover all of unicode require post name entries greater than
32767. While the standard strictly requires this limit. Common implementations
support it (it was an artifact of 16-bit windows).
Permitting this variance from the specification.

BUG=None
TEST=unifont.ttf passes validation

[bradn, 2013-11-16 22:56:29]

Hi Yusuke,

I happened to run into Paul Hardy (maintainer of unifont) this weekend.
He mentioned that chromium does not accept unifont for use as a web font due to
a strict interpretation of the otf font spec.

Is it possible we can loosen this requirement?
I'm confess I'm largely ignorant of the details of this format so I defer to
your wisdom.

-BradN

[Yusuke Sato, 2013-11-18 01:21:08]

Thanks for the patch, Brad.

LG, but I'd defer to ksakamoto@ who is a current owner of the module.

[unifoundry_gmail.com, 2013-11-18 05:01:53]

On Sun, Nov 17, 2013 at 5:21 PM, <yusukes@chromium.org> wrote:

> Thanks for the patch, Brad.
>
> LG, but I'd defer to ksakamoto@ who is a current owner of the module.
>
>
I have just written a report on GNU's Savannah that provides a more
thorough description of this issue:

https://savannah.gnu.org/support/index.php?108441

This issue of Unifont having more than 2^15 glyphs (but fewer than 2^16
glyphs) also applies to the other large font you have blacklisted, Wen Quan
Yi (wqy-zenhei.ttf), with over 40,000 glyphs at available point sizes.  I
request that whatever you implement to allow Unifont to be un-blacklisted
also be implemented for Wen Quan Yi because both fonts have the same issue
of > 32,768 glyphs.  I am copying Qianqian Fang, the creator of Wen Quan
Yi, in this email.

Let me know if I can provide any further information beyond what I
described in the Savannah report above.


Paul Hardy

https://codereview.chromium.org/68173026/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Kunihiko Sakamoto, 2013-11-19 06:23:27]

LGTM

Thank you for the patch.
The change looks good. I will land this after doing some security tests.

[Kunihiko Sakamoto, 2013-11-19 07:22:52]

On 2013/11/19 06:23:27, ksakamoto wrote:
> LGTM
> 
> Thank you for the patch.
> The change looks good. I will land this after doing some security tests.

Landed in https://code.google.com/p/ots/source/detail?r=106

[unifoundry_gmail.com, 2013-11-19 13:18:24]

On Mon, Nov 18, 2013 at 11:22 PM, <ksakamoto@chromium.org> wrote:
>
>
> Landed in https://code.google.com/p/ots/source/detail?r=106


Thank you!

https://codereview.chromium.org/68173026/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.