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)
Background
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:2 Changed on 09/14/2016 at 11:07:59 AM by kzar
comment:3 Changed on 09/14/2016 at 02:47:15 PM by abpbot
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 1.12.2.1670, Chrome 45, 51, 54, Opera 35, 40 and Safari 7, 9.
A commit referencing this issue has landed:
Issue 4429 - Update dependencies for latest Js Shell