diff --git a/chrome/content/composer.js b/chrome/content/composer.js index 06cae51..bedd66c 100644 --- a/chrome/content/composer.js +++ b/chrome/content/composer.js @@ -135,11 +135,6 @@ function init() { typeGroup.appendChild(typeNode); } - let collapseDefault = E("collapseDefault"); - collapseDefault.label = collapseDefault.getAttribute(aup.prefs.fastcollapse ? "label_no" : "label_yes"); - E("collapse").value = ""; - E("collapse").setAttribute("label", collapseDefault.label); - updatePatternSelection(); document.getElementById("disabledWarning").hidden = aup.prefs.enabled; @@ -184,10 +179,7 @@ function updateFilter() if (E("matchCase").checked) options.push("match-case"); - let collapse = E("collapse"); - disableElement(collapse, type == "whitelist", "value", ""); - if (collapse.value != "") - options.push(collapse.value); + disableElement(false, type == "whitelist", "value", ""); let enabledTypes = []; let disabledTypes = []; diff --git a/chrome/content/composer.xul b/chrome/content/composer.xul index d83a1d1..3ffb41d 100644 --- a/chrome/content/composer.xul +++ b/chrome/content/composer.xul @@ -103,16 +103,6 @@ - - diff --git a/chrome/content/filterClasses.js b/chrome/content/filterClasses.js index 78754b1..21ff7e3 100644 --- a/chrome/content/filterClasses.js +++ b/chrome/content/filterClasses.js @@ -378,7 +378,6 @@ RegExpFilter.fromText = function(text) let matchCase = null; let domains = null; let thirdParty = null; - let collapse = null; let options; if (Filter.optionsRegExp.test(text)) { @@ -409,10 +408,6 @@ RegExpFilter.fromText = function(text) thirdParty = true; else if (option == "~THIRD_PARTY") thirdParty = false; - else if (option == "COLLAPSE") - collapse = true; - else if (option == "~COLLAPSE") - collapse = false; } } @@ -445,7 +440,7 @@ RegExpFilter.fromText = function(text) try { - return new constructor(origText, regexp, contentType, matchCase, domains, thirdParty, collapse); + return new constructor(origText, regexp, contentType, matchCase, domains, thirdParty, false); } catch (e) { @@ -481,25 +476,17 @@ RegExpFilter.typeMap = { * @param {Boolean} matchCase see RegExpFilter() * @param {String} domains see RegExpFilter() * @param {Boolean} thirdParty see RegExpFilter() - * @param {Boolean} collapse defines whether the filter should collapse blocked content, can be null * @constructor * @augments RegExpFilter */ -function BlockingFilter(text, regexp, contentType, matchCase, domains, thirdParty, collapse) +function BlockingFilter(text, regexp, contentType, matchCase, domains, thirdParty, false) { RegExpFilter.call(this, text, regexp, contentType, matchCase, domains, thirdParty); - - this.collapse = collapse; } BlockingFilter.prototype = { - __proto__: RegExpFilter.prototype, - - /** - * Defines whether the filter should collapse blocked content. Can be null (use the global preference). - * @type Boolean - */ - collapse: null + __proto__: RegExpFilter.prototype + }; aup.BlockingFilter = BlockingFilter; diff --git a/chrome/content/overlay.js b/chrome/content/overlay.js index 4a4106d..5f8fefe 100644 --- a/chrome/content/overlay.js +++ b/chrome/content/overlay.js @@ -542,7 +542,6 @@ function aupFillPopup(popup) { elements.enabled.setAttribute("checked", aupPrefs.enabled); elements.frameobjects.setAttribute("checked", aupPrefs.frameobjects); - elements.slowcollapse.setAttribute("checked", !aupPrefs.fastcollapse); elements.showintoolbar.setAttribute("checked", aupPrefs.showintoolbar); elements.showinstatusbar.setAttribute("checked", aupPrefs.showinstatusbar); diff --git a/chrome/content/overlayGeneral.xul b/chrome/content/overlayGeneral.xul index 6b2ffae..3d5bf52 100644 --- a/chrome/content/overlayGeneral.xul +++ b/chrome/content/overlayGeneral.xul @@ -57,7 +57,6 @@ - diff --git a/chrome/content/policy.js b/chrome/content/policy.js index f387516..19f93f5 100644 --- a/chrome/content/policy.js +++ b/chrome/content/policy.js @@ -118,10 +118,9 @@ var policy = * @param node {nsIDOMElement} * @param contentType {String} * @param location {nsIURI} - * @param collapse {Boolean} true to force hiding of the node * @return {Boolean} false if the node is blocked */ - processNode: function(wnd, node, contentType, location, collapse) { + processNode: function(wnd, node, contentType, location, false) { var topWnd = wnd.top; if (!topWnd || !topWnd.location || !topWnd.location.href) return true; diff --git a/chrome/content/settings.js b/chrome/content/settings.js index 5b26ceb..9471ba8 100644 --- a/chrome/content/settings.js +++ b/chrome/content/settings.js @@ -1085,7 +1085,6 @@ function fillOptionsPopup() { E("aup-enabled").setAttribute("checked", prefs.enabled); E("frameobjects").setAttribute("checked", prefs.frameobjects); - E("slowcollapse").setAttribute("checked", !prefs.fastcollapse); E("showintoolbar").setAttribute("checked", prefs.showintoolbar); E("showinstatusbar").setAttribute("checked", prefs.showinstatusbar); } diff --git a/chrome/locale/en-US/composer.dtd b/chrome/locale/en-US/composer.dtd index 21a3664..c2fbf3f 100644 --- a/chrome/locale/en-US/composer.dtd +++ b/chrome/locale/en-US/composer.dtd @@ -63,10 +63,3 @@ - - - - - - - diff --git a/chrome/locale/en-US/settings.dtd b/chrome/locale/en-US/settings.dtd index 4590649..681affe 100644 --- a/chrome/locale/en-US/settings.dtd +++ b/chrome/locale/en-US/settings.dtd @@ -81,8 +81,6 @@ - - diff --git a/defaults/preferences/autoproxy.js b/defaults/preferences/autoproxy.js index cfcf68a..655a8af 100644 --- a/defaults/preferences/autoproxy.js +++ b/defaults/preferences/autoproxy.js @@ -3,7 +3,6 @@ pref("extensions.autoproxy.currentVersion", "0.0"); pref("extensions.autoproxy.enabled", true); pref("extensions.autoproxy.frameobjects", true); -pref("extensions.autoproxy.fastcollapse", false); pref("extensions.autoproxy.showintoolbar", true); pref("extensions.autoproxy.showinstatusbar", false); pref("extensions.autoproxy.detachsidebar", false); diff --git a/test_locales.pl b/test_locales.pl index 88ea46d..3384bc8 100755 --- a/test_locales.pl +++ b/test_locales.pl @@ -13,11 +13,11 @@ ['aup:settings:cut.accesskey', 'aup:settings:copy.accesskey', 'aup:settings:paste.accesskey', 'aup:settings:remove.accesskey', 'aup:settings:menu.find.accesskey', 'aup:settings:menu.findagain.accesskey'], ['aup:settings:filter.accesskey', 'aup:settings:enabled.accesskey', 'aup:settings:hitcount.accesskey', 'aup:settings:lasthit.accesskey', 'aup:settings:sort.accesskey'], ['aup:settings:sort.none.accesskey', 'aup:settings:filter.accesskey', 'aup:settings:enabled.accesskey', 'aup:settings:hitcount.accesskey', 'aup:settings:lasthit.accesskey', 'aup:settings:sort.ascending.accesskey', 'aup:settings:sort.descending.accesskey'], - ['aup:settings:enable.accesskey', 'aup:settings:showintoolbar.accesskey', 'aup:settings:showinstatusbar.accesskey', 'aup:settings:objecttabs.accesskey', 'aup:settings:collapse.accesskey'], + ['aup:settings:enable.accesskey', 'aup:settings:showintoolbar.accesskey', 'aup:settings:showinstatusbar.accesskey', 'aup:settings:objecttabs.accesskey'], ['aup:settings:gettingStarted.accesskey', 'aup:settings:faq.accesskey', 'aup:settings:filterdoc.accesskey', 'aup:settings:about.accesskey'], ['aup:findbar:next.accesskey', 'aup:findbar:previous.accesskey', 'aup:findbar:highlight.accesskey'], ['aup:subscription:location.accesskey', 'aup:subscription:title.accesskey', 'aup:subscription:autodownload.accesskey', 'aup:subscription:enabled.accesskey'], - ['aup:composer:filter.accesskey', 'aup:composer:preferences.accesskey', 'aup:composer:type.filter.accesskey', 'aup:composer:type.whitelist.accesskey', 'aup:composer:custom.pattern.accesskey', 'aup:composer:anchor.start.accesskey', 'aup:composer:anchor.end.accesskey', 'aup:composer:domainRestriction.accesskey', 'aup:composer:firstParty.accesskey', 'aup:composer:thirdParty.accesskey', 'aup:composer:matchCase.accesskey', 'aup:composer:collapse.accesskey'], + ['aup:composer:filter.accesskey', 'aup:composer:preferences.accesskey', 'aup:composer:type.filter.accesskey', 'aup:composer:type.whitelist.accesskey', 'aup:composer:custom.pattern.accesskey', 'aup:composer:anchor.start.accesskey', 'aup:composer:anchor.end.accesskey', 'aup:composer:domainRestriction.accesskey', 'aup:composer:firstParty.accesskey', 'aup:composer:thirdParty.accesskey', 'aup:composer:matchCase.accesskey'], ); my @must_equal = (