Closed
Bug 1736582
Opened 3 years ago
Closed 3 years ago
Initial implementation of `scripting.registerContentScripts()`
Categories
(WebExtensions :: General, task, P2)
WebExtensions
General
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)
(deleted),
text/x-phabricator-request
|
Details |
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Severity: -- → N/A
Updated•3 years ago
|
Points: --- → 3
Assignee | ||
Comment 1•3 years ago
|
||
Assignee | ||
Comment 2•3 years ago
|
||
The current WIP adds scripting.registerContentScripts()
and scripting.getRegisteredContentScripts() but it lacks a lot of tests. I am also unclear about the
persistAcrossSessions` option, so that isn't supported yet.
Updated•3 years ago
|
Attachment #9255916 -
Attachment description: WIP: Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu! → WIP: Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu!
Assignee | ||
Updated•3 years ago
|
Summary: Implement `scripting.registerContentScripts()` → Initial implementation of `scripting.registerContentScripts()`
Updated•3 years ago
|
Attachment #9255916 -
Attachment description: WIP: Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu! → Bug 1736582 - Initial implementation of `scripting.registerContentScripts()`. r?robwu!
Pushed by wdurand@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e079032cf055
Initial implementation of `scripting.registerContentScripts()`. r=robwu
Comment 4•3 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox99:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Assignee | ||
Comment 5•3 years ago
|
||
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
Comment 6•3 years ago
|
||
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)
Assignee | ||
Updated•3 years ago
|
Flags: needinfo?(wdurand)
Updated•3 years ago
|
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•