Summary: | REGRESSION: Multiple svg layout tests have become flaky timeouts | ||
---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> |
Component: | Animations | Assignee: | Nobody <webkit-unassigned> |
Status: | NEW --- | ||
Severity: | Normal | CC: | dino, realdawei, simon.fraser, tsavell, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | Other | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Description
Ryan Haddad
2018-07-23 15:53:26 PDT
Perhaps related, I noticed that svg/zoom/page/zoom-svg-through-object-with-percentage-size.xhtml was reported as missing expected results on this test run, but the test isn't new and does indeed have an iOS expected.txt file: https://build.webkit.org/results/Apple%20iOS%2011%20Simulator%20Release%20WK2%20(Tests)/r234110%20(6375)/results.html One of the earliest timeouts I see was with svg/animations/end-event-syncbase.svg, which blames https://trac.webkit.org/log/?verbose=on&rev=234078&stop_rev=234077 https://build.webkit.org/builders/Apple%20High%20Sierra%20Release%20WK2%20(Tests)/builds/5757 This change was related to animations: "Remove completed animations from GraphicsLayer, thus avoiding excessive backing store allocation" https://trac.webkit.org/changeset/234073/webkit (In reply to Ryan Haddad from comment #3) > This change was related to animations: > "Remove completed animations from GraphicsLayer, thus avoiding excessive > backing store allocation" > https://trac.webkit.org/changeset/234073/webkit That shouldn't have any impact on SVG animations. After further testing and help from Johnathan, I found that running the directories storage/indexeddb/modern/ and svg/ in succession with one test runner reproduced the issue. I bisected the storage/indexeddb/modern/ directory down to find that storage/indexeddb/modern/opendatabase-after-storage-crash.html is the casue of the flakey timouts. If you run this test in front tof the svg/ directory there is a chance of a timout occuring. Test Command Line: Run-webkit-tests --root testbuild-234505 storage/indexeddb/modern/opendatabase-after-storage-crash.html svg/ --child-processes 1 -v I then marked the test as skip locally and ran both directories again 10 times with no timouts occuring. Next steps are to mark the test as skip for now and work to find a proper solution. |