Closed Bug 1333621 Opened 7 years ago Closed 7 years ago

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

Categories

(Core :: JavaScript: Internationalization API, defect)

Product:
Core
Component:
JavaScript: Internationalization API
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: zbraniecki, Assigned: anba)

References

Details

Attachments

(1 file, 1 obsolete file)

bug1333621.patch
4.28 KB, patch
anba
: review+
Details | Diff | Splinter Review
Depends on: 1328386
Attached 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+
Attached patch bug1333621.patch — — Splinter Review 
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+
Keywords: checkin-needed
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
https://hg.mozilla.org/mozilla-central/rev/d09d07b4d50d
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: