If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Move the JSCompartment to the end of the arglist for JS-implemented WebIDL, and make it optional

RESOLVED FIXED in mozilla27

Status

()

Core
DOM
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

(Blocks: 1 bug)

unspecified
mozilla27
x86
Mac OS X
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

This way C++ callers can call into JS-implemented WebIDL methods sanely and by default get the right behavior: null compartment, so exception reported before returning.
Created attachment 806266 [details] [diff] [review]
Move the JSCompartment argument to the end of the list for JS-implemented WebIDL, so C++ callers can call into it sanely too.
Attachment #806266 - Flags: review?(peterv)
Attachment #806266 - Flags: review?(peterv) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/a354bd50d7f4
Flags: in-testsuite-
Whiteboard: [need review]
Target Milestone: --- → mozilla27
https://hg.mozilla.org/mozilla-central/rev/a354bd50d7f4
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.