Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proposal: Extension Loading Priority #557

Open
erosman opened this issue Mar 4, 2024 · 3 comments
Open

Proposal: Extension Loading Priority #557

erosman opened this issue Mar 4, 2024 · 3 comments
Labels
neutral: firefox Not opposed or supportive from Firefox neutral: safari Not opposed or supportive from Safari spec clarification Needs clarification when specified

Comments

@erosman
Copy link

erosman commented Mar 4, 2024

Proposal: Extension Loading Priority

There are extensions that due to their specific functions, need to be loaded as soon as possible.
For example, extensions with the following permissions would need to be prioritised:

  • privacy
  • proxy

It would be good to have a consensus on the issue.

@dotproto dotproto added needs-triage: chrome Chrome needs to assess this issue for the first time needs-triage: safari Safari needs to assess this issue for the first time needs-triage: firefox Firefox needs to assess this issue for the first time and removed needs-triage labels Mar 9, 2024
@xeenon xeenon added neutral: safari Not opposed or supportive from Safari and removed needs-triage: safari Safari needs to assess this issue for the first time labels Mar 14, 2024
@xeenon
Copy link
Collaborator

xeenon commented Mar 14, 2024

Safari does not support privacy or proxy permissions, but we do load extensions right away at launch to have content blockers and injected scripts ready before pages load.

@Rob--W Rob--W added spec clarification Needs clarification when specified neutral: firefox Not opposed or supportive from Firefox and removed needs-triage: firefox Firefox needs to assess this issue for the first time labels Mar 14, 2024
@Rob--W
Copy link
Member

Rob--W commented Mar 14, 2024

Added spec clarification Needs clarification when specified because a call out to early startup behavior in the spec, even a non-normative note would be useful. It is not clear whether we can require browser implementations to activate on early startup due to the performance impact any such guarantees would have, so I expect that we cannot do more other than adding a non-normative note.

In practice, Firefox tries to make sure that extensions are loaded in situations where needed. The proxy and webRequest APIs specifically block the startup (implemented in https://bugzilla.mozilla.org/show_bug.cgi?id=1447551). Due to performance concerns we haven't implemented that for declarativeNetRequest yet (https://bugzilla.mozilla.org/show_bug.cgi?id=1821676).

@erosman
Copy link
Author

erosman commented Mar 14, 2024

It should also be noted that the priority should also apply when compared with other extensions. That is to say, extensions with above specific functions should be loaded BEFORE other extensions.

@oliverdunk oliverdunk removed the needs-triage: chrome Chrome needs to assess this issue for the first time label Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
neutral: firefox Not opposed or supportive from Firefox neutral: safari Not opposed or supportive from Safari spec clarification Needs clarification when specified
Projects
None yet
Development

No branches or pull requests

5 participants