Opened 2 years ago

Last modified 2 years ago

#5061 closed change

Update adblockpluscore dependency to 3bdddf0e8343 — at Version 4

Reported by: kzar Assignee: kzar
Priority: P2 Milestone: Adblock-Plus-1.13.3-for-Chrome-Opera
Module: Platform Keywords:
Cc: sebastian, trev Blocked By: #4878
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29398651/

Description (last modified by kzar)

Background

With #4878 we started using ESLint for the code in adblockpluscore. We want to update the dependency in adblockpluschrome to use those as soon as possible, they will require lots of testing.

As part of those changes we started using the alternative syntax of Cu.import.

For example this:

Cu.import("resource://gre/modules/XPCOMUtils.jsm", {});

Became this:

const {XPCOMUtils} = Cu.import("resource://gre/modules/XPCOMUtils.jsm", {});

I've also been asked to update the dependency past the ESLint changes, to include the most recent ones.

Full list of changes imported (irrelevant ones struck-through):

  1. Issue 4838 - Use nodeunit framework for integration tests running in browser
  2. Issue 4956 - Fix test_runner.js
  3. Issue 4962 - Fix the tests.
  4. Issue 4878 - Start using ESLint for adblockpluscore
  5. Noissue - Updated copyright year
  6. Issue 5037 - Look for "[-abp-properties=" to recognize the CSS filter
  7. Issue 4919 - Make getBackupFiles in lib/filterStorage.js asynchronous
  • The first three are related to the unit/browser tests in adblockpluscore and are irrelevant to adblockpluschrome.
  • The fourth is the ESLint changes themselves.
  • The fifth change is only to the copyright headers in all files and shouldn't change how adblockpluschrome behaves at all.
  • The sixth change is a small alteration to how CSS property filters are parsed.
  • The final change is to the getBackupFiles FilterStorage function which is only used by the Firefox extension, not adblockpluschrome.

What to change

  • Update the adblockpluscore dependency in adblockpluschrome to 3bdddf0e8343.
  • Update the Cu.import stub in lib/compat.js.

Hints for testers

  • Test CSS property filters still work properly.
  • The ESLint changes will need a thorough test of pretty much all code paths, on both a new and old version of Chrome. I recommend testing that at the same time as the changes for #4864.

Change History (4)

comment:1 Changed 2 years ago by kzar

  • Ready set

comment:2 Changed 2 years ago by kzar

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

comment:3 Changed 2 years ago by kzar

  • Description modified (diff)
  • Summary changed from Update adblockpluscore dependency to 68a4dc4e79e8, including ESLint changes to Update adblockpluscore dependency to 3bdddf0e8343

comment:4 Changed 2 years ago by kzar

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