Summary: | [ Mac wk2 Debug ] http/tests/inspector/network/intercept tests are crashing with alert - WTFCrash + 14 (Assertions.cpp:295) - WTF::CompletionHandler<void (WebCore::ResourceRequest const&)>::~CompletionHandler | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jason Lawrence <Lawrence.j> | ||||||||||
Component: | Web Inspector | Assignee: | Pavel Feldman <pfeldman> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | ews-watchlist, hi, inspector-bugzilla-changes, joepeck, pfeldman, webkit-bot-watchers-bugzilla, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Mac | ||||||||||||
OS: | macOS 10.14 | ||||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=207446 | ||||||||||||
Attachments: |
|
Description
Jason Lawrence
2020-06-16 09:36:02 PDT
ASSERTION FAILED: Completion handler should always be called !m_function /Volumes/Data/slave/catalina-debug/build/WebKitBuild/Debug/usr/local/include/wtf/CompletionHandler.h(53) : WTF::CompletionHandler<void (const WebCore::ResourceRequest &)>::~CompletionHandler() Created attachment 402027 [details]
Patch
Created attachment 402033 [details]
Patch
Comment on attachment 402033 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402033&action=review r=me As an alternative (not saying you should do this, just thinking out loud), we could create a `CompletionHandler::markAsCompleted` or `CompletionHandler::clear` that inspector code could call to satisfy the assertion, but that is more work than this solution so what you have is fine too IMO :) > Source/WebCore/ChangeLog:3 > + Web Inspector: replace completion handler with a function in interception. This could use a comment/description. Something like: ``` Don't use a `CompletionHandler` as it asserts that it's been called when it's destroyed. Both `Network.interceptRequestWithResponse` and `Network.interceptRequestWithError` essentially "skip" the network pipeline, so the `CompletionHandler` is not invoked for those commands. ``` > Source/WebKit/ChangeLog:8 > + * WebProcess/Network/WebLoaderStrategy.cpp: This doesn't appear to have changed. Created attachment 402041 [details]
Patch
All Done. Comment on attachment 402041 [details]
Patch
r=me
Committed r263119: <https://trac.webkit.org/changeset/263119> All reviewed patches have been landed. Closing bug and clearing flags on attachment 402041 [details]. |