Bug 214574

Summary: [Mac] svg/animations/smil-leak-list-property-instances.svg is a flaky failure
Product: WebKit Reporter: Hector Lopez <hector_i_lopez>
Component: New BugsAssignee: Fujii Hironori <Hironori.Fujii>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, ehutchison, Hironori.Fujii, jlewis3, sabouhallawa, tsavell, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
tsavell: commit-queue-
mark as flaky
none
Update Test Expectations
none
Patch
none
Patch none

Description Hector Lopez 2020-07-20 14:52:02 PDT 
svg/animations/smil-leak-list-property-instances.svg

Test is flaky failing in macOS wk2.

History:
https://results.webkit.org/?suite=layout-tests&test=svg%2Fanimations%2Fsmil-leak-list-property-instances.svg

Diff:
--- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/svg/animations/smil-leak-list-property-instances-expected.txt
+++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/svg/animations/smil-leak-list-property-instances-actual.txt
@@ -1 +1 @@
-PASS
+FAIL: 3 extra live node(s)
Comment 1 Radar WebKit Bug Importer 2020-07-20 14:52:31 PDT 
<rdar://problem/65849502>
Comment 2 Hector Lopez 2020-07-20 15:01:37 PDT 
Created attachment 404764 [details]
Patch
Comment 3 Truitt Savell 2020-07-20 15:04:34 PDT 
It looks like this is only failing on release builds. Could you add [ Release ] to the front of your expectation so we maintain coverage of Debug?
Comment 4 Hector Lopez 2020-07-20 15:20:01 PDT 
Created attachment 404767 [details]
mark as flaky
Comment 5 EWS 2020-07-20 15:57:19 PDT 
Committed r264628: <https://trac.webkit.org/changeset/264628>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 404767 [details].
Comment 6 Alexey Proskuryakov 2020-07-20 19:31:44 PDT 
Re-opening, because this isn't fixed.
Comment 7 Eric Hutchison 2021-07-20 13:29:55 PDT 
Created attachment 433896 [details]
Update Test Expectations
Comment 8 Eric Hutchison 2021-07-20 13:30:44 PDT 
Test is also flaky failing on Debug.

Diff: 
--- /Volumes/Data/worker/bigsur-debug-applesilicon-tests-wk2/build/layout-test-results/svg/animations/smil-leak-list-property-instances-expected.txt
+++ /Volumes/Data/worker/bigsur-debug-applesilicon-tests-wk2/build/layout-test-results/svg/animations/smil-leak-list-property-instances-actual.txt
@@ -1 +1 @@
-PASS
+FAIL: -12 extra live node(s)

Updating test expectations to include debug.
Comment 9 Eric Hutchison 2021-07-20 13:37:49 PDT 
reproduced using run-webkit-tests --debug --force --iterations 500 --exit-after-n-failures 2 svg/animations/smil-leak-list-property-instances.svg
Comment 10 Truitt Savell 2021-07-20 13:56:42 PDT 
Comment on attachment 433896 [details]
Update Test Expectations

Clearing flags on attachment: 433896

Committed r280104 (239821@main): <https://commits.webkit.org/239821@main>
Comment 11 Fujii Hironori 2022-07-04 14:50:06 PDT 
Created attachment 460660 [details]
Patch
Comment 12 Fujii Hironori 2022-07-04 14:54:19 PDT 
Created attachment 460661 [details]
Patch
Comment 13 Fujii Hironori 2022-07-04 21:06:05 PDT 
*** Bug 174218 has been marked as a duplicate of this bug. ***
Comment 14 EWS 2022-07-04 22:24:13 PDT 
Committed 252128@main (c05bfb093f5d): <https://commits.webkit.org/252128@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 460661 [details].