Opened on 08/14/2018 at 12:13:30 PM
Closed on 09/17/2018 at 01:53:45 PM
#6860 closed change (fixed)
Update adblockpluscore dependency to revision git:2551978
Reported by: | sergz | Assignee: | hfiguiere |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Libadblockplus | Keywords: | |
Cc: | hfiguiere | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | no | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
https://gitlab.com/eyeo/adblockplus/libadblockplus/merge_requests/15 |
Description (last modified by hfiguiere)
Background
There were a lot of changes in the core, most notably they concern the memory usage and support of new filters, like snippets.
The list of changes imported by this is (these are issue numbers extracted from commits):
46ffcc4 | Noissue - Remove extra blank line in lib/matcher.js | Manish Jethani |
4b3326c | Noissue - Return value of filter match expression directly | Manish Jethani |
655ff97 | Noissue - Define RegExpFilter's 0 getter statically | Manish Jethani |
9ac10ab | Noissue - Check for comment indicator first while parsing filter text | Manish Jethani |
821d706 | Noissue - Delete element hiding domain entry once all filters removed | Manish Jethani |
d811099 | Noissue - Avoid :-abp-properties() legacy syntax parsing for most filters | Manish Jethani |
2fc20a4 | Noissue - Update comment on :-abp-properties() DOM dependence | Manish Jethani |
a2773c0 | Noissue - Add a test to confirm an evil rewrite gets cancelled | Hubert Figuière |
5ecee61 | Noissue - Add @link tag for JSDoc references | Manish Jethani |
42b20fe | Noissue - Remove unnecessary references to undefined | Manish Jethani |
be95169 | Noissue - Remove redundant assignment to zero | Manish Jethani |
ef3749f | Noissue - Add criteria test for ~example.com##world | Manish Jethani |
81c64f3 | Noissue - Check only first two characters for exception marker | Manish Jethani |
df18e31 | Noissue - Remove workaround for WebKit getter misoptimization | Manish Jethani |
e04e8ca | Noissue - Avoid setting ActiveFilter.domainSource | Manish Jethani |
00210e9 | Noissue - Avoid setting BlockingFilter instance properties | Manish Jethani |
5bd94d6 | Noissue - Avoid delete operator on RegExpFilter.regexpSource | Manish Jethani |
4e136fa | Noissue - Remove ElemHideBase.selectorDomains | Manish Jethani |
8be0963 | Noissue - Avoid redundant iterations in subscriptions loop | Manish Jethani |
f77da12 | Noissue - Discard regular expression source when no longer needed | Manish Jethani |
f75f0ce | Noissue - Fix JSDoc type for exceptions in lib/elemHide.js | Manish Jethani |
60a41e8 | Noissue - Make private functions inaccessible via ElemHide object | Manish Jethani |
42fbb57 | Noissue - Update JSDoc to mark optionals in lib/filterClasses.js | Manish Jethani |
efd99fe | Noissue - Use default parameters | Manish Jethani |
4222d52 | Noissue - Update JSDoc to mark optionals in lib/elemHide.js | Manish Jethani |
aef75c4 | Noissue - Update JSDoc @type for Matcher.filterByKeyword | Manish Jethani |
c9221c1 | Noissue - Use plural for element hiding domains | Manish Jethani |
8028eea | Noissue - Compare individual characters instead of substring | Manish Jethani |
7b7286a | Noissue - Trim spaces around element hiding emulation selectors | Manish Jethani |
66b307d | Noissue - Check for null and blank string instead of undefined | Manish Jethani |
5efe12d | Noissue - Use Map to store EventEmitter listeners | Dave Barker |
95a7b57 | Noissue - Access document via this | Manish Jethani |
314bda3 | Noissue - Allow identifying the element in case of error in test | Hubert Figuière |
ff18e95 | Noissue - Change specialization for "ABPVN List+EasyList" | Sergei Zabolotskikh |
0d3c182 | Noissue - Fix naming inconsistency in ContainsSelector | Manish Jethani |
7af872b | Noissue - Fix the escaping of '{' and '}' in CSS selectors | Hubert Figuière |
2b57122 | Noissue - Fixed typo with getLocalizedTexts function Review: https://codereview.adblockplus.org/29567746/ | Dave Barker |
662ce93 | Noissue - Updated recommended subscriptions | Wladimir Palant |
What to change
Adapt the code in accordance with the made changes in the core.
- Change require() in lib/compat.js to strip the leading ./ from module names`
- Change isListedFilter() in lib/api.js since Filter.subscription is no longer a property but a generator function.
- Change isListedSubscription() in lib/api.js since FilterStorage.knownSubscriptions is a Map object and not an array anymore.
- Change lib/filterUpdateRegistration.js to use the new event API, by listening to all the individual events and binding them to emit filterChange
- Import filterNotifier instead of FilterNotifier (singleton , not class)
- In init.js change the load event listener to use the new event API
- Add the new JS files to the build in libadblockplus.gyp
It also seems it makes sense to implement certain functionality in libadblockplus as the following up issues, e.g. introduce the API to work with snippets.
Attachments (0)
Change History (6)
comment:1 Changed on 09/11/2018 at 04:32:22 PM by hfiguiere
- Owner set to hfiguiere
comment:2 Changed on 09/14/2018 at 03:31:45 PM by hfiguiere
- Description modified (diff)
- Summary changed from Update adblockpluscore dependency to revision git:975f6d79dad9e210058e6db7fef4975b95c6b424 to Update adblockpluscore dependency to revision git:2551978
comment:4 Changed on 09/14/2018 at 06:12:59 PM by hfiguiere
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:6 Changed on 09/17/2018 at 01:53:45 PM by hfiguiere
- Resolution set to fixed
- Status changed from reviewing to closed
Note: See
TracTickets for help on using
tickets.
Merged in https://gitlab.com/eyeo/adblockplus/libadblockplus/commit/ed692a2990ae1e75e2db7a9aa9009f4684de5087