[Webkit-unassigned] [Bug 93528] ASSERTION FAILED: resultAnimationElement->m_animatedType

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Mar 28 02:40:27 PDT 2013


https://bugs.webkit.org/show_bug.cgi?id=93528





--- Comment #12 from David Vest <davve at opera.com>  2013-03-28 02:38:36 PST ---
(From update of attachment 195313)
View in context: https://bugs.webkit.org/attachment.cgi?id=195313&action=review

>>>>>> LayoutTests/svg/animations/animation-dependency-crash.html:14
>>>>>> +    }, 300);
>>>>> 
>>>>> This value is too high and will cause our testing infrastructure to run even slower. Can this be reduced (preferably to 0 or 1?)
>>>> 
>>>> Any lower and it won't crash chrome on my machine. The Gtk port crashes with a zero value, but I erred on the side of caution. So yes, we might lower the value but the test might lose relevance.
>>> 
>>> Another possibility is to make it an SVG animation test, which uses JS to jump the timeline ahead. I have no idea at all whether or not that will trigger the crash.
>> 
>> I think crashing on at least one platform is probably enough test coverage for a crasher. I'm in favor of just using 0 since we will still have coverage, albeit not the best coverage. Stephen and David, what do you think?
> 
> I'm fine with that. I'll upload a new patch.

Sorry. When preparing the patch, I found out I can't reproduce the crash with a 0 or 1 timeout, not even in the gtk port. (For some reason, my <script> didn't run when I placed it below the <svg>, which actually in practice meant I had an infinite timeout value, not a zero one). I will try to find another way of testing this.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list