RESOLVED FIXED 216529
[JSC] Intl.Collator should take collation option 
https://bugs.webkit.org/show_bug.cgi?id=216529
Summary [JSC] Intl.Collator should take collation option
Yusuke Suzuki
Reported 2020-09-14 22:55:18 PDT
https://github.com/tc39/ecma402/issues/380 We have consensus.
Attachments
Patch (10.67 KB, patch)
2020-09-14 23:55 PDT, Yusuke Suzuki 		ross.kirsling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2020-09-14 22:57:14 PDT
https://github.com/tc39/ecma402/pull/459
Yusuke Suzuki
Comment 2 2020-09-14 23:55:10 PDT
Created attachment 408797 [details] Patch
Ross Kirsling
Comment 3 2020-09-15 11:21:39 PDT
Comment on attachment 408797 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=408797&action=review r=me > Source/JavaScriptCore/runtime/IntlCollator.cpp:185 > + throwRangeError(globalObject, scope, "collation must be unicode-locale-identifier's \"type\""_s); Hmm, we shouldn't require the user to know Unicode spec productions. 😅 To match what we've done in other Intl classes, I would suggest "collation is not a well-formed collation value".
Yusuke Suzuki
Comment 4 2020-09-15 13:13:05 PDT
Comment on attachment 408797 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=408797&action=review Thanks! >> Source/JavaScriptCore/runtime/IntlCollator.cpp:185 >> + throwRangeError(globalObject, scope, "collation must be unicode-locale-identifier's \"type\""_s); > > Hmm, we shouldn't require the user to know Unicode spec productions. 😅 > To match what we've done in other Intl classes, I would suggest "collation is not a well-formed collation value". That's good! Changed.
Yusuke Suzuki
Comment 5 2020-09-15 14:58:37 PDT
Committed r267102: <https://trac.webkit.org/changeset/267102>
Radar WebKit Bug Importer
Comment 6 2020-09-15 14:59:16 PDT
<rdar://problem/68943945>
Note You need to log in before you can comment on or make changes to this bug.