Bug 1524674 - Cleanup site data with custom permissions per subdomains - check permissions. r=johannh a=lizzard
authorAndrea Marchesini <amarchesini@mozilla.com>
Mon, 04 Feb 2019 14:25:47 +0000
changeset 515764 ea1d5311b285dcdb93b1b1748dc5d9905fda7993
parent 515763 bf4fec8933af388b2e37a56780dc6895727a8b2d
child 515765 6e1b209f1fd83d990cbfff4740579544e9e88fcf
push id1953
push userffxbld-merge
push dateMon, 11 Mar 2019 12:10:20 +0000
treeherdermozilla-release@9c35dcbaa899 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjohannh, lizzard
bugs1524674
milestone66.0
Bug 1524674 - Cleanup site data with custom permissions per subdomains - check permissions. r=johannh a=lizzard Differential Revision: https://phabricator.services.mozilla.com/D18497
browser/modules/Sanitizer.jsm
--- a/browser/modules/Sanitizer.jsm
+++ b/browser/modules/Sanitizer.jsm
@@ -713,21 +713,17 @@ async function sanitizeOnShutdown(progre
     if (!isSupportedURI(permission.principal.URI)) {
       continue;
     }
 
     log("Custom session cookie permission detected for: " + permission.principal.URI.spec);
 
     // We use just the URI here, because permissions ignore OriginAttributes.
     let principals = await getAllPrincipals(permission.principal.URI);
-    let promises = [];
-    principals.forEach(principal => {
-      promises.push(sanitizeSessionPrincipal(principal));
-    });
-    await Promise.all(promises);
+    await maybeSanitizeSessionPrincipals(principals);
   }
 
   if (Sanitizer.shouldSanitizeNewTabContainer) {
     sanitizeNewTabSegregation();
     removePendingSanitization("newtab-container");
   }
 
   if (Sanitizer.shouldSanitizeOnShutdown) {