Backed out changeset 2c633215c514 (bug 1482055) for causing bug 1500505.
authorRyan VanderMeulen <ryanvm@gmail.com>
Fri, 19 Oct 2018 13:32:56 -0400
changeset 500579 4888629028695c78b394cef273c00e70ae4b87df
parent 500578 18859d2fec94f35e924e9093a99169623e0b2d78
child 500580 01378c910610cd214b2838650d0d2b7218fa8b5d
child 500717 9e9ca4e2ef17b85d0e49979049b227f3bd9915b1
push id1864
push userffxbld-merge
push dateMon, 03 Dec 2018 15:51:40 +0000
treeherdermozilla-release@f040763d99ad [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1482055, 1500505
milestone64.0a1
backs out2c633215c5148a0ea416237b749032e44e7708fb
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset 2c633215c514 (bug 1482055) for causing bug 1500505.
browser/modules/Sanitizer.jsm
--- a/browser/modules/Sanitizer.jsm
+++ b/browser/modules/Sanitizer.jsm
@@ -706,24 +706,16 @@ async function sanitizeOnShutdown(progre
 }
 
 async function sanitizeSessionPrincipals() {
   if (Services.prefs.getIntPref(PREF_COOKIE_LIFETIME,
                                 Ci.nsICookieService.ACCEPT_NORMALLY) != Ci.nsICookieService.ACCEPT_SESSION) {
     return;
   }
 
-  // When PREF_COOKIE_LIFETIME is set to ACCEPT_SESSION, any new cookie will be
-  // marked as session only. But we don't touch the existing ones. For this
-  // reason, here we delete any existing cookie, at shutdown.
-  await new Promise(resolve => {
-    Services.clearData.deleteData(Ci.nsIClearDataService.CLEAR_COOKIES,
-                                  resolve);
-  });
-
   let principals = await new Promise(resolve => {
     quotaManagerService.getUsage(request => {
       if (request.resultCode != Cr.NS_OK) {
         // We are probably shutting down. We don't want to propagate the
         // error, rejecting the promise.
         resolve([]);
         return;
       }