Bug 259528 - REGRESSION(266045@main?): [ macOS WK2 ] fast/attachment/mac/wide-attachment-image-controls-basic.html is a flaky failure
Summary: REGRESSION(266045@main?): [ macOS WK2 ] fast/attachment/mac/wide-attachment-i...
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: Gerald Squelart
URL:
Keywords: InRadar
Depends on: 259094
Blocks:
  Show dependency treegraph
 
Reported: 2023-07-26 11:54 PDT by Ben Schwartz
Modified: 2024-01-11 16:18 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 Ben Schwartz 2023-07-26 11:54:05 PDT
fast/attachment/mac/wide-attachment-image-controls-basic.html

This test is a flaky failure on macOS Monterey and above.

HISTORY:

https://results.webkit.org/?suite=layout-tests&test=fast%2Fattachment%2Fmac%2Fwide-attachment-image-controls-basic.html

TEXT DIFF:

     RenderBody {BODY} at (8,8) size 784x82
       RenderAttachment {ATTACHMENT} at (1,1) size 266x80
         RenderFlexibleBox {DIV} at (0,0) size 266x80 [bgcolor=#0000000D]
-          RenderGrid {DIV} at (14,14) size 0x52
-            RenderImage {IMG} at (0,26) size 0x0
-          RenderFlexibleBox {DIV} at (32,0) size 210x80
-            RenderGrid {DIV} at (0,40) size 210x0
-      RenderText {#text} at (268,26) size 4x18
-        text run at (268,26) width 4: " "
-      RenderImage {IMG} at (272,20) size 20x20
+          RenderGrid {DIV} at (14,14) size 52x52
+            RenderImage {IMG} at (0,0) size 52x52
+          RenderFlexibleBox {DIV} at (84,0) size 158x80
+            RenderGrid {DIV} at (0,40) size 158x0
+      RenderText {#text} at (268,52) size 4x18
+        text run at (268,52) width 4: " "
+      RenderImage {IMG} at (272,46) size 20x20
       RenderText {#text} at (0,0) size 0x0
 layer at (9,9) size 266x80
   RenderBlock (relative positioned) {DIV} at (0,0) size 266x80 [color=#00000000]

DIFF URL:

https://build.webkit.org/results/Apple-Ventura-Release-AppleSilicon-WK2-Tests/266314@main%20(4170)/fast/attachment/mac/wide-attachment-image-controls-basic-pretty-diff.html

REPRODUCIBILITY:

I was able to reproduce this bug on macOS Ventura Release ToT running the test as follows:

run-webkit-tests --iterations 500 --verbose fast/attachment/mac/wide-attachment-image-controls-basic.html

REGRESSION:

I was able to bisect a regression point via local testing and the flakiness dashboard. This test reproduced at 266045@main, but it did not reproduce at 266044@main. Changes at 266045@main appear to be related to this error, but more investigation is needed to determine for sure.
Comment 1 Radar WebKit Bug Importer 2023-07-26 11:54:20 PDT
<rdar://problem/112918166>
Comment 2 Ben Schwartz 2023-07-26 12:10:25 PDT
I have marked this test as a flaky failure while this issue is investigated. (Link: https://github.com/WebKit/WebKit/pull/16107)
Comment 3 EWS 2023-07-26 12:13:31 PDT
Test gardening commit 266321@main (6b6e228f29e0): <https://commits.webkit.org/266321@main>

Reviewed commits have been landed. Closing PR #16107 and removing active labels.
Comment 4 Gerald Squelart 2023-07-26 20:30:40 PDT
At first glance, looking at the history of this failure, it seems to start at https://commits.webkit.org/266038@main https://bugs.webkit.org/show_bug.cgi?id=259094
That patch changed the IMG size to 0x52, and the flaky "failure" goes back to a more expected 52x52.
To be continued...
Comment 5 Ben Schwartz 2023-07-31 15:02:46 PDT
Upon further inspection, it appears that this issue occurs only on WK2. Amending previous gardening commit to reflect the correct expectation.

HISTORY:

https://results.webkit.org/?suite=layout-tests&test=fast%2Fattachment%2Fmac%2Fwide-attachment-image-controls-basic.html&flavor=wk2

I have marked this test as a flaky failure on macOS WK2 while this issue is investigated. (Link: https://github.com/WebKit/WebKit/pull/16107)
Comment 6 EWS 2023-08-01 09:07:51 PDT
Test gardening commit 266477@main (20e7648a111f): <https://commits.webkit.org/266477@main>

Reviewed commits have been landed. Closing PR #16248 and removing active labels.
Comment 7 Gerald Squelart 2023-12-27 20:02:58 PST
Pull request: https://github.com/WebKit/WebKit/pull/22255
Comment 8 EWS 2023-12-31 22:28:51 PST
Committed 272548@main (44fccba78690): <https://commits.webkit.org/272548@main>

Reviewed commits have been landed. Closing PR #22255 and removing active labels.
Comment 9 EWS 2024-01-11 14:08:05 PST
Committed 272938@main (6eee6a333d53): <https://commits.webkit.org/272938@main>

Reviewed commits have been landed. Closing PR #22677 and removing active labels.
Comment 10 EWS 2024-01-11 16:18:08 PST
Test gardening commit 272944@main (069950e20dd5): <https://commits.webkit.org/272944@main>

Reviewed commits have been landed. Closing PR #22685 and removing active labels.