Remove param from refreshUpdateStatus

RESOLVED FIXED in Firefox 39

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

unspecified
mozilla39
x86_64
Windows 8.1
Points:
---

Firefox Tracking Flags

(firefox36 wontfix, firefox37 wontfix, firefox38 wontfix, firefox39 fixed, firefox-esr31 unaffected, firefox-esr38 wontfix)

Details

Attachments

(1 attachment, 1 obsolete attachment)

The patch in bug 1044443 makes it so the refreshUpdateStatus nsIUpdate param is not necessary and it should be removed. It wasn't removed as part of bug 1044443 since a beta patch was needed and that would have required an idl change. It also provided more assurance that the patch actually fixed the bug via crash-stats since the same patch was landed on nightly.
Created attachment 8570711 [details] [diff] [review]
patch
Attachment #8570711 - Flags: review?(spohl.mozilla.bugs)
Comment on attachment 8570711 [details] [diff] [review]
patch

Meh... forgot tests
Attachment #8570711 - Attachment is obsolete: true
Attachment #8570711 - Flags: review?(spohl.mozilla.bugs)
Attachment #8570718 - Flags: review?(spohl.mozilla.bugs) → review+
Pushed to fx-team
https://hg.mozilla.org/integration/fx-team/rev/d1675b777f95
status-firefox36: --- → wontfix
status-firefox37: --- → wontfix
status-firefox38: --- → wontfix
status-firefox-esr31: --- → unaffected
status-firefox-esr38: --- → wontfix
Target Milestone: --- → mozilla39
https://hg.mozilla.org/mozilla-central/rev/d1675b777f95
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.