1181740 - Replace nsAutoMicroTask with a class that does both a microtask and an entry script that takes ownership of error reporting

Status: NEW

(Core :: DOM: Core & HTML, defect)

Description

[Boris Zbarsky [:bzbarsky]]

This is a followup for bug 1181619.

There shouldn't be any cases in which we do a microtask but not an autoentryscript, in practice.  We should enforce that.

This does require that we stop using nsAutoMicroTask manually around calls to XPCOM callbacks or something.  I'm not quite sure how to do that in practice for XPCOM callbacks not backed by a scripted thing, or whether that's even important.  The most obvious case here, of course, is calling an XPCOM event listener not implemented as XPCWrappedJS that then internally happens to call into script.  Is it OK for a microtask checkpoint to happen at that internal call?

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

"There shouldn't be any cases in which we do a microtask but not an autoentryscript" sounds ok.


You mean some C++ event listener which calls into JS? Some example please

Comment 2

[Boris Zbarsky [:bzbarsky]]

> You mean some C++ event listener which calls into JS? 

Yes.

> Some example please

I don't have one offhand.