Open Bug 1055755 Opened 6 years ago Updated 5 years ago

Remove |existing| argument from JSCompartment::wrap

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set

Tracking

()

People

(Reporter: efaust, Unassigned)

Details

Attachments

(1 file)

Since billm cleaned up dead compartment GC logic, we no longer need to avoid allocating an object during wrapper remapping, so we can just wrap again and swap every time, instead.
Attached patch PatchSplinter Review
Attachment #8475442 - Flags: review?(wmccloskey)
Attachment #8475442 - Flags: review?(wmccloskey) → review+
(In reply to Wes Kocher (:KWierso) from comment #4)
> This also apparently made OSX 10.6 reftests

10.6 *debug* - I know it sounds crazy, but the retriggers on inbound paint a pretty conclusive picture about it.
You need to log in before you can comment on or make changes to this bug.