commit a8c2fe1a361abb926e6dbbce91940a1c56b0b9bf
Author: Alex Catarineu <[email protected]>
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)

_______________________________________________
tor-commits mailing list
[email protected]
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits

Reply via email to