Port |Bug 474043 - Get rid of mimeTypes.rdf|

RESOLVED FIXED in Thunderbird 61.0

Status

enhancement
RESOLVED FIXED
Last year
11 months ago

People

(Reporter: jorgk, Assigned: jorgk)

Tracking

unspecified
Thunderbird 61.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

I'll start with the package manifests, let's see what else is required.
M-C removed nsHandlerService and then renamed nsHandlerService-json to nsHandlerService. So the net effect is to remove nsHandlerService-json.

Marco, what else do we need to do?
Looks like we have a few references to mimeTypes.rdf still:
https://dxr.mozilla.org/comm-central/search?q=mimeTypes.rdf&redirect=false

Aceman, any insight into mimeTypes.rdf?
Flags: needinfo?(mak77)
Flags: needinfo?(acelists)
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/6111e84d64ef
Port bug 474043: Adjust names of nsHandlerService in package manifests. rs=bustage-fix
nsHandlerService stopped using mimetypes.rdf some time ago, in Firefox 55. It moved to a json implementation. The old implementation was retained just to allow for migrating to the new store.
Bug 474043 is just removing leftover code, though if you need to upgrade from TB52 to this version, you may still need the old code. We landed this for Firefox 61 because we're past the 60 ESR, so I hope you are also past the TB60 merge point.
You can probably also remove any leftover at that point.
Flags: needinfo?(mak77)
Yeah, if this still works in TB60 the code should automatically migrate all users to the json file. We can then remove the migration code in 61.
Probably the test file at mailnews/test/resources/mailTestUtils.js may need some update.
And we need to know from SM guys what they plan to do in the future, so we can know what to do with https://dxr.mozilla.org/comm-central/source/mail/components/migration/src/nsSeamonkeyProfileMigrator.cpp#34 .
Flags: needinfo?(acelists)
Aceman, can you please handle the remaining removal.
Damn, I got this wrong, the thing is now called "HandlerService" without the "ns" prefix :-((
Attachment #8959486 - Attachment is obsolete: true
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/2a3870ccd7e1
Backed out changeset 6111e84d64ef. a=jorgk
https://hg.mozilla.org/comm-central/rev/76ab402c1342
Port bug 474043: Adjust name of nsHandlerService in package manifests. rs=bustage-fix
I think this was done already. The remaining work is bug 1456405.
Assignee: nobody → jorgk
Status: NEW → RESOLVED
Closed: 11 months ago
Keywords: leave-open
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 61.0
You need to log in before you can comment on or make changes to this bug.