Opened on 10/02/2015 at 02:31:25 PM
Closed on 01/14/2016 at 10:54:52 AM
Last modified on 01/28/2016 at 07:45:31 AM
#3153 closed defect (fixed)
Remove duplicated handling of add-subscription in Platform
Reported by: | greiner | Assignee: | greiner |
---|---|---|---|
Priority: | P2 | Milestone: | Adblock-Plus-1.10.1-for-Chrome-Opera-Safari |
Module: | Platform | Keywords: | goodfirstbug |
Cc: | sebastian, mapx | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Scheer | Verified working: | yes |
Review URL(s): |
Description (last modified by sebastian)
Environment
Chrome 45
Adblock Plus 1.9.2.1481
How to reproduce
- Go to https://adblockplus.org/en/features#tracking
- Click on "open this dialog"
Observed behaviour
Options page is opened twice with only the second instance showing the confirmation dialog.
Expected behaviour
Options page is opened once and shows the confirmation dialog.
Background
With the dependency update in #3087 we integrated code from #1528 that includes opening the options page when the message responder receives the "add-subscription" event. This event, however, is already handled by the background page. This causes the options page to be opened twice whenever an abp:subscribe link is clicked.
What to change
Remove the handling of "add-subscription" in adblockpluschrome/background.js.
Attachments (0)
Change History (9)
comment:1 Changed on 10/06/2015 at 09:45:55 AM by greiner
- Cc sebastian added
comment:2 Changed on 10/19/2015 at 07:08:39 AM by sebastian
- Description modified (diff)
- Keywords goodfirstbug added
- Priority changed from Unknown to P2
- Ready set
comment:3 Changed on 01/04/2016 at 03:52:48 PM by greiner
- Owner set to greiner
comment:4 Changed on 01/04/2016 at 04:06:10 PM by greiner
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:5 Changed on 01/13/2016 at 03:02:13 PM by greiner
comment:6 Changed on 01/13/2016 at 03:04:07 PM by mapx
- Cc mapx added
comment:7 Changed on 01/14/2016 at 10:54:52 AM by greiner
- Resolution set to fixed
- Status changed from reviewing to closed
comment:8 Changed on 01/26/2016 at 11:10:32 AM by sebastian
- Milestone set to Adblock-Plus-1.10.1-for-Chrome-Opera-Safari
comment:9 Changed on 01/28/2016 at 07:45:31 AM by scheer
- Tester changed from Unknown to Scheer
- Verified working set
Done. The options page opens once when opening 'Open this dialog' from https://adblockplus.org/en/features#tracking.
Chrome 44,45,46
Adblock Plus 1.10.0.1534
See also #3510