Created attachment 8875546 [details] [diff] [review] patch This global variable causes trouble for Quantum DOM since we'll have one context per thread. It's pretty easy to remove it in place of AutoJSAPI though. One note: I also removed the code that stores the context in the user pointer of an async task (see StartAsyncTaskCallback). This code is only used for an assertion, and this assertion will no longer be valid in Quantum DOM. An async task that is created on one thread can finish on another thread.
Attachment #8875546 - Flags: review?(continuation)
Attachment #8875546 - Flags: review?(continuation) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/ccc624eb77e1 Remove nsJSEnvironment::sContext (r=mccr8)
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.