1190328 - Test coverage for tabs extension API

Status: RESOLVED FIXED

(WebExtensions :: General, defect, P3)

Description

[Gabor Krizsanits (INACTIVE)]

https://wiki.mozilla.org/Browser_Extensions#List_of_supported_APIs

Comment 1

[Kris Maglione [:kmag]]

This is still missing coverage for:

* The |onCreated| and |onRemoved| events.
* The |onUpdated| event with an image change.
* The |onUpdated| progress listener with (stateFlags & Ci.nsIWebProgressListener.STATE_IS_WINDOW) ever being false.
* The |onLocationChange| listener with a "complete", rather than "loading", status.
* |tabs.create| without the |active| or |selected| states.
* |tabs.create| called before the browser window has fully loaded, and does not yet have a |gBrowser| global.
* |tabs.update| without an explicit tab ID.
* The |reload|, |getAllInWindow|, and |insertCSS| API methods.
* |tabs.query| with a non-matching |lastFocusedWindow| or |windowType| filter.
* The |_execute| method with |code|, |allFrames|, |matchAboutBlank|, and |runAt| properties.
* |tabs.connect| without an explicit |name|, or with a |frameId|.
* |tabs.sendMessage| with a |frameId|.

I think we should probably split this into several bugs.


https://people.mozilla.org/~kmaglione/webextension-test-coverage/browser/components/extensions/ext-tabs.js.html