Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#4587 closed defect (fixed)

Blockable items list calls reloadDisabledFilters() unnecessarily often

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

Description (last modified by trev)


Firefox 52.0a1 (2016-10-28 nightly) - multiprocess enabled
Adblock Plus
macOS 10.12

How to reproduce

  1. Press Ctrl/Cmd-Shift-V to open Blockable items list.
  2. Load

Observed behaviour

The page loads considerably slower than with Blockable items list closed.

Expected behaviour

There shouldn't be a significant slowdown.


Profiling shows that the blockable items list keeps calling reloadDisabledFilters() function. This is supposed to happen when filters change, it is currently being triggered by filter hits as well however.

Hints for testers

This change affects Blockable items list displaying disabled filters, the functionality should stay unchanged. If a disabled filter would normally match an allowed request, the Blockable items list should indicate that - and it should consider filters that were disabled while the list is already open. This functionality ignores subscriptions and filter groups that are completely disabled, so changes here should be reflected properly as well.

Change History (6)

comment:1 Changed 4 years ago by trev

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

comment:3 Changed 4 years ago by trev

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

comment:4 Changed 4 years ago by Ross

The slowdown feels fixed.

I'm not sure on the behaviour of filters that have been disabled from the blockable items list? For example, on Heise, if I disable the .promoad filter, it is completely removed from the blockable items list and does not reappear, say, for the user to reenable, is that intended?

Firefox Nightly 53.0a1 (2017-11-17) / OS X 10.10

comment:5 Changed 4 years ago by trev

Yes, disabled element hiding filters cannot be displayed, it only works for blocking filters.

comment:6 Changed 4 years ago by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Okay, thank you for the clarification.

Firefox 38 / 49 / Windows 7

Note: See TracTickets for help on using tickets.