Whitelisted AFS ads are not showing up

[fhd]

Environment

Adblock Plus for IE 1.2.690 with EasyList Germany and Acceptable Ads enabled.

How to reproduce

1. Go to netzwelt.de
2. Search for any term

Observed behaviour

The search ads are not showing up.

Expected behaviour

The search ads should show up.

Hints for testers

The change implemented here might affect other websites in unexpected ways, in particular by causing legitimate content to be blocked even though things worked correctly before. The chances that this actually happens seem to be rather low however.

[fhd]

This was committed without issue in 1.2: ​https://hg.adblockplus.org/adblockplusie/rev/7020c275e744

We should reland this referencing this issue.

[trev]

The original commit was backed out in ​https://hg.adblockplus.org/adblockplusie/rev/b241d73f074d. Please reland with proper issue reference.

[sergz]

The commit ​https://hg.adblockplus.org/adblockplusie/rev/7020c275e744 introduces another trouble.
It allows a lot of ads, which were blocked before.

Env

Acceptable ads is disabled.

How to reproduce

1. Go to ​http://seasonvar.ru/
2. Open any serial page

Observed behaviour

A lot of ads is shown, check the left side.

Expected behaviour

The ads should be blocked.

[fhd]

That's bad, seems like that commit Wladimir reverted did have side effects :(

Can you look into this Sergey? If this is really the cause of Acceptable Ads showing up when they have been disabled, we should definitely find a proper solution quickly - or maybe just make a 1.2 fix release that reverts this.

[arthur]

I cannot reproduce this. Are you using RU AdList+EasyList, Sergey?

[oleksandr]

I don't see any ads there either with RU AdList + EasyList or just plain EasyList (IE11, Windows 8.1). Works as expecteed.

[arthur]

With EasyList only I only get the sometimes.

[arthur]

Regarding #1210:

In the screenshot you can see how izito.de currently looks for me with ABP 1.2 on IE 11.

It looks like this hiding filter ##a[href^="http://www.google.com/aclk?"] is still being applied in the ad iframe although it shouldn't:

1. The responsible whitelist for the ad iframe (@@||google.com/uds/afs$document,subdocument,domain=zoeken.nl) should make this sure via the $document option but apparently doesn't.
2. Even if the $document option doesn't work, this element hiding whitelist in EasyList should show it: www.google.com#@#a[href^="http://www.google.com/aclk?"]

[sergz]

After discussing it in IRC some things are clear. Seems it work, I need to perform additional testing.

Despite of it I would clarify the code, I cannot find the further usage of CFilter::contentTypeAny & ~CFilter::contentTypeSubdocument; which looks like here was an idea but then something changed. Do I correctly understand that we actually should return CFilter::contentTypeAny (which is 2^16-1 which is a union of all other types) without subtracting any type. We might can in theory return something like CFilter::contentTypeXmlHttpRequest & CFilter::contentTypeDocument (BTW, xmlhttprequest refers to requests started by the XMLHttpRequest object, not a request which queries for xml file, but it's another issue) and feed it to the matcher via serializing to the string like "xmlhttprequest,document". For CFilter::contentTypeAny the content type string would include all types. The matcher firstly checks whether some of these types is in the white list, if so return <not-blocked, filter-entry>, otherwise check the blocking entries. As I understand now it works similar but expects only one type in the contentType.
What do you think about it, what was the aim of CFilter::contentTypeAny, why do we subtract CFilter::contentTypeSubdocument and what happened with the idea?

[oleksandr]

This issue was fixed, since issues that were blocking it are fixed now. Current issue was just a meta-issue for those. However this one had real world manifestations, and could be observed by the user, unlike the underlying issues.