1236122 - Complete test coverage for the WebExtension webRequest API.

Status: RESOLVED WORKSFORME

(WebExtensions :: General, defect, P3)

Description

[Kris Maglione [:kmag]]

This is still missing coverage for:

* Events with filters with |types|, |tabId|, or |windowId| properties.
* Attempt to use blocking listeners without the appropriate permissions.
* Initial process data with any initial content policy data.
* |addContentPolicy| and |WebRequest:RemoveContentPolicy| calls where |contentPolicies.size| is anything other than 0.
* |shouldLoad| calls with non-blocking policies.
* |typeMatches| calls with non-null |filterTypes|.
* |urlMatches| calls with a null |urlFilter|.
* |addListener| without a |urls| property in the filter.
* The |getHeaders| method.
* |runChannelListener| with |opts.requestHeaders| or |opts.responseHeaders| in the listener options, or |result.requestHeaders|, |result.responseHeaders|, or |result.cancel| in the listener result.
* |examine| calls with |onStart.size| of 0, and |onStop.size| of both 0 and non-0.


https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/components/extensions/ext-webRequest.js.html
https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/modules/addons/WebRequestContent.js.html
https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/modules/addons/WebRequestCommon.jsm.html
https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/modules/addons/WebRequest.jsm.html

Comment 1

[Shane Caraveo (:mixedpuppy)]

We have pretty extensive coverage at this point.