Closed Bug 1736582 Opened 3 years ago Closed 3 years ago

Initial implementation of `scripting.registerContentScripts()`

Categories

(WebExtensions :: General, task, P2)

task
Points:
3

Tracking

(firefox99 fixed)

RESOLVED FIXED
99 Branch
Tracking Status
firefox99 --- fixed

People

(Reporter: willdurand, Assigned: willdurand)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete, Whiteboard: [mv3-m2])

Attachments

(1 file)

Severity: -- → N/A
Points: --- → 3

The current WIP adds scripting.registerContentScripts() and scripting.getRegisteredContentScripts() but it lacks a lot of tests. I am also unclear about thepersistAcrossSessions` option, so that isn't supported yet.

Attachment #9255916 - Attachment description: WIP: Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu! → WIP: Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu!
Blocks: 1751436
Summary: Implement `scripting.registerContentScripts()` → Initial implementation of `scripting.registerContentScripts()`
Attachment #9255916 - Attachment description: WIP: Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu! → Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu!
Blocks: 1755976
Blocks: 1755978
Depends on: 1756495
Blocks: 1756758
Pushed by wdurand@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e079032cf055 Initial implementation of `scripting.registerContentScripts()`. r=robwu
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Regressions: 1756871

For this new API method, we do not support persistAcrossSessions yet. This parameter is currently required and should be set to false, see Bug 1758007.

In addition, we are currently not injecting CSS "before any DOM is constructed or displayed for the page" (quoting Chrome's docs). We will fix this problem in Bug 1759117.

Keywords: dev-doc-needed

Draft of the MDN documentation page is ready for review in the Scripting API documentation #14725 PR. Details about the non-support of persistAcrossSessions will be included in the browser compatibility details.

Flags: needinfo?(wdurand)
Flags: needinfo?(wdurand)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: