Preserve platform specific names in fonts.

[ILOVEPIE]

Description

This PR redoes the names table handling to preserve platform specific naming schemes for fonts.

Motivation and Context

I have a project that depends on this working correctly. This closes issue #527.

How Has This Been Tested?

All test cases pass.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I did npm run test and all tests passed green (including code styling checks).
• I have added tests to cover my changes.
• I have updated the README accordingly.
• I have read the CONTRIBUTING document.

[ILOVEPIE]

@Connum sorry, can you re-do your review on this repository.

[Connum]

As I didn't have access to the former review, I did it again quickly - hope I caught everything! :)

[ILOVEPIE]

@Connum when you get the time, I've updated it to the latest master and fixed those issues you mentioned, needs another review.

[Connum]

I noticed that the changes break the font inspector when running npm run start. I made a PR in your branch to fix this.

[Connum]

Looks good!

[ILOVEPIE]

Just need this to be signed off again, as I bumped the version to 2.0.0 because of the breaking change. @Connum

[yne]

@ILOVEPIE Could you share the google-closure-compiler command line you used ?

I can't find a way to run it without heavily modifying the opentype.js source code

[ILOVEPIE]

@yne I didn't use closure compiler on opentype.js, that externs file is a file that is used by other projects to inform their closure compiler as to what functions/types/properties opentype.js exposes.