Closed Bug 1214936 Opened 5 years ago Closed 5 years ago

Make the ArrayBuffer constructor throw if invoked without 'new'

Categories

(Core :: JavaScript: Standard Library, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: efaust, Unassigned)

References

Details

(Keywords: dev-doc-complete, site-compat)

Attachments

(1 file)

Attached patch PatchSplinter Review
By spec, the ArrayBuffer constructor is not callable. Presently, this is only a warning.
Attachment #8673979 - Flags: review?(evilpies)
Blocks: 1062075
Comment on attachment 8673979 [details] [diff] [review]
Patch

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

Nice. Please create copy of my test from bug 980945 and put it under ecma_6/ArrayBuffer. You can also remove WarnIfNotConstructing from NativeObject-inl.h now.
Attachment #8673979 - Flags: review?(evilpies) → review+
Keywords: site-compat
https://hg.mozilla.org/mozilla-central/rev/e5dcb06052bf
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.