| Summary: | String.prototype.trim removes U+200B from strings. | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Lasse R.H. Nielsen <lrn> | ||||
| Component: | JavaScriptCore | Assignee: | Geoffrey Garen <ggaren> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | ap, ggaren, m.goleb+bugzilla, oliver | ||||
| Priority: | P2 | ||||||
| Version: | 525.x (Safari 3.2) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
Adding this character was intentional, see discussion in bug 26590. Seems like maybe we should remove it now to match the new standard. Looks like bug 26590 was written to match a draft specification, but no ratified specification ever included u200b, Unicode does not include u200b in the 'Separator, Space' category, and Firefox does not include it either. Created attachment 230402 [details]
Patch
Comment on attachment 230402 [details]
Patch
r=me
Committed r167951: <http://trac.webkit.org/changeset/167951> |
In the current version of Safari on a Mac, the String.prototype.trim function removes "\u200b" characters from the string. That character is not recognized as whitespace by the ECMAScript 5 spec (or the ES6 drafts), and should not be removed. Example code: alert("\u200b".trim().length); // should alert 1, actually alerts 0. Apologies if this has been fixed in a nightly build.