Closed Bug 902921 Opened 11 years ago Closed 7 years ago

Refactor XPIProvider to move "is a theme" logic from XPIProvider to AddonInternal

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED INACTIVE

People

(Reporter: Irving, Unassigned)

References

Details

The logic for enabling / disabling themes (to make sure exactly one theme is active at any time) is spread through XPIProvider.jsm, leading to problems like bug 715680. Perhaps we should refactor this into methods on AddonInternal.
Yeah this code is kinda crazy, mainly because of the need to work with background themes. At one point I know Blair was working on making background themes work with any theme, if that happens this all becomes much much simpler. If that is still on the radar it may be worth just waiting till that happens.
Yea, bug 520124 involves a significant refactor of all the theme stuff - UI and API. Hopefully this bug will be a WONTFIX when that other bug is done (will leave it open until we know so).
Depends on: 520124
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.