Bug 248735 - New broken test: http/wpt/service-workers/fetch-service-worker-preload-changing-request.https.html fails
Summary: New broken test: http/wpt/service-workers/fetch-service-worker-preload-changi...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
: 257963 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-12-03 21:25 PST by Simon Fraser (smfr)
Modified: 2023-08-28 02:46 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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