Opened on 01/18/2019 at 11:12:33 AM

Closed on 08/29/2019 at 05:43:18 PM

#7224 closed defect (rejected)

Generic filter text logged if specific filter with same selector matches

Reported by: greiner Assignee:
Priority: P3 Milestone:
Module: Platform Keywords: closed-in-favor-of-gitlab
Cc: kzar, sebastian, arthur, mjethani Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Environment

Ubuntu 16.04
Chrome 71
Adblock Plus 3.4.2

How to reproduce

  1. Add the following filters:
    ##body
    example.com##body
    @@||example.com^$generichide
    
  2. Open a new tab and open the Adblock Plus DevTools panel
  3. Navigate to http://example.com/

Observed behaviour

DevTools panel lists the following filters:

  • @@||example.com^$generichide
  • ##body

Expected behaviour

DevTools panel lists the following filters:

  • @@||example.com^$generichide
  • example.com##body

Attachments (0)

Change History (4)

comment:1 Changed on 01/19/2019 at 12:33:36 AM by sebastian

  • Cc mjethani added

We cannot trace the chain of events from an applied element hiding filter -> injected selector -> blocked element. So the best we can do is looking for which injected selector there is a hidden element on the page, and then find a filter that is active on the given domain and has the same selector. Some edge cases giving inaccurate results might be inevitable. However, maybe a workaround for this particular case could be if we have multiple candidates to prefer domain-specific over generic filters.

comment:2 Changed on 01/22/2019 at 12:01:58 PM by greiner

Good idea, preferring domain-specific filters should be sufficient to avoid this issue.

comment:3 Changed on 01/22/2019 at 07:02:21 PM by sebastian

  • Priority changed from Unknown to P3
  • Ready set

comment:4 Changed on 08/29/2019 at 05:43:18 PM by sebastian

  • Keywords closed-in-favor-of-gitlab added
  • Resolution set to rejected
  • Status changed from new to closed

Sorry, but we switched to GitLab. If this issue is still relevant, please file it again in the new issue tracker.

Add Comment

Modify Ticket

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