Opened on 06/07/2017 at 05:31:22 AM
Closed on 06/13/2017 at 06:49:28 AM
Last modified on 07/07/2017 at 11:20:00 AM
#5300 closed change (fixed)
Distinguish OBJECT and OBJECT_SUBREQUEST requests on Firefox
Reported by: | sebastian | Assignee: | jsonesen |
---|---|---|---|
Priority: | P2 | Milestone: | Adblock-Plus-1.13.3-for-Chrome-Opera |
Module: | Platform | Keywords: | goodfirstbug |
Cc: | trev, jsonesen | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Ross | Verified working: | yes |
Review URL(s): |
Description
Background
As Chrome cannot distinguish between requests loading files handled by a plugin like Flash (i.e. object) and subsequent requests caused by such a file (i.e. object_subrequest) these types are mapped to each other. However, apparently, despite being undocumented, these requests are reported with different types to Firefox WebExtensions.
What to change
Check chrome.webRequest.ResourceType, and don't alias the OBJECT and OBJECT_SUBREQUEST types in RegExpFilter.typeMap, if object_subrequest is supported.
Attachments (0)
Change History (7)
comment:1 Changed on 06/07/2017 at 12:41:37 PM by jsonesen
- Owner set to jsonesen
comment:2 Changed on 06/12/2017 at 06:17:24 AM by jsonesen
comment:3 Changed on 06/12/2017 at 11:01:34 AM by trev
More detailed resource types landed in Firefox 54, so the current stable isn't supporting OBJECT_SUBREQUEST yet.
comment:4 Changed on 06/13/2017 at 06:47:16 AM by abpbot
A commit referencing this issue has landed:
Issue 5300 - Add feature detection for object_subrequest
comment:5 Changed on 06/13/2017 at 06:49:28 AM by jsonesen
- Resolution set to fixed
- Review URL(s) modified (diff)
- Status changed from new to closed
comment:6 Changed on 06/13/2017 at 08:18:03 AM by sebastian
- Milestone set to Adblock-Plus-for-Chrome-Opera-next
comment:7 Changed on 07/07/2017 at 11:20:00 AM by Ross
- Tester changed from Unknown to Ross
- Verified working set
Implemented. This hasn't broken anything as far as I can tell so will mark verified. I need to check this properly once #5316 is implemented.
ABP 1.13.2.1785
Firefox 55 (Developer) / Windows 7
Chrome 49 / 58 / Windows 7
@trev:
On Friday I was looking into the ResourceType property of the browser.webRequest object. However I was unable to find the object_subrequest key value there. Although I thought I remembered seeing it previously, this was done in the debug console in firefox for the extension. Is it possible I am doing something wrong here?