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: call loadUrl when opening new windows from links #34159
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.
lgtm, just had a question about the test
ddbdac0
to
eccb487
Compare
Release Notes Persisted
|
I was unable to backport this PR to "18-x-y" cleanly; |
I have automatically backported this PR to "19-x-y", please check out #34189 |
Is this going to be backported to Electron 18? |
@codebytere has manually backported this PR to "18-x-y", please check out #34910 |
* fix: call loadUrl when opening new windows from links * spec: add regression test
Description of Change
Closes #34152.
Fixes an issue where windows opened from new links wouldn't properly load URLs. This happened as a result of cleanup in d44a187 - a section of code was removed which loaded the url for non-guest windows in
openGuestWindow
. It's actually the case that this code is necessary when we're opening new windows from links as well. When we open a new window from a link viaShift+LeftClick
, this routes throughOpenUrlFromTab
, at which point the browser process is responsible for initiating navigation in the new window. This PR restores that code and updates the comment accordingly.Checklist
npm test
passesRelease Notes
Notes: Fixes an issue where some
BrowserWindow
s opened from new links wouldn't properly load URLs.