1678109 - get rid of the mailnews IOUtils.jsm

Status: RESOLVED FIXED

(Thunderbird :: General, task, P1)

Description

[Magnus Melin [:mkmelin] (away until Aug 10)]

We need to get rid of the mailnews IOUtils.jsm: https://searchfox.org/comm-central/rev/e3363365eed24f91277ac9946d8e61d357d8547d/mailnews/base/src/IOUtils.jsm

OS.File is being phased out in favour of toolkit IOUtils.jsm - see bug 986145. It would be confusing, and maybe not even possible to use them both with the same name. Anyway, there's not a whole lot of useful stuff in the mailnews IOUtils.jsm so replacing should be fairly painless.

Comment 1

[Wayne Mery (:wsmwk)]

bug 986145 has a significant amount completed. Do we need to target v91 for our completion?

Comment 2

[Beth Rennie [:beth] (she/her)]

IOUtils described in bug 986145 isn't a JSM, but a webidl module and parts of it are available as early as 80. You should however still be able to import it, though may have to do so under a different name to avoid conflicts.

Comment 3

[Magnus Melin [:mkmelin] (away until Aug 10)]

Attachment: Bug 1678109 - get rid of the mailnews IOUtils.jsm module. r=freaktechnik

Depends on D108932

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/08e7efbb4e58
get rid of the mailnews IOUtils.jsm module. r=freaktechnik

Comment 5

[alta88]

If esr78 is a watershed, then the feed migration code should now be removed, not renovated.

Comment 6

[Magnus Melin [:mkmelin] (away until Aug 10)]

Watersheds can only handle automatic updates, so if someone does a manual update from something earlier...
It wasn't a big enough deal to convert so just went with that.

Comment 7

[[e-mail]]

Printing seems to be broken; printUtils.js seems to have been depending on the Services declared in msgPrintEngine.js that was removed here.

Comment 8

[Magnus Melin [:mkmelin] (away until Aug 10)]

I see. I had removed it because there were errors about re-declaring non-configurable property Services (something else already defined it). Seems there is bug 1701172 for it.