Opened on 04/13/2016 at 11:35:24 AM
Closed on 04/14/2016 at 11:52:14 AM
#3924 closed defect (fixed)
Wrong domain entered into whitelist after using back button
Reported by: | Ross | Assignee: | kzar |
---|---|---|---|
Priority: | P2 | Milestone: | Adblock-Plus-1.12-for-Chrome-Opera-Safari |
Module: | Platform | Keywords: | |
Cc: | sebastian, kzar | Blocked By: | |
Blocking: | Platform: | Safari | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by kzar)
Environment
Reproduced in:
ABP 1.11.0.1593
Safari 8.0.6 / OS X 10.10.3 (Also Safari 9.0)
Not reproduced in:
Chrome 43 / Ubuntu 14.04
How to reproduce
- Navigate to the devbuilds page (https://adblockplus.org/en/development-builds).
- Select the [Older builds] link next for Safari.
- Press the Back button in Safari to navigate back to the devbuilds page.
- Select [ABP Icon] > [Enabled on this site] to disable on this site.
- Select [ABP Icon] > [Options] > [Whitelisted domains].
Observed behaviour
downloads.adblockplus.org was added to the list of whitelisted domains.
Expected behaviour
adblockplus.org should have been added to the list of whitelisted domains instead.
Attachments (0)
Change History (8)
comment:1 Changed on 04/13/2016 at 12:21:01 PM by kzar
- Cc sebastian kzar added
- Component changed from Unknown to Platform
- Platform changed from Unknown / Cross platform to Safari
comment:2 Changed on 04/13/2016 at 02:39:30 PM by kzar
- Description modified (diff)
- Owner set to kzar
- Priority changed from Unknown to P2
- Ready set
comment:4 Changed on 04/13/2016 at 03:36:11 PM by kzar
I attempted to pin down when the regression was introduced, but it looks more like a long standing bug, I could reproduce even with 1.9.4. (I stopped trying to bisect at that point!)
I'm not able to reproduce the issue on Chrome either, so this indeed looks like it's Safari specific. Continuing to investigate...
comment:5 Changed on 04/13/2016 at 03:58:05 PM by kzar
The content script is simply not being loaded again, which means the loading message is not dispatched and the background page isn't aware that the page has changed. Will experiment with listening for different events (from both background and content script) to see if there's a way we can catch this situation.
comment:6 Changed on 04/14/2016 at 09:00:01 AM by kzar
- Review URL(s) modified (diff)
- Status changed from new to reviewing
Looks like we can listen for the pageshow event, if persisted is true then we know to send the loading message again.
comment:7 Changed on 04/14/2016 at 11:49:40 AM by abpbot
A commit referencing this issue has landed:
https://hg.adblockplus.org/adblockpluschrome/rev/5511dd4badc7
comment:8 Changed on 04/14/2016 at 11:52:14 AM by kzar
- Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
- Resolution set to fixed
- Status changed from reviewing to closed
Can reproduce as described with Safari 9.0.