256893 2023-05-17 06:27:14 -0700 Make sure to run a service worker before firing its activate event 2023-06-25 15:10:20 -0700 1 1 1 Unclassified WebKit Service Workers WebKit Local Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=258021 https://bugs.webkit.org/show_bug.cgi?id=258502 InRadar P2 Normal --- 1 youennf youennf fujii twisniewski webkit-bug-importer oldest_to_newest 1955896 0 youennf 2023-05-17 06:27:14 -0700 Make sure to run a service worker before firing its activate event 1955897 1 youennf 2023-05-17 06:27:31 -0700 <rdar://109411104> 1955900 2 youennf 2023-05-17 06:37:13 -0700 Pull request: https://github.com/WebKit/WebKit/pull/13961 1956414 3 ews-feeder 2023-05-19 02:07:36 -0700 Committed 264242@main (ff4eced3ff23): <https://commits.webkit.org/264242@main> Reviewed commits have been landed. Closing PR #13961 and removing active labels. 1960418 4 twisniewski 2023-06-07 14:09:09 -0700 Has the new WPT in this patch been sent to be synced upstream? I'm not seeing it on wpt.fyi or wpt.live. 1960423 5 twisniewski 2023-06-07 14:16:51 -0700 (I also note that `withIframe` isn't defined in the test itself, so it would be good to fix that before sending a PR). 1961312 6 youennf 2023-06-13 06:22:12 -0700 (In reply to Thomas Wisniewski [:twisniewski] from comment #4) > Has the new WPT in this patch been sent to be synced upstream? I'm not > seeing it on wpt.fyi or wpt.live. The test is in LayoutTests/http/wpt so it is separate from wpt.fyi. I guess we could upstream it though to help other browsers. Would that be of interest? withIFrame is defined in a JS file in wpt repo. 1961318 7 twisniewski 2023-06-13 06:47:15 -0700 Ah I see, thanks for the explanation! Yes, it would be great to have as much coverage in WPTs as possible instead of browser-specific test infrastructure, so that we can all ensure interop. Even where it isn't currently possible (due to lacking low-level APIs and such), it would be good to let the WPT team know what's missing to make it possible eventually. This doesn't have to be a high priority of course, but every little bit helps (especially with interop202x [1] being a thing; the better our coverage now, the less likely service workers will become a future focus area in that project).