Fix removal of quotes from property names

[boopathi]

• Fix don't try and minify props that contain unicode #416
• Fix Only allow safe identifiers as object keys (Fixes #416) #445
• Fix Syntax errors after minifying unicode #415

[boopathi]

@mathiasbynens Would like you to review this. There are a few more corner cases missing I suppose - for example - .123. But, is there a esutils API like - isPropertyNameES5 like isIdentifierES5 ?

[boopathi]

Or would it suffice to only transform valid Identifiers (esutils.keyword.isIdentifierNameES5) and not care about the other corner cases for property names - for example .12e3 or 12e34, default, import ?

[mathiasbynens]

There’s no isPropertyName* API in esutils (yet). I wonder how @Constellation and @michaelficarra feel about potentially adding it.

[michaelficarra]

@mathiasbynens Are you thinking of something like this? https://github.com/shapesecurity/shift-codegen-js/blob/0d09bd8a881c62c1bb85d6ff3cfa09697867ed0a/src/minimal-codegen.js#L636-L643

[mathiasbynens]

@michaelficarra Exactly!

[codecov]

Codecov Report

Merging #543 into master will decrease coverage by 0.52%.
The diff coverage is 54%.

@@            Coverage Diff             @@
##           master     #543      +/-   ##
==========================================
- Coverage   83.39%   82.86%   -0.53%     
==========================================
  Files          42       44       +2     
  Lines        2860     2900      +40     
  Branches     1003     1012       +9     
==========================================
+ Hits         2385     2403      +18     
- Misses        284      303      +19     
- Partials      191      194       +3

Impacted Files	Coverage Δ
...n-transform-property-literals/src/property-name.js	100% <100%> (ø)
...el-plugin-transform-property-literals/src/index.js	100% <100%> (ø)	⬆️
...orm-property-literals/src/escape-string-literal.js	30.3% <30.3%> (ø)
packages/babel-plugin-minify-simplify/src/index.js	80.74% <0%> (+0.12%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e2b315...8cc15b7. Read the comment docs.

[boopathi]

@michaelficarra Thanks a lot. I used the code from that repository and have also added the license. Please review the changes.