Polyfill browser.webRequest.ResourceType for Edge

[kzar]

Background

Both Firefox and Chrome provide an Object browser.webRequest.ResourceType which lists the support resource types which can be used to filter webRequest events. Edge does not provide that, but does list the valid resource types when an listener is incorrectly created.

What to change

• In adblockpluschrome/polyfill.js, if browser.webRequest.ResourceType isn't defined, attempt to create a listener for the browser.webRequest.onBeforeRequest event with an invalid resource type filter. Then attempt to obtain the list of valid resource types from the exception's message.

For example:

browser.webRequest.onBeforeRequest.addListener(details => {}, {urls:["<all_urls>"], types:["foo"]})

Results in the following exception on Edge 42.17134.1.0:

Error: Invalid value for argument 2. Property 'types': Value must be one of: [main_frame,sub_frame,stylesheet,script,image,object,xmlhttprequest,fetch,ping,other].

From that, browser.webRequest.ResourceType should look something like this:

{MAIN_FRAME: "main_frame",
SUB_FRAME: "sub_frame",
STYLESHEET: "stylesheet",
SCRIPT: "script",
IMAGE: "image",
OBJECT: "object",
XMLHTTPREQUEST: "xmlhttprequest",
FETCH: "fetch",
PING: "ping",
OTHER: "other"}

• Modify any code which assumes browser.webRequest.ResourceType doesn't exist on Edge, for example in adblockpluschrome/lib/requestBlocker.js.

• Modify webRequest listeners (e.g. in adblockpluschrome/ext/background.js) that we've so far avoided passing the types filter for, to use that instead of manually checking the event's type property.

Notes

• Edge seems to provide the exception as a String, where as the other browsers provide it as an Object which contains a message property which references the String.
• No need to handle them, but here's how the exception message looks on
  • Chrome 72.0.3626.119: Error at parameter 'filter': Error at property 'types': Error at index 0: Value must be one of csp_report, font, image, main_frame, media, object, other, ping, script, stylesheet, sub_frame, websocket, xmlhttprequest.
  • Firefox 64.0: Type error for parameter filter (Error processing types.0: Invalid enumeration value "foo") for webRequest.onBeforeRequest.

Hints for testers

• Ensure request blocking works properly on Chrome (both new and old), Firefox and Edge.
• Ensure that #7305 is still fixed.
• Ensure there aren't any new exceptions thrown by the content scripts.

[abpbot]

A commit referencing this issue has landed:
​Issue 7384 - Polyfill webRequest.ResourceType and Object.values

[abpbot]

A commit referencing this issue has landed:
​Issue 7384 - Only polyfill webRequest.ResourceType in background page

[Ross]

Done. Working as described.

ABP 3.5.2.2339
Microsoft Edge 44.17763.1.0