Closed Bug 1026860 Opened 7 years ago Closed 7 years ago
This Objects map from moz JSComponent Loader
Kyle added this in bug 810987, and it has caused (bug 980537) us (bug 950653) endless (bug 1023899) pain. It also makes it impossible/unsafe to unload modules in b2g (see bug 1004295). And it isn't GGC-safe (bug 983399). I described the basic setup in bug 980537 comment 28. Digging in a bit more, it seems like we could instead just check if the function is parented to a FakeBackstagePass, and if so, use that as the target. I'm going to prototype this and see what breaks.
Compile fail: https://tbpl.mozilla.org/?tree=Try&rev=0eded3bbac13
Attachment #8442109 - Flags: review?(khuey)
Attachment #8442109 - Flags: review?(khuey) → review+
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Comment on attachment 8442109 [details] [diff] [review] Remove mThisObjects and use the function parent instead. v1 This bug seems to have fixed the very serious b2g stability issues in bug 950653. It also gives us a _much_ simpler and safer setup. I recommend we take it for b2g 1.4 and 2.0. [Approval Request Comment] Bug caused by (feature/regressing bug #): Unknown User impact if declined: Crashes Testing completed: Landed on m-c last week, appears to fix the crashes. Risk to taking this patch (and alternatives if risky): Surprisingly, very low-risk. String or UUID changes made by this patch: None
Comment on attachment 8442109 [details] [diff] [review] Remove mThisObjects and use the function parent instead. v1 Aurora+ b2g30- Given that the branch stability is currently meeting our partner's expectations, we are not taking any additional speculative patches at this point. (I use the term speculative based on the comments in bug 950653.)
You need to log in before you can comment on or make changes to this bug.