[tbb-commits] [tor-browser/tor-browser-68.1.0esr-9.0-2] fixup! Bug 1330467 - part 7. Confirm FPI in permission manager tests; r=Ehsan

gk at torproject.org gk at torproject.org
Tue Sep 3 07:57:12 UTC 2019


commit a8c2fe1a361abb926e6dbbce91940a1c56b0b9bf
Author: Alex Catarineu <acat at torproject.org>
Date:   Tue Sep 3 09:41:37 2019 +0200

    fixup! Bug 1330467 - part 7. Confirm FPI in permission manager tests; r=Ehsan
---
 extensions/permissions/test/unit/test_permmanager_defaults.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/extensions/permissions/test/unit/test_permmanager_defaults.js b/extensions/permissions/test/unit/test_permmanager_defaults.js
index cfd42f49e92a..c586982a133c 100644
--- a/extensions/permissions/test/unit/test_permmanager_defaults.js
+++ b/extensions/permissions/test/unit/test_permmanager_defaults.js
@@ -156,11 +156,12 @@ add_task(async function do_test() {
     Ci.nsIPermissionManager.ALLOW_ACTION,
     pm.testPermissionFromPrincipal(principal4, TEST_PERMISSION)
   );
-  // make sure principals with a firstPartyDomain use different permissions
+  // make sure principals with userContextId use the same permissions
   Assert.equal(
     Ci.nsIPermissionManager.ALLOW_ACTION,
     pm.testPermissionFromPrincipal(principal6, TEST_PERMISSION)
   );
+  // make sure principals with a firstPartyDomain use different permissions
   Assert.equal(
     Ci.nsIPermissionManager.UNKNOWN_ACTION,
     pm.testPermissionFromPrincipal(principal7, TEST_PERMISSION)



More information about the tbb-commits mailing list