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
fix: assign session to remote webContents #23989
Conversation
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.
Looks good from here Eryk 🚀 👍 🏆
web_contents->SetUserAgentOverride(blink::UserAgentOverride::UserAgentOnly( | ||
GetBrowserContext()->GetUserAgent()), | ||
false); | ||
Init(isolate); | ||
AttachAsUserData(web_contents); | ||
InitZoomController(web_contents, gin::Dictionary::CreateEmpty(isolate)); | ||
|
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.
Nit: needless whitespace change (and another one below)
Release Notes Persisted
|
I was unable to backport this PR to "9-x-y" cleanly; |
I have automatically backported this PR to "10-x-y", please check out #24064 |
* fix: assign session to remote webContents * fix: test name
* fix: assign session to remote webContents * fix: test name
Description of Change
This PR ensures that remote webContents (such as devtools and extension background pages) get initialized with
session_
set.Could this be backported to 9-x-y and 10-x-y, please?
Closes #23615
Ref #19447
Checklist
npm test
passesRelease Notes
Notes: Fixed no
session
in webContents of type remote.