Opened on 09/14/2016 at 09:06:19 AM

Closed on 09/14/2016 at 02:53:56 PM

Last modified on 10/25/2016 at 10:36:32 AM

#4429 closed change (fixed)

Update Js Shell and related dependencies

Reported by: kzar Assignee: kzar
Priority: P3 Milestone: Adblock-Plus-1.12.4-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: sebastian, kzar, rraceanu Blocked By: #4353, #4376
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: yes
Review URL(s):

Description (last modified by kzar)


For issue #4353 we have been working towards updating Js Shell. Since we were previously using quite an old version this has required changes to adblockpluschrome, adblockpluscore, adblockplustests to remove non-standard / deprecated syntax that's no longer supported. We've also obviously needed to make changes to the jshydra repository and therefore the buildtools repository.

What to change

  • Update the following dependencies:
    • adblockpluscore to hg:103fdc164738 git:199dffd. (Which also includes an unrelated change to Firefox-specific element hiding code.)
    • adblockplustests to hg:592c6d74a9cc git:cfbe11f. (Which also includes a few unrelated buildtools / metadata changes.)
    • buildtools to hg:400d571dc98d git:ee6620c. (Which also includes an unrelated change to exception catching syntax.)
  • Add adblockpluscore/lib/coreUtils.js to lib/adblockplus.js in the [convert_js] section of metadata.common.

Hints for testers

This is not one that you can easily test directly, other than to check the extension works as normal! If this had gone wrong it would likely completely break the extension.)

Attachments (0)

Change History (7)

comment:1 Changed on 09/14/2016 at 09:43:04 AM by kzar

  • Description modified (diff)

comment:2 Changed on 09/14/2016 at 11:07:59 AM by kzar

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

comment:3 Changed on 09/14/2016 at 02:47:15 PM by abpbot

A commit referencing this issue has landed:
Issue 4429 - Update dependencies for latest Js Shell

comment:4 Changed on 09/14/2016 at 02:53:56 PM by kzar

  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed on 09/14/2016 at 02:55:15 PM by kzar

  • Ready set

comment:6 Changed on 10/25/2016 at 08:33:02 AM by kzar

  • Cc rraceanu added
  • Description modified (diff)

(Added a note about testing Robert.)

comment:7 Changed on 10/25/2016 at 10:36:32 AM by rraceanu

  • Verified working set

No issues encountered on ABP version, Chrome 45, 51, 54, Opera 35, 40 and Safari 7, 9.

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 kzar.
Note: See TracTickets for help on using tickets.