277234 2024-07-28 12:55:53 -0700 http/tests/canvas/ctx.2d-canvas-style-no-document-leak.html is flaky failure 2024-07-29 14:12:58 -0700 1 1 1 Unclassified WebKit Tools / Tests WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=256262 https://bugs.webkit.org/show_bug.cgi?id=277263 InRadar P2 Normal --- 1 fujii rreno rreno webkit-bug-importer oldest_to_newest 2048975 0 fujii 2024-07-28 12:55:53 -0700 http/tests/canvas/ctx.2d-canvas-style-no-document-leak.html is flaky failure History: https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fcanvas%2Fctx.2d-canvas-style-no-document-leak.html Buildbot: builder Apple-Sonoma-Release-WK2-Tests build 2831 : 278780@main https://build.webkit.org/#/builders/926/builds/2831 https://build.webkit.org/results/Apple-Sonoma-Release-WK2-Tests/278780@main%20(2831)/http/tests/canvas/ctx.2d-canvas-style-no-document-leak-diff.txt --- /Volumes/Data/worker/Apple-Sonoma-Release-WK2-Tests/build/layout-test-results/http/tests/canvas/ctx.2d-canvas-style-no-document-leak-expected.txt +++ /Volumes/Data/worker/Apple-Sonoma-Release-WK2-Tests/build/layout-test-results/http/tests/canvas/ctx.2d-canvas-style-no-document-leak-actual.txt @@ -6,7 +6,7 @@ PASS internals.isDocumentAlive(frameDocumentID) is true PASS internals.isDocumentAlive(frameDocumentID) is true PASS internals.isDocumentAlive(frameDocumentID) is true -PASS The iframe document didn't leak. +FAIL The iframe document leaked. PASS successfullyParsed is true TEST COMPLETE 2048977 1 fujii 2024-07-28 12:58:43 -0700 This test was added by 263774@main. 2048978 2 fujii 2024-07-28 13:29:29 -0700 This test case has only a single iframe. It should create more iframes like document-leak-test.js does. 2049106 3 rreno 2024-07-29 08:26:17 -0700 Yes, this test should be rewritten to use the document-leak-test.js helper. I'll put a patch up for review shortly. Thanks for catching that. There's a couple of other tests I can rewrite - the one for ReportingObserver callbacks (which is where document-leak-test.js was adopted from), geolocation callbacks, and probably the one for MediaSession Action Handlers. 2049109 4 webkit-bug-importer 2024-07-29 08:44:05 -0700 <rdar://problem/132725257> 2049113 5 rreno 2024-07-29 08:49:09 -0700 Pull request: https://github.com/WebKit/WebKit/pull/31395 2049200 6 ews-feeder 2024-07-29 14:12:56 -0700 Committed 281523@main (148e56013b45): <https://commits.webkit.org/281523@main> Reviewed commits have been landed. Closing PR #31395 and removing active labels.