Opened on 09/25/2017 at 07:36:06 AM
Closed on 09/25/2017 at 12:06:19 PM
Last modified on 09/26/2017 at 06:26:37 AM
#5766 closed defect (invalid)
Youtube ad on main page is not blocked
Reported by: | asmirnov | Assignee: | |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Libadblockplus-Android | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Platform: | Android | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Environment
Updated libadblockplus (#5556 done), any locale (Robert tested with German(?), Anton with Russian).
How to reproduce
- Go to http://m.youtube.com
- Make sure you can see "Ad" or " Реклама" text with ad banner shown
- Uncheck AA checkbox and retry. Does not help.
...
Observed behaviour
One can see "Ad" or " Реклама" text with ad banner shown.
Expected behaviour
No banners are shown
Attachments (6)
Change History (11)
Changed on 09/25/2017 at 07:36:23 AM by asmirnov
Changed on 09/25/2017 at 07:36:42 AM by asmirnov
log file
comment:1 Changed on 09/25/2017 at 07:36:59 AM by asmirnov
- Ready set
comment:2 Changed on 09/25/2017 at 10:49:57 AM by asmirnov
i've checked blocking rules are working, elemhide is working so it seems to be filters issue.
For some reason setting mobile agent in desktop browser (Mozilla/5.0 (Linux; Android 5.0; SM-N9005 Build/LRX21V; wv) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/61.0.3163.98 Mobile Safari/537.36) does not help to reproduce the case on desktop (the ad is not shown) and detect URL to be blocked or element to be hidden for now. To be continued
Changed on 09/25/2017 at 11:33:57 AM by asmirnov
DOM
comment:3 Changed on 09/25/2017 at 11:35:32 AM by asmirnov
BTW this is really helpful https://developers.google.com/web/tools/chrome-devtools/remote-debugging/webviews
Changed on 09/25/2017 at 11:43:51 AM by asmirnov
div classname seems to be different all the time (circumvention?)
Changed on 09/25/2017 at 12:05:36 PM by asmirnov
google made it looking as regular search result
comment:4 Changed on 09/25/2017 at 12:06:19 PM by asmirnov
- Resolution set to invalid
- Status changed from new to closed
comment:5 Changed on 09/26/2017 at 06:26:20 AM by asmirnov
Reply from Monzta:
"Hi Anton,
This should get rid of them: https://github.com/easylist/easylist/commit/1c12fba53ed1535b7b0c71332746a012c25c3a04
Thanks,
Arthur"
Changed on 09/26/2017 at 11:45:57 AM by asmirnov
checked fixed
screenshot