Add Intl.RelativeTimeFormat.prototype[@@toStringTag]

RESOLVED FIXED in Firefox 63

Status

()

RESOLVED FIXED
5 months ago
4 months ago

People

(Reporter: anba, Assigned: khyperia)

Tracking

Trunk
mozilla63
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 months ago
Add Intl.RelativeTimeFormat.prototype[@@toStringTag], but probably wait until <https://github.com/tc39/proposal-intl-relative-time/issues/72> was resolved.
(Assignee)

Updated

4 months ago
Assignee: nobody → khyperia
Status: NEW → ASSIGNED
(Assignee)

Comment 1

4 months ago
Created attachment 9002846 [details] [diff] [review]
Add @@toStringTag in Intl.RelativeTimeFormat.
(Assignee)

Updated

4 months ago
Attachment #9002846 - Flags: review?(jwalden+bmo)
(Assignee)

Comment 2

4 months ago
Note that the value of it actually is "Intl.RelativeTimeFormat", unlike what seems like other Intl.* objects, which are "Object".

https://tc39.github.io/proposal-intl-relative-time/#sec-Intl.RelativeTimeFormat.prototype-toStringTag
Comment on attachment 9002846 [details] [diff] [review]
Add @@toStringTag in Intl.RelativeTimeFormat.

Review of attachment 9002846 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/tests/non262/Intl/RelativeTimeFormat/relativetimeformat.js
@@ +9,5 @@
>  
>  addIntlExtras(Intl);
>  
>  rtf = new Intl.RelativeTimeFormat("en-us");
> +assertEq(rtf[Symbol.toStringTag], "Intl.RelativeTimeFormat");

Instead, delete these lines from js/src/tests/jstests.list:

# https://bugzilla.mozilla.org/show_bug.cgi?id=1473228
skip script test262/intl402/RelativeTimeFormat/prototype/toStringTag/toString.js
skip script test262/intl402/RelativeTimeFormat/prototype/toStringTag/toStringTag.js
Attachment #9002846 - Flags: review?(jwalden+bmo) → review+
(Assignee)

Comment 4

4 months ago
Created attachment 9002912 [details] [diff] [review]
Add @@toStringTag in Intl.RelativeTimeFormat.
(Assignee)

Updated

4 months ago
Attachment #9002846 - Attachment is obsolete: true
(Assignee)

Updated

4 months ago
Attachment #9002912 - Flags: review+
(Assignee)

Updated

4 months ago
Keywords: checkin-needed

Comment 5

4 months ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c1e8bad5cea6
Add @@toStringTag in Intl.RelativeTimeFormat. r=jorendorff
Keywords: checkin-needed

Comment 6

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c1e8bad5cea6
Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
status-firefox63: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.