js_InitClass says: 3518 if (!proto->scope()->ensureEmptyScope(cx, clasp)) 3519 goto bad; without first ensuring that proto->scope() is mutable. This causes an assertion during GC in debug builds if you manually hack JS_SetGCZeal(cx, 2) into jsapi-tests/tests.h. Easy patch coming.
Created attachment 458470 [details] [diff] [review] v1 (Locking here, because we may have called the constructor on this proto, which could have executed code on another context or whatever.)
Assignee: general → jorendorff
Attachment #458470 - Flags: review?(brendan)
Comment on attachment 458470 [details] [diff] [review] v1 Hurt me with more conflicts in my mq for bug 558451 -- all good ;-). /be
Attachment #458470 - Flags: review?(brendan) → review+
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.