Bug 74788 – REGRESSION(r103130) - svg/W3C-SVG-1.1/animate-elem-*-t.svg asserting

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: SVG

[Adrienne Walker]

A number of svg animation tests are intermittently crashing on an assert in debug in Chromium's layout tests, although the release tests continue to pass.

ASSERTION FAILED: m_activeState != Active
/b/build/slave/webkit-mac-latest-dbg/build/src/third_party/WebKit/Source/WebCore/WebCore.gyp/../svg/animation/SVGSMILElement.cpp(921) : void WebCore::SVGSMILElement::progress(WebCore::SMILTime, WebCore::SVGSMILElement *)
1   0x53462dec WebCore::SVGSMILElement::progress(WebCore::SMILTime, WebCore::SVGSMILElement*)
2   0x53450374 WebCore::SMILTimeContainer::updateAnimations(WebCore::SMILTime, double, WTF::String const&)
3   0x5344f73b WebCore::SMILTimeContainer::timerFired(WebCore::Timer<WebCore::SMILTimeContainer>*)
4   0x5345af17 WebCore::Timer<WebCore::SMILTimeContainer>::fired()
5   0x51e7119e WebCore::ThreadTimers::sharedTimerFiredInternal()
6   0x51e70f2f WebCore::ThreadTimers::sharedTimerFired()
7   0x53842689 webkit_glue::WebKitPlatformSupportImpl::DoTimeout()
8   0x538431d7 base::BaseTimer<webkit_glue::WebKitPlatformSupportImpl, false>::TimerTask::Run()

Here's a sampling of tests that have been failing.  Win, Linux, and Mac all seem affected.

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=svg%2FW3C-SVG-1.1%2Fanimate-elem-46-t.svg%2Csvg%2FW3C-SVG-1.1%2Fanimate-elem-44-t.svg%2Csvg%2FW3C-SVG-1.1%2Fanimate-elem-52-t.svg%2Csvg%2FW3C-SVG-1.1%2Fanimate-elem-41-t.svg%2Csvg%2FW3C-SVG-1.1%2Fanimate-elem-81-t.svg%2Csvg%2FW3C-SVG-1.1%2Fanimate-elem-65-t.svg%2Csvg%2FW3C-SVG-1.1%2Fanimate-elem-82-t.svg

From the potential revision ranges where the above tests started crashing, the culprit is r103130 or before.  In looking at http://trac.webkit.org/log/?action=stop_on_copy&mode=stop_on_copy&rev=103130&stop_rev=103100&limit=100&verbose=on, this seems the most likely revision.

I was able to repro this locally on Chromium's debug Linux DRT by running all the animate-elem*.svg tests with child-processes=1, where one test consistently failed.  If I reverted this patch, it succeeded in 10/10 runs.

[Adrienne Walker]

Committed r103156: <http://trac.webkit.org/changeset/103156>

[Ryosuke Niwa]

It appears that these tests are no longer asserting.

[Ryosuke Niwa]

Hm... it appears that some of these crashes are still happening. I've removed them all in http://trac.webkit.org/changeset/103645 but had to add some of them back in http://trac.webkit.org/changeset/103654.

[John Knottenbelt]

svg/W3C-SVG-1.1/animate-elem-60-t.svg is flakily asserting on linux dgb:

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=svg%2FW3C-SVG-1.1%2Fanimate-elem-60-t.svg

[Stephen Chenney]

Taking ownership, as it seems the fastest way to get to the bottom of this. I'll first update expectations to mark it failing.

[Stephen Chenney]

Created attachment 122020 [details]
Patch

Expectations update to stop build failures

[WebKit Review Bot]

Comment on attachment 122020 [details]
Patch

Clearing flags on attachment: 122020

Committed r104715: <http://trac.webkit.org/changeset/104715>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Stephen Chenney]

Still need to actually fix it.

[Stephen Chenney]

Marking all bugs reporting assertions in WebCore::SVGSMILElement::progress as duplicates.

*** This bug has been marked as a duplicate of bug 64671 ***