Open Bug 1471300 Opened 2 years ago Updated 2 years ago

Remove LightweightThemeManager

Categories

(Toolkit :: Add-ons Manager, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: ntim, Unassigned)

References

(Blocks 1 open bug)

Details

LightweightThemeManager isn't involved in the flow of WebExtension themes at all, so we can remove it once lightweight themes are unsupported.
Priority: -- → P3
Depends on: 1524660

(In reply to Tim Nguyen :ntim from comment #0)

LightweightThemeManager isn't involved in the flow of WebExtension themes at
all

It still is on Android, isn't it? So that'd have to be rewired, too, before being able to get rid of the LWTManager.

(In reply to Jan Henning [:JanH] from comment #1)

(In reply to Tim Nguyen :ntim from comment #0)

LightweightThemeManager isn't involved in the flow of WebExtension themes at
all

It still is on Android, isn't it? So that'd have to be rewired, too, before being able to get rid of the LWTManager.

I think you mean LightweightThemeConsumer.jsm, not LightweightThemeManager.jsm :)

(In reply to Tim Nguyen :ntim from comment #2)

I think you mean LightweightThemeConsumer.jsm, not LightweightThemeManager.jsm :)

No, I do mean the Manager.

(In reply to Jan Henning [:JanH] from comment #3)

(In reply to Tim Nguyen :ntim from comment #2)

I think you mean LightweightThemeConsumer.jsm, not LightweightThemeManager.jsm :)

No, I do mean the Manager.

That also needs to be rewired on the Desktop version too, so it will be done at the same time.

You need to log in before you can comment on or make changes to this bug.