Created attachment 685908 [details] [diff] [review] patch It would be good to assert that we only try to mark objects from the runtime's main thread. This is helpful both on general principle, and to diagnose crashes decoder is seeing that seem to indicate off thread Ion compilation is trying to mark things.
Attachment #685908 - Flags: review?(wmccloskey)
Comment on attachment 685908 [details] [diff] [review] patch It looks like assertValidThread doesn't need to be JS_FRIEND_API.
Attachment #685908 - Flags: review?(wmccloskey) → review+
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.