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: support chrome.scripting
extension APIs
#39395
Conversation
ad0243c
to
762d1ce
Compare
385065d
to
f3577e1
Compare
9998354
to
f3577e1
Compare
f3577e1
to
21286cf
Compare
21286cf
to
564ab5c
Compare
564ab5c
to
a9e1472
Compare
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.
API LGTM
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.
API LGTM
Release Notes Persisted
|
I have automatically backported this PR to "27-x-y", please check out #39675 |
I have automatically backported this PR to "26-x-y", please check out #39676 |
I have automatically backported this PR to "25-x-y", please check out #39677 |
@codebytere Is there a list of what V3 APIs are currently supported? I have looked around but have yet to be able to locate it. |
@jasonfill i've tried to tune up this doc to reflect current support - what would help that doc be more effective for you? |
feat: support chrome.scripting extension APIs
Description of Change
Closes #37876.
This PR adds support for
chrome.scripting
.From reference docs:
This represents an important step towards proper support of Manifest V3. Specifically -
chrome.scripting
supersedeschrome.tabs.executeScript
, which is gone in Manifest V3. It's commonly used in several DevTools extensions, including React DevTools.The source code added here is essentially unchanged from upstream - we need to add it to our api definitions because it lives upstream in
chrome/browser/extensions/api
, which we do not depend on (and shouldn't, since many of the api implementations therein involve concepts not transferable to Electron without significant modification).Checklist
npm test
passesRelease Notes
Notes: Added support for
chrome.scripting
extension APIs.