Opened on 10/29/2015 at 08:12:46 AM
Closed on 11/10/2017 at 11:17:30 AM
#3244 closed defect (rejected)
Pop-up blocker unit tests broken in E10S mode
Reported by: | trev | Assignee: | erikvold |
---|---|---|---|
Priority: | P4 | Milestone: | |
Module: | Extensions-for-Adblock-Plus | Keywords: | e10s |
Cc: | Blocked By: | #3495 | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Environment
Firefox 45.0a1
How to reproduce
- Make sure "Enable multi-process Nightly" is enabled in Firefox preferences.
- Go into Adblock Plus unit tests preferences, select popupBlocker and click Run.
Observed behaviour
The test opens pop-up windows but doesn't close them. Closing them manually makes the test complete, albeit with failures.
Expected behaviour
The test should run through without issues, as it does when E10S is disabled.
Attachments (0)
Change History (8)
comment:1 Changed on 11/30/2015 at 09:13:24 PM by erikvold
- Owner set to erikvold
comment:3 Changed on 12/04/2015 at 08:59:54 PM by erikvold
comment:4 Changed on 12/04/2015 at 09:01:23 PM by erikvold
Just for the record, these where the errors that I saw in the console before my patch is applied:
unsafe CPOW usage remote-browser.xml:164:0
unsafe CPOW usage popupBlocker.js:119:4
TypeError: browser.contentWindow.addEventListener is not a function popupBlocker.js:93:1
GET
http://127.0.0.1:1234/redirect [HTTP/1.1 302 Moved Temporarily 4ms]
GET
http://127.0.0.1:1234/test [HTTP/1.1 200 OK 7ms]
unsafe CPOW usage remote-browser.xml:164:0
unsafe CPOW usage popupBlocker.js:119:4
TypeError: browser.contentWindow.addEventListener is not a function
comment:5 Changed on 01/09/2016 at 12:31:20 AM by erikvold
- Blocked By 3495 added
comment:6 Changed on 02/10/2016 at 01:00:26 AM by erikvold
- Resolution set to fixed
- Status changed from new to closed
comment:7 Changed on 02/23/2016 at 02:32:11 PM by trev
- Resolution fixed deleted
- Status changed from closed to reopened
It seems that there are two issues with the solution here, too bad I didn't test it earlier:
- The tests won't work with the pop-up blocker enabled (previously we were simulating proper user clicks that the built-in pop-up blocker wouldn't catch).
- It seems that tabs.on("ready") functionality in the Add-on SDK relies on E10S compatibility shims. Since Adblock Plus no longer uses them the test won't work in E10S mode.
comment:8 Changed on 11/10/2017 at 11:17:30 AM by trev
- Resolution set to rejected
- Status changed from reopened to closed
Mass-closing all bugs in the Extensions for Adblock Plus module, these extensions no longer work as of Adblock Plus 3.0 / Firefox 57.
https://codereview.adblockplus.org/29331996