| Summary: | REGRESSION(250469@main) svg/animations/animation-leak-list-property-instances.html is randomly failing | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Fujii Hironori <Hironori.Fujii> | ||||||
| Component: | Animations | Assignee: | Fujii Hironori <Hironori.Fujii> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | ap, darin, dino, graouts, koivisto, svillar, webkit-bug-importer, ysuzuki | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=242231 https://bugs.webkit.org/show_bug.cgi?id=114280 |
||||||||
| Attachments: |
|
||||||||
|
Description
Fujii Hironori
2022-06-30 23:44:44 PDT
250480@main Buildbot: builder Apple-BigSur-Debug-WK2-Tests build 6491 https://build.webkit.org/#/builders/23/builds/6491 --- /Volumes/Data/worker/Apple-BigSur-Debug-WK2-Tests/build/layout-test-results/svg/animations/animation-leak-list-property-instances-expected.txt +++ /Volumes/Data/worker/Apple-BigSur-Debug-WK2-Tests/build/layout-test-results/svg/animations/animation-leak-list-property-instances-actual.txt @@ -3,5 +3,5 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". -PASS 0 is 0 +FAIL -63 should be 0. Was -63. Another test failure. Buildbot: builder GTK-Linux-64-bit-Release-Skip-Failing-Tests build 6406: 252088@main https://build.webkit.org/#/builders/199/builds/6406 --- /home/buildbot/worker/GTK-Linux-64-bit-Release-Skip-Failing-Tests/build/layout-test-results/svg/animations/animation-leak-list-property-instances-expected.txt +++ /home/buildbot/worker/GTK-Linux-64-bit-Release-Skip-Failing-Tests/build/layout-test-results/svg/animations/animation-leak-list-property-instances-actual.txt @@ -3,8 +3,9 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". -PASS 0 is 0 +FAIL -7 should be 0. Was -7. PASS successfullyParsed is true +Some tests failed. TEST COMPLETE Bug 242231 fixed some obviously mistakes in the test, but now it's failing for apparently more substantial reasons. Created attachment 460659 [details]
Patch
Comment on attachment 460659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=460659&action=review > LayoutTests/svg/animations/animation-leak-list-property-instances-expected.txt:6 > +PASS 0 is <= 5 This won’t work to make the test less flaky. Because if we have 1 this line will say "Pass 1 is <= 5" and the test will fail. Comment on attachment 460659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=460659&action=review >> LayoutTests/svg/animations/animation-leak-list-property-instances-expected.txt:6 >> +PASS 0 is <= 5 > > This won’t work to make the test less flaky. Because if we have 1 this line will say "Pass 1 is <= 5" and the test will fail. This needs to be changed so that the number of objects is not logged. Comment on attachment 460659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=460659&action=review >>> LayoutTests/svg/animations/animation-leak-list-property-instances-expected.txt:6 >>> +PASS 0 is <= 5 >> >> This won’t work to make the test less flaky. Because if we have 1 this line will say "Pass 1 is <= 5" and the test will fail. > > This needs to be changed so that the number of objects is not logged. Good catch. Will fix. Thank you very much. Created attachment 460669 [details]
Patch
Comment on attachment 460669 [details]
Patch
r=me
Committed 252153@main (1b783a18fdac): <https://commits.webkit.org/252153@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 460669 [details]. |