Opened on 01/02/2015 at 01:04:06 PM

Closed on 10/09/2019 at 08:27:51 PM

#1747 closed change (rejected)

GetElementHidingSelectors should return an empty list if the website is whitelisted by any option

Reported by: sergz Assignee:
Priority: Unknown Milestone:
Module: Core Keywords: closed-in-favor-of-gitlab
Cc: greiner Blocked By: #1711
Blocking: Platform: Unknown
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



Since this question periodically arises it would be nice to discus it to better understand whether we need it or not and the reasons of the final decision.

Currently the client of libadblockplus and of the core library has to firstly test whether the website is whitelisted or not and only then call GetElementHidingSelectors if necessary.


  • avoiding of the same logic on different clients


  • the current version seems good from the API fatness point of view, there are no auxiliary parameters, merely what we need, there is no additional logic (see Pros) which can limit the usage of it.
  • the current version seems useful to show potentially blockable items, otherwise the API could be overcomplicated with additional flags/parameters
  • for the performance reason we should not call it twice or more for the same website, at the same time we need to know exactly whether the website is whitelisted or not.

Please leave your thoughts.

Attachments (0)

Change History (3)

comment:1 Changed on 01/05/2015 at 04:25:15 PM by fhd

Note that it's not just GetElementHidingSelectors, it's the same with Matches - the logic has to be in the client currently. If we were to change this, we should change it everywhere IMO, everything else seems confusing. But I suppose you're not a fan of that since you only propose it for GetElementHidingSelectors?

comment:2 Changed on 10/09/2019 at 11:33:28 AM by greiner

  • Cc greiner added
  • Component changed from Unknown to Core
  • Tester set to Unknown
  • Verified working unset

comment:3 Changed on 10/09/2019 at 08:27:51 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
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.