719154 - Remove support for nested event loop filters

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Benjamin Smedberg]

Attachment: Remove nested event loop filtering, rev. 1

With the removal of XPCOM proxies, we can now safely also remove the rather complicated thread support for nested event loop filters. These never worked well because native events weren't usefully filtered, and even if we want to end up with something like DOM event sources as in the HTML spec, the current code will not be useful at all.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Could you ask somebody else to review this?

Comment 2

[Benjamin Smedberg]

Comment on attachment 589582 [details] [diff] [review]
Remove nested event loop filtering, rev. 1

Apparently I need to find somebody to own/be a peer of this code.

Comment 3

[Justin Lebar (not reading bugmail)]

Comment on attachment 589582 [details] [diff] [review]
Remove nested event loop filtering, rev. 1

This patch is clearly fine in itself, but I admit that I don't know what an xpcom proxy is or why their removal means we can remove this code.  I'm reading bug 675221, but I may be at it for a while.  :)

Comment 4

[Justin Lebar (not reading bugmail)]

Okay, I see.  Or at least, think I've seen enough!  :)

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/1b05e9751f34