Bug 256613 - REGRESSION (263724@main): [ iOS macOS wk2 ] imported/w3c/web-platform-tests/screen-orientation/fullscreen-interactions.html is a flaky failure
Summary: REGRESSION (263724@main): [ iOS macOS wk2 ] imported/w3c/web-platform-tests/s...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-05-10 14:23 PDT by Karl Rackler
Modified: 2023-05-11 16:10 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 Karl Rackler 2023-05-10 14:23:56 PDT
Description:
imported/w3c/web-platform-tests/screen-orientation/fullscreen-interactions.html

This is a consistent failure on iOS, a flaky failure on macOS debug wk2, and a consistent failure on macOS release wk2.

This issue can be bisected to 263724@main using command:
run-webkit-tests --release --iterations=1 --ios-simulator imported/w3c/web-platform-tests/screen-orientation/fullscreen-interactions.html

History:
https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fscreen-orientation%2Ffullscreen-interactions.html

Diff:
--- /Volumes/Data/worker/Apple-iOS-16-Simulator-Release-GPUProcess-WK2-Tests/build/layout-test-results/imported/w3c/web-platform-tests/screen-orientation/fullscreen-interactions-expected.txt
+++ /Volumes/Data/worker/Apple-iOS-16-Simulator-Release-GPUProcess-WK2-Tests/build/layout-test-results/imported/w3c/web-platform-tests/screen-orientation/fullscreen-interactions-actual.txt
@@ -1,5 +1,5 @@
 
 
-FAIL Fully unlocking the screen orientation causes a pending lock to be aborted assert_unreached: Should have rejected: undefined Reached unreachable code
-FAIL Fully unlocking the screen orientation causes a pending lock in a nested browsing context to be aborted promise_rejects_dom: function "function () { throw e }" threw object "SecurityError: Locking the screen orientation is only allowed when in fullscreen" that is not a DOMException AbortError: property "code" is equal to 18, expected 20
+FAIL Fully unlocking the screen orientation causes a pending lock to be aborted promise_test: Unhandled rejection with value: object "TypeError: Type error"
+FAIL Fully unlocking the screen orientation causes a pending lock in a nested browsing context to be aborted promise_test: Unhandled rejection with value: object "TypeError: Type error"
Comment 1 Radar WebKit Bug Importer 2023-05-10 14:24:20 PDT
<rdar://problem/109172161>
Comment 2 Karl Rackler 2023-05-10 14:27:04 PDT
This appears to be a re-baseline.
Comment 3 Karl Rackler 2023-05-10 14:46:57 PDT
Unsure if re-baseline will resolve this issue.  I have marked this test as a failure while this issue is investigated.
Comment 4 EWS 2023-05-10 14:52:36 PDT
Test gardening commit 263931@main (ea163072541f): <https://commits.webkit.org/263931@main>

Reviewed commits have been landed. Closing PR #13722 and removing active labels.
Comment 5 Ryosuke Niwa 2023-05-10 21:03:13 PDT
There is nothing to be fixed here. We just need to ignore this console error message since it only appears intermittently.
Comment 6 Ryosuke Niwa 2023-05-10 21:06:27 PDT
Pull request: https://github.com/WebKit/WebKit/pull/13736
Comment 7 EWS 2023-05-10 21:10:30 PDT
Committed 263945@main (79f43cff248f): <https://commits.webkit.org/263945@main>

Reviewed commits have been landed. Closing PR #13736 and removing active labels.
Comment 8 Alexey Proskuryakov 2023-05-11 16:05:11 PDT
Note that this test was and remains a flaky failure on iOS simulator (where it actually matters). 

Not the same as this bug, and I didn't check if we had a separate bug about that.
Comment 9 Ryosuke Niwa 2023-05-11 16:08:27 PDT
(In reply to Alexey Proskuryakov from comment #8)
> Note that this test was and remains a flaky failure on iOS simulator (where
> it actually matters). 
> 
> Not the same as this bug, and I didn't check if we had a separate bug about
> that.

Yeah, I notice that now. I guess we should track it in a separate bug.
Comment 10 Ryosuke Niwa 2023-05-11 16:10:38 PDT
(In reply to Ryosuke Niwa from comment #9)
> (In reply to Alexey Proskuryakov from comment #8)
> > Note that this test was and remains a flaky failure on iOS simulator (where
> > it actually matters). 
> > 
> > Not the same as this bug, and I didn't check if we had a separate bug about
> > that.
> 
> Yeah, I notice that now. I guess we should track it in a separate bug.

Tracking this in the bug 256671.