Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-08 10:44:56 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1703320 - Ensure Background Task is scheduled if and only if it ought to be scheduled r=nalexander,application-update-reviewers

Browse Source 
Depends on D111014

Differential Revision: https://phabricator.services.mozilla.com/D111015
This commit is contained in:
Kirk Steuber 2021-04-07 17:19:22 +00:00
parent 812308277d
commit 1751681fd0
1 changed files with 11 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
toolkit/mozapps/update/BackgroundUpdate.jsm
View File

@ -330,10 +330,15 @@ var BackgroundUpdate = {
`${SLUG}: checking eligibility before scheduling background update task`
);
const previousEnabled = Services.prefs.getBoolPref(
"app.update.background.previous.enabled",
false
);
let previousEnabled;
let successfullyReadPrevious;
try {
previousEnabled = await TaskScheduler.taskExists(this.taskId);
successfullyReadPrevious = true;
} catch (ex) {
successfullyReadPrevious = false;
}
const previousReasons = Services.prefs.getCharPref(
"app.update.background.previous.reasons",
null
@ -404,12 +409,6 @@ var BackgroundUpdate = {
}
let updatePreviousPrefs = () => {
// Squirrel away our previous values: keeping them allows us to witness both the rising edge
// (disabled -> enabled) and the falling (enabled -> disabled) edge.
Services.prefs.setBoolPref(
"app.update.background.previous.enabled",
!reasons.length
);
if (reasons.length) {
Services.prefs.setCharPref(
"app.update.background.previous.reasons",
@ -429,7 +428,7 @@ var BackgroundUpdate = {
)}'`
);
if (previousEnabled) {
if (!successfullyReadPrevious || previousEnabled) {
await TaskScheduler.deleteTask(this.taskId);
log.debug(
`${SLUG}: witnessed falling (enabled -> disabled) edge; deleted task ${this.taskId}.`
@ -441,7 +440,7 @@ var BackgroundUpdate = {
return false;
}
if (previousEnabled) {
if (successfullyReadPrevious && previousEnabled) {
log.info(
`${SLUG}: background update was previously enabled; not registering task.`
);