Closed Bug 873136 Opened 8 years ago Closed 8 years ago
Don't store an extra JSRuntime* in the minor GC tracer
Note, there is yet another extra JSRuntime* in the AutoTraceSession. This is very silly.
Attachment #750529 - Flags: review?(wmccloskey)
Attachment #750529 - Flags: review?(wmccloskey) → review+
Reluctantly backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/b54ce66659aa - one of the four things in that push, despite having been green on try, picked up causing failures in Android 2.2 reftest-1, reftest-2, and reftest-4 by the time it landed. Since I always blame the need to clobber when it's Android, I clobbered and retriggered on your push, and they still failed.
I'm going to reland this now because this patch only affects ggc enabled builds: https://hg.mozilla.org/integration/mozilla-inbound/rev/ad25f39d0f18 I trychose the wrong tests apparently, but here is a green try anyway: https://tbpl.mozilla.org/?tree=Try&rev=a704f20883bf
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.