Open Bug 1768882 Opened 3 years ago Updated 2 years ago

Add Registration for cleanup calls to Runnables/lambdas to nsIGlobalObject

Categories

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

enhancement

Tracking

()

People

(Reporter: jesup, Assigned: jesup)

References

Details

Attachments

(3 obsolete files)

We need to clean up various things before an nsIGlobalObject stops working, such as Close()ing actors which may cause in-flight IPCs using promises to abort. This is needed both in Workers and Windows, and the cleanup call to walk the list needs to be called a goodtime (before the event loop is dead for certain, perhaps when storage shuts down).

If we have needs for two different sets/times for cleanup, we may need some sort of phase argument (say Storage/Quota shutdown vs final shutdown).

Attachment #9276110 - Attachment description: WIP: Bug 1768882: Hook up shutdown for nsIGlobalObject cleanup registrations r=#dom-reviewers → Bug 1768882: Hook up shutdown for nsIGlobalObject cleanup registrations r=#dom-reviewers
Attachment #9276109 - Attachment description: WIP: Bug 1768882: Add shutdown callbacks with runnables to nsIGlobalObject r=#dom-reviewers → Bug 1768882: Add shutdown callbacks with runnables to nsIGlobalObject r=#dom-reviewers
Attachment #9276135 - Attachment description: WIP: Bug 1768882: Hook up OPFS Parent actor shutdown to QuotaClient shutdown r=#dom-storage-reviewers,janv → Bug 1768882: Hook up OPFS Parent actor shutdown to QuotaClient shutdown r=#dom-storage-reviewers,janv

So we're trying to capture what state? Shutdown or Window/Worker being closed?
The naming is confusing.

Attachment #9276109 - Attachment description: Bug 1768882: Add shutdown callbacks with runnables to nsIGlobalObject r=#dom-reviewers → Bug 1768882: Add shutdown callbacks with runnables to nsIGlobalObject r=#dom-storage-reviewers,smaug
Attachment #9276110 - Attachment description: Bug 1768882: Hook up shutdown for nsIGlobalObject cleanup registrations r=#dom-reviewers → Bug 1768882: Hook up shutdown for nsIGlobalObject cleanup registrations r=#dom-storage-reviewers,asuth
Attachment #9276110 - Attachment is obsolete: true
Attachment #9276135 - Attachment is obsolete: true
Attachment #9276109 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: