Opened 15 months ago
Last modified 13 months ago
#6051 new defect
[webextension] Blocked iframes are not collapsed (hidden)
Reported by: | gymka | Assignee: | |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Platform | Keywords: | externaldependencies |
Cc: | kzar, sebastian, mapx, Lain_13, dimisa, arthur | Blocked By: | |
Blocking: | Platform: | Firefox | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by kzar)
Environment
Firefox 57
Adblock Plus 3.0.1
Windows 10 OR Linux (Debian Buster)
How to reproduce
- Add the filter ||alio.lt/infoNewAuto1200.html.
- Browse to http://margevicius.lt/adas.html
Observed behaviour
While the iframe is blocked from loading, the element itself is not collapsed (hidden). So it remains on the page, but empty.
Expected behaviour
The iframe element should be hidden as well as its src request blocked.
Notes
- This works as expected on Chrome.
- On both platforms a SUBDOCUMENT request to https://www.alio.lt/infoNewAuto1200.html is reported as being blocked.
- The problem seems to be caused by the fact that Firefox doesn't fire a load event for the blocked iframe where as Chrome does. On Chrome the frame is replaced and ends up with the location chrome-error://chromewebdata/ for which the load event is fired. On Firefox the frame isn't being replaced, the location is considered about:blank and therefore the load event is never fired.
Change History (19)
comment:1 Changed 15 months ago by gymka
comment:2 Changed 15 months ago by mapx
- Cc trev kzar sebastian mapx added
comment:3 follow-up: ↓ 4 Changed 15 months ago by kzar
- Component changed from Unknown to Platform
- Description modified (diff)
- Summary changed from adb not see iframe with https:// or // url to [webextensions] Adblock Plus doesn't block iframe with https:// or // url
Which operating system are you using?
comment:4 in reply to: ↑ 3 Changed 15 months ago by gymka
comment:5 Changed 15 months ago by gymka
change line ||alio.lt/infoNewNT1200.html. to ||alio.lt/infoNewNTFooter1200.html. it's my error, but i can't modify original post
in original page there is both iframes, in my test page only 1
comment:6 Changed 15 months ago by kzar
- Description modified (diff)
Done. (I think it was my error, I adjusted the filters when I edited the description earlier!)
comment:7 Changed 15 months ago by kzar
- Summary changed from [webextensions] Adblock Plus doesn't block iframe with https:// or // url to [webextension] Adblock Plus doesn't block iframe with https:// or // url
comment:8 Changed 15 months ago by kzar
- Owner set to kzar
- Priority changed from Unknown to P2
- Ready set
Please could you modify your test page so that it has two iframes, one using the // prefix and the other using the https:// prefix? That would make it easier for people to see the problem.
I can reproduce this, according to our devtools pane Adblock Plus blocks the SUBDOCUMENT request of http://www.alio.lt/infoNewNTFooter1200.html (note the http prefix despite the page using https), the iframe element is left empty on the page but not collapsed.
I'll investigate further next week.
comment:9 Changed 15 months ago by gymka
when i'm testing in test page looks like problem is not about "" or "https" but with all url's, simply adblock plus 3.0.1 rules works differently then old version. old version blocked iframe and removed whitespace, 3.0.1 version only blocks iframe, but leaves whitespace.
issue is not with "src" protocol. then i tested in real life there was a lot of ads, so i assumed it's protocol fault.
comment:10 Changed 15 months ago by kzar
- Description modified (diff)
- Summary changed from [webextension] Adblock Plus doesn't block iframe with https:// or // url to [webextension] Blocked iframes are not collapsed (hidden)
when i'm testing in test page looks like problem is not about "" or "https" but with all url's, simply adblock plus 3.0.1 rules works differently then old version. old version blocked iframe and removed whitespace, 3.0.1 version only blocks iframe, but leaves whitespace.
issue is not with "src" protocol. then i tested in real life there was a lot of ads, so i assumed it's protocol fault.
Aha, we're getting somewhere now, I've updated the issue again. Please could you update the test page so that there's only one iframe with the src of https://www.alio.lt/infoNewAuto1200.html ?
I'm going to start investigating what's going on today.
comment:11 Changed 15 months ago by kzar
- Description modified (diff)
comment:12 Changed 15 months ago by kzar
- Description modified (diff)
comment:13 Changed 15 months ago by kzar
- Description modified (diff)
comment:14 Changed 15 months ago by kzar
- Keywords externaldependencies added
I've filed a bug report with Mozilla about this: https://bugzilla.mozilla.org/show_bug.cgi?id=1418975
comment:15 Changed 15 months ago by kzar
- Owner kzar deleted
comment:16 Changed 15 months ago by Lain_13
Add me and dimisa into CC here please. Actually, is there a way to add myself into CC on issues without a request?
comment:17 Changed 15 months ago by mapx
- Cc Lain_13 dimisa added
comment:18 Changed 14 months ago by fhd
- Cc trev removed
comment:19 Changed 13 months ago by arthur
- Cc arthur added
for easier testing, my test page http://margevicius.lt/adas.html