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

Remove invalid assertion hit on 30 API tests since 255719@main #5550

Conversation

@achristensen07 achristensen07 self-assigned this Oct 19, 2022
@achristensen07 achristensen07 added WebKit Local Build WebKit Misc. For miscellaneous bugs in the WebKit framework (and not JavaScriptCore or WebCore). labels Oct 19, 2022
@@ -46,8 +46,6 @@ void FrameLoadState::removeObserver(Observer& observer)

void FrameLoadState::didStartProvisionalLoad(const URL& url)
{
ASSERT(m_provisionalURL.isEmpty());
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this assertion is invalid. We hit it once in a while and every time it is a valid bug.

Seems to indicate duplicate didStartProvisionalLoad calls or a provisional load starting without the previous provisional load being cancelled first.

@achristensen07 achristensen07 force-pushed the eng/Remove-invalid-assertion-hit-on-30-API-tests-since-255719main branch from c8cb29c to 4f9c36f Compare October 19, 2022 19:40
Copy link
Contributor

@cdumez cdumez left a comment

Choose a reason for hiding this comment

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

LGTM

@achristensen07 achristensen07 added the merge-queue Applied to send a pull request to merge-queue label Oct 19, 2022
https://bugs.webkit.org/show_bug.cgi?id=246759
rdar://101344112

Reviewed by Chris Dumez.

* Source/WebKit/UIProcess/FrameLoadState.cpp:
(WebKit::FrameLoadState::didStartProvisionalLoad):

Canonical link: https://commits.webkit.org/255755@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/Remove-invalid-assertion-hit-on-30-API-tests-since-255719main branch from 4f9c36f to 1431af1 Compare October 19, 2022 20:56
@webkit-commit-queue
Copy link
Collaborator

Committed 255755@main (1431af1): https://commits.webkit.org/255755@main

Reviewed commits have been landed. Closing PR #5550 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit 1431af1 into WebKit:main Oct 19, 2022
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebKit Misc. For miscellaneous bugs in the WebKit framework (and not JavaScriptCore or WebCore).
Projects
None yet
4 participants