939196 - Allow nsThread to have nested event queues

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Attachment: Patch, v1

This basically reverts the changes from bug 719154 but it does away with the complicated threadfilter stuff. This allows nsThread to nest event queues and generates a new nsIEventTarget for each queue.

Comment 1

[Benjamin Smedberg]

Comment on attachment 833026 [details] [diff] [review]
Patch, v1

I wonder if it wouldn't be good for popEventQueue() to take the nsIEventTarget as a parameter so we can assert that we're popping things in the correct order. Also I presume these methods shouldn't be scriptable so can we add [noscript] markers?

r=me with that or a really good reason why we shouldn't do it. This looks pretty good for how bad it seemed on IRC!

Comment 2

[Ben Turner (not reading bugmail, use the needinfo flag!)]

https://hg.mozilla.org/integration/b2g-inbound/rev/499de7433a6a

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/499de7433a6a