RESOLVED FIXED247894
REGRESSION (255498@main): [ iOS ] TestWebKitAPI.ServiceWorkers.SuspendServiceWorkerProcessBasedOnClientProcessesWithoutSeparateServiceWorkerProcess is a consistent failure 
https://bugs.webkit.org/show_bug.cgi?id=247894
Summary REGRESSION (255498@main): [ iOS ] TestWebKitAPI.ServiceWorkers.SuspendService...
Karl Rackler
Reported 2022-11-14 07:44:31 PST
Description: TestWebKitAPI.ServiceWorkers.SuspendServiceWorkerProcessBasedOnClientProcessesWithoutSeparateServiceWorkerProcess Reproduction steps: I can reproduce this on 255498@main but not on 255497@main or earlier. Command: run-api-tests --iterations=5 --ios-simulator --debug TestWebKitAPI.ServiceWorkers.SuspendServiceWorkerProcessBasedOnClientProcessesWithoutSeparateServiceWorkerProcess History: https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.ServiceWorkers.SuspendServiceWorkerProcessBasedOnClientProcessesWithoutSeparateServiceWorkerProcess Result: Failed TestWebKitAPI.ServiceWorkers.SuspendServiceWorkerProcessBasedOnClientProcessesWithoutSeparateServiceWorkerProcess LEAK: 1 WebProcessPool /Volumes/Data/worker/Apple-iOS-16-Simulator-Debug-Build/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ServiceWorkerBasic.mm:1757 Value of: waitUntilEvaluatesToTrue([&] { return [webViewToUpdate _hasServiceWorkerForegroundActivityForTesting]; }) Actual: false Expected: true
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-11-14 07:45:09 PST
<rdar://problem/102320977>
Miguel Salinas
Comment 2 2022-11-14 15:04:00 PST
Pull request: https://github.com/WebKit/WebKit/pull/6490
EWS
Comment 3 2022-11-16 14:48:08 PST
Committed 256753@main (c38857287101): <https://commits.webkit.org/256753@main> Reviewed commits have been landed. Closing PR #6490 and removing active labels.
Miguel Salinas
Comment 4 2022-11-19 12:40:17 PST
Re-opening for pull request https://github.com/WebKit/WebKit/pull/6675
EWS
Comment 5 2022-11-29 14:30:20 PST
Committed 257141@main (e33ada3ec1ab): <https://commits.webkit.org/257141@main> Reviewed commits have been landed. Closing PR #6675 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.