Bug 283144 – REGRESSION (Sequoia 24A335): [ Sequoia wk2 ] 2x storageAccess/grant-storage-access-under-opener*.html and 2x storageAccess/request-and-grant-access-then-navigate*.html are a consistent failure

• Status: NEW
• Component: New Bugs

[Karl Rackler]

Description:
http/tests/storageAccess/grant-storage-access-under-opener-at-popup-user-gesture-ephemeral.https.html
http/tests/storageAccess/grant-storage-access-under-opener-at-popup-user-gesture.https.html
http/tests/storageAccess/request-and-grant-access-then-navigate-cross-site-should-not-have-access.https.html
http/tests/storageAccess/request-and-grant-access-then-navigate-same-site-should-have-access.https.html

These tests have been a constant failure on Sequoia ToT since the bring-up on Sequoia 24A335.

The EWS queue from history shows that after updating from Sequoia 24A335 -> 24A348 the tests began consistently passing on a clean-tree with results from the run-layout-tests-without-change step.

This issue can be reproduced using the command:
run-webkit-tests --release --no-retry --force --iterations=1 http/tests/storageAccess/grant-storage-access-under-opener-at-popup-user-gesture-ephemeral.https.html http/tests/storageAccess/grant-storage-access-under-opener-at-popup-user-gesture.https.html http/tests/storageAccess/request-and-grant-access-then-navigate-cross-site-should-not-have-access.https.html http/tests/storageAccess/request-and-grant-access-then-navigate-same-site-should-have-access.https.html 

History:
https://results.webkit.org/?suite=layout-tests&suite=layout-tests&suite=layout-tests&suite=layout-tests&test=http%2Ftests%2FstorageAccess%2Fgrant-storage-access-under-opener-at-popup-user-gesture-ephemeral.https.html&test=http%2Ftests%2FstorageAccess%2Fgrant-storage-access-under-opener-at-popup-user-gesture.https.html&test=http%2Ftests%2FstorageAccess%2Frequest-and-grant-access-then-navigate-cross-site-should-not-have-access.https.html&test=http%2Ftests%2FstorageAccess%2Frequest-and-grant-access-then-navigate-same-site-should-have-access.https.html&version_name=Sequoia&flavor=wk1&flavor=wk2

Diff:
http/tests/storageAccess/grant-storage-access-under-opener-at-popup-user-gesture-ephemeral.https.html
@@ -13,5 +13,5 @@
 Frame: '<!--frame1-->'
 --------
 Should receive first-party cookie.
-Received cookie named 'firstPartyCookie'.
-Client-side document.cookie: firstPartyCookie=value
+Did not receive cookie named 'firstPartyCookie'.
+Client-side document.cookie:

http/tests/storageAccess/grant-storage-access-under-opener-at-popup-user-gesture.https.html
@@ -13,5 +13,5 @@
 Frame: '<!--frame1-->'
 --------
 Should receive first-party cookie.
-Received cookie named 'firstPartyCookie'.
-Client-side document.cookie: firstPartyCookie=value
+Did not receive cookie named 'firstPartyCookie'.
+Client-side document.cookie:

http/tests/storageAccess/request-and-grant-access-then-navigate-cross-site-should-not-have-access.https.html
@@ -5,7 +5,7 @@
 
 PASS document.cookie ==
 PASS Storage access was granted.
-PASS document.cookie == cookieSetClientSideAfterGrantedStorageAccess=value; firstPartyCookie=value
+PASS document.cookie == firstPartyCookie=value
 PASS No storage access. document.cookie == , cookies seen server-side == "No cookies"
 PASS successfullyParsed is true

http/tests/storageAccess/request-and-grant-access-then-navigate-same-site-should-have-access.https.html
@@ -5,8 +5,8 @@
 
 PASS document.cookie ==
 PASS Storage access was granted.
-PASS document.cookie == cookieSetClientSideAfterGrantedStorageAccess=value; firstPartyCookie=value
-PASS Has storage access. document.cookie == cookieSetClientSideAfterGrantedStorageAccess=value; firstPartyCookie=value, cookies seen server-side == {"cookieSetClientSideAfterGrantedStorageAccess":"value","firstPartyCookie":"value"}
+PASS document.cookie == firstPartyCookie=value
+PASS Has storage access. document.cookie == firstPartyCookie=value, cookies seen server-side == {"firstPartyCookie":"value"}
 PASS successfullyParsed is true
 
 TEST COMPLETE

[Radar WebKit Bug Importer]

<rdar://problem/139916862>

[Karl Rackler]

The long-term resolution involves updating post-commit testers to Sequoia 24B83 to effectively address this issue. In the interim, I will implement test expectations until the completion of this task.

[EWS]

Test gardening commit 286619@main (bbe4a02c53a6): <https://commits.webkit.org/286619@main>

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