Opened 2 years ago

Last modified 2 years ago

#6784 closed change

Update adblockpluscore and adblockplusui dependencies for snippet filters, and the anti-circumvention filter list — at Version 23

Reported by: hfiguiere Assignee: hfiguiere
Priority: P2 Milestone: Adblock-Plus-3.3-for-Chrome-Opera-Firefox
Module: Platform Keywords:
Cc: mjethani, sebastian, kzar, saroyanm, wspee, greiner Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29833597/

Description (last modified by hfiguiere)

Background

We've done some work in the adblockplusui and adblockpluscore repositories towards adding the anti-circumvention filter list, and support for snippet filters. We'd now like to update the dependencies in adblockpluschrome to include that work.

Here are the changes for adblockpluscore:

  • Issue #6690 - Always ignore trailing dot in document domain
  • Issue #6737 - Use String.includes instead of String.indexOf
  • Issue #6559 - Use Map object for known subscriptions, change SpecialSubscription.defaultsMap to Map object
  • Issue #6735 - Store domains in lower case
  • Issue #6619 - Qualify CSS selectors in document style sheet correctly
  • Issue #6727 - Use string rather than map for single-domain filters
  • Issue #6437 - Skip elements not affected by DOM mutations
  • Issue #6689 - Add type property to Subscription class, added anti-CV filter list subscription
  • Issue #6538, #6781 - Add snippets
  • Issue #6733 - Allow empty values in filter options

Here is the change for adblockplusui:

  • Issue #6739 - Adapted code to work with Map implementation of FilterStorage.knownSubscriptions

What to change

Update the adblockpluscore dependency to hg:8e466288a0c3 git:f289a76, and the adblockplusui dependency to hg:9a652397b9af git:2fe1f42.

Hint for testers

TBD (a lot of things)

Change History (23)

comment:1 Changed 2 years ago by hfiguiere

  • Description modified (diff)

comment:2 Changed 2 years ago by hfiguiere

  • Description modified (diff)
  • Summary changed from Upgrade adblockplus core for anti-circumvention and snippets (rev TBD) to Upgrade adblockplus core for anti-circumvention and snippets hg:ba2fb17a1dc1

comment:3 Changed 2 years ago by hfiguiere

  • Description modified (diff)

comment:4 Changed 2 years ago by hfiguiere

We need to update adblockplusui at the same time to get https://issues.adblockplus.org/ticket/6739 that is required.

comment:5 Changed 2 years ago by sebastian

  • Cc saroyanm wspee added

I discussed that with Winsley before. Right after the 3.2 release, #6739 (and only this change) will be merged in the adblockplusui Mercurial repository (they do their development on GitLab and only merge stuff to Mercurial for us to integrate). Then we can update adblockpluscore to the latest revision, and simultaneously update the adblockplusui dependency without pulling in unrelated UI changes at the same time, while not breaking anytinhg (hopefully).

For reference, #6739 (in adblockplusui) and #6559 (in adblockpluscore) have a mutual depdency, i.e. if those changes aren't pulled in simultaneously things will break.

comment:6 Changed 2 years ago by wspee

  • Cc greiner added

comment:7 Changed 2 years ago by hfiguiere

  • Description modified (diff)
  • Summary changed from Upgrade adblockplus core for anti-circumvention and snippets hg:ba2fb17a1dc1 to Upgrade adblockplus core for anti-circumvention and snippets hg:5f851931ffea

comment:8 Changed 2 years ago by hfiguiere

  • Description modified (diff)
  • Summary changed from Upgrade adblockplus core for anti-circumvention and snippets hg:5f851931ffea to Upgrade adblockplus core for anti-circumvention and snippets hg:bec7c35902d3

comment:9 Changed 2 years ago by hfiguiere

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

comment:10 Changed 2 years ago by hfiguiere

  • Owner set to hfiguiere

comment:11 Changed 2 years ago by hfiguiere

  • Description modified (diff)

comment:12 Changed 2 years ago by greiner

See also duplicate ticket #6753.

comment:13 Changed 2 years ago by hfiguiere

I looked for one before even filing this one :-(

comment:14 Changed 2 years ago by hfiguiere

  • Description modified (diff)

comment:15 Changed 2 years ago by hfiguiere

  • Description modified (diff)

comment:16 Changed 2 years ago by hfiguiere

  • Description modified (diff)

comment:17 Changed 2 years ago by mjethani

The last patch for the Snippets feature has landed, but I'm also working on the implementation of a couple of "snippets" and would like to include those as well.

Please wait, I'll update this issue with the final commit hash.

comment:18 Changed 2 years ago by mjethani

  • Description modified (diff)
  • Summary changed from Upgrade adblockplus core for anti-circumvention and snippets hg:bec7c35902d3 to Upgrade adblockplus core for anti-circumvention and snippets hg:8e466288a0c3

comment:19 Changed 2 years ago by mjethani

On second thoughts, we should update the dependency now. I have updated the issue.

comment:20 Changed 2 years ago by kzar

  • Summary changed from Upgrade adblockplus core for anti-circumvention and snippets hg:8e466288a0c3 to Update adblockpluscore and adblockplusui dependencies for snippet filters, and the anti-circumvention filter list

comment:21 Changed 2 years ago by kzar

  • Description modified (diff)

comment:22 Changed 2 years ago by kzar

  • Description modified (diff)

comment:23 Changed 2 years ago by hfiguiere

  • Description modified (diff)
Note: See TracTickets for help on using tickets.