Bug 238309

Summary: PushAPI.firePushEventWithNoPagesTimeout fails in debug builds
Product: WebKit Reporter: Ben Nham <nham>
Component: Service WorkersAssignee: Ben Nham <nham>
Status: RESOLVED FIXED    
Severity: Normal CC: nham, ryanhaddad, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Ben Nham 2022-03-23 21:47:25 PDT 
PushAPI.firePushEventWithNoPagesTimeout fails in debug builds with this assertion:

ASSERTION FAILED: m_connection                                          
/Volumes/Data/src/webkit/OpenSource/Source/WebKit/UIProcess/AuxiliaryProcessProxy.h(80) : IPC::Connection *WebKit::AuxiliaryProcessProxy::connection() const
1   0x117861389 WTFCrash
2   0x13447ef2b WTFCrashWithInfo(int, char const*, char const*, int)
3   0x135531ae7 WebKit::AuxiliaryProcessProxy::connection() const
4   0x136200842 WebKit::NetworkProcessProxy::terminate()
5   0x1362fa633 WebKit::WebsiteDataStore::terminateNetworkProcess()
6   0x1355d5cf5 -[WKWebsiteDataStore(WKPrivate) _terminateNetworkProcess]
7   0x108a67278 terminateNetworkProcessWhileRegistrationIsStored(WKWebViewConfiguration*)
8   0x108a69280 PushAPI_firePushEventWithNoPagesTimeout_Test::TestBody()
9   0x108fbba8b void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*)
10  0x108f82ffa void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*)
11  0x108f82f43 testing::Test::Run()
12  0x108f840c1 testing::TestInfo::Run()
13  0x108f85164 testing::TestSuite::Run()
14  0x108f941fb testing::internal::UnitTestImpl::RunAllTests()
15  0x108fc08eb bool testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*)
16  0x108f93bba bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*)
17  0x108f93aa5 testing::UnitTest::Run()
18  0x1086f5071 RUN_ALL_TESTS()
19  0x1086f5006 TestWebKitAPI::TestsController::run(int, char**)
20  0x108f52de2 main
21  0x10f22ccd0
worker/0 TestWebKitAPI.PushAPI.firePushEventWithNoPagesTimeout Crashed
Comment 1 Ben Nham 2022-03-23 22:06:51 PDT 
Created attachment 455605 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2022-03-23 22:07:28 PDT 
<rdar://problem/90746582>
Comment 3 youenn fablet 2022-03-24 02:46:08 PDT 
Regression from added assertion in https://bugs.webkit.org/show_bug.cgi?id=236844.
Comment 4 EWS 2022-03-24 10:21:41 PDT 
Committed r291802 (248830@main): <https://commits.webkit.org/248830@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 455605 [details].
Comment 5 Ryan Haddad 2022-03-24 11:05:28 PDT 
*** Bug 238241 has been marked as a duplicate of this bug. ***