Bug 1914417 - Refactor notification permission checker using principa…

…l partition key r=asuth,bvandersloot This patch keeps the current status where Firefox and Chrome both allows notification permission in ABA context although the spec does not cover it. (whatwg/notifications#177, also listed in https://privacycg.github.io/storage-partitioning/#remaining-user-agent-state) The child patch in bug 1914203 will add a telemetry to see whether the usage is low enough so that we can block ABA access, to make it more consistent with storage partitioning. Differential Revision: https://phabricator.services.mozilla.com/D219908 UltraBlame original commit: bd858e1f75435642347157a152c8e28d612e1895
marco-c · Sep 25, 2024 · 5a27f9a · 5a27f9a
1 parent dacce8c
commit 5a27f9a
Show file tree

Hide file tree

Showing 17 changed files with 950 additions and 775 deletions.
diff --git a/browser/base/content/test/alerts/browser_notification_close.js b/browser/base/content/test/alerts/browser_notification_close.js
@@ -59,7 +59,7 @@ let
 notificationURL
 =
 "
-http
+https
 :
 /
 /

diff --git a/browser/base/content/test/alerts/browser_notification_do_not_disturb.js b/browser/base/content/test/alerts/browser_notification_do_not_disturb.js
@@ -39,7 +39,7 @@ const
 PAGE
 =
 "
-http
+https
 :
 /
 /

diff --git a/browser/base/content/test/alerts/browser_notification_open_settings.js b/browser/base/content/test/alerts/browser_notification_open_settings.js
@@ -7,7 +7,7 @@ var
 notificationURL
 =
 "
-http
+https
 :
 /
 /

diff --git a/browser/base/content/test/alerts/browser_notification_remove_permission.js b/browser/base/content/test/alerts/browser_notification_remove_permission.js
@@ -36,7 +36,7 @@ var
 notificationURL
 =
 "
-http
+https
 :
 /
 /

diff --git a/browser/base/content/test/alerts/browser_notification_tab_switching.js b/browser/base/content/test/alerts/browser_notification_tab_switching.js
@@ -39,7 +39,7 @@ var
 notificationURL
 =
 "
-http
+https
 :
 /
 /