Opened on 01/21/2015 at 01:55:27 PM
Closed on 01/27/2015 at 10:51:07 AM
#1843 closed defect (fixed)
"Block element" dialog keeps showing when extension is disabled/uninstalled/reloaded
| Reported by: | greiner | Assignee: | sebastian |
|---|---|---|---|
| Priority: | P3 | Milestone: | Adblock-Plus-1.8.11-for-Chrome-Opera-Safari |
| Module: | Platform | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Platform: | Unknown | |
| Ready: | yes | Confidential: | no |
| Tester: | Verified working: | ||
| Review URL(s): | |||
Description (last modified by sebastian)
Environment
Ubuntu 14.04
Chrome 39
How to reproduce
- Go to any webpage (e.g. google.com)
- Click on "Block element" in the Adblock Plus icon popup
- Select any element on the page (block.html is shown within a frame)
- Disable the Adblock Plus extension
- Go back to the webpage
Observed behaviour
- block.html is still shown on the page
- Neither clicking on "Cancel" nor "Add" closes the dialog
This behavior persists even after the extension was reenabled.
Expected behaviour
All DOM modifications should be undone when the extension is disabled/uninstalled/reloaded.
Attachments (0)
Change History (7)
comment:1 Changed on 01/21/2015 at 02:54:33 PM by sebastian
comment:2 Changed on 01/21/2015 at 02:54:45 PM by sebastian
- Cc sebastian added
comment:3 Changed on 01/22/2015 at 03:52:59 PM by greiner
Unfortunately, there doesn't appear to be an event we could listen to for that purpose. However, there is a relevant Chromium issue and until then we could at least handle the messaging errors appropriately e.g. by overlaying a message for the user with an "OK" button to, upon clicking, undo any DOM modifications.
comment:4 Changed on 01/22/2015 at 04:01:44 PM by sebastian
As I said you might still need to communicate between frames to undo all DOM modifications. And waiting for an error (usually triggered by user interaction) isn't as smooth as immediately reverting DOM modifications when the extension is unloaded. This is what the user expects when he disables/uninstalls an extension after all. Also you might not be able to catch those errors, if they occur asynchronously.
However, it might be possible to create a port with chrome.runtime.connect() and listen to it's onDisconnected event. It should be fired in the content script when the background page dies. Then we could implement ext.onExtensionUnloaded using ports under the hood.
comment:5 Changed on 01/23/2015 at 09:45:53 AM by sebastian
- Cc sebastian removed
- Description modified (diff)
- Owner set to sebastian
- Priority changed from Unknown to P3
- Ready set
- Summary changed from Cannot close block.html frame if extension got disabled while it was open to "Block element" dialog keeps showing when extension is disabled/uninstalled/reloaded
Awesome, that actually works.
comment:6 Changed on 01/23/2015 at 09:47:48 AM by sebastian
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:7 Changed on 01/27/2015 at 10:51:07 AM by sebastian
- Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
- Resolution set to fixed
- Status changed from reviewing to closed
When the extension is disabled/removed/relaoded, all attempts to message the background page fail. Hence the dialog won't be functional anymore. Technically we could close the dialog without messaging, however the selected elements would still stay highlighted. Since they might be in a different frame, messaging is required here. Also the "Add" button would still not work. So this isn't a solution.
If there is a way to get notified in the content script when the extension is unloaded, we should automatically remove the dialog and undo the highlighting instead. If there isn't such an option, I'd rather in the favor to close this issue as rejected.