Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 41c888f493fe21f4a759e4dee01532181f629833
      
https://github.com/WebKit/WebKit/commit/41c888f493fe21f4a759e4dee01532181f629833
  Author: Youenn Fablet <youe...@gmail.com>
  Date:   2023-07-03 (Mon, 03 Jul 2023)

  Changed paths:
    M LayoutTests/TestExpectations
    M 
LayoutTests/http/wpt/service-workers/fetch-service-worker-preload-changing-request.https-expected.txt
    M 
LayoutTests/http/wpt/service-workers/fetch-service-worker-preload-changing-request.https.html
    M Source/WebCore/Modules/fetch/FetchRequest.cpp

  Log Message:
  -----------
  New broken test: 
http/wpt/service-workers/fetch-service-worker-preload-changing-request.https.html
 fails
https://bugs.webkit.org/show_bug.cgi?id=248735
rdar://problem/103220705

Reviewed by Alex Christensen.

When calling fetch(event.request), we are creating a new FetchRequest.
To be able to reuse the preload, we need to set the navigation preload 
identifier in the new FetchRequest.
We make sure to not set it when the request is changed, in particular its 
headers.

Covered by tests no longer failing.
CoveWe update 
LayoutTests/http/wpt/service-workers/fetch-service-worker-preload-changing-request.https.html
 since it must not reuse preload in that case.
Cove
Cove* LayoutTests/TestExpectations:
Cove* 
LayoutTests/http/wpt/service-workers/fetch-service-worker-preload-changing-request.https-expected.txt:
* 
LayoutTests/http/wpt/service-workers/fetch-service-worker-preload-changing-request.https.html:
* Source/WebCore/Modules/fetch/FetchRequest.cpp:
(WebCore::FetchRequest::initializeWith):

Canonical link: https://commits.webkit.org/265705@main


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to