Bug 284812
Summary: | [GTK Debug] A few API tests times out with assertion error "ASSERTION FAILED: !didReceiveInvalidMessage" | ||
---|---|---|---|
Product: | WebKit | Reporter: | zsun |
Component: | New Bugs | Assignee: | zsun |
Status: | NEW | ||
Severity: | Normal | CC: | csaavedra, fujii.hironori, kkinnunen, zsun |
Priority: | P2 | ||
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
zsun
ASSERTION FAILED: !didReceiveInvalidMessage
/app/webkit/Source/WebKit/Platform/IPC/Connection.cpp(1454) : void IPC::Connection::dispatchMessage(WTF::UniqueRef<IPC::Decoder>)
Affected tests -
/webkit/WebKitWebView/sync-request-on-max-conns
/webkit/WebKitWebView/stop-loading
/webkit/WebKitWebProcessExtension/user-messages
/webkit/WebKitWebView/run-async-js-functions
/webkit/WebKitWebView/navigation-after-session-restore
/webkit/WebKitWebsiteData/storage
/webkit/WebKitWebView/stop-loading
/webkit/WebKitConsoleMessage/security-error
/webkit/WebKitFrame/uri
/webkit/WebKitFrame/javascript-values
example bots output: https://build.webkit.org/#/builders/63/builds/15096
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Kimmo Kinnunen
Idea of the assertion is that it makes no sense that one would receive an invalid message. This is a programming error. Typically it is caused by sending a message to a receiver that does not exist at the time of the receipt, which does not typically make sense.
Code should be typically organized so that the sender is the one removing the target endpoint.
zsun
This issue might be related -
https://bugs.webkit.org/show_bug.cgi?id=265994
The tricky part I found is that the sender is not aware that the receiver has been removed (via removeWorkQueueMessageReceiver). In the case we describe in bug 265994, any suggestions that we could inform the sender that the receiver does not exist any more? Thank you!
zsun
Pull request: https://github.com/WebKit/WebKit/pull/38129
zsun
Also happens to -
/webkit/WebKitWebContext/languages
/webkit/WebKitPolicyClient/new-window-policy
EWS
Test gardening commit 288155@main (146fa28a329d): <https://commits.webkit.org/288155@main>
Reviewed commits have been landed. Closing PR #38129 and removing active labels.