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):



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

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?

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.

Firefox 55 (Developer) / Windows 7
Chrome 49 / 58 / Windows 7

Add Comment

Modify Ticket

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