Issue 3258 - Blockable items: fix Size column

lib/child/requestNotifier.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
@@ skipping 259 matching lines @@
       responseID,
       size
     });
   }
 };
 
 /**
  * Attaches request data to a DOM node.
  * @param {Node} node   node to attach data to
  * @param {Window} topWnd   top-level window the node belongs to
- * @param {String} contentType   request type, e.g. "IMAGE"
- * @param {String} docDomain  domain of the document that initiated the request
- * @param {Boolean} thirdParty  will be true if a third-party server has been requested
- * @param {String} location   the address that has been requested
- * @param {Filter} filter   filter applied to the request or null if none
- */
-RequestNotifier.addNodeData = function(node, topWnd, contentType, docDomain, thirdParty, location, filter)
+ * @param {Object} hitData
+ * @param {String} hitData.contentType   request type, e.g. "IMAGE"
+ * @param {String} hitData.docDomain  domain of the document that initiated the request
+ * @param {Boolean} hitData.thirdParty  will be true if a third-party server has been requested
+ * @param {String} hitData.location   the address that has been requested
+ * @param {String} hitData.filter   filter applied to the request or null if none
+ * @param {String} hitData.filterType  type of filter applied to the request
+ */
+RequestNotifier.addNodeData = function(node, topWnd, {contentType, docDomain, thirdParty, location, filter, filterType})
 {
   let entry = {
     id: ++requestEntryMaxId,
     type: contentType,
     docDomain, thirdParty, location, filter
   };
 
   let existingData = nodeData.get(node);
   if (typeof existingData == "undefined")
   {
@@ skipping 10 matching lines @@
     windowStats.set(topWnd.document, {
       items: 0,
       hidden: 0,
       blocked: 0,
       whitelisted: 0,
       filters: {}
     });
   }
 
   let stats = windowStats.get(topWnd.document);
-  let filterType = (filter ? filter.type : null);
   if (filterType != "elemhide" && filterType != "elemhideexception" && filterType != "cssproperty")
     stats.items++;
   if (filter)
   {
     if (filterType == "blocking")
       stats.blocked++;
     else if (filterType == "whitelist" || filterType == "elemhideexception")
       stats.whitelisted++;
     else if (filterType == "elemhide" || filterType == "cssproperty")
       stats.hidden++;
 
-    if (filter.text in stats.filters)
-      stats.filters[filter.text]++;
+    if (filter in stats.filters)
+      stats.filters[filter]++;
     else
-      stats.filters[filter.text] = 1;
+      stats.filters[filter] = 1;
   }
 
   // Notify listeners
   for (let notifier of notifiers.values())
     if (!notifier.window || notifier.window == topWnd)
       notifier.notifyListener(node, entry);
 }