Bug 208167 - ASSERTION FAILED: Completion handler should always be called on inspector/dom-debugger/attribute-modified-style.html
Summary: ASSERTION FAILED: Completion handler should always be called on inspector/dom...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Service Workers (show other bugs)
Version: WebKit Nightly Build
Hardware: Mac macOS 10.14
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-02-24 16:34 PST by Jason Lawrence
Modified: 2020-03-02 02:52 PST (History)
5 users (show)

See Also:


Attachments
crash log (86.16 KB, text/plain)
2020-02-24 16:34 PST, Jason Lawrence
no flags Details
Patch (1.56 KB, patch)
2020-03-02 01:56 PST, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Lawrence 2020-02-24 16:34:32 PST
Created attachment 391597 [details]
crash log

inspector/dom-debugger/attribute-modified-style.html

Description:
This test is flaky failing on Mac wk2 Debug. The first crash that I see is on r256912.

History:
https://results.webkit.org/?suite=layout-tests&test=inspector%2Fdom-debugger%2Fattribute-modified-style.html&platform=mac&limit=50000&flavor=wk2&style=debug

Crash log attached.
Comment 1 Jason Lawrence 2020-02-24 16:48:12 PST
I have marked this test as crashing while this issue is investigated.
https://trac.webkit.org/changeset/257282/webkit
Comment 2 Radar WebKit Bug Importer 2020-02-24 16:51:19 PST
<rdar://problem/59747016>
Comment 3 Alexey Proskuryakov 2020-02-24 18:43:43 PST
stderr:
ASSERTION FAILED: Completion handler should always be called
!m_function
/Volumes/Data/slave/mojave-debug/build/WebKitBuild/Debug/usr/local/include/wtf/CompletionHandler.h(53) : WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData> &&)>::~CompletionHandler()
1   0x129b6d9f9 WTFCrash
2   0x103321ee3 WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)>::~CompletionHandler()
3   0x103320ba5 WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)>::~CompletionHandler()
4   0x103914e59 WTF::KeyValuePair<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType>, WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)> >::~KeyValuePair()
5   0x103914da5 WTF::KeyValuePair<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType>, WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)> >::~KeyValuePair()
6   0x103914d11 WTF::HashTable<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType>, WTF::KeyValuePair<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType>, WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)> >, WTF::KeyValuePairKeyExtractor<WTF::KeyValuePair<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType>, WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)> > >, WTF::ObjectIdentifierHash<WebCore::ServiceWorkerJobIdentifierType>, WTF::HashMap<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType>, WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)>, WTF::ObjectIdentifierHash<WebCore::ServiceWorkerJobIdentifierType>, WTF::HashTraits<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType> >, WTF::HashTraits<WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)> > >::KeyValuePairTraits, WTF::HashTraits<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType> > >::deallocateTable(WTF::KeyValuePair<WTF::ObjectIdentifier<WebCore::ServiceWorkerJobIdentifierType>, WTF::CompletionHandler<void (std::experimental::fundamentals_v3::expected<bool, WebCore::ExceptionData>&&)> >*)
Comment 4 Alexey Proskuryakov 2020-02-24 18:47:38 PST
This shows up as a recent regression (a week ago) on this test, however it's clearly a culprit of some SW test. So the crash may have just moved.
Comment 5 Alexey Proskuryakov 2020-02-24 18:48:11 PST
http://trac.webkit.org/r256900 doesn't look entirely unrelated though...
Comment 6 Jason Lawrence 2020-02-25 07:18:13 PST
I was able to reproduce this issue with r257181 by running the command below.
run-webkit—tests inspector/dom-debugger/attribute-modified-style.html --iterations 4000 --exit-after-n-failures 5 --force --platform mac-mojave --debug -g

Output:
[1052/4000] inspector/dom-debugger/attribute-modified-style.html failed unexpectedly (text diff)
Interrupted, exiting ...
1831 tests ran as expected, 1 didn't (2168 didn't run):
Comment 7 Jason Lawrence 2020-02-25 09:05:10 PST
Correction. I was not able to reproduce the issue, I got a failure not a crash. Please disregard my comment on 2020-02-25 07:18:13 PST.
Comment 8 Alexey Proskuryakov 2020-02-29 09:28:41 PST
Youenn fixed this via bug 208208. Still need to unmark the test. Jason or Youenn, could you please take care of that?
Comment 9 youenn fablet 2020-03-02 01:56:57 PST
Created attachment 392122 [details]
Patch
Comment 10 WebKit Commit Bot 2020-03-02 02:52:09 PST
Comment on attachment 392122 [details]
Patch

Clearing flags on attachment: 392122

Committed r257706: <https://trac.webkit.org/changeset/257706>
Comment 11 WebKit Commit Bot 2020-03-02 02:52:10 PST
All reviewed patches have been landed.  Closing bug.