Closed Bug 1758007 Opened 2 years ago Closed 2 years ago

Support for `persistAcrossSessions: false` in `scripting.RegisteredContentScript`

Categories

(WebExtensions :: General, task)

task

Tracking

(firefox100 fixed)

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: robwu, Assigned: willdurand)

References

(Blocks 1 open bug)

Details

(Whiteboard: [addons-jira])

Attachments

(1 file)

Scripts in the scripting API should be persistent by default, with them being optionally non-persistent with the persistAcrossSessions flag.

In our initial implementation, they are not persisted, but the persistAcrossSessions flag is not recognized.

We should update the schema (scripting.json) and tests to support persistAcrossSessions: false. And since the default (persistent scripts) is not supported, we should throw unless persistAcrossSessions: false is set.

To allow extensions to feature-detect the availability of this functionality, it is desirable if the API throws when persistAcrossSessions is not set or true.

This could be done by defining the type of persistAcrossSessions to be required, and an enum of type boolean with value false (example).

Assignee: nobody → wdurand
Attachment #9267002 - Attachment description: WIP: Bug 1758007 - Add support for `persistAcrossSessions: false` in `scripting.RegisteredContentScript`. r?robwu! → Bug 1758007 - Add support for `persistAcrossSessions: false` in `scripting.RegisteredContentScript`. r?robwu!
Pushed by wdurand@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/380a0e054b78
Add support for `persistAcrossSessions: false` in `scripting.RegisteredContentScript`. r=robwu
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: