Update Intl's PluralRules to throw if ctor called without new.

RESOLVED FIXED in Firefox 55

Status

()

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: gandalf, Assigned: anba)

Tracking

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Updated

2 years ago
Depends on: 1328386
Assignee

Comment 1

2 years ago
Posted patch bug1333621.patch (obsolete) — Splinter Review
Here's the patch if the PR gets accepted.
Assignee: nobody → andrebargull
Status: NEW → ASSIGNED
Attachment #8830873 - Flags: review?(gandalf)
Comment on attachment 8830873 [details] [diff] [review]
bug1333621.patch

sweet, thank you!
Attachment #8830873 - Flags: review?(gandalf) → review+
Assignee

Comment 3

2 years ago
Updated patch to add a missing test exclusion for test262, pending https://github.com/tc39/test262/pull/947. Carrying r+ from gandalf.
Attachment #8830873 - Attachment is obsolete: true
Attachment #8855273 - Flags: review+
Assignee

Updated

2 years ago
Keywords: checkin-needed

Comment 5

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d09d07b4d50d
Throw when calling Intl.PluralRules as a function. r=gandalf
Keywords: checkin-needed

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d09d07b4d50d
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.