Remove legacy aboutDialog-appUpdater.js implementation
Categories
(Toolkit :: Application Update, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox82 | --- | fixed |
People
(Reporter: adw, Assigned: adw)
References
Details
Attachments
(1 file)
Bug 1599360 added a new AppUpdater.jsm that we intend to use in an experiment targeted for 72 (bug 1564506). We wanted test coverage for it, so we rewrote aboutDialog-appUpdater.js to use it. But we also didn't want that rewrite landing at the end of a cycle, so we kept around the old aboutDialog-appUpdater.js. The new one is currently preffed off, and our tests test both versions.
Now that we're at the beginning of a new cycle, we should remove the old aboutDialog-appUpdater.js and use the new version always.
Assignee | ||
Comment 1•5 years ago
|
||
This patch reverts the relevant parts of D54837
Comment 2•4 years ago
|
||
This will need to be reviewed by someone else. Perhaps mhowell or bytesized
Updated•4 years ago
|
Updated•4 years ago
|
Comment 3•4 years ago
|
||
adw: could this be a good mentored ticket, a pre-cursor to https://bugzilla.mozilla.org/show_bug.cgi?id=1643309? If yes, could you provide a sketch of what needs to be done?
Assignee | ||
Comment 4•4 years ago
|
||
The patch is done, it just needs to be unbitrotted (if it is bitrotted, hopefully not), and then I need to request review as Robert mentioned in comment 2. I'll go ahead and do that.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 5•4 years ago
|
||
Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5ce71a4b0949 Remove legacy aboutDialog-appUpdater.js implementation r=bytesized
Comment 7•4 years ago
|
||
bugherder |
Description
•