Bug 312949 – [iOS] fast/mediasession/metadata/artworkdownload.html is a flaky text failure

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

[Diego De La Toba]

fast/mediasession/metadata/artworkdownload.html is a flaky text failure on iOS.


HISTORY:
https://results.webkit.org/?suite=layout-tests&test=fast%2Fmediasession%2Fmetadata%2Fartworkdownload.html


DIFF:
--- /Volumes/Data/worker/Apple-iOS-26-Simulator-Release-WK2-Tests/build/layout-test-results/fast/mediasession/metadata/artworkdownload-expected.txt
+++ /Volumes/Data/worker/Apple-iOS-26-Simulator-Release-WK2-Tests/build/layout-test-results/fast/mediasession/metadata/artworkdownload-actual.txt
@@ -1,3 +1,4 @@
+CONSOLE MESSAGE: A VideoFrame was destroyed without having been closed explicitly
 
 PASS ensure loading artwork image method operates properly
 PASS ensure loading invalid artwork causes a failure


DIFF URL:
https://build.webkit.org/results/Apple-iOS-26-Simulator-Release-WK2-Tests/311710%40main%20(2692)/fast/mediasession/metadata/artworkdownload-pretty-diff.html


REPRODUCTION:
I was able to reproduce this on iOS 26.4 release ToT with the following:

run-webkit-tests --no-build --no-retry --no-show-results --exit-after-n-failures=1 --expect-pass --iterations=143 --force --ios-simulator --release fast/mediasession/metadata/artworkdownload.html

I am going to mark expectations as pass fail while this pends investigation.

[Radar WebKit Bug Importer]

<rdar://problem/175302325>

[Diego De La Toba]

Test gardening pull request: https://github.com/WebKit/WebKit/pull/63289

[EWS]

Test gardening commit 311741@main (f6c6dec54ba9): <https://commits.webkit.org/311741@main>

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

[Diego De La Toba]

The test gets a `VideoFrame` from `loadArtworkImage()` but never calls `.close()` on it. This causes the console warning but all three tests still pass. Looks like we just need to close the VideoFrame after checking its properties. This can be in the test itself but should arguably be cleaned up in internals resetToConsistentState as well.

[Diego De La Toba]

Pull request: https://github.com/WebKit/WebKit/pull/63348

[EWS]

Committed 311886@main (f3607e114d57): <https://commits.webkit.org/311886@main>

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