Issue 3208 - Don`t use numerical content types outside nsIContentPolicy.shouldLoad

chrome/content/ui/composer.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 var nodes = null;
 var item = null;
 var advancedMode = false;

[Thomas Greiner, 2015/11/02 18:54:36]

How is this change related to the issue?

[Wladimir Palant, 2015/11/03 11:26:49]

It is only related to testing here. We have an event handler in the dialog
expecting advancedMode to be global, switching to Advanced Mode isn't possible
if it isn't (that's https://bugzilla.mozilla.org/show_bug.cgi?id=1202902
striking again).

Let me know if you prefer this to be fixed separately.

[Thomas Greiner, 2015/11/03 11:54:54]

I see. In that case fine to fix it separately.

[Wladimir Palant, 2015/11/04 09:30:41]

Ok, I removed that change here and created
https://issues.adblockplus.org/ticket/3263. All the other "changes" displayed by
the interdiff are due to rebasing.

 
 function init()
 {
   [nodes, item] = window.arguments;
 
   E("filterType").value = (!item.filter || item.filter.disabled || item.filter instanceof WhitelistFilter ? "filterlist" : "whitelist");
   E("customPattern").value = item.location;
 
   let insertionPoint = E("customPatternBox");
   let addSuggestion = function(address)
@@ skipping 82 matching lines @@
     addSuggestion(item.location);
     E("patternGroup").value = "";
   }
   if (Prefs.composer_default == 0)
     E("customPattern").focus();
   else
     E("patternGroup").focus();
 
   let types = [];
   for (let type of Policy.localizedDescr.keys())
     types.push(type);

[Thomas Greiner, 2015/11/02 18:54:36]

Detail: What about writing `let types =
Array.from(Policy.localizedDescr.keys());` instead? It more clearly communicates
the intent behind those three lines and squashes them into a single one.

[Wladimir Palant, 2015/11/03 11:26:49]

Nice one, I didn't know about Array.from() yet. However, the way this code is
changed in https://codereview.adblockplus.org/29329473/ I don't think that this
makes sense.

[Thomas Greiner, 2015/11/03 11:54:54]

I agree that, with that change in mind, it's not necessary here.

   types.sort();
 
   let docDomain = item.docDomain;
   let thirdParty = item.thirdParty;
 
   if (docDomain)
     docDomain = docDomain.replace(/^www\./i, "").replace(/\.+$/, "");
   if (docDomain)
     E("domainRestriction").value = docDomain;
 
@@ skipping 260 matching lines @@
  * Selects or unselects all type checkboxes except those
  * that are disabled.
  */
 function selectAllTypes(/**Boolean*/ select)
 {
   for (let typeNode = E("typeGroup").firstChild; typeNode; typeNode = typeNode.nextSibling)
     if (typeNode.getAttribute("disabled") != "true")
       typeNode.checked = select;
   updateFilter();
 }