Bug 248735

Summary:	New broken test: http/wpt/service-workers/fetch-service-worker-preload-changing-request.https.html fails
Product:	WebKit	Reporter:	Simon Fraser (smfr) <simon.fraser>
Component:	Tools / Tests	Assignee:	youenn fablet <youennf>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	erik.witt, rik, simon.fraser, webkit-bug-importer, youennf
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=242910

Description Simon Fraser (smfr) 2022-12-03 21:25:47 PST

https://ews-build.s3-us-west-2.amazonaws.com/macOS-AppleSilicon-Big-Sur-Debug-WK2-Tests-EWS/d33837a3-45214/results.html

Diff is
@@ -1,5 +1,5 @@
 
 
 PASS Setup activating worker
-PASS Service worker load uses preload through calling fetch on the fetch event request
+FAIL Service worker load uses preload through calling fetch on the fetch event request assert_equals: expected "1" but got "2"

Comment 1 Simon Fraser (smfr) 2022-12-03 21:27:54 PST

http/wpt/service-workers/fetch-service-worker-preload.https.html also fails:

@@ -1,5 +1,5 @@
 
 
 PASS Setup activating worker
-PASS Service worker load uses preload through calling fetch on the fetch event request
+FAIL Service worker load uses preload through calling fetch on the fetch event request assert_equals: expected "use-preload" but got "nothing"

Comment 2 Alexey Proskuryakov 2022-12-04 18:03:18 PST

Failing most of the time since added in 257110@main.

Comment 3 Radar WebKit Bug Importer 2022-12-10 21:26:17 PST

<rdar://problem/103220705>

Comment 4 youenn fablet 2023-06-21 00:32:47 PDT

*** Bug 257963 has been marked as a duplicate of this bug. ***

Comment 5 youenn fablet 2023-06-21 02:29:40 PDT

Pull request: https://github.com/WebKit/WebKit/pull/15135

Comment 6 EWS 2023-07-03 00:26:10 PDT

Committed 265705@main (41c888f493fe): <https://commits.webkit.org/265705@main>

Reviewed commits have been landed. Closing PR #15135 and removing active labels.

Comment 7 erik.witt 2023-08-28 02:46:36 PDT

Hey!

Will this fix be part of the release 17?

It seems to be fixed in technology preview Release 177 (Safari 17.0, WebKit 18617.1.4.3) but I cannot find the problem number in the release notes https://developer.apple.com/documentation/safari-release-notes/safari-17-release-notes