Opened on 06/14/2017 at 03:31:10 AM
Closed on 09/07/2017 at 11:54:46 PM
Last modified on 09/23/2017 at 02:56:34 PM
#5315 closed change (fixed)
Add support for Microsoft Edge
Reported by: | oleksandr | Assignee: | oleksandr |
---|---|---|---|
Priority: | P2 | Milestone: | Adblock-Plus-1.13.4-for-Chrome-Opera |
Module: | Platform | Keywords: | |
Cc: | kzar, sebastian | Blocked By: | |
Blocking: | Platform: | Edge | |
Ready: | yes | Confidential: | no |
Tester: | Ross | Verified working: | yes |
Review URL(s): |
Description (last modified by oleksandr)
Background
We have a version of ABP for Microsoft Edge under the edge bookmark currently. With Windows 10 Creators Update, the amount of Edge specific changes is quite minimal, so we can drop the bookmark and just operate from one version of platform code.
What to change
Move the necessary changes from the edge bookmark to master.
Hints for testers
The changes for this ticket touched the following code paths, so it would be preferable to them:
- We are overriding the core namespace for extensions API. So please make sure that basic functionality like popup page population, localization of options page works.
- We have changed the way detection of default language works. So please make sure the correct subscription is selected for a language, options page is in correct language, and that left-to-right and right-to-left languages are working fine.
- We have changed how options page is being opened. Please make sure both options page and first run page can be opened as expected.
- We have touched the code that deals with our WebSocket wrapper, so checking if WebSocket blocking works would be great. Also if OBJECT and OBJECT_SUBREQUEST filter types work fine.
Attachments (0)
Change History (14)
comment:2 Changed on 07/11/2017 at 04:20:28 PM by kzar
- Cc kzar sebastian added
- Priority changed from Unknown to P2
- Ready set
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:3 Changed on 07/28/2017 at 11:43:29 AM by abpbot
comment:4 Changed on 07/28/2017 at 12:05:28 PM by kzar
So most fixes have made it into master with the above commit. Ollie's now looking into the storage workaround, when he's done we'll either rebase the IndexDB commit or come up with a different workaround and push that to the edge bookmark.
comment:5 Changed on 08/24/2017 at 11:38:12 AM by oleksandr
- Resolution set to fixed
- Status changed from reviewing to closed
The IndexedDB change is described in #5562. Closing this ticket.
comment:6 Changed on 08/29/2017 at 12:49:51 PM by kzar
- Resolution fixed deleted
- Status changed from closed to reopened
I think this issue should remain open until the edge branch is no longer necessary at all. We're nearly there, but not quite.
comment:7 Changed on 08/29/2017 at 12:57:40 PM by sebastian
comment:9 Changed on 09/05/2017 at 10:18:35 PM by sebastian
- Milestone set to Adblock-Plus-for-Chrome-Opera-next
comment:10 Changed on 09/05/2017 at 11:18:01 PM by sebastian
I suppose this issue can be closed now? But how about adding a "Hints for testers" section, highlighting the changed code paths which need to be tested on Chrome/Opera as well, in order to make sure that we didn't introduce any bugs there?
comment:11 Changed on 09/06/2017 at 09:31:32 AM by kzar
Well don't we still need the edge branch for the storage workaround? IMO we shouldn't close this issue until that's no longer the case.
comment:12 Changed on 09/06/2017 at 05:04:46 PM by sebastian
Well, if this issue is supposed to be a meta issue (IMO we don't need one) for everything Microsoft Edge until we abandon the edge bookmark, then there has to be a separate issue for the changes merged upstream in https://codereview.adblockplus.org/29458601/, so that this code gets tested.
comment:13 Changed on 09/07/2017 at 11:54:46 PM by oleksandr
- Description modified (diff)
- Resolution set to fixed
- Status changed from reopened to closed
comment:14 Changed on 09/23/2017 at 02:56:34 PM by Ross
- Tester changed from Unknown to Ross
- Verified working set
Child tickets verified. Localization, right-to-left text, options page opening, websocket/object/subrequest work as expected.
ABP 0.9.11.1849
Edge 40 / Windows 10
ABP 1.13.3.1838
Chrome 49 / 61 / Windows 10
Opera 36 / 47 / Windows 10
ABP 2.99.0.1838beta
Firefox 53 / 55 / Windows 10
A commit referencing this issue has landed:
Issue 5315 - Add support for Microsoft Edge