RESOLVED FIXED227154
Add assertion in RunLoop::dispatch 
https://bugs.webkit.org/show_bug.cgi?id=227154
Summary Add assertion in RunLoop::dispatch
Alex Christensen
Reported 2021-06-17 16:52:08 PDT
Add assertion in RunLoop::dispatch
Attachments
Patch (1.54 KB, patch)
2021-06-17 16:53 PDT, Alex Christensen 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2021-06-17 16:53:37 PDT
Created attachment 431741 [details] Patch
Geoffrey Garen
Comment 2 2021-06-17 16:55:53 PDT
Comment on attachment 431741 [details] Patch r=me
Alex Christensen
Comment 3 2021-06-17 17:01:00 PDT
http://trac.webkit.org/r279025
Radar WebKit Bug Importer
Comment 4 2021-06-17 17:02:29 PDT
<rdar://problem/79477948>
Chris Dumez
Comment 5 2021-06-17 17:03:14 PDT
I bet Sihui just fixed it via: commit 9c3f87df21fba60bd607cb2[3a70b6fad5ac56926 Author: sihui_liu@apple.com <sihui_liu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Date: Thu Jun 10 23:55:52 2021 +0000 Regression(r278449): add null check for completionHandler in WebIDBServer::close https://bugs.webkit.org/show_bug.cgi?id=226900 rdar://79160392 Reviewed by Chris Dumez. * NetworkProcess/IndexedDB/WebIDBServer.cpp: (WebKit::WebIDBServer::close): And your assertion wouldn't have caught it I believe?
Note You need to log in before you can comment on or make changes to this bug.