1478404 - Remove all traces of _xpcom_categories

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P5)

Description

[:Gijs (he/him)]

Pointed out by Florian in bug 1477670 - AFAICT bug 568691 ( https://hg.mozilla.org/mozilla-central/rev/b9a87d218a73 ) removed support for XPCOMUtils.generateModule, and ever since then the _xpcom_categories garbage has been a complete no-op. Kris, am I missing something? If not, we should nuke all remaining references ( https://searchfox.org/mozilla-central/search?q=_xpcom_categories ) from orbit.

Comment 1

[Kris Maglione [:kmag]]

Yup, this has been dead code for years. And the in-tree references are... super dodgy. If the thing actually worked, we'd wind up with a "service" category entry named "true" with some random contract ID as its value depending on registration order...

So, yeah, nuke it.

Comment 2

[:Gijs (he/him)]

Attachment: Bug 1478404 - remove _xpcom_categories annotations as they're dead code, r?florian

Support for the _xpcom_categories property was removed in Bug 568691,
but we left a bunch of consumers behind, and it's been cargo-culted a bit more.
We should remove the remaining remnants.

Comment 3

[Phabricator Automation]

Comment on attachment 8995201 [details]
Bug 1478404 - remove _xpcom_categories annotations as they're dead code, r?florian

Florian Quèze [:florian] has approved the revision.

https://phabricator.services.mozilla.com/D2429

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6499bc583b57
remove _xpcom_categories annotations as they're dead code, r=florian

Comment 5

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/6499bc583b57

Comment 6

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/activity-stream/pull/4280

Comment 7

[[github robot]]

Commit pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/ce7ded0ac20f37f5a4d1e8c68664ca134e7faa77
chore(mc): Port Bug 1478404 - remove _xpcom_categories annotations as they're dead code, r=florian (#4280)