Omnibus ad-blocking bug for February 2010

RESOLVED FIXED

Status

Camino Graveyard
Annoyance Blocking
RESOLVED FIXED
9 years ago
8 years ago

People

(Reporter: Smokey Ardisson (offline for a while; not following bugs - do not email), Assigned: Smokey Ardisson (offline for a while; not following bugs - do not email))

Tracking

({fixed1.8.1.24})

Details

(Whiteboard: [camino-2.0.3])

Attachments

(1 attachment, 1 obsolete attachment)

Comment 1

9 years ago
On some forums (vBulletin 4.0.1), forum post appear empty.
The div wrapping the message body:
<div class="content has_ad">

blocked by div[class$="_ad"], 

http://www.bikeforums.net/
http://www.monachos.net/

As reported in the forums
http://forums.mozillazine.org/viewtopic.php?f=12&t=1726625
http://forums.mozillazine.org/viewtopic.php?f=12&t=1731945

Comment 2

8 years ago
The ads (googleads afaict) in the affected blocks are wrapped in class="floatedAd" and already blocked.
Created attachment 428370 [details] [diff] [review]
Preliminary Omnibus Patch

This fixes the vBulletin "has_ad" issue, which has the potential to be fairly widespread (1Password forums have the class on their divs without even having ads!).  It also blocks the ads inside the formerly-missing posts on bikeforum.net.

I'll leave this for a few days before asking for review to see if any other ads to be blocked come up before the end of the month, but I wanted to get this patch up here now so that I don't forget to get it in at the end of the month if I get bogged down later in the week.
Assignee: nobody → alqahira
Status: NEW → ASSIGNED
We need to get this in 1.6.11, too.
Flags: camino2.0.3?
Flags: camino1.6.11?
(In reply to comment #2)
> The ads (googleads afaict) in the affected blocks are wrapped in
> class="floatedAd" and already blocked.

They weren't in my testing, which is why I added that rule.

From today's double-checking, I've also added div[class^="googleAdBlock"] to take out some additional right-hand column ads on bikeforums (the right-hand column ads seem to alternate between Flash and Google). I checked both threads today and got Flash on the right in one and some different Google ads on the right in the other.

Updated

8 years ago
Blocks: 545942

Comment 6

8 years ago
A monstrosity on the NYTimes:

<!-- ADXINFO classification="interstitial-multimedia" campaign="i_int-PRLFashionWeek03_1254154_nyt8"--><!-- BEGIN RICH-MEDIA ShortTail Media CODE -->
<script type="text/javascript" language="JavaScript">
rnum=Math.round(Math.random() * 100000);
ts=String.fromCharCode(60);
if (window.self != window.top) {nf=''} else {nf='NF/'};
document.write(ts+'script src="http://ads.shorttail.net/cgi-bin/ads/ad17922k.cgi/v=2.3S/sz=1x1A/'+rnum+'/'+nf+'RETURN-CODE/JS/">'+ts+'/script>');
</script><noscript><a href="http://ads.shorttail.net/ads/ad17922k-map.cgi/ns/v=2.3S/sz=1x1A/" target="_top">
<img src="http://ads.shorttail.net/cgi-bin/ads/ad17922k.cgi/ns/v=2.3S/sz=1x1A/" border="0" alt="Click Here" /></a>
</noscript>
<!-- END ShortTail Media CODE -->

It showed up as blocked Flash floating over everything; blocking "ads.shorttail.net" is probably what we want to do here. (Not sure why "/ads/" isn't causing it to be blocked already, but whatever.)

cl
(In reply to comment #6)
> A monstrosity on the NYTimes:
[...] 
> It showed up as blocked Flash floating over everything; blocking
> "ads.shorttail.net" is probably what we want to do here. (Not sure why "/ads/"
> isn't causing it to be blocked already, but whatever.)

I have no idea what that is, because there's no Flash at all in the snippet, and the embedded URLs all redirect to an empty .gif file (and browsing the NYTimes doesn't trigger any such Flash for me.)
Created attachment 429447 [details] [diff] [review]
Feb 2010 Omnibus ad-blocking, v2

Now with fixes for comment 5 and bug 545942 (where blocking really horked the layout of the poorly-designed macwelt.de but which was fixable thanks to some CSS from philippe).
Attachment #428370 - Attachment is obsolete: true
Attachment #429447 - Flags: superreview?(stuart.morgan+bugzilla)

Comment 9

8 years ago
Comment on attachment 429447 [details] [diff] [review]
Feb 2010 Omnibus ad-blocking, v2

sr=smorgan
Attachment #429447 - Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Landed on cvs trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
I'm not exactly sure what we want to do here for 1.6.11; we'll need to build before the typical 10 days-2 weeks have passed, but we really want to get the vBulletin fix on 1.6.11 before we EOL it.

I guess I'm inclined to let this patch bake this week and then land it on the 18branch.
Landed this on the MOZILLA_1_8_BRANCH (and the CAMINO_2_0_BRANCH, so as not to leave things hanging).
Flags: camino2.0.3?
Flags: camino2.0.3+
Flags: camino1.6.11?
Flags: camino1.6.11+
Keywords: fixed1.8.1.24
Whiteboard: [camino-2.0.3]
You need to log in before you can comment on or make changes to this bug.