Opened on 09/15/2015 at 10:26:50 AM

Closed on 12/01/2015 at 10:36:59 AM

Last modified on 12/01/2015 at 10:46:45 AM

#3073 closed defect (fixed)

Adblock Browser doesn't display acceptable ads on ebay.com (mobile version) and walmart.com

Reported by: philll Assignee:
Priority: P1 Milestone: Adblock-Browser-for-iOS-1.2.0
Module: Adblock-Browser-for-iOS Keywords: salsita
Cc: fhd, philll, sebastian, oleksandr, mario, sergz, Shikitita Blocked By:
Blocking: Platform: Adblock Browser for iOS
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by mario)

Environment

Iphone 5S
iOS 8.4
Adblock Browser (core 1.3.0) build qa-455
Easylist
Acceptable Ads enabled

How to reproduce

  1. Go to http://www.ebay.com/sch/i.html?_odkw=&_osacat=33829&_from=R40&_trksid=p2045573.m570.l1311.R5.TR12.TRC2.A0.H0.Xiphone.TRS0&_nkw=iphone+6&_sacat=33829 or http://www.walmart.com/search/?query=car%20rental&typeahead=car%20rental
  2. Scroll all the way down

Observed behaviour

Under the paging bar, there is an empty space.

Expected behaviour

Google sponsored links should show as when you open the page with Google Chrome for Desktop and ABP enabled or with ABP disabled in Adblock Browser for iOS.

Remark for testers

The observed behaviour was fixed for the following use cases:

The observed behaviour was not fixed for the desktop version of http://www.ebay.com/ as it was removed from the scope of this issue. However this problem was moved to #3360.

Attachments (1)

ebay_advertisement.png (421.4 KB) - added by mario on 11/30/2015 at 08:49:07 AM.
eBay Advertisement

Download all attachments as: .zip

Change History (25)

comment:1 Changed on 09/15/2015 at 10:27:59 AM by philll

  • Description modified (diff)

comment:2 Changed on 09/15/2015 at 12:13:52 PM by vojtab

comment:3 Changed on 09/15/2015 at 08:17:42 PM by pavelz

  • Cc fhd philll sebastian added

comment:4 Changed on 09/15/2015 at 08:19:24 PM by pavelz

So switching the fallback type to other did not fix ebay sponsored links. Comparing with your IE blocker behavior would be useful confirmation. So leaving it at image for now, will be working on more deterministic solution toward the next release.

comment:6 Changed on 09/16/2015 at 07:36:35 AM by philll

With Adblock Plus for IE 1.4.798, the whitelisted ads on both sites show as expected with Acceptable Ads enabled in a 64 bit IE11 on 64 bit Win7.

comment:7 Changed on 09/16/2015 at 07:45:10 AM by sebastian

  • Cc oleksandr added

Note that the logic in IE is more complex than just the function you linked, that if I understand correctly, only tries to associate types to file extensions as a last resort, while headers are considered first. Maybe Ollie, can explain the logic we use in IE to determine the type of a request better?

Last edited on 09/16/2015 at 07:45:23 AM by sebastian

comment:8 Changed on 09/22/2015 at 10:48:02 AM by oleksandr

Yes, in IE we essentially try to get the mime type first, and only if we fail to get the mime type we try to get the content type from URL. Here is the full function:
https://github.com/adblockplus/adblockplusie/blob/master/src/plugin/PluginWbPassThrough.cpp#L199
Note that we search for the extension first in the URL and then in the query string.
We also assume that requests that do not have parent requests are XMLHttpRequest.

comment:9 Changed on 09/22/2015 at 11:42:49 AM by oleksandr

By "get the mime type", I mean we try to extract the value of HTTP Accept header of the request. We do this before the request is actually issues, obviously.

comment:10 Changed on 10/06/2015 at 07:32:39 AM by mario

  • Cc mario added

comment:11 Changed on 10/06/2015 at 07:57:39 AM by mario

  • Component changed from Unknown to Adblock-Browser-for-iOS

comment:12 Changed on 10/12/2015 at 11:11:18 AM by mario

  • Keywords salsita added

comment:13 Changed on 10/13/2015 at 12:01:02 PM by Shikitita

I'm regressing this issue at the moment, in regards to Walmart, and the website's behaviour is really unreliable. The acceptable ads are never displayed with the first results of the very first search on the website. However, they are sometimes when refreshing the page or when disabling and enabling adblock plus.

It might be random, unfortunately, but the acceptable ads are displayed, so I will accept it in Pivotal.

comment:14 Changed on 10/26/2015 at 08:38:17 AM by pavelz

  • Resolution set to fixed
  • Status changed from new to closed

comment:15 Changed on 10/26/2015 at 10:34:40 AM by mario

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:16 Changed on 10/26/2015 at 10:35:07 AM by mario

  • Status changed from reopened to reviewing

comment:17 Changed on 10/26/2015 at 10:58:36 AM by mario

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:18 Changed on 11/04/2015 at 10:30:39 AM by mario

  • Milestone set to Adblock-Browser-for-iOS-next

comment:19 Changed on 11/27/2015 at 01:57:27 PM by Shikitita

  • Resolution fixed deleted
  • Status changed from closed to reopened

I'm afraid this issue is still occurring but only for the classic/desktop site, not for the mobile site. Acceptable Ads are only displayed when ABP is disabled, regardless if the AA option was disabled or enabled previously.

comment:20 Changed on 11/27/2015 at 03:01:22 PM by sergz

  • Cc sergz added

comment:21 Changed on 11/30/2015 at 08:44:20 AM by pavelz

  • Cc Shikitita added

Sponsored ads on Walmart stopped displaying for me completely - even with another browser. Might be a change in location awareness. But you may have been talking only about eBay, as i see an obvious way of switching to Classic Site there, not on Walmart. So regarding eBay, i need to know how a sponsored ad ON CLASSIC SITE is supposed to look like. I don't see anything like Acceptable Ad on eBay classic site - even with real desktop browsers.

Changed on 11/30/2015 at 08:49:07 AM by mario

eBay Advertisement

comment:22 Changed on 11/30/2015 at 08:50:18 AM by mario

Replying to pavelz:

So regarding eBay, i need to know how a sponsored ad ON CLASSIC SITE is supposed to look like. I don't see anything like Acceptable Ad on eBay classic site - even with real desktop browsers.

  1. Go to ebay.com
  2. Switch to desktop site at the bottom of the page (if necessary)
  3. Search for "iPhone"
  4. "Sponsored Links" are displayed at the bottom of the page right below the search results

-> See screenshot

Last edited on 11/30/2015 at 09:09:45 AM by mario

comment:23 Changed on 12/01/2015 at 10:36:59 AM by mario

  • Description modified (diff)
  • Resolution set to fixed
  • Status changed from reopened to closed

I've removed eBay desktop (only remaining problem) from the scope of this issue and moved it to #3360. I've also outlined the change of scope in the issue's description.
Fixes for eBay mobile and Walmart mobile have landed and this very issue can be closed.

comment:24 Changed on 12/01/2015 at 10:46:45 AM by mario

  • Summary changed from Adblock Browser doesn't display acceptable ads on ebay.com and walmart.com to Adblock Browser doesn't display acceptable ads on ebay.com (mobile version) and walmart.com

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.