2021618 - Geckoview's DelayedInit queue isn't processed till the end if a queued init errors

Status: NEW

(GeckoView :: General, defect, P3)

Description

[Beatriz Rizental]

This https://searchfox.org/firefox-main/source/mobile/shared/components/geckoview/GeckoViewStartup.sys.mjs#406 call throws on my local dev build.

That prevents the DelayedInit queue from being completely flushed. If a new pending init is added to the queue it would be cleared, but it doesn't seem that happens.

Note, this prevents all the InitLater calls from geckoview.js from running and probably other important calls. From my logs, the faulty one is the second queued init (deterministically), so anything queued after it is not running.

I will send a suggested patch fixing this.

Comment 1

[Beatriz Rizental]

Attachment: Bug 2021618 - Keep processing pending inits after failure.r=owlish

Comment 2

[[:owlish] 🦉 PST]

Hi Beatriz, I wonder if turning off browser.crashReports.onDemand pref would solve this issue for you?

Comment 3

[[:owlish] 🦉 PST]

If it doesn't help, can you show us the stacktrace of the crash? Thank you!

Comment 4

[Beatriz Rizental]

Hi Beatriz, I wonder if turning off browser.crashReports.onDemand pref would solve this issue for you?

It doesn't.

If it doesn't help, can you show us the stacktrace of the crash? Thank you!

This is the error I get from the line I pointed to:

Error: Failed to load resource://gre/modules/RemoteSettingsCrashPull.sys.mjs"

Even if we fix this specific error, wouldn't it be better to also catch error on the DelayedInit module? It's doesn't seem ok to let a single error prevent the whole queue from running.