Bug 312949
| Summary: | [iOS] fast/mediasession/metadata/artworkdownload.html is a flaky text failure | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Diego De La Toba <d_delatoba> |
| Component: | New Bugs | Assignee: | Diego De La Toba <d_delatoba> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | annevk, eric.carlson, jer.noble, sirhc.schroeder, webkit-bot-watchers-bugzilla, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
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.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
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.