#2151 closed defect (fixed)
$sitekey ignored for $elemhide filters
Reported by: | greiner | Assignee: | greiner |
---|---|---|---|
Priority: | P1 | Milestone: | Adblock-Plus-2.6.9-for-Firefox |
Module: | Adblock-Plus-for-Firefox | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Platform: | Firefox | |
Ready: | yes | Confidential: | no |
Tester: | Verified working: | ||
Review URL(s): |
Description
Environment
Firefox 36.0.1
Adblock Plus 2.6.8.3914
How to reproduce
- Add the following custom filter: @@*$elemhide,sitekey=MHwwDQYJKoZIhvcNAQEBBQADawAwaAJhALky0YRGoxmI1Wj/uDqZwxIfLA4oU/0pT09NC4IyT87Esd4jJU6TIQL9sg1L2KHyg8MD4gY9GWoFNLR2j+0cWIhihnFscBQTUxsSM/uzJF8jqK5FVaR4pyYunqGr+uJ+qQIDAQAB
- Go to http://muvie.com
- Open blockable items panel
Observed behaviour
The blockable items list show one or more element hiding filter as active on the page.
Expected behaviour
The blockable items list shows no element hiding filters as active on the page.
Background
This issue goes back to #432 which made use of the new $sitekey behavior ,as introduced in #431, in the Firefox version.
What to change
Append the sitekey as an additional parameter to the following call in lib/contentPolicy.js:
match = defaultMatcher.matchesAny(testWndLocation, "ELEMHIDE", parentDocDomain, false);
Change History (5)
comment:1 Changed 5 years ago by trev
- Platform changed from Unknown to Firefox/Firefox Mobile
- Priority changed from Unknown to P1
- Ready set
comment:2 Changed 5 years ago by greiner
- Owner set to greiner
comment:3 Changed 5 years ago by greiner
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed 5 years ago by greiner
- Milestone set to Adblock-Plus-for-Firefox-next
- Resolution set to fixed
- Status changed from reviewing to closed
comment:5 Changed 5 years ago by philll
- Platform changed from Firefox/Firefox Mobile to Firefox
Made Firefox and Firefox mobile available as seperate platforms.
Note: See
TracTickets for help on using
tickets.
Fixed: https://hg.adblockplus.org/adblockplus/rev/24a1d79fec93