Regression range: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2776db2defa2&tochange=f5a48a82dca6
Status: UNCONFIRMED → NEW
Ever confirmed: true
Peter/jst: can you take a look at this? Seems to be related to changes made by either: bug 461566 - Don't call FindTearoff when not needed and cache XPCNativeInterfaces in quickstubs bug 461563 - Allow WrapNative to return a jsval without the wrapper
Definitely a regression from bug 461566.
Assignee: nobody → peterv
Status: NEW → ASSIGNED
Created attachment 380643 [details] [diff] [review] v1 I think we should wait to get an interface as late as possible, because if we have classinfo we don't need it (so only do it if we don't). The automarking pointer isn't completely free, but we shouldn't hit this case that often. Jst or mrbkap, whoever gets here first should feel free to just r/sr :-). Not sure how I can test this, needs a custom component. Also not sure that we should block. I'd take the fix for sure, but I think this might be a rare crash.
I hope the fix is low risk enough to go into the 1.9.1 branch, because I'd hate for my extension to not be compatible with FF 3.5. However, if it can't make it into the branch, can anyone see a way to workaround this crash with respect the attached sample extension?
Comment on attachment 380643 [details] [diff] [review] v1 r+sr=jst Peterv, I do think we should get this in for 1.9.1, but let's start with getting it in to the trunk ASAP for baking.
Attachment #380643 - Flags: approval1.9.1? → approval1.9.1+
Comment on attachment 380643 [details] [diff] [review] v1 And approving so that this can land on the trunk for baking...
Severity: normal → critical
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
OS: Windows XP → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
I think comment 6 implies that this is a blocker, jst, please flip the flag to - and flip wanted 1.9.1.x to + if that wasn't your intent.
Flags: blocking1.9.1? → blocking1.9.1+
Crash Signature: [@ XPCNativeSet::NewInstance(XPCCallContext&, XPCNativeInterface**, unsigned short) ]
You need to log in before you can comment on or make changes to this bug.