Closed Bug 1555070 Opened 3 months ago Closed 3 months ago

Remove unused _showUpdateNotification code.

Categories

(Firefox :: General, task, P3)

task

Tracking

()

RESOLVED FIXED
Firefox 69
Tracking Status
firefox69 --- fixed

People

(Reporter: rstrong, Assigned: rstrong)

References

Details

Attachments

(1 file)

The following code that was added in bug 538331 at the request of Firefox UX has never been used and should be removed.
https://searchfox.org/mozilla-central/source/browser/components/BrowserGlue.jsm#1946-2038

Per bug 538331 comment #64 the tests are also disabled in bug 1077643 though I haven't checked if they are still disabled.

This does not affect the what's new page which is used and is located in BrowserContentHandler.jsm
https://searchfox.org/mozilla-central/source/browser/components/BrowserContentHandler.jsm#140

I asked Mossop as Firefox module owner whether he is ok with removing this code and he is fine with removing it.

This code was added over 9 years ago per a request from Firefox UX but it has never been used.
This does not affect the What's New page.

Priority: -- → P3
Type: defect → task
Pushed by rstrong@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9b19d5f72273
Remove unused _showUpdateNotification function. r=mossop
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69
You need to log in before you can comment on or make changes to this bug.