Issue 394 - hit statistics tool data collection

lib/contentPolicy.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 90 matching lines @@
    * @return {Object} An object containing properties allow, collapse and hits
    *                  indicating how this request should be handled.
    */
   shouldAllow: function({contentType, location, frames, isPrivate})
   {
     let hits = [];
 
     function addHit(frameIndex, contentType, docDomain, thirdParty, location, filter)
     {
       if (filter && !isPrivate)
-        FilterStorage.increaseHitCount(filter);
+      {
+        let topWindowLocation = frames[frames.length-1].location;
+        FilterStorage.increaseHitCount(filter, getHostname(topWindowLocation));

[Wladimir Palant, 2016/02/15 12:25:40]

getHostname(topWindowLocation) should be identical to docDomain.

[saroyanm, 2016/02/19 17:38:11]

Done.

+      }
       hits.push({
         frameIndex, contentType, docDomain, thirdParty, location,
         filter: filter ? filter.text : null,
         filterType: filter ? filter.type : null
       });
     }
 
     function response(allow, collapse)
     {
       return {allow, collapse, hits};
@@ skipping 273 matching lines @@
   {
     // EffectiveTLDService throws on IP addresses, just compare the host name
     let host = "";
     try
     {
       host = uri.host;
     } catch (e) {}
     return host != docDomain;
   }
 }