1797974 - Remove unused isWinVistaOrHigher from DownloadsCommon

Status: RESOLVED FIXED

(Toolkit :: Downloads API, task, P5)

Description

[Marco Bonardo [:mak]]

the isWinVistaOrHigher helper in DownloadsCommon is not used anywhere and should be removed https://searchfox.org/mozilla-central/rev/968bd894205cf4f579d94ac4e175cc3187458605/browser/components/downloads/DownloadsCommon.jsm#775

Comment 1

[Ho Cheung]

Hello @mak, I want to take over and deal with this bug, what should I pay attention to, is it enough to just remove the "isWinVistaOrHigher" parameter?

Comment 2

[Marco Bonardo [:mak]]

this is pure code removal, sorry for missing the question.

Comment 3

[Ho Cheung]

(In reply to Marco Bonardo [:mak] from comment #2)

this is pure code removal, sorry for missing the question.

Can't seem to find the DownloadsCommon.jsm file now, it has been removed in the new version?

Comment 4

[Marco Bonardo [:mak]]

it was likely renamed to .sys.mjs
You can search on searchfor.org for isWinVistaOrHigher

Comment 5

[Ho Cheung]

Attachment: Bug 1797974 - Remove unused isWinVistaOrHigher from DownloadsCommon. r=mak

isWinVistaOrHigher in DownloadsCommon is no longer used, and the
relevant code will be deleted through this commit.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3eb07a1d79b0
Remove unused isWinVistaOrHigher from DownloadsCommon. r=mak

Comment 7

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/3eb07a1d79b0