clear storage on load+unload

packages/special-pages/pages/duckplayer/src/js/index.js

@@ -37,6 +37,7 @@ import {
 } from '../../../../../messaging/index.js'
 import { DuckPlayerPageMessages, UserValues } from './messages'
 import { html } from '../../../../../../src/dom-utils'
+import { initStorage } from './storage'
 
 // for docs
 export { DuckPlayerPageMessages, UserValues }
@@ -761,3 +762,5 @@ document.addEventListener('DOMContentLoaded', () => {
     })
     MouseMove.init()
 })
+
+initStorage()

packages/special-pages/pages/duckplayer/src/js/storage.js

@@ -0,0 +1,32 @@
+function deleteStorage (subject) {
+    Object.keys(subject).forEach((key) => {
+        if (key.indexOf('yt-player') === 0) {
+            return
+        }
+        subject.removeItem(key)
+    })
+}
+
+function deleteAllCookies () {
+    const cookies = document.cookie.split(';')
+    for (let i = 0; i < cookies.length; i++) {
+        const cookie = cookies[i]
+        const eqPos = cookie.indexOf('=')
+        const name = eqPos > -1 ? cookie.substr(0, eqPos) : cookie
+        document.cookie = name + '=;expires=Thu, 01 Jan 1970 00:00:00 GMT;domain=youtube-nocookie.com;path=/;'
+    }
+}
+
+export function initStorage () {
+    window.addEventListener('unload', () => {
+        deleteStorage(localStorage)
+        deleteStorage(sessionStorage)
+        deleteAllCookies()
+    })
+
+    window.addEventListener('load', () => {
+        deleteStorage(localStorage)
+        deleteStorage(sessionStorage)
+        deleteAllCookies()
+    })
+}

packages/special-pages/tests/duckplayer.spec.js

@@ -28,6 +28,12 @@ test.describe('duckplayer iframe', () => {
         await duckplayer.hasShownErrorMessage()
         await duckplayer.hasNotAddedIframe()
     })
+    test('clears storage', async ({ page }, workerInfo) => {
+        const duckplayer = DuckPlayerPage.create(page, workerInfo)
+        await duckplayer.openWithVideoID()
+        await duckplayer.withStorageValues()
+        await duckplayer.storageClearedAfterReload()
+    })
 })
 
 test.describe('duckplayer toolbar', () => {

packages/special-pages/tests/page-objects/duck-player.js

@@ -247,6 +247,21 @@ export class DuckPlayerPage {
         })
     }
 
+    async withStorageValues () {
+        await this.page.evaluate(() => {
+            localStorage.setItem('foo', 'bar')
+            localStorage.setItem('yt-player-other', 'baz')
+        })
+    }
+
+    async storageClearedAfterReload () {
+        await this.page.reload()
+        const storaget = await this.page.evaluate(() => localStorage)
+        expect(storaget).toMatchObject({
+            'yt-player-other': 'baz'
+        })
+    }
+
     /**
      * We test the fully built artifacts, so for each test run we need to
      * select the correct HTML file.