Closed Bug 1506915 Opened Last year Closed Last year

Move app update auto enabled / disabled to a jsm

Categories

(Toolkit :: Application Update, defect, P1)

59 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

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

References

Details

Attachments

(1 file)

I noticed that telemetry checks whether app update auto is enabled or not early on. I'd like to move this code to a jsm since that instantiates the app update service which takes several ms to load due to the file size of nsUpdateService.js and is why nsUpdateServiceStub.js exists to prevent loading nsUpdateService.js during startup unless it is necessary.
Priority: -- → P1
This moves getting and setting the value of app.update.auto from nsUpdateService.js to UpdateUtils.jsm so nsUpdateService.js isn't loaded during startup
Changes the browser_policy_app_update_URL.js test to check the default pref for app.update.url since the default pref is read by app update
Fixes a NS_NOINTERFACE error in app update xpcshell tests for AppInfo not implementing nsIPlatformInfo
Filed bug 1510798 for Thunderbird
Updated bug 1501790 for Seamonkey to include these changes.
Pushed by rstrong@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4939e4a1cc20
Move app update auto enabled / disabled to a jsm. r=bytesized,chutten
https://hg.mozilla.org/mozilla-central/rev/4939e4a1cc20
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.