248173 2022-11-21 05:53:03 -0800 Workers: Simplify early dedicated worker termination 2022-11-22 13:23:17 -0800 1 1 1 Unclassified WebKit New Bugs Safari Technology Preview Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 ahmad.saleem792 webkit-unassigned webkit-bug-importer oldest_to_newest 1913851 0 ahmad.saleem792 2022-11-21 05:53:03 -0800 Hi Team, While going through Blink commits, I found another simplification in workers termination, which can be brought to Webkit: Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=180532 Although there is no test case to confirm whether Webkit suffer from some issue due to it or not but since it is just simplification, it seems to be safe. I can do draft PR and run via EWS to see if there is any issue across multiple platforms. https://github.com/WebKit/WebKit/blob/a845b7e4ec0490d1d8f5d7e7f5ddf44fef0b6f17/Source/WebCore/workers/WorkerMessagingProxy.cpp#L330 ^ Remove this bit from here and move it here: https://github.com/WebKit/WebKit/blob/a845b7e4ec0490d1d8f5d7e7f5ddf44fef0b6f17/Source/WebCore/workers/WorkerMessagingProxy.cpp#LL122 Just wanted to raise bug. Thanks! 1914159 1 ews-feeder 2022-11-22 13:22:27 -0800 Committed 256949@main (0987e578a6a8): <https://commits.webkit.org/256949@main> Reviewed commits have been landed. Closing PR #6700 and removing active labels. 1914160 2 webkit-bug-importer 2022-11-22 13:23:17 -0800 <rdar://problem/102612212>