Closed Bug 1960233 Opened 1 year ago Closed 1 year ago

Switch to using type-preserving declareLazy() in toolkit/extensions

Categories

(WebExtensions :: General, task, P2)

Product:
WebExtensions
Component:
General
Type:
task

Tracking

(firefox139 fixed)

Status:
RESOLVED FIXED
Milestone:
139 Branch
Tracking Flags:
Tracking Status
firefox139 --- fixed

People

(Reporter: zombie, Assigned: zombie)

References

Details

(Whiteboard: [addons-jira])

Attachments

(1 file)

Bug 1960233 - Switch to using type-preserving declareLazy() r?robwu
48 bytes, text/x-phabricator-request
Details | Review
No description provided.
Component: Lint and Formatting → General
Product: Developer Infrastructure → WebExtensions
Whiteboard: [addons-jira]

The intention of this patch is to result in zero functional changes.

eslint-disable mozilla/valid-lazy is temporary until we fugure out
what to do with it in bug 1955492.

XPCOMUtils being imported in a few more places is temporary until
the new design of declareLazy() is moved to ChromeUtils.

Assignee: nobody → tomica
Status: NEW → ASSIGNED
Severity: -- → N/A
Priority: -- → P2
Pushed by tjovanovic@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6cb483de4159 Switch to using type-preserving declareLazy() r=robwu

https://hg.mozilla.org/mozilla-central/rev/6cb483de4159

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox139: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 139 Branch

Lead to an increase of 4.1Kb/0.3% on Base content JS opt fission on Windows.

Regressions: 1962342
Blocks: 1965726
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: