No description provided.
Assignee: nobody → gal
blocking2.0: --- → final+
This fixes this compartment mismatch, but I wonder if we need to push this JS_WrapValue() further down to cover other places where this could show up. In this particular case we have an InstallTrigger object that we've just wrapped with WrapNative() and the jsval we get back from it is a chrome object, with code running in the testcase compartment.
Attachment #509794 - Flags: review?(mrbkap)
Attachment #509794 - Flags: review?(mrbkap) → review+
Landed: http://hg.mozilla.org/mozilla-central/rev/bb740aa9e48e And for the record, resolving the InstallTrigger property wasn't the problem here, the declaration of "external" was the trigger here, as evident by deeper digging by mrbkap. window.external is something that the sidebar code exposes (who knew?), and declaring it makes it enumerable. InstallTrigger.install() enumerates all properties on the given argument (window in this case), which ends up resolving window.external, which in turn leads to this compartment mismatch.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.