1548869 - Blocklist profile-after-change observer is set up incorrectly

Status: RESOLVED FIXED

(Toolkit :: Blocklist Implementation, defect)

Description

[Andrew Swan [:aswan]]

Prior to bug 1524688, this observer was set up for non-desktop builds:
https://searchfox.org/mozilla-central/rev/da054397cdad072c8c61b463d374bc012b5f12aa/toolkit/mozapps/extensions/extensions.manifest#3-5

But the registration logic was inadvertently flipped when this was converted to static registration:
https://searchfox.org/mozilla-central/rev/b59a99943de4dd314bae4e44ab43ce7687ccbbec/toolkit/mozapps/extensions/components.conf#14-16

I think this means we're loading the blocklist earlier than needed on desktop and later than intended on android.

Comment 1

[Kris Maglione [:kmag]]

Attachment: Bug 1548869: Fix blocklist startup category registration. r=aswan

We're supposed to register this category for everything other than desktop
Firefox. Instead, we currently register it only for desktop Firefox.

Comment 2

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/559ed8ce1baad23807527b737ddd519919f63986
Bug 1548869: Fix blocklist startup category registration. r=aswan

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/559ed8ce1baa