Bug 222917 - REGRESSION (r274033): [ macOS/iOS ] model-element/model-element-ready-load-aborted.html is text failing
Summary: REGRESSION (r274033): [ macOS/iOS ] model-element/model-element-ready-load-ab...
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: Tim Horton
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-03-08 09:29 PST by Robert Jenner
Modified: 2021-03-08 17:50 PST (History)
5 users (show)

See Also:


Attachments
222917-test list (454.93 KB, text/plain)
2021-03-08 12:06 PST, Robert Jenner
no flags Details
Patch (1.67 KB, patch)
2021-03-08 15:45 PST, Tim Horton
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Jenner 2021-03-08 09:29:37 PST
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
Comment 1 Radar WebKit Bug Importer 2021-03-08 09:29:58 PST
<rdar://problem/75173628>
Comment 2 Robert Jenner 2021-03-08 09:32:02 PST
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.
Comment 3 Robert Jenner 2021-03-08 12:06:41 PST
Created attachment 422600 [details]
222917-test list

Test list for reproducing "model-element/model-element-ready-load-aborted.html" text failure.
Comment 4 Robert Jenner 2021-03-08 12:26:55 PST
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.
Comment 5 Tim Horton 2021-03-08 14:02:17 PST
Sam and I think we see the mistake in the test
Comment 6 Tim Horton 2021-03-08 15:45:01 PST
Created attachment 422626 [details]
Patch
Comment 7 EWS 2021-03-08 17:50:05 PST
Committed r274126: <https://commits.webkit.org/r274126>

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