RESOLVED FIXED 203995
Regression r252209: Layout Test fast/workers/worker-close-more.html is failing 
https://bugs.webkit.org/show_bug.cgi?id=203995
Summary Regression r252209: Layout Test fast/workers/worker-close-more.html is failing
Aakash Jain
Reported 2019-11-08 07:05:13 PST
The following layout test is failing on all the platforms: fast/workers/worker-close-more.html Flakiness Dashboard: https://results.webkit.org/?suite=layout-tests&test=fast%2Fworkers%2Fworker-close-more.html https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fworkers%2Fworker-close-more.html Probable cause: It passed in r252208 and failed in r252210. Most likely a regression from r252209 https://trac.webkit.org/changeset/252209/webkit https://bugs.webkit.org/show_bug.cgi?id=203960 'Port MessagePort to the HTML5 event loop'
Attachments
Patch (1.51 KB, patch)
2019-11-08 09:15 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2019-11-08 07:05:39 PST
Sample failure: https://ews-build.webkit.org/results/macOS-High-Sierra-Debug-WK1-Tests-EWS/r383034-6067-clean-tree/results.html Diff: PASS: workerThreadCount = 0 PASS: MessagePort multiple message test, received message: echo_ping1 +FAIL: The MessagePort message was dispatched after a JS fragment that called 'close()' exited. +PASS: MessagePort messages were not delivered asfter close(). +DONE PASS: MessagePort messages were not delivered asfter close(). DONE
Radar WebKit Bug Importer
Comment 2 2019-11-08 07:08:04 PST
<rdar://problem/57020558>
Chris Dumez
Comment 3 2019-11-08 08:05:21 PST
Looking.
Chris Dumez
Comment 4 2019-11-08 08:15:44 PST
I think WorkerMessagingProxy::workerGlobalScopeClosed() needs to be ported to the Event Loop too, or the ordering between posting a message and closing the worker global scope is no longer correct. I will confirm.
Chris Dumez
Comment 5 2019-11-08 09:15:12 PST
Created attachment 383130 [details] Patch
WebKit Commit Bot
Comment 6 2019-11-08 10:24:30 PST
Comment on attachment 383130 [details] Patch Clearing flags on attachment: 383130 Committed r252242: <https://trac.webkit.org/changeset/252242>
WebKit Commit Bot
Comment 7 2019-11-08 10:24:32 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.