1703302 - Replace app.background.experimental with app.update.background.scheduling.enabled

Status: RESOLVED FIXED

(Toolkit :: Application Update, task)

Description

[Nick Alexander :nalexander [he/him]]

Now that --backgroundtask backgroundupdate and task scheduling have landed, we can display the UI more generally. That means removing app.update.background.experimental (and have it just be AppConstants.MOZ_UPDATE_AGENT, I guess).

Comment 1

[Robin Steuber (she/her) [:bytesized]]

Attachment: Bug 1703302 - Remove app.background.experimental to show background update UI unconditionally r=nalexander!

Comment 2

[Nick Alexander :nalexander [he/him]]

Kirk, can you verify that the BackgroundAppUpdate=true policy always shows and disables the "When not running" check box? When BackgroundAppUpdate=false do we hide the check box or always show it and disable it?

Comment 3

[Robin Steuber (she/her) [:bytesized]]

We actually have an automated test for this, here. It passes with this patch applied.

When any policy affecting the control is active, we hide the whole control. This is consistent with what we do for other update controls that can be affected by policy.

Comment 4

[Nick Alexander :nalexander [he/him]]

(In reply to Kirk Steuber (he/him) [:bytesized] from comment #3)

We actually have an automated test for this, here. It passes with this patch applied.

When any policy affecting the control is active, we hide the whole control. This is consistent with what we do for other update controls that can be affected by policy.

Thanks for confirming our standard in this space.

Comment 5

[Robin Steuber (she/her) [:bytesized]]

After talking this over with Nick, we want this to continue to be pref'ed off for now. But we would like it to be controlled by the same pref that controls the Background Update Task scheduling default: app.update.background.scheduling.enabled. This will allow us to use that pref with Normandy when we roll this feature out, enabling both the default scheduling and the UI.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2a1b2c95538e
Remove app.background.experimental to show background update UI unconditionally r=nalexander,preferences-reviewers

Comment 7

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/2a1b2c95538e