131097 2014-04-02 02:33:25 -0700 ASSERTION FAILED: !begin.isIndefinite() in WebCore::SVGSMILElement::resolveFirstInterval 2014-04-24 09:28:11 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 116980 1 mhodovan.u-szeged webkit-unassigned andersca commit-queue d-r eric fmalita gyuyoung.kim koivisto pdr rhodovan.u-szeged sam schenney sergio tgergely.u-szeged zimmermann oldest_to_newest 996644 0 228376 mhodovan.u-szeged 2014-04-02 02:33:25 -0700 Created attachment 228376 Test case The failing test case: <svg> <animatemotion begin="-13 ;indefinite ;" max="0h"> </animatemotion> </svg> The backtrace: #1 0x00007ffff1638939 in WebCore::SVGSMILElement::resolveFirstInterval (this=0x785470) at /home/martin/Data/WebKit/Source/WebCore/svg/animation/SVGSMILElement.cpp:840 #2 0x00007ffff16353c0 in WebCore::SVGSMILElement::insertedInto (this=0x785470, rootParent=...) at /home/martin/Data/WebKit/Source/WebCore/svg/animation/SVGSMILElement.cpp:256 #3 0x00007ffff09e6265 in WebCore::ChildNodeInsertionNotifier::notifyNodeInsertedIntoDocument (this=0x7fffffffd140, node=...) at /home/martin/Data/WebKit/Source/WebCore/dom/ContainerNodeAlgorithms.h:200 #4 0x00007ffff09e641d in WebCore::ChildNodeInsertionNotifier::notify (this=0x7fffffffd140, node=...) at /home/martin/Data/WebKit/Source/WebCore/dom/ContainerNodeAlgorithms.h:228 #5 0x00007ffff09e3e9a in WebCore::ContainerNode::parserAppendChild (this=0x6d2ab0, newChild=...) at /home/martin/Data/WebKit/Source/WebCore/dom/ContainerNode.cpp:744 #6 0x00007ffff0d12c94 in WebCore::insert (task=...) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLConstructionSite.cpp:96 #7 0x00007ffff0d12ced in WebCore::executeInsertTask (task=...) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLConstructionSite.cpp:103 #8 0x00007ffff0d12ef3 in WebCore::executeTask (task=...) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLConstructionSite.cpp:141 #9 0x00007ffff0d13218 in WebCore::HTMLConstructionSite::executeQueuedTasks (this=0x6e8e68) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLConstructionSite.cpp:191 #10 0x00007ffff0d40564 in WebCore::HTMLTreeBuilder::constructTree (this=0x6e8e50, token=0x7fffffffd2c0) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLTreeBuilder.cpp:366 #11 0x00007ffff0d1b6a8 in WebCore::HTMLDocumentParser::constructTreeFromHTMLToken (this=0x6f3c10, rawToken=...) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLDocumentParser.cpp:352 #12 0x00007ffff0d1b32f in WebCore::HTMLDocumentParser::pumpTokenizer (this=0x6f3c10, mode=WebCore::HTMLDocumentParser::AllowYield) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLDocumentParser.cpp:309 #13 0x00007ffff0d1ab35 in WebCore::HTMLDocumentParser::pumpTokenizerIfPossible (this=0x6f3c10, mode=WebCore::HTMLDocumentParser::AllowYield) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLDocumentParser.cpp:189 #14 0x00007ffff0d1bbef in WebCore::HTMLDocumentParser::append (this=0x6f3c10, inputSource=...) at /home/martin/Data/WebKit/Source/WebCore/html/parser/HTMLDocumentParser.cpp:428 #15 0x00007ffff09f8f01 in WebCore::DecodedDataDocumentParser::flush (this=0x6f3c10, writer=...) at /home/martin/Data/WebKit/Source/WebCore/dom/DecodedDataDocumentParser.cpp:60 #16 0x00007ffff0e7f1a3 in WebCore::DocumentWriter::end (this=0x735490) at /home/martin/Data/WebKit/Source/WebCore/loader/DocumentWriter.cpp:245 #17 0x00007ffff0e69b69 in WebCore::DocumentLoader::finishedLoading (this=0x7353f0, finishTime=0) at /home/martin/Data/WebKit/Source/WebCore/loader/DocumentLoader.cpp:440 #18 0x00007ffff0e698d2 in WebCore::DocumentLoader::notifyFinished (this=0x7353f0, resource=0x73e750) at /home/martin/Data/WebKit/Source/WebCore/loader/DocumentLoader.cpp:374 #19 0x00007ffff0f129c8 in WebCore::CachedResource::checkNotify (this=0x73e750) at /home/martin/Data/WebKit/Source/WebCore/loader/cache/CachedResource.cpp:332 #20 0x00007ffff0f12aa6 in WebCore::CachedResource::finishLoading (this=0x73e750) at /home/martin/Data/WebKit/Source/WebCore/loader/cache/CachedResource.cpp:348 #21 0x00007ffff0f0f44e in WebCore::CachedRawResource::finishLoading (this=0x73e750, data=0x74eb10) at /home/martin/Data/WebKit/Source/WebCore/loader/cache/CachedRawResource.cpp:97 #22 0x00007ffff0eca590 in WebCore::SubresourceLoader::didFinishLoading (this=0x7406a0, finishTime=0) at /home/martin/Data/WebKit/Source/WebCore/loader/SubresourceLoader.cpp:310 #23 0x00007ffff0ec6847 in WebCore::ResourceLoader::didFinishLoading (this=0x7406a0, finishTime=0) at /home/martin/Data/WebKit/Source/WebCore/loader/ResourceLoader.cpp:508 #24 0x00007ffff17b51ed in WebCore::readCallback (asyncResult=0x77e9e0, data=0x740ac0) at /home/martin/Data/WebKit/Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:1324 #25 0x00007fffebe85aaa in async_ready_callback_wrapper (source_object=0x66e1b0, res=0x77e9e0, user_data=0x740ac0) at ginputstream.c:519 #26 0x00007fffebea547b in g_task_return_now (task=0x77e9e0) at gtask.c:1108 #27 0x00007fffebea5499 in complete_in_idle_cb (task=0x77e9e0) at gtask.c:1117 #28 0x00007fffeb8f5536 in g_main_dispatch (context=0x6692e0) at gmain.c:3065 #29 g_main_context_dispatch (context=context@entry=0x6692e0) at gmain.c:3641 #30 0x00007fffecbd6708 in _ecore_glib_select__locked (ecore_timeout=<optimized out>, efds=<optimized out>, wfds=0x7fffffffda30, rfds=0x7fffffffd9b0, ecore_fds=8, ctx=<optimized out>) at ecore_glib.c:171 #31 _ecore_glib_select (ecore_fds=8, rfds=0x7fffffffd9b0, wfds=0x7fffffffda30, efds=<optimized out>, ecore_timeout=<optimized out>) at ecore_glib.c:205 #32 0x00007fffecbd0b37 in _ecore_main_select (timeout=<optimized out>) at ecore_main.c:1466 #33 0x00007fffecbd16c5 in _ecore_main_loop_iterate_internal (once_only=once_only@entry=0) at ecore_main.c:1894 #34 0x00007fffecbd19c7 in ecore_main_loop_begin () at ecore_main.c:956 #35 0x00007ffff76a122b in WTF::RunLoop::run () at /home/martin/Data/WebKit/Source/WTF/wtf/efl/RunLoopEfl.cpp:51 #36 0x00007ffff763347d in WebKit::WebProcessMainEfl (argc=2, argv=0x7fffffffdeb8) at /home/martin/Data/WebKit/Source/WebKit2/WebProcess/efl/WebProcessMainEfl.cpp:126 #37 0x0000000000400840 in main (argc=2, argv=0x7fffffffdeb8) at /home/martin/Data/WebKit/Source/WebKit2/efl/MainEfl.cpp:30 1003636 1 tgergely.u-szeged 2014-04-24 08:43:52 -0700 According to the SVG 1.1 animation reference (http://www.w3.org/TR/SVG11/animate.html#TimingAttributes), max "Value must be greater than 0.". Seems to be a boundary-value error, as wk works fine for negative values (ignores them). 1003637 2 230080 tgergely.u-szeged 2014-04-24 08:44:43 -0700 Created attachment 230080 patch 1003665 3 230080 commit-queue 2014-04-24 09:28:06 -0700 Comment on attachment 230080 patch Clearing flags on attachment: 230080 Committed r167761: <http://trac.webkit.org/changeset/167761> 1003666 4 commit-queue 2014-04-24 09:28:11 -0700 All reviewed patches have been landed. Closing bug. 228376 2014-04-02 02:33:25 -0700 2014-04-02 02:33:25 -0700 Test case test.html text/html 85 mhodovan.u-szeged PHN2Zz4KICAgIDxhbmltYXRlbW90aW9uIGJlZ2luPSItMTMgO2luZGVmaW5pdGUgOyIgbWF4PSIw aCI+IDwvYW5pbWF0ZW1vdGlvbj4KPC9zdmc+Cg== 230080 2014-04-24 08:44:43 -0700 2014-04-24 09:28:06 -0700 patch amination-max-attribute-zero.patch text/plain 3405 tgergely.u-szeged ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCA4YzNlODA4Li4xZWMzZTY2IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTQtMDQt MjQgIFRhbWFzIEdlcmdlbHkgIDx0Z2VyZ2VseS51LXN6ZWdlZEBwYXJ0bmVyLnNhbXN1bmcuY29t PgorCisgICAgICAgIEFTU0VSVElPTiBGQUlMRUQ6ICFiZWdpbi5pc0luZGVmaW5pdGUoKSBpbiBX ZWJDb3JlOjpTVkdTTUlMRWxlbWVudDo6cmVzb2x2ZUZpcnN0SW50ZXJ2YWwuCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMzEwOTcKKworICAgICAgICBS ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUZXN0IGFkZGVkLgorCisgICAg ICAgICogc3ZnL2FuaW1hdGlvbnMvc21pbC1hbmltYXRpb24tbWF4LWF0dHJpYnV0ZS16ZXJvLWNy YXNoLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogc3ZnL2FuaW1hdGlvbnMvc21pbC1h bmltYXRpb24tbWF4LWF0dHJpYnV0ZS16ZXJvLWNyYXNoLnN2ZzogQWRkZWQuCisKIDIwMTQtMDQt MjMgIFByYXZlZW4gUiBKYWRoYXYgIDxwcmF2ZWVuLmpAc2Ftc3VuZy5jb20+CiAKICAgICAgICAg W01lZGlhU3RyZWFtXSBJbXBsZW1lbnQgTWVkaWFTdHJlYW0gYWN0aXZlIGF0dHJpYnV0ZQpkaWZm IC0tZ2l0IGEvTGF5b3V0VGVzdHMvc3ZnL2FuaW1hdGlvbnMvc21pbC1hbmltYXRpb24tbWF4LWF0 dHJpYnV0ZS16ZXJvLWNyYXNoLWV4cGVjdGVkLnR4dCBiL0xheW91dFRlc3RzL3N2Zy9hbmltYXRp b25zL3NtaWwtYW5pbWF0aW9uLW1heC1hdHRyaWJ1dGUtemVyby1jcmFzaC1leHBlY3RlZC50eHQK bmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMC4uMTIyMWVjYgotLS0gL2Rldi9udWxs CisrKyBiL0xheW91dFRlc3RzL3N2Zy9hbmltYXRpb25zL3NtaWwtYW5pbWF0aW9uLW1heC1hdHRy aWJ1dGUtemVyby1jcmFzaC1leHBlY3RlZC50eHQKQEAgLTAsMCArMSBAQAorUEFTUzogbm90IGNy YXNoaW5nIGluIGRlYnVnLgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvc3ZnL2FuaW1hdGlvbnMv c21pbC1hbmltYXRpb24tbWF4LWF0dHJpYnV0ZS16ZXJvLWNyYXNoLnN2ZyBiL0xheW91dFRlc3Rz L3N2Zy9hbmltYXRpb25zL3NtaWwtYW5pbWF0aW9uLW1heC1hdHRyaWJ1dGUtemVyby1jcmFzaC5z dmcKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMC4uOTVmOGFlZAotLS0gL2Rldi9u dWxsCisrKyBiL0xheW91dFRlc3RzL3N2Zy9hbmltYXRpb25zL3NtaWwtYW5pbWF0aW9uLW1heC1h dHRyaWJ1dGUtemVyby1jcmFzaC5zdmcKQEAgLTAsMCArMSwxMiBAQAorPD94bWwgdmVyc2lvbj0i MS4wIiBlbmNvZGluZz0iVVRGLTgiPz4KKzxzdmcgdmVyc2lvbj0iMS4xIiB4bWxucz0iaHR0cDov L3d3dy53My5vcmcvMjAwMC9zdmciPgorICA8IS0tIFRlc3QgZm9yIGh0dHBzOi8vYnVncy53ZWJr aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMzEwOTcgLS0+CisgIDxyZWN0PgorICAgIDxhbmltYXRl IGF0dHJpYnV0ZU5hbWU9ImZpbGwiIGJlZ2luPSItMTsgaW5kZWZpbml0ZSIgbWF4PSIwIiBmaWxs PSJmcmVlemUiIHZhbHVlcz0iIzAwMDsjZmZmIi8+CisgIDwvcmVjdD4KKyAgPHRleHQ+UEFTUzog bm90IGNyYXNoaW5nIGluIGRlYnVnLjwvdGV4dD4KKyAgPHNjcmlwdD4KKyAgICBpZiAod2luZG93 LnRlc3RSdW5uZXIpCisgICAgICB0ZXN0UnVubmVyLmR1bXBBc1RleHQoKTsKKyAgPC9zY3JpcHQ+ Cis8L3N2Zz4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9X ZWJDb3JlL0NoYW5nZUxvZwppbmRleCA1NTY4NGRkLi5lOGI1NDBhIDEwMDY0NAotLS0gYS9Tb3Vy Y2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0x LDMgKzEsMTggQEAKKzIwMTQtMDQtMjQgIFRhbWFzIEdlcmdlbHkgIDx0Z2VyZ2VseS51LXN6ZWdl ZEBwYXJ0bmVyLnNhbXN1bmcuY29tPgorCisgICAgICAgIEFTU0VSVElPTiBGQUlMRUQ6ICFiZWdp bi5pc0luZGVmaW5pdGUoKSBpbiBXZWJDb3JlOjpTVkdTTUlMRWxlbWVudDo6cmVzb2x2ZUZpcnN0 SW50ZXJ2YWwuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD0xMzEwOTcKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg ICBBY2NvcmRpbmcgdG8gc21pbCBhbmltYXRpb24gcmVmZXJlbmNlLCBtYXggYXR0cmlidXRlIGNh bm5vdCBiZSAwLgorCisgICAgICAgIFRlc3Q6IHN2Zy9hbmltYXRpb25zL3NtaWwtYW5pbWF0aW9u LW1heC1hdHRyaWJ1dGUtemVyby1jcmFzaC5zdmcKKworICAgICAgICAqIHN2Zy9hbmltYXRpb24v U1ZHU01JTEVsZW1lbnQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6U1ZHU01JTEVsZW1lbnQ6Om1h eFZhbHVlKToKKyAgICAgICAgICBjaGFuZ2VkIChyZXN1bHQgPCAwKSB0byAocmVzdWx0IDw9IDAp CisKIDIwMTQtMDQtMjMgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t PgogCiAgICAgICAgIFtHVEtdIEhUTUwgTWVkaWEgY2FwdHVyZSBhdHRyaWJ1dGUgaXMgYSBib29s ZWFuIHNpbmNlIHIxNjM5NTgKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3N2Zy9hbmltYXRp b24vU1ZHU01JTEVsZW1lbnQuY3BwIGIvU291cmNlL1dlYkNvcmUvc3ZnL2FuaW1hdGlvbi9TVkdT TUlMRWxlbWVudC5jcHAKaW5kZXggNmI4MDRhMS4uMmE0NDZlNiAxMDA2NDQKLS0tIGEvU291cmNl L1dlYkNvcmUvc3ZnL2FuaW1hdGlvbi9TVkdTTUlMRWxlbWVudC5jcHAKKysrIGIvU291cmNlL1dl YkNvcmUvc3ZnL2FuaW1hdGlvbi9TVkdTTUlMRWxlbWVudC5jcHAKQEAgLTY4MCw3ICs2ODAsNyBA QCBTTUlMVGltZSBTVkdTTUlMRWxlbWVudDo6bWF4VmFsdWUoKSBjb25zdAogICAgICAgICByZXR1 cm4gbV9jYWNoZWRNYXg7CiAgICAgY29uc3QgQXRvbWljU3RyaW5nJiB2YWx1ZSA9IGZhc3RHZXRB dHRyaWJ1dGUoU1ZHTmFtZXM6Om1heEF0dHIpOwogICAgIFNNSUxUaW1lIHJlc3VsdCA9IHBhcnNl Q2xvY2tWYWx1ZSh2YWx1ZSk7Ci0gICAgcmV0dXJuIG1fY2FjaGVkTWF4ID0gKHJlc3VsdC5pc1Vu cmVzb2x2ZWQoKSB8fCByZXN1bHQgPCAwKSA/IFNNSUxUaW1lOjppbmRlZmluaXRlKCkgOiByZXN1 bHQ7CisgICAgcmV0dXJuIG1fY2FjaGVkTWF4ID0gKHJlc3VsdC5pc1VucmVzb2x2ZWQoKSB8fCBy ZXN1bHQgPD0gMCkgPyBTTUlMVGltZTo6aW5kZWZpbml0ZSgpIDogcmVzdWx0OwogfQogICAgIAog U01JTFRpbWUgU1ZHU01JTEVsZW1lbnQ6Om1pblZhbHVlKCkgY29uc3QK