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
- 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
- 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:
- Mobile version of http://www.ebay.com/
- Mobile version of http://www.walmart.com/ (no desktop version available on mobile phones)
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)
Change History (25)
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:5 Changed on 09/15/2015 at 08:20:53 PM by pavelz
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?
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.
- Go to ebay.com
- Switch to desktop site at the bottom of the page (if necessary)
- Search for "iPhone"
- "Sponsored Links" are displayed at the bottom of the page right below the search results
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
PT ticket https://www.pivotaltracker.com/story/show/103402340