Opened on 03/28/2014 at 09:31:06 PM
Closed on 01/21/2016 at 12:50:49 PM
#230 closed defect (worksforme)
ABP for opera does not block ads in speed dial
Reported by: | =Vitali= | Assignee: | |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | Platform | Keywords: | externaldependency |
Cc: | smultron45@gmail.com, trev, sebastian | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by mapx)
Environment
opera 20
ABP with ruadlist - easylist
How to reproduce
- add rutracker.org in speed dial window
Observed behaviour
the thumbnail shows the site with ads
Expected behaviour
the thumbnail should be without ads (like in a normal tab)
see also
https://adblockplus.org/forum/viewtopic.php?f=14&t=19998
Attachments (0)
Change History (8)
comment:1 Changed on 03/28/2014 at 09:48:06 PM by mapx
- Description modified (diff)
- Summary changed from ABP IN OPERA! to ABP for opera does not block ads in speed dial
comment:2 Changed on 03/28/2014 at 09:53:34 PM by mapx
- Priority changed from Unknown to P3
comment:3 Changed on 03/31/2014 at 07:09:15 AM by philll
- Component changed from Extensions-for-Adblock-Plus to Unknown
- Ready set
comment:4 Changed on 04/02/2014 at 08:17:02 AM by mapx
- Cc smultron45@gmail.com added
comment:5 Changed on 05/07/2014 at 08:22:16 AM by trev
- Cc trev added
comment:6 Changed on 05/07/2014 at 09:41:30 AM by trev
- Cc sebastian added
- Component changed from Unknown to Platform
comment:7 Changed on 06/19/2014 at 06:37:57 PM by sebastian
- Keywords externaldependency added
Treating -1 as a real tab might not be a suitable solution. The frame hierarchy stored inside Adblock Plus, would get replaced every time another page loads. So when multiple pages with tabId == -1 load at the same time, things will go wrong. Also I'm not sure whether it is worth messing with other requests, sent in the background by extensions and Opera itself.
However, I have filed a bug at Opera (DNA-22176) to use individual tabIds for each page when loading its preview on the pre-dial page.
comment:8 Changed on 01/21/2016 at 12:50:49 PM by sebastian
- Platform set to Unknown / Cross platform
- Resolution set to worksforme
- Status changed from new to closed
- Tester set to Unknown
I cannot reproduce this issue anymore as of Opera 34. It seems that they don't render live previews on the speed dial (aka new tab) page anymore, but rather use static images.
I confirmed that the requests initiated by the thumbnail generation have tabId == -1 meaning that they aren't associated with any tab - not even a virtual one. There is a real frame structure however, frameId and parentFrameId are being set correctly. The request types are set correctly, meaning there is a main_frame request first followed by requests for individual resources. For some reason, there is one more request issued for the page URL with type other before the actual load starts, this one has frameId == -1.
Altogether this looks like we might want to start treating tabId == -1 as a real tab and track the frame structure for it. The issue is that requests made by extensions aren't really distinguishable - these have frameId == 0 and would look like they come from the page being loaded for thumbnail generation. The other obvious issue - it isn't possible to distinguish requests produced by two different thumbnails loading in parallel, both obviously have the same tab ID and Adblock Plus will necessarily associate the requests incorrectly. Still, I'm not so much worried about false positives during thumbnail generation, the potential issue here is rather mistakenly blocking requests initiated by extensions or the browser.
Note that while content scripts apparently do run while thumbnails are being generated, I was unable to send a message to the background page - which might have something to do with the fact that they are associated with tab -1.