Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1b0d8f7b12c4b763fadd606974c5167284436989
      
https://github.com/WebKit/WebKit/commit/1b0d8f7b12c4b763fadd606974c5167284436989
  Author: Youenn Fablet <youe...@gmail.com>
  Date:   2023-01-26 (Thu, 26 Jan 2023)

  Changed paths:
    M Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServer.cpp
    M Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServer.h
    M 
Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServerConnection.cpp
    M 
Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServerToContextConnection.cpp

  Log Message:
  -----------
  REGRESSION(?): [iOS] ASSERTION FAILED: result in 
WebKit::NetworkProcess::allowsFirstPartyForCookies()
https://bugs.webkit.org/show_bug.cgi?id=248065
rdar://problem/102490719

Reviewed by Alex Christensen.

We may reuse a process for running shared workers with different first party 
cookie URLs.
We therefore need for each shared worker to make sure the NetworkProcess 
allowed first party URLs map is correctly populated.
We move the population at the place where we launch the shared worker instead 
of the time where the worker process connection is created.
This aligns with how this is done with service workers.

Covered by 
imported/w3c/web-platform-tests/service-workers/cache-storage/cross-partition.https.tentative.html
 no longer crashing.

* Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServer.cpp:
(WebKit::WebSharedWorkerServer::didFinishFetchingSharedWorkerScript):
(WebKit::WebSharedWorkerServer::createContextConnection):
* Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServer.h:
* 
Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServerToContextConnection.cpp:
(WebKit::WebSharedWorkerServerToContextConnection::launchSharedWorker):

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


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

Reply via email to