Bug 1631234 - Part 1: Add a test case to check we won't hit the assertion when we have service workers registered in the tracker who has storage access permission. r=baku

We add a test to ensure we won't hit the assertion here.
https://searchfox.org/mozilla-central/rev/a4d62e09a4c46aef918667fa759bf9ae898dc258/dom/clients/manager/ClientSource.cpp#265-268

Differential Revision: https://phabricator.services.mozilla.com/D71564

Author: artines1

toolkit/components/antitracking/test/browser/browser.ini

@@ -102,6 +102,8 @@ support-files = subResources.sjs
 skip-if = fission
 support-files = tracker.js
 [browser_userInteraction.js]
+[browser_serviceWorkersWithStorageAccessGranted.js]
+skip-if = fission
 [browser_storageAccessDoorHanger.js]
 skip-if = fission
 [browser_storageAccessPromiseRejectHandlerUserInteraction.js]

toolkit/components/antitracking/test/browser/browser_serviceWorkersWithStorageAccessGranted.js

@@ -0,0 +1,82 @@
+/** This tests that the service worker can be used if we have storage access
+ *  permission. We manually write the storage access permission into the
+ *  permission manager to simulate the storage access has been granted. We would
+ *  test the service worker twice. The fist time is to check the service
+ *  work is allowed. The second time is to load again and check it won't hit
+ *  assertion, this assertion would only be hit if we have registered a service
+ *  worker, see Bug 1631234. */
+
+/* import-globals-from antitracking_head.js */
+
+add_task(async _ => {
+  // Manually add the storage permission.
+  PermissionTestUtils.add(
+    TEST_DOMAIN,
+    "3rdPartyStorage^https://tracking.example.org",
+    Services.perms.ALLOW_ACTION
+  );
+
+  registerCleanupFunction(_ => {
+    Services.perms.removeAll();
+  });
+
+  AntiTracking._createTask({
+    name:
+      "Test that we can use service worker if we have the storage access permission",
+    cookieBehavior: BEHAVIOR_REJECT_TRACKER,
+    allowList: false,
+    callback: async _ => {
+      await navigator.serviceWorker
+        .register("empty.js")
+        .then(
+          _ => {
+            ok(true, "ServiceWorker can be used!");
+          },
+          _ => {
+            ok(false, "ServiceWorker can be used!");
+          }
+        )
+        .catch(e => ok(false, "Promise rejected: " + e));
+    },
+    extraPrefs: [
+      ["dom.serviceWorkers.exemptFromPerDomainMax", true],
+      ["dom.serviceWorkers.enabled", true],
+      ["dom.serviceWorkers.testing.enabled", true],
+    ],
+    expectedBlockingNotifications: 0,
+    runInPrivateWindow: false,
+    iframeSandbox: null,
+    accessRemoval: null,
+    callbackAfterRemoval: null,
+  });
+
+  AntiTracking._createTask({
+    name:
+      "Test again to check if we can still use service worker without hit the assertion.",
+    cookieBehavior: BEHAVIOR_REJECT_TRACKER,
+    allowList: false,
+    callback: async _ => {
+      await navigator.serviceWorker
+        .register("empty.js")
+        .then(
+          _ => {
+            ok(true, "ServiceWorker can be used!");
+          },
+          _ => {
+            ok(false, "ServiceWorker can be used!");
+          }
+        )
+        .catch(e => ok(false, "Promise rejected: " + e));
+    },
+    extraPrefs: [
+      ["dom.serviceWorkers.exemptFromPerDomainMax", true],
+      ["dom.serviceWorkers.enabled", true],
+      ["dom.serviceWorkers.testing.enabled", true],
+    ],
+    expectedBlockingNotifications: 0,
+    runInPrivateWindow: false,
+    iframeSandbox: null,
+    accessRemoval: null,
+    callbackAfterRemoval: null,
+  });
+});