Bug 1297905 - Remove usage of getAllDisabledHosts in sanitize implementations. r=liuche,markh

Bug 1058438 moved disabled hosts to the permission manager which are already cleared by these modules.

MozReview-Commit-ID: InprrYLvjMR

--HG--
extra : rebase_source : 806c2542cbab953b74ad82611501ecac32400930
This commit is contained in:
Matthew Noorenberghe 2016-08-24 17:41:10 -07:00
parent 7d3cddeb71
commit 3803d21cec
2 changed files with 0 additions and 21 deletions

View File

@ -538,20 +538,6 @@ Sanitizer.prototype = {
seenException = ex;
}
try {
// Clear "Never remember passwords for this site", which is not handled by
// the permission manager
// (Note the login manager doesn't support date ranges yet, and bug
// 1058438 is calling for loginSaving stuff to end up in the
// permission manager)
let hosts = Services.logins.getAllDisabledHosts();
for (let host of hosts) {
Services.logins.setLoginSavingEnabled(host, true);
}
} catch (ex) {
seenException = ex;
}
try {
// Clear site security settings - no support for ranges in this
// interface either, so we clearAll().

View File

@ -95,13 +95,6 @@ Sanitizer.prototype = {
.getService(Ci.nsIContentPrefService2)
.removeAllDomains(null);
// Clear "Never remember passwords for this site", which is not handled by
// the permission manager
var hosts = Services.logins.getAllDisabledHosts({})
for (var host of hosts) {
Services.logins.setLoginSavingEnabled(host, true);
}
// Clear site security settings
var sss = Cc["@mozilla.org/ssservice;1"]
.getService(Ci.nsISiteSecurityService);