Bug 184659 – [Web Animation] Continuous DocumentTimeline::performInvalidationTask() dispatch on washingtonpost.com

• Component: Animations
• Status: NEW

[Zan Dobersek]

On the GTK and WPE ports, after scrolling through article pages down to the comment section, some undetermined animation content sets up an infinite scheduling of the DocumentTimeline::performInvalidationTask().

This is the backtrace that showcases the issue:

#0  0x00007f0806c14c40 in WebCore::DocumentTimeline::scheduleInvalidationTaskIfNeeded() () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#1  0x00007f0806c14da8 in WebCore::DocumentTimeline::currentTime() () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#2  0x00007f0806c11683 in WebCore::WebAnimation::currentTime(WebCore::WebAnimation::RespectHoldTime) const () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#3  0x00007f0806c1173b in WebCore::WebAnimation::currentTime() const () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#4  0x00007f0807a41f62 in WebCore::AnimationEffectReadOnly::localTime() const () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#5  0x00007f0807a42968 in WebCore::AnimationEffectReadOnly::phase() const () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#6  0x00007f0807a42a94 in WebCore::AnimationEffectReadOnly::activeTime() const () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#7  0x00007f0807a42e9b in WebCore::AnimationEffectReadOnly::currentIteration() const () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#8  0x00007f0807a43f2d in WebCore::DeclarativeAnimation::invalidateDOMEvents(WTF::Seconds) () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#9  0x00007f0806c11f26 in WebCore::DocumentTimeline::performInvalidationTask() () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#10 0x00007f080730daec in WebCore::TaskDispatcher<WebCore::Timer>::dispatchOneTask() () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#11 0x00007f080730dbf7 in WebCore::TaskDispatcher<WebCore::Timer>::sharedTimerFired() () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#12 0x00007f0807338d5a in WebCore::ThreadTimers::sharedTimerFiredInternal() () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#13 0x00007f08086aaa33 in WTF::RunLoop::TimerBase::TimerBase(WTF::RunLoop&)::{lambda(void*)#1}::_FUN(void*) () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#14 0x00007f07fda0656a in g_main_dispatch (context=0x55efcd13f450) at /home/zan/Work/webkit/git/WebKitBuild/DependenciesWPE/Source/glib-2.54.3/glib/gmain.c:3142
#15 g_main_context_dispatch (context=context@entry=0x55efcd13f450) at /home/zan/Work/webkit/git/WebKitBuild/DependenciesWPE/Source/glib-2.54.3/glib/gmain.c:3795
#16 0x00007f07fda068f8 in g_main_context_iterate (context=0x55efcd13f450, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>)
    at /home/zan/Work/webkit/git/WebKitBuild/DependenciesWPE/Source/glib-2.54.3/glib/gmain.c:3868
#17 0x00007f07fda06c12 in g_main_loop_run (loop=0x55efcd13f590) at /home/zan/Work/webkit/git/WebKitBuild/DependenciesWPE/Source/glib-2.54.3/glib/gmain.c:4064
#18 0x00007f08086aae50 in WTF::RunLoop::run() () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#19 0x00007f0806668849 in int WebKit::ChildProcessMain<WebKit::WebProcess, WebKit::WebProcessMain>(int, char**) () from /home/zan/Work/webkit/git/WebKitBuild/Release/lib/libWPEWebKit-0.1.so.1
#20 0x00007f07fc6932e1 in __libc_start_main (main=0x55efcbdc6ba0 <main>, argc=4, argv=0x7ffeb33c5de8, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7ffeb33c5dd8)
    at ../csu/libc-start.c:291
#21 0x000055efcbdc6c2a in _start ()

Not yet tested on the Mac port. Disabling Web Animations support as a runtime-enabled feature avoids this issue.

[Antoine Quint]

Zan, is this still current? I expect this to no longer be the case.