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: |
|
Description
Lasse R.H. Nielsen
2014-03-13 02:58:48 PDT
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> |