Bug 266518 – REGRESSION (271259@main?): [ iOS Sonoma Debug arm64 ] ASSERTION FAILED: WTF::CompletionHandler<void (std::optional<WTF::String>)>::~CompletionHandler() in TestWebKitAPI.WKWebExtensionAPITabs.SendMessageWithPromiseReply result of timeout

• Component: WebKit Extensions
• Status: RESOLVED FIXED

[Marta Darbinyan]

Description:
TestWebKitAPI.WKWebExtensionAPITabs.SendMessageWithPromiseReply is timing out on iOS17 and Sonoma Debug arm64. First commit with timeout is 271259@main. The change can be possible culprit since it updates web extension API.


History:
https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.WKWebExtensionAPITabs.SendMessageWithPromiseReply&version_name=iOS%2017&version_name=Sonoma&style=debug&architecture=arm64

Result:

ASSERTION FAILED: Completion handler should always be called
        !m_function
        /Volumes/Data/worker/Apple-Sonoma-Debug-Build/build/WebKitBuild/Debug/usr/local/include/wtf/CompletionHandler.h(65) : WTF::CompletionHandler<void (std::optional<WTF::String>)>::~CompletionHandler()
        1   0x10851ed74 WTFCrash
        2   0x13c1253e8 WTF::CompletionHandler<void (std::__1::optional<WTF::String>)>::~CompletionHandler()
        3   0x13bf10c8c WTF::CompletionHandler<void (std::__1::optional<WTF::String>)>::~CompletionHandler()
        4   0x13bfa3958 WebKit::WebExtensionContextProxy::internalDispatchRuntimeMessageEvent(WebCore::DOMWrapperWorld&, WTF::String const&, std::__1::optional<WTF::ObjectIdentifierGeneric<WebKit::WebExtensionFrameIdentifierType, WTF::ObjectIdentifierMainThreadAccessTraits>>, WebKit::WebExtensionMessageSenderParameters const&, WTF::CompletionHandler<void (std::__1::optional<WTF::String>)>&&)::$_7::~$_7()
        5   0x13bfa1d7c WebKit::WebExtensionContextProxy::internalDispatchRuntimeMessageEvent(WebCore::DOMWrapperWorld&, WTF::String const&, std::__1::optional<WTF::ObjectIdentifierGeneric<WebKit::WebExtensionFrameIdentifierType, WTF::ObjectIdentifierMainThreadAccessTraits>>, WebKit::WebExtensionMessageSenderParameters const&, WTF::CompletionHandler<void (std::__1::optional<WTF::String>)>&&)::$_7::~$_7()
        6   0x13bfa1d54 __Block_byref_object_dispose_
        7   0x18e5e1aa8 _Block_object_dispose
        8   0x1393fd024 __destroy_helper_block_ea8_32r
        9   0x18e5e217c _call_dispose_helpers_excp
        10  0x18e5e1f48 _Block_release
        11  0x139416454 __destroy_helper_block_ea8_32s
        12  0x18e5e217c _call_dispose_helpers_excp
        13  0x18e5e1f48 _Block_release
        14  0x108564aa4 WTF::RetainPtr<void const*>::~RetainPtr()
        15  0x108564850 WTF::RetainPtr<void const*>::~RetainPtr()
        16  0x109b66ed0 WTF::VectorDestructor<true, WTF::RetainPtr<void const*>>::destruct(WTF::RetainPtr<void const*>*, WTF::RetainPtr<void const*>*)
        17  0x109b66e44 WTF::VectorTypeOperations<WTF::RetainPtr<void const*>>::destruct(WTF::RetainPtr<void const*>*, WTF::RetainPtr<void const*>*)
        18  0x109b6b84c WTF::Vector<WTF::RetainPtr<void const*>, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>::shrink(unsigned long)
        19  0x109b6b6c0 WTF::Vector<WTF::RetainPtr<void const*>, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>::shrinkCapacity(unsigned long)
        20  0x109b290b0 WTF::Vector<WTF::RetainPtr<void const*>, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc>::clear()
        21  0x109b28f54 JSC::Heap::releaseDelayedReleasedObjects()
        22  0x10a26c1dc JSC::JSLock::willReleaseLock()
        23  0x10a26c0d0 JSC::JSLock::unlock(long)
        24  0x10a26b8a4 JSC::JSLock::unlock()
        25  0x108dc9344 WTF::Locker<JSC::JSLock>::unlock()
        26  0x10a2dfacc WTF::Locker<JSC::JSLock>::~Locker()
        27  0x108da8e70 WTF::Locker<JSC::JSLock>::~Locker()
        28  0x10a2c61d8 JSC::JSRunLoopTimer::timerDidFire()
        29  0x10a2c58e8 JSC::JSRunLoopTimer::Manager::timerDidFire()
        30  0x10a2c5594 JSC::JSRunLoopTimer::Manager::timerDidFireCallback()
        31  0x10a2cfd54 decltype(*std::declval<JSC::JSRunLoopTimer::Manager*&>().*std::declval<void (JSC::JSRunLoopTimer::Manager::*&)()>()()) std::__1::__invoke[abi:v160006]<void (JSC::JSRunLoopTimer::Manager::*&)(), JSC::JSRunLoopTimer::Manager*&, void>(void (JSC::JSRunLoopTimer::Manager::*&)(), JSC::JSRunLoopTimer::Manager*&)

[Radar WebKit Bug Importer]

<rdar://problem/119740501>

[Timothy Hatcher]

Pull request: https://github.com/WebKit/WebKit/pull/21946

[EWS]

Committed 272236@main (fdd1ff775c0c): <https://commits.webkit.org/272236@main>

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