All of this stuff appears to be happening on the worker cx. I can't quite tell whether there are ever multiple globals/compartments involved for a given cx - it seems like there might be with sub-workers, but I can't quite tell. Either way though, it seems like all of these would be better off just pulling the global off the current compartment rather than grabbing the default one stashed on the context, especially the aforementioned API is going away.

There are no shared globals/compartments/contexts at all. If we are running with a worker cx then its global object can only be the worker associated with that cx.

(In reply to ben turner [:bent] from comment #2) > There are no shared globals/compartments/contexts at all. If we are running > with a worker cx then its global object can only be the worker associated > with that cx. In that case, these patches should be totally fine (they pass mochitests, anyway). I'll upload now.

This makes workers the first consumer to be weaned off this API. \o/

This is green.

Thanks for the review ben - I know you're busy. :-) remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/cbebcd9f2531 remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/469e647ccbf5

https://hg.mozilla.org/mozilla-central/rev/cbebcd9f2531 https://hg.mozilla.org/mozilla-central/rev/469e647ccbf5