Opened on 05/11/2017 at 07:57:13 PM
Closed on 09/14/2017 at 04:20:22 AM
#5236 closed change (duplicate)
Update the abp2blocklist dependency to SOME_REVISION for Safari
Reported by: | mjethani | Assignee: | |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | Platform | Keywords: | |
Cc: | sebastian, kzar | Blocked By: | #3673 |
Blocking: | Platform: | Safari | |
Ready: | no | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by kzar)
Background
The content blocking API in Safari imposes a limit of 50,000 rules. When using EasyList and similar large subscriptions that can be quickly hit. Therefore for #3673 we have started merging similar rules together. We now need to update the abp2blocklist dependency in the safari branch of adblockpluschrome.
Additionally the following unrelated abp2blocklist changes will be included:
- SOME_CHANGE
- SOME_CHANGE
What to change
- Update abp2blocklist dependency to SOME_REVISION.
- Pass in the merge option to contentBlockerList.generateRules, if it remains an option.
- Optionally, if it makes sense, do not pass in the merge option if we know the rule set is already small enough.
Hints for testers
- Test that the experimental content blocking option still works in Safari.
- Test that a combined filter list such as EasyList Hebrew + Easylist can now be added.
- Test that SOME_CHANGE that was also included with the dependency update didn't break SOME_FEATURE.
Attachments (0)
Change History (2)
Note: See
TracTickets for help on using
tickets.
I've updated the issue description and title to show what's normally included in dependency update issues. We'll need to come back to it and fill in the details once the related abp2blocklist change actually lands.