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: deprecate <webview>.getWebContents() #20726
Conversation
811cc0e
to
93622dc
Compare
93622dc
to
e62c260
Compare
what's still |
@nornagon the tests will have to be updated to not use the deprecated API |
/cc @electron/wg-api |
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.
The previous API would only allow getting guest webcontents for the current host, right? Do we want to add some example code for checking that?
@nornagon yes, that's why I wanted to just make |
e62c260
to
996f446
Compare
e3aad9c
to
135910d
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
bcf3ec5
to
8817cce
Compare
8817cce
to
6b59b58
Compare
6b59b58
to
fa5bb17
Compare
@codebytere are you ok with merging the PR? |
Release Notes Persisted
|
I have automatically backported this PR to "8-x-y", please check out #21039 |
Description of Change
<webview>.getWebContents()
APIremote.webContents.fromId(<webview>.getWebContentsId())
as replacementChecklist
npm test
passesRelease Notes
Notes: Deprecated
<webview>.getWebContents()
as it depends on theremote
module.