Closed Bug 1794657 Opened 2 years ago Closed 2 years ago

Remove the main-thread-only host->policy map from ExtensionPolicyService

Categories

(WebExtensions :: General, enhancement)

enhancement

Tracking

(firefox107 fixed)

RESOLVED FIXED
107 Branch
Tracking Status
firefox107 --- fixed

People

(Reporter: nika, Assigned: nika)

References

Details

Attachments

(2 files)

In https://phabricator.services.mozilla.com/D158884, kmag mentioned that he'd be comfortable getting rid of the main thread copy of the table if we made the threadsafe version of the table be guarded by an RWLock.

This will require the changes in bug 1794644 to allow the StaticRWLock type to be used more generally.

We're still holding strong references to the WebExtensionPolicy objects through
the mExtensions map, so we don't need to worry about the policy instances
dying earlier after this change.

Depends on D159086

Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/18fab9aeb0ec
Part 1: Use a StaticRWLock in the ExtensionPolicyService, r=kmag
https://hg.mozilla.org/integration/autoland/rev/639c1a648ed7
Part 2: Remove the main-thread-only host map from ExtensionPolicyService, r=kmag
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: