fix: Revert "fix: no need for double reload of SW in Chrome (#1251)" (#1257)

This reverts commit fa2eb8fe52.

src/routes/_pages/settings/index.html

@@ -36,7 +36,7 @@
     methods: {
       reload (event) {
         event.preventDefault()
-        location.reload()
+        document.location.reload(true)
       }
     }
   }

src/routes/_utils/serviceWorkerClient.js

@@ -1,81 +1,17 @@
 import { snackbar } from '../_components/snackbar/snackbar'
 
-// A lot of this code is borrowed from https://github.com/GoogleChromeLabs/squoosh/blob/53b46f8/src/lib/offliner.ts
+function onUpdateFound (registration) {
-// Service Workers are hard!
+  const newWorker = registration.installing
 
-// Tell the service worker to skip waiting
+  newWorker.addEventListener('statechange', async () => {
-async function skipWaiting () {
+    if (newWorker.state === 'installed' && navigator.serviceWorker.controller) {
-  const reg = await navigator.serviceWorker.getRegistration()
+      snackbar.announce('App update available.', 'Reload', () => document.location.reload(true))
-  if (!reg || !reg.waiting) {
-    return
     }
-  reg.waiting.postMessage('skip-waiting')
-}
-
-// Wait for an installing worker
-async function installingWorker (reg) {
-  if (reg.installing) {
-    return reg.installing
-  }
-  return new Promise((resolve) => {
-    reg.addEventListener(
-      'updatefound',
-      () => resolve(reg.installing),
-      { once: true }
-    )
   })
 }
 
-// Wait a service worker to become waiting
-async function updateReady (reg) {
-  if (reg.waiting) {
-    return
-  }
-  const installing = await installingWorker(reg)
-  return new Promise((resolve) => {
-    const listener = () => {
-      if (installing.state === 'installed') {
-        installing.removeEventListener('statechange', listener)
-        resolve()
-      }
-    }
-    installing.addEventListener('statechange', listener)
-  })
-}
-
-(async () => {
 if ('serviceWorker' in navigator) {
-    navigator.serviceWorker.register('/service-worker.js')
+  navigator.serviceWorker.register('/service-worker.js').then(registration => {
-
+    registration.addEventListener('updatefound', () => onUpdateFound(registration))
-    const hasController = !!navigator.serviceWorker.controller
-
-    navigator.serviceWorker.addEventListener('controllerchange', () => {
-      if (!hasController) { // first install
-        return
-      }
-
-      location.reload()
-    })
-
-    // If we don't have a controller, we don't need to check for updates – we've just loaded from the
-    // network.
-    if (!hasController) {
-      return
-    }
-
-    const reg = await navigator.serviceWorker.getRegistration()
-    if (!reg) { // SW not registered yet
-      return
-    }
-
-    // Look for updates
-    await updateReady(reg)
-
-    // Ask the user if they want to update.
-    snackbar.announce('App update available.', 'Reload', () => {
-      // Tell the waiting worker to activate, this will change the controller and cause a reload (see
-      // 'controllerchange')
-      /* no await */ skipWaiting()
   })
 }
-})()

src/service-worker.js

@@ -35,6 +35,7 @@ self.addEventListener('install', event => {
       caches.open(WEBPACK_ASSETS).then(cache => cache.addAll(webpackAssets)),
       caches.open(ASSETS).then(cache => cache.addAll(assets))
     ])
+    self.skipWaiting()
   })())
 })
 
@@ -242,11 +243,3 @@ self.addEventListener('notificationclick', event => {
     }
   })())
 })
-
-self.addEventListener('message', (event) => {
-  switch (event.data) {
-    case 'skip-waiting':
-      self.skipWaiting()
-      break
-  }
-})