Skip to content
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: resolve loadURL properly for in-page navigations #36151

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Oct 26, 2022

Backport of #36129

See that PR for details.

Notes: Fixed spurious promise rejection in webContents.loadURL when navigating to a hash.

Co-authored-by: Jeremy Rose <jeremya@chromium.org>
@trop trop bot requested a review from nornagon October 26, 2022 20:58
@trop trop bot added 22-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Oct 26, 2022
Copy link
Contributor

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

newly added test fails

@nornagon
Copy link
Member

@jkleinsc it ... does?

@nornagon
Copy link
Member

@jkleinsc huh... okay, maybe there's flake there, since it seems to be passing now. Since this is already landed on main, can I follow up with deflake?

@jkleinsc jkleinsc merged commit 4a13d27 into 22-x-y Nov 1, 2022
@jkleinsc jkleinsc deleted the trop/22-x-y-bp-fix-resolve-loadurl-properly-for-in-page-navigations-1666817890664 branch November 1, 2022 15:08
@release-clerk
Copy link

release-clerk bot commented Nov 1, 2022

Release Notes Persisted

Fixed spurious promise rejection in webContents.loadURL when navigating to a hash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
22-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants