12974 2007-03-05 14:55:40 -0800 Repro ASSERT failure when repeatedly running svg/custom/js-update-path-changes.svg 2007-03-13 10:07:41 -0700 1 1 1 Unclassified WebKit SVG 523.x (Safari 3) Mac OS X 10.4 RESOLVED FIXED InRadar P1 Normal --- 1 ggaren zimmermann zimmermann oldest_to_newest 20370 0 ggaren 2007-03-05 14:55:40 -0800 This does the trick: run-webkit-tests --debug --no-launch-safari --verbose --nthly=100000 svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg svg/custom/js-update-path-changes.svg 20390 1 mitz 2007-03-05 23:20:28 -0800 ASSERTION FAILED: extensions->genericContext<SVGPathSeg>(obj) == context (WebCore/bindings/js/JSSVGPathSegListCustom.cpp:52 void WebCore::updatePathSegContextMap(KJS::ExecState*, WebCore::SVGPathSegList*, WebCore::SVGPathSeg*)) 20126 2 mrowe 2007-03-07 06:48:12 -0800 <rdar://problem/5045719> 19497 3 zimmermann 2007-03-11 13:14:52 -0700 Thanks for the report! I investigated, and found the problem. Trying to come up with a patch soon. Niko 19500 4 13587 zimmermann 2007-03-11 14:01:22 -0700 Created attachment 13587 Initial patch 19258 5 13587 mjs 2007-03-12 17:04:02 -0700 Comment on attachment 13587 Initial patch r=me 19125 6 sam 2007-03-13 10:07:41 -0700 Landed in r20151. 13587 2007-03-11 14:01:22 -0700 2007-03-12 17:04:02 -0700 Initial patch FixAssert.diff text/plain 6929 zimmermann SW5kZXg6IGJpbmRpbmdzL3NjcmlwdHMvQ29kZUdlbmVyYXRvckpTLnBtCj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t IGJpbmRpbmdzL3NjcmlwdHMvQ29kZUdlbmVyYXRvckpTLnBtCShyZXZpc2lvbiAyMDEwMykKKysr IGJpbmRpbmdzL3NjcmlwdHMvQ29kZUdlbmVyYXRvckpTLnBtCSh3b3JraW5nIGNvcHkpCkBAIC0x LDUgKzEsNSBAQAogIwotIyBDb3B5cmlnaHQgKEMpIDIwMDUsIDIwMDYgTmlrb2xhcyBaaW1tZXJt YW5uIDx6aW1tZXJtYW5uQGtkZS5vcmc+CisjIENvcHlyaWdodCAoQykgMjAwNSwgMjAwNiwgMjAw NyBOaWtvbGFzIFppbW1lcm1hbm4gPHppbW1lcm1hbm5Aa2RlLm9yZz4KICMgQ29weXJpZ2h0IChD KSAyMDA2IEFuZGVycyBDYXJsc3NvbiA8YW5kZXJzY2FAbWFjLmNvbT4KICMgQ29weXJpZ2h0IChD KSAyMDA2IFNhbXVlbCBXZWluaWcgPHNhbS53ZWluaWdAZ21haWwuY29tPgogIyBDb3B5cmlnaHQg KEMpIDIwMDYgQWxleGV5IFByb3NrdXJ5YWtvdiA8YXBAd2Via2l0Lm9yZz4KQEAgLTE2Niw2ICsx NjYsMTYgQEAKICAgICByZXR1cm4gMDsKIH0KIAorc3ViIENyZWF0ZVNWR0NvbnRleHRJbnRlcmZh Y2VOYW1lCit7CisgICAgbXkgJHR5cGUgPSBzaGlmdDsKKworICAgIHJldHVybiAkdHlwZSBpZiAk Y29kZUdlbmVyYXRvci0+SXNTVkdBbmltYXRlZFR5cGUoJHR5cGUpOworICAgIHJldHVybiAiU1ZH UGF0aFNlZyIgaWYgJHR5cGUgPX4gL15TVkdQYXRoU2VnLyBhbmQgJHR5cGUgbmUgIlNWR1BhdGhT ZWdMaXN0IjsKKworICAgIHJldHVybiAiIjsKK30KKwogc3ViIEFkZEluY2x1ZGVzRm9yVHlwZQog ewogICAgIG15ICR0eXBlID0gJGNvZGVHZW5lcmF0b3ItPlN0cmlwTW9kdWxlKHNoaWZ0KTsKQEAg LTcyOSw4ICs3MzksOSBAQAogICAgIGlmICghJGhhc1BhcmVudCkgewogICAgICAgICBwdXNoKEBp bXBsQ29udGVudCwgIiR7Y2xhc3NOYW1lfTo6fiRjbGFzc05hbWUoKVxuIik7CiAKLSAgICAgICAg aWYgKCRjb2RlR2VuZXJhdG9yLT5Jc1NWR0FuaW1hdGVkVHlwZSgkaW50ZXJmYWNlTmFtZSkpIHsK LSAgICAgICAgICAgIHB1c2goQGltcGxDb250ZW50LCAie1xuICAgIFNWR0RvY3VtZW50RXh0ZW5z aW9uczo6Zm9yZ2V0R2VuZXJpY0NvbnRleHQ8JGludGVyZmFjZU5hbWU+KG1faW1wbC5nZXQoKSk7 XG4iKTsgICAKKyAgICAgICAgbXkgJGNvbnRleHRJbnRlcmZhY2VOYW1lID0gQ3JlYXRlU1ZHQ29u dGV4dEludGVyZmFjZU5hbWUoJGludGVyZmFjZU5hbWUpOworICAgICAgICBpZiAoJGNvbnRleHRJ bnRlcmZhY2VOYW1lIG5lICIiKSB7CisgICAgICAgICAgICBwdXNoKEBpbXBsQ29udGVudCwgIntc biAgICBTVkdEb2N1bWVudEV4dGVuc2lvbnM6OmZvcmdldEdlbmVyaWNDb250ZXh0PCRjb250ZXh0 SW50ZXJmYWNlTmFtZT4obV9pbXBsLmdldCgpKTtcbiIpOyAgIAogICAgICAgICAgICAgcHVzaChA aW1wbENvbnRlbnQsICIgICAgU2NyaXB0SW50ZXJwcmV0ZXI6OmZvcmdldERPTU9iamVjdChtX2lt cGwuZ2V0KCkpO1xufVxuXG4iKTsKICAgICAgICAgfSBlbHNlIHsKICAgICAgICAgICAgIHB1c2go QGltcGxDb250ZW50LCAie1xuICAgIFNjcmlwdEludGVycHJldGVyOjpmb3JnZXRET01PYmplY3Qo bV9pbXBsLmdldCgpKTtcbn1cblxuIik7CkBAIC05NDgsMTYgKzk1OSwxNyBAQAogICAgICAgICAg ICAgfQogICAgICAgICAgICAgcHVzaChAaW1wbENvbnRlbnQsICIgICAgfVxuIik7ICMgZW5kIHN3 aXRjaAogCisgICAgICAgICAgICBteSAkY29udGV4dEludGVyZmFjZU5hbWUgPSBDcmVhdGVTVkdD b250ZXh0SW50ZXJmYWNlTmFtZSgkaW50ZXJmYWNlTmFtZSk7CiAgICAgICAgICAgICBpZiAoJGlu dGVyZmFjZU5hbWUgZXEgIkRPTVdpbmRvdyIpIHsKICAgICAgICAgICAgICAgICBwdXNoKEBpbXBs Q29udGVudCwgIiAgICAvLyBGSVhNRTogSGFjayB0byBwcmV2ZW50IHVudXNlZCB2YXJpYWJsZSB3 YXJuaW5nIC0tIHJlbW92ZSBvbmNlIERPTVdpbmRvdyBpbmNsdWRlcyBhIHNldHRhYmxlIHByb3Bl cnR5XG4iKTsKICAgICAgICAgICAgICAgICBwdXNoKEBpbXBsQ29udGVudCwgIiAgICAodm9pZClp bXA7XG4iKTsKLSAgICAgICAgICAgIH0gZWxzaWYgKCRpbnRlcmZhY2VOYW1lID1+IC9eU1ZHUGF0 aFNlZy8gb3IgJGNvZGVHZW5lcmF0b3ItPklzU1ZHQW5pbWF0ZWRUeXBlKCRpbnRlcmZhY2VOYW1l KSkgeworICAgICAgICAgICAgfSBlbHNpZiAoJGNvbnRleHRJbnRlcmZhY2VOYW1lIG5lICIiKSB7 CiAgICAgICAgICAgICAgICAgcHVzaChAaW1wbENvbnRlbnQsICIgICAgQVNTRVJUKGV4ZWMgJiYg ZXhlYy0+ZHluYW1pY0ludGVycHJldGVyKCkpO1xuIik7CiAgICAgICAgICAgICAgICAgcHVzaChA aW1wbENvbnRlbnQsICIgICAgRnJhbWUqIGFjdGl2ZUZyYW1lID0gc3RhdGljX2Nhc3Q8U2NyaXB0 SW50ZXJwcmV0ZXIqPihleGVjLT5keW5hbWljSW50ZXJwcmV0ZXIoKSktPmZyYW1lKCk7XG4iKTsK ICAgICAgICAgICAgICAgICBwdXNoKEBpbXBsQ29udGVudCwgIiAgICBpZiAoIWFjdGl2ZUZyYW1l KVxuICAgICAgICByZXR1cm47XG5cbiIpOwogICAgICAgICAgICAgICAgIHB1c2goQGltcGxDb250 ZW50LCAiICAgIFNWR0RvY3VtZW50RXh0ZW5zaW9ucyogZXh0ZW5zaW9ucyA9IChhY3RpdmVGcmFt ZS0+ZG9jdW1lbnQoKSA/IGFjdGl2ZUZyYW1lLT5kb2N1bWVudCgpLT5hY2Nlc3NTVkdFeHRlbnNp b25zKCkgOiAwKTtcbiIpOwotICAgICAgICAgICAgICAgIHB1c2goQGltcGxDb250ZW50LCAiICAg IGlmIChleHRlbnNpb25zICYmIGV4dGVuc2lvbnMtPmhhc0dlbmVyaWNDb250ZXh0PCRpbnRlcmZh Y2VOYW1lPihpbXApKSB7XG4iKTsKLSAgICAgICAgICAgICAgICBwdXNoKEBpbXBsQ29udGVudCwg IiAgICAgICAgY29uc3QgU1ZHRWxlbWVudCogY29udGV4dCA9IGV4dGVuc2lvbnMtPmdlbmVyaWND b250ZXh0PCRpbnRlcmZhY2VOYW1lPihpbXApO1xuIik7CisgICAgICAgICAgICAgICAgcHVzaChA aW1wbENvbnRlbnQsICIgICAgaWYgKGV4dGVuc2lvbnMgJiYgZXh0ZW5zaW9ucy0+aGFzR2VuZXJp Y0NvbnRleHQ8JGNvbnRleHRJbnRlcmZhY2VOYW1lPihpbXApKSB7XG4iKTsKKyAgICAgICAgICAg ICAgICBwdXNoKEBpbXBsQ29udGVudCwgIiAgICAgICAgY29uc3QgU1ZHRWxlbWVudCogY29udGV4 dCA9IGV4dGVuc2lvbnMtPmdlbmVyaWNDb250ZXh0PCRjb250ZXh0SW50ZXJmYWNlTmFtZT4oaW1w KTtcbiIpOwogICAgICAgICAgICAgICAgIHB1c2goQGltcGxDb250ZW50LCAiICAgICAgICBBU1NF UlQoY29udGV4dCk7XG5cbiIpOwogICAgICAgICAgICAgICAgIHB1c2goQGltcGxDb250ZW50LCAi ICAgICAgICBjb250ZXh0LT5ub3RpZnlBdHRyaWJ1dGVDaGFuZ2UoKTtcbiIpOwogICAgICAgICAg ICAgICAgIHB1c2goQGltcGxDb250ZW50LCAiICAgIH1cblxuIik7CkluZGV4OiBiaW5kaW5ncy9q cy9KU1NWR1BhdGhTZWdMaXN0Q3VzdG9tLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBiaW5kaW5ncy9qcy9K U1NWR1BhdGhTZWdMaXN0Q3VzdG9tLmNwcAkocmV2aXNpb24gMjAxMDMpCisrKyBiaW5kaW5ncy9q cy9KU1NWR1BhdGhTZWdMaXN0Q3VzdG9tLmNwcAkod29ya2luZyBjb3B5KQpAQCAtNTcsMjMgKzU3 LDEzIEBACiAgICAgY29udGV4dC0+bm90aWZ5QXR0cmlidXRlQ2hhbmdlKCk7CiB9CiAKLXN0YXRp YyB2b2lkIHJlbW92ZUZyb21QYXRoU2VnQ29udGV4dE1hcChFeGVjU3RhdGUqIGV4ZWMsIFNWR1Bh dGhTZWdMaXN0KiBsaXN0LCBTVkdQYXRoU2VnKiBvYmopCitzdGF0aWMgdm9pZCByZW1vdmVGcm9t UGF0aFNlZ0NvbnRleHRNYXAoU1ZHUGF0aFNlZ0xpc3QqIGxpc3QsIFNWR1BhdGhTZWcqIG9iaikK IHsKLSAgICBBU1NFUlQoZXhlYyAmJiBleGVjLT5keW5hbWljSW50ZXJwcmV0ZXIoKSk7Ci0gICAg RnJhbWUqIGFjdGl2ZUZyYW1lID0gc3RhdGljX2Nhc3Q8U2NyaXB0SW50ZXJwcmV0ZXIqPihleGVj LT5keW5hbWljSW50ZXJwcmV0ZXIoKSktPmZyYW1lKCk7Ci0gICAgaWYgKCFhY3RpdmVGcmFtZSkK LSAgICAgICAgcmV0dXJuOworCWNvbnN0IFNWR0VsZW1lbnQqIGNvbnRleHQgPSBsaXN0LT5jb250 ZXh0KCk7CisJQVNTRVJUKGNvbnRleHQpOwogCi0gICAgY29uc3QgU1ZHRWxlbWVudCogY29udGV4 dCA9IGxpc3QtPmNvbnRleHQoKTsKLSAgICBBU1NFUlQoY29udGV4dCk7Ci0KLSAgICBTVkdEb2N1 bWVudEV4dGVuc2lvbnMqIGV4dGVuc2lvbnMgPSAoYWN0aXZlRnJhbWUtPmRvY3VtZW50KCkgPyBh Y3RpdmVGcmFtZS0+ZG9jdW1lbnQoKS0+YWNjZXNzU1ZHRXh0ZW5zaW9ucygpIDogMCk7Ci0gICAg aWYgKGV4dGVuc2lvbnMpIHsKLSAgICAgICAgaWYgKGV4dGVuc2lvbnMtPmhhc0dlbmVyaWNDb250 ZXh0PFNWR1BhdGhTZWc+KG9iaikpCi0gICAgICAgICAgICBleHRlbnNpb25zLT5yZW1vdmVHZW5l cmljQ29udGV4dDxTVkdQYXRoU2VnPihvYmopOwotICAgIH0KLQotICAgIGNvbnRleHQtPm5vdGlm eUF0dHJpYnV0ZUNoYW5nZSgpOworCVNWR0RvY3VtZW50RXh0ZW5zaW9uczo6Zm9yZ2V0R2VuZXJp Y0NvbnRleHQob2JqKTsKKwljb250ZXh0LT5ub3RpZnlBdHRyaWJ1dGVDaGFuZ2UoKTsKIH0KIAog SlNWYWx1ZSogSlNTVkdQYXRoU2VnTGlzdDo6Y2xlYXIoRXhlY1N0YXRlKiBleGVjLCBjb25zdCBM aXN0JiBhcmdzKQpAQCAtODQsNyArNzQsNyBAQAogCiAgICAgdW5zaWduZWQgaW50IG5yID0gaW1w LT5udW1iZXJPZkl0ZW1zKCk7CiAgICAgZm9yICh1bnNpZ25lZCBpbnQgaSA9IDA7IGkgPCBucjsg aSsrKQotICAgICAgICByZW1vdmVGcm9tUGF0aFNlZ0NvbnRleHRNYXAoZXhlYywgaW1wLCBpbXAt PmdldEl0ZW0oaSwgZWMpLmdldCgpKTsKKyAgICAgICAgcmVtb3ZlRnJvbVBhdGhTZWdDb250ZXh0 TWFwKGltcCwgaW1wLT5nZXRJdGVtKGksIGVjKS5nZXQoKSk7CiAKICAgICBpbXAtPmNsZWFyKGVj KTsKICAgICBzZXRET01FeGNlcHRpb24oZXhlYywgZWMpOwpAQCAtMTgxLDcgKzE3MSw3IEBACiAg ICAgU1ZHUGF0aFNlZ0xpc3QqIGltcCA9IHN0YXRpY19jYXN0PFNWR1BhdGhTZWdMaXN0Kj4oaW1w bCgpKTsKIAogICAgIFJlZlB0cjxTVkdQYXRoU2VnPiBvYmooaW1wLT5yZW1vdmVJdGVtKGluZGV4 LCBlYykpOwotICAgIHJlbW92ZUZyb21QYXRoU2VnQ29udGV4dE1hcChleGVjLCBpbXAsIG9iai5n ZXQoKSk7CisgICAgcmVtb3ZlRnJvbVBhdGhTZWdDb250ZXh0TWFwKGltcCwgb2JqLmdldCgpKTsK IAogICAgIEtKUzo6SlNWYWx1ZSogcmVzdWx0ID0gdG9KUyhleGVjLCBvYmouZ2V0KCkpOwogICAg IHNldERPTUV4Y2VwdGlvbihleGVjLCBlYyk7CkluZGV4OiBDaGFuZ2VMb2cKPT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot LS0gQ2hhbmdlTG9nCShyZXZpc2lvbiAyMDEwMykKKysrIENoYW5nZUxvZwkod29ya2luZyBjb3B5 KQpAQCAtMSwzICsxLDI4IEBACisyMDA3LTAzLTExICBOaWtvbGFzIFppbW1lcm1hbm4gIDx6aW1t ZXJtYW5uQGtkZS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK KyAgICAgICAgRml4ZXM6IGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEy OTc0CisKKyAgICAgICAgQ2FsbCBmb3JnZXRHZW5lcmljQ29udGV4dCBpbiBKU1NWR1BhdGhTZWcg ZGVzdHJ1Y3Rvciwgb3RoZXJ3aGlzZQorICAgICAgICB3ZSdsbCBoaXQgYW4gQVNTRVJUIGluIGEg ZGVidWcgYnVpbGQsIHdoZW4gcnVubmluZyBzdmcvY3VzdG9tL2pzLXVwZGF0ZS1wYXRoLWNoYW5n ZXMuc3ZnCisgICAgICAgIGEgZmV3IGRvemVuIHRpbWVzIGluIGEgc2luZ2xlIFdlYktpdCBpbnN0 YW5jZS4gVGhlIEFTU0VSVCBpcyBnb29kIGFuZCBqdXN0IHdhcm5zIHRoYXQKKyAgICAgICAgdGhl cmUgd2FzIGFscmVhZHkgYSBnZW5lcmljIGNvbnRleHQgcG9pbnRlciByZWdpc3RlcmVkLCBhbmQg dGhlIG5ldyAidG8gYmUgcmVnaXN0ZXJlZCIKKyAgICAgICAgb2JqZWN0IGFscmVhZHkgZXhpc3Rz LCBidXQgcG9pbnRpbmcgdG8gYSBkaWZmZXJlbnQgb2JqZWN0LiBUaGF0J3MgYmVjYXVzZSBnYXJi YWdlIGNvbGxlY3Rpb24KKyAgICAgICAgY2FsbHMgdGhlIEpTU1ZHUGF0aFNlZyBkZXN0cnVjdG9y LCBidXQgdGhhdCBkaWRuJ3QgY2xlYW51cCB0aGUgZ2VuZXJpYyBjb250ZXh0IG1hcC4KKworICAg ICAgICBPbmx5IEpTU1ZHUGF0aFNlZyBpcyBoaXQgYnkgdGhpcyBwcm9ibGVtLCBhcyBpdCdzIHRo ZSBvbmx5IG5vbiBTVkdBbmltYXRlZCogdHlwZQorICAgICAgICB1c2luZyB0aGUgZ2VuZXJpYyBj b250ZXh0IHN5c3RlbSB3aGlsZSB1c2luZyBhIGN1c3RvbSBKU1NWR1BhdGhTZWdMaXN0IGltcGxl bWVudGF0aW9uLgorCisgICAgICAgIEFsc28gY2xlYW51cCBKU1NWR1BhdGhTZWdMaXN0Q3VzdG9t IGNvZGUgdG8gY2FsbCB0aGUgc3RhdGljIGZvcmdldEdlbmVyaWNDb250ZXh0CisgICAgICAgIG1l dGhvZCBpbnN0ZWFkIG9mIGRvaW5nIHRoZSBzYW1lIHVzaW5nIGN1c3RvbSBjb2RlLgorCisgICAg ICAgICogYmluZGluZ3MvanMvSlNTVkdQYXRoU2VnTGlzdEN1c3RvbS5jcHA6CisgICAgICAgIChX ZWJDb3JlOjpyZW1vdmVGcm9tUGF0aFNlZ0NvbnRleHRNYXApOgorICAgICAgICAoV2ViQ29yZTo6 SlNTVkdQYXRoU2VnTGlzdDo6Y2xlYXIpOgorICAgICAgICAoV2ViQ29yZTo6SlNTVkdQYXRoU2Vn TGlzdDo6cmVtb3ZlSXRlbSk6CisgICAgICAgICogYmluZGluZ3Mvc2NyaXB0cy9Db2RlR2VuZXJh dG9ySlMucG06CisKIDIwMDctMDMtMTEgIEFudHRpIEtvaXZpc3RvICA8YW50dGlAYXBwbGUuY29t PgogCiAgICAgICAgIFJldmlld2VkIGJ5IEh5YXR0Lgo=