REOPENED 240987
New test: [macOS WK2] fast/images/heic-as-background-image.html is consistently failing
https://bugs.webkit.org/show_bug.cgi?id=240987
Summary New test: [macOS WK2] fast/images/heic-as-background-image.html is consistent...
Ryan Haddad
Reported 2022-05-26 16:15:39 PDT
images/heic-as-background-image.html has been consisntely failing on macOS WK2 bots since it was added with https://commits.webkit.org/250930@main Image diff link showing that the test is expecting a green square on white background, but getting a blank white image https://build.webkit.org/results/Apple-BigSur-Release-WK2-Tests/250994@main%20(9462)/fast/images/heic-as-background-image-diffs.html https://results.webkit.org/?suite=layout-tests&test=fast/images/heic-as-background-image.html
Attachments
Radar WebKit Bug Importer
Comment 1 2022-05-26 16:15:50 PDT
Ryan Haddad
Comment 2 2022-05-26 17:15:02 PDT
These tests were disabled in https://commits.webkit.org/251031@main Also worth noting, all three tests added/re-enabled with that commit appear to be flakily crashing in VideoToolbox code.. https://build.webkit.org/results/Apple-BigSur-Release-AppleSilicon-WK2-Tests/251011@main%20(8342)/fast/images/animated-heics-draw-crash-log.txt
Nikos Mouchtaris
Comment 3 2022-05-26 17:19:20 PDT
Filed rdar://94005388 on VideoToolbox to investigate this issue.
Nikos Mouchtaris
Comment 4 2022-07-15 12:32:27 PDT
EWS
Comment 5 2022-07-15 13:03:55 PDT
Committed 252517@main (20d56bbc5cbb): <https://commits.webkit.org/252517@main> Reviewed commits have been landed. Closing PR #2465 and removing active labels.
Nikos Mouchtaris
Comment 6 2022-07-15 13:19:27 PDT
This pr is just for data collection. This issue hasn't been resolved yet.
Ryosuke Niwa
Comment 7 2022-07-17 22:03:13 PDT
Re-enabling is good but we probably want failure test expectation so that EWS bots won't get slowed down.
Alexey Proskuryakov
Comment 8 2022-07-17 22:06:52 PDT
Indeed, crashes appear to be gone, but failing in WK2 remains.
Ryosuke Niwa
Comment 9 2022-07-17 22:49:12 PDT
EWS
Comment 10 2022-07-17 22:51:07 PDT
Committed 252559@main (92b4f5717db9): <https://commits.webkit.org/252559@main> Reviewed commits have been landed. Closing PR #2504 and removing active labels.
Ryosuke Niwa
Comment 11 2022-07-17 22:51:58 PDT
Once again, the above commit was a test expectation update.
Ryosuke Niwa
Comment 12 2022-07-17 23:55:29 PDT
(In reply to EWS from comment #10) > Committed 252559@main (92b4f5717db9): > <https://commits.webkit.org/252559@main> > > Reviewed commits have been landed. Closing PR #2504 and removing active > labels. Oops, this is wrong. I need ImageOnlyFailure instead :(
Ryosuke Niwa
Comment 13 2022-07-18 00:01:50 PDT
EWS
Comment 14 2022-07-18 00:03:44 PDT
Committed 252562@main (668b1631918d): <https://commits.webkit.org/252562@main> Reviewed commits have been landed. Closing PR #2505 and removing active labels.
Ryosuke Niwa
Comment 15 2022-07-18 00:05:54 PDT
Once again, the above commit just updated test expectation.
Note You need to log in before you can comment on or make changes to this bug.