Filter hit counters aren't cleared

[Berroll]

Environment

I'm using Adblock Plus version 2.6 in Firefox 24.5.0 ESR (Ubuntu GNU/Linux 12.04.4 LTS).
I use RuAdList+EasyList, Fanboy Social Blocklist, EasyPrivacy and some custom filters.

How to reproduce

When I open Filter Settings > Personal Filters, select all my custom filters and use "Clear hit counters" (sorry if the menu names aren't correct as I'm translating from Russian), the counters are cleared and show all zeroes.

Observed behaviour

When I restart Firefox and open the same settings tab, all my counters are restored back with the same numbers which were shown before clearing them.

Expected behaviour

The counters should remain cleared so I can see which filters aren't used anymore and remove them.

I also tried clearing these counters separately, not all together, but after browser restart they revert back to their former state.

[philll]

I cannot reproduce your issue exactly, but i noticed that the custom filter hits are 1 instead of 0 after a Firefox restart without having visited any page after clearing.

[beelzy]

I was able to reproduce this problem and find out why. Apparently, resetHitCounter is counted as a 'hitCount' action, so setDirty is only increased by 0.002, which is not enough to call SaveToDisk (unless you wait long enough or remove enough entries or setDirty is already high enough). You can still reset the hit counters by clearing the history as a workaround until this gets fixed. This problem also appears to affect subscription filters.

My suggestion would be to make resetHitCount have its own action, or give it another one that exists and requires the same behavior or call SaveToDisk when you click on the close button. I know this isn't tagged as goodfirstbug, so I'm unsure if you want me to work on this, but that's what I found out about it.

[trev]

Yes, hit counts are generally not considered important enough to trigger saving to disk immediately. When clearing all hit counts we call FilterListener.setDirty(0); explicitly because of that, we might want to do it here as well - or just make FilterStorage.resetHitCounts() always do it.

[mapx]

unfortunately it's not only about hits but all other settings (filter subscriptions, custom filters)
see
https://issues.adblockplus.org/ticket/513

[trev]

Not the same issue. Hit counts not being saved immediately is expected, other data - not so much.

[saroyanm]

Revision:
​https://hg.adblockplus.org/adblockplus/rev/13e2b53e6ed8

[philll]

Made Firefox and Firefox mobile available as seperate platforms.