Closed Bug 816160 Opened 8 years ago Closed 8 years ago

hasXPConnectImpls should imply that both arguments and return values might not be new binding objects

Categories

(Core :: DOM: Core & HTML, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla20

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Right now any time hasXPConnectImpls is set we also have to set prefable to true to make return values work and we have to set castable to false to make arguments work because we don't allow prefable castable arguments.

But we already handle non-new-binding castable arguments when hasXPConnectImpls, so there's no need to fail prefable castable arguments if hasXPConnectImpls is set.  And we can similarly use hasXPConnectImpls for the return values instead of actually making things prefable, which speeds up Wrap() a tad.

Patch coming up that does just that.
Attachment #686175 - Flags: review?(peterv) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/f9764b2ccbac
Flags: in-testsuite-
Whiteboard: [need review]
Target Milestone: --- → mozilla20
https://hg.mozilla.org/mozilla-central/rev/f9764b2ccbac
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.