Bug 237116 - REGRESSION(r290347-r290348) [ iOS ] http/tests/events/device-orientation-motion-non-secure-context.html is a constant text failure
Summary: REGRESSION(r290347-r290348) [ iOS ] http/tests/events/device-orientation-moti...
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: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-02-23 15:05 PST by Dawn Morningstar
Modified: 2022-02-23 16:45 PST (History)
4 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-02-23 15:05:15 PST
http/tests/events/device-orientation-motion-non-secure-context.html
Is a constant failure on iOS.

HISTORY:
https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fevents%2Fdevice-orientation-motion-non-secure-context.html

DIFF:
--- /Volumes/Data/worker/ios-simulator-15-debug-tests-wk2/build/layout-test-results/http/tests/events/device-orientation-motion-non-secure-context-expected.txt
+++ /Volumes/Data/worker/ios-simulator-15-debug-tests-wk2/build/layout-test-results/http/tests/events/device-orientation-motion-non-secure-context-actual.txt
@@ -6,11 +6,12 @@
 
 
 * Registering device orientation listener
-PASS lastConsoleMessage is "No device orientation events will be fired, reason: Browsing context is not secure."
+FAIL lastConsoleMessage should be No device orientation events will be fired, reason: Browsing context is not secure. (of type string). Was null (of type object).
 
 * Registering device motion listener
-PASS lastConsoleMessage is "No device motion events will be fired, reason: Browsing context is not secure."
+FAIL lastConsoleMessage should be No device motion events will be fired, reason: Browsing context is not secure. (of type string). Was null (of type object).
 PASS successfullyParsed is true
+Some tests failed.
 
 TEST COMPLETE
 
DIFF-URL:
https://build.webkit.org/results/Apple-iOS-15-Simulator-Debug-WK2-Tests/r290384%20(1710)/http/tests/events/device-orientation-motion-non-secure-context-diff.txt
Comment 1 Radar WebKit Bug Importer 2022-02-23 15:06:03 PST
<rdar://problem/89380424>
Comment 2 Dawn Morningstar 2022-02-23 15:16:01 PST
Failure was reproduced at iOS production TOT using:
run-webkit-tests http/tests/events/device-orientation-motion-non-secure-context.html

History shows regression range between r290344-r290348.
https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fevents%2Fdevice-orientation-motion-non-secure-context.html

Test passed successfully locally on iOS at r290343.
Comment 3 Dawn Morningstar 2022-02-23 15:25:14 PST
Test was passed locally at iOS r290346, and failed locally at iOS r290348. Narrowing regression point.
Comment 4 Dawn Morningstar 2022-02-23 15:37:22 PST
I believe regression point is r290348 due to modification of line: 

2113	        printWarningMessage(makeString("No device orientation events will be fired, reason: ", innerMessage, "."));
Comment 5 Darin Adler 2022-02-23 16:06:59 PST
The problem seems to be that "lastConsoleMessage" does not see this warning message now.
Comment 6 Dawn Morningstar 2022-02-23 16:45:50 PST
Marking as resolved fixed with this revert: https://trac.webkit.org/changeset/290399/webkit