Bug 222917 – REGRESSION (r274033): [ macOS/iOS ] model-element/model-element-ready-load-aborted.html is text failing

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Robert Jenner]

model-element/model-element-ready-load-aborted.html

is a consistent text failure in macOS. 

HISTORY URL:
https://results.webkit.org/?suite=layout-tests&test=model-element%2Fmodel-element-ready-load-aborted.html

TEXT DIFF:
--- /Volumes/Data/worker/bigsur-release-tests-wk2/build/layout-test-results/model-element/model-element-ready-load-aborted-expected.txt
+++ /Volumes/Data/worker/bigsur-release-tests-wk2/build/layout-test-results/model-element/model-element-ready-load-aborted-actual.txt
@@ -1,3 +1,3 @@
 This test passes if you see the word "Passed" below:
 
-Passed
+Failed. Model should not have loaded, but did.

History indicates failures consistently started occurring at r274033.It appears there were changes made that appear related to this test.
https://trac.webkit.org/changeset/274033/webkit

[Radar WebKit Bug Importer]

<rdar://problem/75173628>

[Robert Jenner]

I haven't reproduced the issue yet, but it does appear to have a clear regression point. Running the test standalone does not produce the failure.

[Robert Jenner]

Created attachment 422600 [details]
222917-test list

Test list for reproducing "model-element/model-element-ready-load-aborted.html" text failure.

[Robert Jenner]

I was able to reproduce the failure at tip of tree using the attached test, and running it as:

run-webkit-test --root <path to revision> <path to test list> --debug --child-process=1

By eliminating tests from the test list, I was able to deduce that when the test: model-element/model-element-graphics-layers.html is ran before: model-element/model-element-ready-load-aborted.html it causes a consistent text failure. 

When the test: model-element/model-element-graphics-layers.html is removed, and the entire suite of 7000+ tests is ran again without it, model-element/model-element-ready-load-aborted.html passes.

[Tim Horton]

Sam and I think we see the mistake in the test

[Tim Horton]

Created attachment 422626 [details]
Patch

[EWS]

Committed r274126: <https://commits.webkit.org/r274126>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 422626 [details].