Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

Commit 052d9fd

Browse files
authored
Add missing default values in background.js (#20044)
* Add missing default for the applied_rulesets * Update background.js * Update background.js * Update background.js
1 parent acff1ec commit 052d9fd

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

chromium/background-scripts/background.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,8 @@ function initializeStoredGlobals() {
9696
});
9797
}
9898

99-
let upgradeToSecureAvailable;
99+
/** @type {boolean} */
100+
let upgradeToSecureAvailable = false;
100101

101102
function getUpgradeToSecureAvailable() {
102103
if (typeof browser !== 'undefined') {
@@ -276,7 +277,7 @@ BrowserSession.prototype = {
276277

277278
// sort by ruleset names alphabetically, case-insensitive
278279
if (this.getTab(tabId, "applied_rulesets", null)) {
279-
let rulesets = this.getTab(tabId, "applied_rulesets");
280+
let rulesets = this.getTab(tabId, "applied_rulesets", null);
280281
let insertIndex = 0;
281282

282283
const ruleset_name = ruleset.name.toLowerCase();
@@ -407,7 +408,7 @@ function onBeforeRequest(details) {
407408
return redirectOnCancel(shouldCancel, details.url);
408409
}
409410

410-
if (browserSession.getRequest(details.requestId, "redirect_count") >= 8) {
411+
if (browserSession.getRequest(details.requestId, "redirect_count", 0) >= 8) {
411412
util.log(util.NOTE, "Redirect counter hit for " + uri.href);
412413
urlBlacklist.add(uri.href);
413414
rules.settings.domainBlacklist.add(uri.hostname);

0 commit comments

Comments
 (0)