1878635 - Remove Deprecated.sys.mjs

Status: RESOLVED FIXED

(Toolkit :: General, task, P3)

Description

[Mark Banner (:standard8)]

We want to remove Deprecated.sys.mjs.

It is only currently used in two instances which have been around for a long time and are no longer needed. The devtools one is also incorrect as it doesn't specify the uri argument.

They were probably there due to the pre-FF 57 extensions which may have been able to call that code.

However, we don't need them now, and it doesn't seem useful to keep the module around, so we should remove the module and all references to it.

A couple of useful links for looking at this:

• https://searchfox.org/mozilla-central/search?q=Deprecated.warning&path=&case=true&regexp=false
• https://searchfox.org/mozilla-central/search?q=Deprecated.sys.mjs&path=&case=true&regexp=false
• https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.html

Note: please don't ask to be assigned to the bug, it will be auto-assigned when the first patch is uploaded. If it is likely to take you a while you are quite welcome to comment that you are working on it.

Comment 1

[Nikki Bernobic [:echrs]]

Attachment: Bug 1878635 - Remove Deprecated.sys.mjs r?standard8!

Comment 2

[Nikki Bernobic [:echrs]]

Attachment: Bug 1878635 - Remove Deprecated.sys.mjs r?standard8!

Comment 3

[Nikki Bernobic [:echrs]]

Attachment: Bug 1878635 - Remove Deprecated.sys.mjs r?standard8!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/47a0e9d6438f
Remove Deprecated.sys.mjs r=Standard8,devtools-reviewers,nchevobbe

Comment 5

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/47a0e9d6438f