Closed Bug 1555077 Opened 7 years ago Closed 7 years ago

Remove broken call to ShowUpdateElevationRequired in nsUpdateDriver.cpp

Categories

(Toolkit :: Application Update, defect, P2)

68 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

Details

Attachments

(1 file)

At some point the code stopped working in that the do_GetService for nsIUpdatePrompt no longer succeeds. It isn't entirely broken because UpdateService.jsm still displays the UI and it works though it does display it after startup now. This is also for an edgecase that most users won't see.

The UX flow will be changed in Bug 1521632 so beyond removing the call I don't think anything else needs to be done in this bug.

At some point the do_GetService call for nsIUpdatePrompt stopped working.
Since this code is being reworked in a different bug just remove the code.

Pushed by rstrong@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4c3805e7eb8b Remove broken call to ShowUpdateElevationRequired in nsUpdateDriver.cpp. r=spohl
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: