Bug 237603 - [ iOS ] 2 imported/w3c/web-platform-tests/content-security-policy/connect-src/* tests are flaky text failures
Summary: [ iOS ] 2 imported/w3c/web-platform-tests/content-security-policy/connect-src...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-03-08 06:36 PST by Dawn Morningstar
Modified: 2022-03-10 14:34 PST (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dawn Morningstar 2022-03-08 06:36:29 PST
imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-allowed.sub.html
imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-blocked.sub.html 
are both flaky text failures on iOS queues. 

Regression may be around r289514. Further investigation is required.

HISTORY:
https://results.webkit.org/?suite=layout-tests&suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fcontent-security-policy%2Fconnect-src%2Fshared-worker-connect-src-allowed.sub.html&test=imported%2Fw3c%2Fweb-platform-tests%2Fcontent-security-policy%2Fconnect-src%2Fshared-worker-connect-src-blocked.sub.html&platform=ios

imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-allowed.sub.html
DIFF:
--- /Volumes/Data/worker/ipados-simulator-15-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-allowed.sub-expected.txt
+++ /Volumes/Data/worker/ipados-simulator-15-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-allowed.sub-actual.txt
@@ -1,3 +1,3 @@
 
-FAIL Expecting logs: ["xhr allowed","TEST COMPLETE"] assert_unreached: unexpected log: xhr blocked Reached unreachable code
+FAIL Expecting logs: ["xhr allowed","TEST COMPLETE"] assert_unreached: Logging timeout, expected logs xhr allowed,TEST COMPLETE not sent. Reached unreachable code
 
DIFF-URL:
https://build.webkit.org/results/Apple-iPadOS-15-Simulator-Release-WK2-Tests/r290762%20(1185)/imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-allowed.sub-diff.txt



imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-blocked.sub.html 
DIFF:
--- /Volumes/Data/worker/ipados-simulator-15-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-blocked.sub-expected.txt
+++ /Volumes/Data/worker/ipados-simulator-15-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-blocked.sub-actual.txt
@@ -1,5 +1,5 @@
 This test loads a shared worker, delivered with its own policy. The worker should be blocked from making an XHR as that policy specifies a connect-src 'none', though this resource's policy is connect-src *. No report should be sent since the worker's policy doesn't specify a report-uri.
 
 
-PASS Expecting logs: ["xhr blocked","TEST COMPLETE"]
+FAIL Expecting logs: ["xhr blocked","TEST COMPLETE"] assert_unreached: Logging timeout, expected logs xhr blocked,TEST COMPLETE not sent. Reached unreachable code
 
DIFF-URL:
https://build.webkit.org/results/Apple-iPadOS-15-Simulator-Release-WK2-Tests/r290762%20(1185)/imported/w3c/web-platform-tests/content-security-policy/connect-src/shared-worker-connect-src-blocked.sub-diff.txt
Comment 1 Radar WebKit Bug Importer 2022-03-08 06:36:58 PST
<rdar://problem/89965072>
Comment 2 Dawn Morningstar 2022-03-09 11:49:43 PST
marking expectations until further investigation can occur.
Comment 3 Dawn Morningstar 2022-03-09 12:01:38 PST
Expectations updated with r291059
Comment 4 Ryan Haddad 2022-03-09 14:36:54 PST
Based on commits that landed shortly before the first failure, https://commits.webkit.org/247042@main seems like it could be related as it was about the SharedWorker process on iOS.
Comment 5 Chris Dumez 2022-03-09 14:42:22 PST
(In reply to Ryan Haddad from comment #4)
> Based on commits that landed shortly before the first failure,
> https://commits.webkit.org/247042@main seems like it could be related as it
> was about the SharedWorker process on iOS.

Flaky timeouts could indeed indicate that we're failing to take a process assertion on iOS. I can investigate.
Comment 6 Chris Dumez 2022-03-10 14:34:46 PST
(In reply to Chris Dumez from comment #5)
> (In reply to Ryan Haddad from comment #4)
> > Based on commits that landed shortly before the first failure,
> > https://commits.webkit.org/247042@main seems like it could be related as it
> > was about the SharedWorker process on iOS.
> 
> Flaky timeouts could indeed indicate that we're failing to take a process
> assertion on iOS. I can investigate.

I haven't been able to reproduce so far, even by forcing the shared worker into its own separate process (which is more likely to trigger process suspension).