We're not ready to ship service workers on ESR yet. We need to disable on 60 ESR.

This is the disable patch we used for 52 ESR. It should still mostly work. Ryan, do you have an ESR tree you can try it out on?

This doesn't apply to ESR60. Note that you could rebase on top of mozilla-release or the FIREFOX_60b_RELBRANCH branch on mozilla-beta if you don't have a clone of that particular tree readily available.

I'll try to get a try build pushed this morning.

The patch failed to apply for a few reasons: - Lots of tests were converted to async function - dom.serviceWorkers.openWindow.enabled was removed from the tree in FF59

Comment on attachment 8972572 [details] [diff] [review] disable.patch Andrea, can you sign off on this? Its blocking the release process that needs to move forward ASAP. Thanks.

Add a commit message. Fix an eslint error flagged by try build.

Fix more tests. Carrying r+ forward since this is all testonly pref flipping. https://treeherder.mozilla.org/#/jobs?repo=try&revision=1458a1a577d0844871bc8b5c5ca495a7e4fcdc59

Ryan, there are a few orange in the try in comment 8, but they look unrelated to service workers to me. Can you land this in esr?

please set the approval-mozilla-esr60 flag on the patch and fill out the form :)

Comment on attachment 8972605 [details] [diff] [review] Disable service workers and push notifications on 60 ESR. r=baku [Approval Request Comment] If this is not a sec:{high,crit} bug, please state case for ESR consideration: Service workers need to be disabled on ESR. Architectural changes will make it difficult to backport fixes to 60 esr. User impact if declined: Less maintainable ESR. Fix Landed on Version: This is a standalone patch for ESR. Risk to taking this patch (and alternatives if risky): Minimal. We have disabled service workers on every ESR. String or UUID changes made by this patch: None

Comment on attachment 8972605 [details] [diff] [review] Disable service workers and push notifications on 60 ESR. r=baku thanks, approved for 60.0esr

Posted the site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2018/service-workers-and-push-notifications-are-disabled-on-firefox-60-esr/ Release Note Request (optional, but appreciated) [Why is this notable]: These features will not be available in Firefox 60 ESR. Both the 45 ESR note [1] and 52 ESR notes [2] mentioned the same disability. [Affects Firefox for Android]: No [Suggested wording]: Service workers and push notifications disabled [Links (documentation, blog post, etc)]: Not necessary, but the compatibility note can be linked if needed [1] https://www.mozilla.org/en-US/firefox/52.0esr/releasenotes/ [2] https://www.mozilla.org/en-US/firefox/45.0esr/releasenotes/

This is included in the ESR60 relnotes as well.