130184 2014-03-13 02:58:48 -0700 String.prototype.trim removes U+200B from strings. 2014-04-29 12:47:49 -0700 1 1 1 Unclassified WebKit JavaScriptCore 525.x (Safari 3.2) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 lrn ggaren ap ggaren m.goleb+bugzilla oliver oldest_to_newest 989975 0 lrn 2014-03-13 02:58:48 -0700 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. 1005049 1 ap 2014-04-29 12:17:12 -0700 Adding this character was intentional, see discussion in bug 26590. Seems like maybe we should remove it now to match the new standard. 1005059 2 ggaren 2014-04-29 12:30:40 -0700 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. 1005063 3 230402 ggaren 2014-04-29 12:42:02 -0700 Created attachment 230402 Patch 1005065 4 230402 msaboff 2014-04-29 12:46:12 -0700 Comment on attachment 230402 Patch r=me 1005067 5 ggaren 2014-04-29 12:47:49 -0700 Committed r167951: <http://trac.webkit.org/changeset/167951> 230402 2014-04-29 12:42:02 -0700 2014-04-29 12:46:12 -0700 Patch bug-130184-20140429124139.patch text/plain 4656 ggaren SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTY3OTUwKQorKysgU291cmNl L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBA CisyMDE0LTA0LTI5ICBHZW9mZnJleSBHYXJlbiAgPGdnYXJlbkBhcHBsZS5jb20+CisKKyAgICAg ICAgU3RyaW5nLnByb3RvdHlwZS50cmltIHJlbW92ZXMgVSsyMDBCIGZyb20gc3RyaW5ncy4KKyAg ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzMDE4NAorCisg ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogcnVudGltZS9T dHJpbmdQcm90b3R5cGUuY3BwOgorICAgICAgICAoSlNDOjp0cmltU3RyaW5nKToKKyAgICAgICAg KEpTQzo6aXNUcmltV2hpdGVzcGFjZSk6IERlbGV0ZWQuCisKIDIwMTQtMDQtMjkgIE1hcmsgTGFt ICA8bWFyay5sYW1AYXBwbGUuY29tPgogCiAgICAgICAgIFpvbWJpZnlpbmcgc3dlZXAgc2hvdWxk IGlnbm9yZSByZXRpcmVkIGJsb2Nrcy4KSW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9ydW50 aW1lL1N0cmluZ1Byb3RvdHlwZS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRD b3JlL3J1bnRpbWUvU3RyaW5nUHJvdG90eXBlLmNwcAkocmV2aXNpb24gMTY3OTQ3KQorKysgU291 cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvU3RyaW5nUHJvdG90eXBlLmNwcAkod29ya2luZyBj b3B5KQpAQCAtMTUwNSwxMSArMTUwNSw2IEBAIGVudW0gewogICAgIFRyaW1SaWdodCA9IDIKIH07 CiAKLXN0YXRpYyBpbmxpbmUgYm9vbCBpc1RyaW1XaGl0ZXNwYWNlKFVDaGFyIGMpCi17Ci0gICAg cmV0dXJuIGlzU3RyV2hpdGVTcGFjZShjKSB8fCBjID09IDB4MjAwYjsKLX0KLQogc3RhdGljIGlu bGluZSBKU1ZhbHVlIHRyaW1TdHJpbmcoRXhlY1N0YXRlKiBleGVjLCBKU1ZhbHVlIHRoaXNWYWx1 ZSwgaW50IHRyaW1LaW5kKQogewogICAgIGlmICghY2hlY2tPYmplY3RDb2VyY2libGUodGhpc1Zh bHVlKSkKQEAgLTE1MTcsMTIgKzE1MTIsMTIgQEAgc3RhdGljIGlubGluZSBKU1ZhbHVlIHRyaW1T dHJpbmcoRXhlY1N0YQogICAgIFN0cmluZyBzdHIgPSB0aGlzVmFsdWUudG9TdHJpbmcoZXhlYykt PnZhbHVlKGV4ZWMpOwogICAgIHVuc2lnbmVkIGxlZnQgPSAwOwogICAgIGlmICh0cmltS2luZCAm IFRyaW1MZWZ0KSB7Ci0gICAgICAgIHdoaWxlIChsZWZ0IDwgc3RyLmxlbmd0aCgpICYmIGlzVHJp bVdoaXRlc3BhY2Uoc3RyW2xlZnRdKSkKKyAgICAgICAgd2hpbGUgKGxlZnQgPCBzdHIubGVuZ3Ro KCkgJiYgaXNTdHJXaGl0ZVNwYWNlKHN0cltsZWZ0XSkpCiAgICAgICAgICAgICBsZWZ0Kys7CiAg ICAgfQogICAgIHVuc2lnbmVkIHJpZ2h0ID0gc3RyLmxlbmd0aCgpOwogICAgIGlmICh0cmltS2lu ZCAmIFRyaW1SaWdodCkgewotICAgICAgICB3aGlsZSAocmlnaHQgPiBsZWZ0ICYmIGlzVHJpbVdo aXRlc3BhY2Uoc3RyW3JpZ2h0IC0gMV0pKQorICAgICAgICB3aGlsZSAocmlnaHQgPiBsZWZ0ICYm IGlzU3RyV2hpdGVTcGFjZShzdHJbcmlnaHQgLSAxXSkpCiAgICAgICAgICAgICByaWdodC0tOwog ICAgIH0KIApJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91 dFRlc3RzL0NoYW5nZUxvZwkocmV2aXNpb24gMTY3OTUwKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTQtMDQtMjkgIEdlb2ZmcmV5 IEdhcmVuICA8Z2dhcmVuQGFwcGxlLmNvbT4KKworICAgICAgICBTdHJpbmcucHJvdG90eXBlLnRy aW0gcmVtb3ZlcyBVKzIwMEIgZnJvbSBzdHJpbmdzLgorICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTMwMTg0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C T0RZIChPT1BTISkuCisKKyAgICAgICAgKiBqcy9zY3JpcHQtdGVzdHMvc3RyaW5nLXRyaW0uanM6 CisgICAgICAgICoganMvc3RyaW5nLXRyaW0tZXhwZWN0ZWQudHh0OgorCiAyMDE0LTA0LTI5ICBB bGV4ZXkgUHJvc2t1cnlha292ICA8YXBAYXBwbGUuY29tPgogCiAgICAgICAgIFJFR1JFU1NJT046 IEludGVybWl0dGVudCBjcmFzaCBpbiBTcGVlY2hTeW50aGVzaXM6OmRpZEZpbmlzaFNwZWFraW5n CkluZGV4OiBMYXlvdXRUZXN0cy9qcy9zdHJpbmctdHJpbS1leHBlY3RlZC50eHQKPT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PQotLS0gTGF5b3V0VGVzdHMvanMvc3RyaW5nLXRyaW0tZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAx Njc5NDcpCisrKyBMYXlvdXRUZXN0cy9qcy9zdHJpbmctdHJpbS1leHBlY3RlZC50eHQJKHdvcmtp bmcgY29weSkKQEAgLTY2LDkgKzY2LDYgQEAgUEFTUyB3aGl0ZXNwYWNlWzE5XS5zLnRyaW1SaWdo dCgpIGlzICcnCiBQQVNTIHdoaXRlc3BhY2VbMjBdLnMudHJpbSgpIGlzICcnCiBQQVNTIHdoaXRl c3BhY2VbMjBdLnMudHJpbUxlZnQoKSBpcyAnJwogUEFTUyB3aGl0ZXNwYWNlWzIwXS5zLnRyaW1S aWdodCgpIGlzICcnCi1QQVNTIHdoaXRlc3BhY2VbMjFdLnMudHJpbSgpIGlzICcnCi1QQVNTIHdo aXRlc3BhY2VbMjFdLnMudHJpbUxlZnQoKSBpcyAnJwotUEFTUyB3aGl0ZXNwYWNlWzIxXS5zLnRy aW1SaWdodCgpIGlzICcnCiBQQVNTIHdzU3RyaW5nLnRyaW0oKSBpcyAnJwogUEFTUyB3c1N0cmlu Zy50cmltTGVmdCgpIGlzICcnCiBQQVNTIHdzU3RyaW5nLnRyaW1SaWdodCgpIGlzICcnCkBAIC0x MDUsNiArMTAyLDkgQEAgUEFTUyB0cmltUmlnaHQuY2FsbCgoe3RvU3RyaW5nOmZ1bmN0aW9uKAog UEFTUyB0cmltLmNhbGwoWydhbicsJ2FycmF5J10pIGlzICdhbixhcnJheScKIFBBU1MgdHJpbUxl ZnQuY2FsbChbJ2FuJywnYXJyYXknXSkgaXMgJ2FuLGFycmF5JwogUEFTUyB0cmltUmlnaHQuY2Fs bChbJ2FuJywnYXJyYXknXSkgaXMgJ2FuLGFycmF5JworUEFTUyB0cmltLmNhbGwoJ+KAiycpIGlz ICfigIsnCitQQVNTIHRyaW1MZWZ0LmNhbGwoJ+KAiycpIGlzICfigIsnCitQQVNTIHRyaW1SaWdo dC5jYWxsKCfigIsnKSBpcyAn4oCLJwogUEFTUyBzdWNjZXNzZnVsbHlQYXJzZWQgaXMgdHJ1ZQog CiBURVNUIENPTVBMRVRFCkluZGV4OiBMYXlvdXRUZXN0cy9qcy9zY3JpcHQtdGVzdHMvc3RyaW5n LXRyaW0uanMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvanMvc2NyaXB0LXRlc3RzL3N0cmlu Zy10cmltLmpzCShyZXZpc2lvbiAxNjc5NDcpCisrKyBMYXlvdXRUZXN0cy9qcy9zY3JpcHQtdGVz dHMvc3RyaW5nLXRyaW0uanMJKHdvcmtpbmcgY29weSkKQEAgLTMzLDcgKzMzLDYgQEAgdmFyIHdo aXRlc3BhY2UgICAgICA9IFsKICAgICB7cyA6ICdcdTMwMDAnLCB0IDogJ0lERU9HUkFQSElDIFNQ QUNFJ30sCiAgICAge3MgOiAnXHUyMDI4JywgdCA6ICdMSU5FIFNFUEFSQVRPUid9LAogICAgIHtz IDogJ1x1MjAyOScsIHQgOiAnUEFSQUdSQVBIIFNFUEFSQVRPUid9LAotICAgIHtzIDogJ1x1MjAw QicsIHQgOiAnWkVSTyBXSURUSCBTUEFDRSAoY2F0ZWdvcnkgQ2YpJ30KIF07CiAKIGZvciAodmFy IGkgPSAwOyBpIDwgd2hpdGVzcGFjZS5sZW5ndGg7IGkrKykgewpAQCAtNjMsMTAgKzYyLDkgQEAg c2hvdWxkQmUoInJpZ2h0VHJpbVN0cmluZy50cmltKCkiLCAgICAgIAogc2hvdWxkQmUoInJpZ2h0 VHJpbVN0cmluZy50cmltTGVmdCgpIiwgICJ0ZXN0U3RyaW5nIik7CiBzaG91bGRCZSgicmlnaHRU cmltU3RyaW5nLnRyaW1SaWdodCgpIiwgInJpZ2h0VHJpbVN0cmluZyIpOwogCi12YXIgdGVzdFZh bHVlcyA9IFsiMCIsICJJbmZpbml0eSIsICJOYU4iLCAidHJ1ZSIsICJmYWxzZSIsICIoe30pIiwg Iih7dG9TdHJpbmc6ZnVuY3Rpb24oKXtyZXR1cm4gJ3dpYmJsZSd9fSkiLCAiWydhbicsJ2FycmF5 J10iXTsKK3ZhciB0ZXN0VmFsdWVzID0gWyIwIiwgIkluZmluaXR5IiwgIk5hTiIsICJ0cnVlIiwg ImZhbHNlIiwgIih7fSkiLCAiKHt0b1N0cmluZzpmdW5jdGlvbigpe3JldHVybiAnd2liYmxlJ319 KSIsICJbJ2FuJywnYXJyYXknXSIsICInXHUyMDBiJyJdOwogZm9yICh2YXIgaSA9IDA7IGkgPCB0 ZXN0VmFsdWVzLmxlbmd0aDsgaSsrKSB7CiAgICAgc2hvdWxkQmUoInRyaW0uY2FsbCgiK3Rlc3RW YWx1ZXNbaV0rIikiLCAiJyIrZXZhbCh0ZXN0VmFsdWVzW2ldKSsiJyIpOwogICAgIHNob3VsZEJl KCJ0cmltTGVmdC5jYWxsKCIrdGVzdFZhbHVlc1tpXSsiKSIsICInIitldmFsKHRlc3RWYWx1ZXNb aV0pKyInIik7CiAgICAgc2hvdWxkQmUoInRyaW1SaWdodC5jYWxsKCIrdGVzdFZhbHVlc1tpXSsi KSIsICInIitldmFsKHRlc3RWYWx1ZXNbaV0pKyInIik7CiB9Ci0K