-
Notifications
You must be signed in to change notification settings - Fork 50
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
[mv3] Support popup windows - via message channels #1167
Conversation
This adds support of manifest v3 to the "popup-window" helpers. This is achieved by reimplementing the popup==>background page communication using message ports. Previously, we used two other mechanisms for this communication: * in the Chrome App build mode we injected callbacks into the popup's global state using the chrome.app.window.create() callback. * in the Chrome manifest v2 Extension build mode we used "setSelfAsOpener" to let the popup call global functions in the opener page itself. Neither works in manifest v3 Extensions, hence this commit implements a new approach that should work in all build modes.
Continuous Integration: All tests passed. |
Continuous Integration: All tests passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM % nits
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
Continuous Integration: All tests passed. |
Continuous Integration: All tests passed. |
I've guarded some code in |
This adds support of manifest v3 to the "popup-window" helpers. This is achieved by reimplementing the popup==>background page communication using message ports.
Previously, we used two other mechanisms for this communication:
Neither works in manifest v3 Extensions, hence this commit implements a new approach that should work in all build modes.