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

feat(extensions): expose ExtensionRegistryObserver events in Session #25385

Merged
merged 4 commits into from Sep 23, 2020

Conversation

samuelmaddock
Copy link
Member

@samuelmaddock samuelmaddock commented Sep 9, 2020

Description of Change

Extensions can be loaded or unloaded for various reasons. In some cases, this can occur by no means of the Electron programmer, such as in the case of chrome.runtime.reload().

In order to be able to manage state about extensions outside of Electron's APIs, events related to loading and unloaded extensions are needed. The only alternative would be to monkey-patch the Session prototype.

The use case I have is to manage state in my own implementation of chrome.browserAction. I need to know when an extension is added or removed to set or delete saved state.

Ref #19447

cc @nornagon @sentialx

Checklist

Release Notes

Notes: Add extension-loaded, extension-unloaded, and extension-ready events to Session.

Extensions can be loaded and unloaded for various reasons. In some cases this can
occur by no means of the Electron programmer, such as in the case of chrome.runtime.reload().

In order to be able to manage state about extensions outside of Electron's APIs, events
reloaded to loading and unloaded are needed.
@samuelmaddock
Copy link
Member Author

Pinging @electron/wg-api since I think this will require their review.

@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Sep 10, 2020
Copy link
Member

@nornagon nornagon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just a few minor comments

docs/api/session.md Outdated Show resolved Hide resolved
docs/api/session.md Outdated Show resolved Hide resolved
shell/browser/api/electron_api_session.cc Show resolved Hide resolved
@jkleinsc
Copy link
Contributor

The API WG approved this at the Sept 21, 2020 meeting.

@jkleinsc jkleinsc merged commit 9d0d9a1 into electron:master Sep 23, 2020
@release-clerk
Copy link

release-clerk bot commented Sep 23, 2020

Release Notes Persisted

Add extension-loaded, extension-unloaded, and extension-ready events to Session.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants