Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#5298 closed change (fixed)

Disable filter hit counts by default

Reported by: trev Assignee: trev
Priority: P1 Milestone: Adblock-Plus-2.9.1-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by trev)

Background indicates that filter hit counts might be responsible for performance issues on some websites. For reference, 250 filter hits will currently cause filters data to be updated on disk. While this is supposed to be a relatively fast operation, for some users apparently it isn't.

What to change

Disable filter hit counting by default which should be beneficial for most users. Only count filter hits for users who enable it manually (that should be filter list authors for most part).

Hints for testers

In addition to performance implications (see #5284, definitely visible on, other websites are not as clear), it should be tested that filter authors can still re-enable filter hit counting in options and it will work.

Change History (7)

comment:1 Changed 3 years ago by trev

  • Ready set

This has been confirmed as the source of at least some recent bug reports. The issue has been made worse by Adblock Plus 2.9 using API to save data which is currently quite inefficient (see and

comment:2 Changed 3 years ago by trev

  • Owner set to trev

comment:3 Changed 3 years ago by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:4 Changed 3 years ago by abpbot

A commit referencing this issue has landed:
Issue 5298 - Disable filter hit counts by default

comment:5 Changed 3 years ago by trev

  • Milestone set to Adblock-Plus-2.9.1-for-Firefox
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:6 Changed 3 years ago by trev

  • Description modified (diff)

comment:7 Changed 3 years ago by rscott

LGTM. In FF 53.0.3/64bit on Ubuntu Xenial, with the out-of-the-box multiprocess setup, these are the performance profiles I get from loading and scrolling to the bottom of the page:

There's a notable user-visible performance improvement as well.

Note: See TracTickets for help on using tickets.