1833990 - Use ChromeUtils.idleDispatch instead of nsIMessageLoop.postIdleTask

Status: RESOLVED FIXED

(GeckoView :: General, task, P3)

Description

[Makoto Kato [:m_kato]]

GeckoView's DelayedInit.jsm is only one consumer for nsIMessageLoop. Actually, ChromeUtils has idleDispatch, or Timer.sys.mjs has requestIdleCallback. So we can replace nsIMessageLoop with ChromeUtils.idleDispatch.

Comment 1

[Makoto Kato [:m_kato]]

Attachment: Bug 1833990 - Use ChromeUtils.idleDispatch instead of nsIMessageLoop.postIdleTask. r=#geckoview-reviewers

GeckoView's DelayedInit.jsm is only one consumer for nsIMessageLoop. This
XPCOM has no unit test now, so we don't know whether this XPCOM works
correctly.

Actually, ChromeUtils has idleDispatch, so we can replace nsIMessageLoop
with ChromeUtils.idleDispatch. It is used on a lot of code.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ddf9ed515362
Use ChromeUtils.idleDispatch instead of nsIMessageLoop.postIdleTask. r=geckoview-reviewers,calu

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/ddf9ed515362