Open Bug 1055755 Opened 6 years ago Updated 5 years ago
Remove |existing| argument from JSCompartment::wrap
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.
Attachment #8475442 - Flags: review?(wmccloskey) → review+
Backed out for Android debug jsreftest permafail. https://hg.mozilla.org/integration/mozilla-inbound/rev/33781a3a5201 https://treeherder.mozilla.org/logviewer.html#?job_id=5128423&repo=mozilla-inbound
This also apparently made OSX 10.6 reftests frequent-but-not-perma-fail: https://treeherder.mozilla.org/ui/logviewer.html#?job_id=5134450&repo=mozilla-inbound
(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.