From 1ea3f1752298413df6638c898d0ff2f6eb0c24d7 Mon Sep 17 00:00:00 2001 From: Sebastian Hengst Date: Wed, 22 Jun 2016 21:16:46 +0200 Subject: [PATCH] Bug 1280775 - null check in removePermission to prevent crash: add test. r=mystor MozReview-Commit-ID: E94VCl0um3T --HG-- extra : rebase_source : f7a94ab729a9d2e09731becde3afb75a19b43cbb --- netwerk/test/unit/test_permmgr.js | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/netwerk/test/unit/test_permmgr.js b/netwerk/test/unit/test_permmgr.js index bb4a2349c7a8..0e735fc91c33 100644 --- a/netwerk/test/unit/test_permmgr.js +++ b/netwerk/test/unit/test_permmgr.js @@ -105,4 +105,15 @@ function run_test() { // test removeAll() pm.removeAll(); do_check_eq(pm.enumerator.hasMoreElements(), false); + + uri = ioService.newURI("https://www.example.com", null, null); + pm.add(uri, "offline-app", pm.ALLOW_ACTION); + principal = secMan.createCodebasePrincipalFromOrigin("https://www.example.com"); + // Remove existing entry. + perm = pm.getPermissionObject(principal, "offline-app", true); + pm.removePermission(perm); + // Try to remove already deleted entry. + perm = pm.getPermissionObject(principal, "offline-app", true); + pm.removePermission(perm); + do_check_eq(pm.enumerator.hasMoreElements(), false); }