Opened on 05/20/2016 at 11:10:43 AM

Closed on 05/20/2016 at 01:58:39 PM

Last modified on 05/26/2016 at 01:12:58 PM

#4054 closed defect (fixed)

Element hiding rules continue to be matched after being removed

Reported by: kzar Assignee: kzar
Priority: P1 Milestone:
Module: Core Keywords:
Cc: sebastian, trev Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

Description (last modified by kzar)


adblockpluschrome after b7884247c8ed

How to reproduce

  1. Enable EasyList and run this code in the console:
var ElemHide = require("elemHide").ElemHide;
  1. Disable EasyList and run the same code again.

Observed behaviour

The results are the same, an array of around 17,000 selectors.

Expected behaviour

The second time a much shorter/empty array should be returned.


When making the changes for #235 we forgot to remove filters from the filtersByDomain lookup again.

Attachments (0)

Change History (8)

comment:1 Changed on 05/20/2016 at 11:36:30 AM by kzar

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

comment:2 Changed on 05/20/2016 at 12:15:48 PM by trev

  • Component changed from Platform to Core

comment:3 Changed on 05/20/2016 at 01:44:33 PM by abpbot

A commit referencing this issue has landed:

comment:4 Changed on 05/20/2016 at 01:58:39 PM by kzar

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed on 05/26/2016 at 12:43:20 PM by kzar

  • Description modified (diff)

comment:6 Changed on 05/26/2016 at 12:43:38 PM by kzar

  • Description modified (diff)

comment:7 Changed on 05/26/2016 at 12:44:54 PM by kzar

  • Description modified (diff)

comment:8 Changed on 05/26/2016 at 01:12:58 PM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Chrome 31, 45, 50 / Windows 7
Chrome 32, 49 / OS X 10.11
Chrome 49 / Ubuntu 14.04
Opera 36 / Ubuntu 14.04

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from kzar.
Note: See TracTickets for help on using tickets.