Open Bug 1542030 Opened 5 years ago Updated 2 years ago

Don't load ext-theme.js for static themes

Categories

(WebExtensions :: Themes, defect, P3)

Product:
WebExtensions
Component:
Themes
Type:
defect

Tracking

(Not tracked)

People

(Reporter: kmag, Unassigned)

References

(Blocks 1 open bug)

Details

The work that we need to do for static themes at startup should be trivial, given that the theme data is already cached. There's no reason to invoke all of the API machinery required to load ext-theme.js in those cases.

We can move the necessary theme parsing logic to a JSM, and handle the trivial cases with an ExtensionData sub-class like we do for dictionaries and langpacks.

No longer blocks: 1492519
Priority: -- → P2
Assignee: kmaglione+bmo → nobody
Priority: P2 → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.