9504 2006-06-19 03:39:06 -0700 img tag rewritten by innerHTML doesn't use image map 2010-07-13 10:09:57 -0700 1 1 1 Unclassified WebKit DOM 420+ Mac All RESOLVED FIXED P2 Normal --- 1 mhinks ap abarth alex ap don.marks emacemac7 eric ian mg05182-webkit shashidaulatabad webkit.review.bot oldest_to_newest 46260 0 mhinks 2006-06-19 03:39:06 -0700 Test case #1: http://www.m-s-d.net/test.htm Test case #2: http://www.m-s-d.net/test2.htm Test case #1 shows a <span> tag being rewritten to contain an img tag with a usemap property and a <map> tag complete with areas. The write works when called from anything other than the function called when the imagemap is clicked. ie. it works on page load, but rewriting the tag via a function called from a click on the imagemap does not. After the rewrite called from within the imagemap click the zonemap does not work. Test case #2 shows a <span> tag being rewritten to contain an img tag with a usemap property. The map is stored inline with the document after the span tag and not generated dynamically. In this case the zonemap never functions correctly. These behaviours work as expected in Firefox and Internet Explorer (tested on WinXP SP2) 46264 1 mhinks 2006-06-19 03:52:56 -0700 My mistake: test case #2 now shows a working example thereby showing that it is the rewrite of the map that causes the failure. (In reply to comment #0) > Test case #1: http://www.m-s-d.net/test.htm > Test case #2: http://www.m-s-d.net/test2.htm > > Test case #1 shows a <span> tag being rewritten to contain an img tag with a > usemap property and a <map> tag complete with areas. The write works when > called from anything other than the function called when the imagemap is > clicked. ie. it works on page load, but rewriting the tag via a function called > from a click on the imagemap does not. After the rewrite called from within the > imagemap click the zonemap does not work. > > Test case #2 shows a <span> tag being rewritten to contain an img tag with a > usemap property. The map is stored inline with the document after the span tag > and not generated dynamically. In this case the zonemap never functions > correctly. > > These behaviours work as expected in Firefox and Internet Explorer (tested on > WinXP SP2) > 16026 2 ap 2007-03-31 12:23:18 -0700 These test cases are now longer available. Reporter, could you please attach them to the bug? 15910 3 mhinks 2007-04-01 09:32:20 -0700 > Test case #1: http://www.m-s-d.net/test.htm > Test case #2: http://www.m-s-d.net/test2.htm > Now at: Test case #1: http://www.2bitpie.net/test.htm Test case #2: http://www.2bitpie.net/test2.htm 100632 4 ap 2008-11-28 13:38:44 -0800 These test cases are now longer available. Reporter, could you please attach them to the bug? 126001 5 ap 2009-06-15 12:21:19 -0700 *** Bug 26277 has been marked as a duplicate of this bug. *** 126002 6 ap 2009-06-15 12:22:21 -0700 Bug 26277 has a test attached. 160418 7 don.marks 2009-11-04 05:49:41 -0800 This problem can be worked around by renaming the old map to something inconsequential before it is replaced. For example: map.name = 'garbage'; mapparent.innerHTML = '<map name="mymap" ...'; 160449 8 don.marks 2009-11-04 08:38:54 -0800 I should add that even though this workaround seems to do the job, there is likely still a small memory leak of a reference to the now-deleted map being kept around. If the map is renamed each time to a unique name, then an even bigger memory leak would likely occur. 235541 9 ap 2010-06-08 14:52:54 -0700 See also: bug 19678. 236082 10 ap 2010-06-09 16:46:27 -0700 *** Bug 19678 has been marked as a duplicate of this bug. *** 236094 11 58314 ap 2010-06-09 17:23:06 -0700 Created attachment 58314 proposed fix I think that the workaround from comment 7 is a good one, there shouldn't be any memory leak. 236163 12 ap 2010-06-09 22:33:45 -0700 Committed <http://trac.webkit.org/changeset/60937>. 236169 13 webkit.review.bot 2010-06-09 23:01:52 -0700 http://trac.webkit.org/changeset/60937 might have broken SnowLeopard Intel Release (Tests) 250589 14 ap 2010-07-13 10:09:57 -0700 Actually landed the test at last, <http://trac.webkit.org/changeset/63217>. 58314 2010-06-09 17:23:06 -0700 2010-06-09 18:51:43 -0700 proposed fix ReplacedMap.txt text/plain 6237 ap SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA2MDkyMykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMjMgQEAKKzIwMTAtMDYtMDkgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEBhcHBs ZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk1MDQKKyAgICAgICAgaW1n IHRhZyByZXdyaXR0ZW4gYnkgaW5uZXJIVE1MIGRvZXNuJ3QgdXNlIGltYWdlIG1hcAorCisgICAg ICAgIFRlc3Q6IGZhc3QvZG9tL3JlcGxhY2VkLWltYWdlLW1hcC5odG1sCisKKyAgICAgICAgKiBo dG1sL0hUTUxNYXBFbGVtZW50LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkhUTUxNYXBFbGVtZW50 Ojp+SFRNTE1hcEVsZW1lbnQpOiBEZXN0cnVjdG9yIGlzIG5vdCBhIGdvb2QgcGxhY2UgdG8gbWFr ZSBjaGFuZ2VzCisgICAgICAgIHRoYXQgYWZmZWN0IHBhZ2UgYmVoYXZpb3IsIGJlY2F1c2UgZGVz dHJ1Y3RvcnMgYXJlIGNhbGxlZCBieSBnYXJiYWdlIGNvbGxlY3Rvci4gRG9uJ3QKKyAgICAgICAg dW5kcmVnaXN0ZXIgdGhlIGltYWdlIG1hcCBmcm9tIGhlcmUuCisgICAgICAgIChXZWJDb3JlOjpI VE1MTWFwRWxlbWVudDo6cGFyc2VNYXBwZWRBdHRyaWJ1dGUpOiBPbmx5IHRlbGwgZG9jdW1lbnQg YWJvdXQgdGhlIG1hcCBpZiBpdCdzCisgICAgICAgIGFjdHVhbGx5IGluIGRvY3VtZW50LiBBIG1h cCBpbiBhIGRldGFjaGVkIHN1YnRyZWUgaXNuJ3QgdXNlZCBieSBGaXJlZm94IGF0IGxlYXN0Lgor ICAgICAgICAoV2ViQ29yZTo6SFRNTE1hcEVsZW1lbnQ6Omluc2VydGVkSW50b0RvY3VtZW50KTog UmVnaXN0ZXIgdGhlIG1hcC4KKyAgICAgICAgKFdlYkNvcmU6OkhUTUxNYXBFbGVtZW50OjpyZW1v dmVkRnJvbURvY3VtZW50KTogVW5yZWdpc3RlciB0aGUgbWFwLgorCisgICAgICAgICogaHRtbC9I VE1MTWFwRWxlbWVudC5oOiBBZGRlZCBpbnNlcnRlZEludG9Eb2N1bWVudC9yZW1vdmVkRnJvbURv Y3VtZW50IG92ZXJyaWRlcy4KKwogMjAxMC0wNi0wOSAgVG9ueSBHZW50aWxjb3JlICA8dG9ueWdA Y2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEFkYW0gQmFydGguCkluZGV4OiBX ZWJDb3JlL2h0bWwvSFRNTE1hcEVsZW1lbnQuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvaHRt bC9IVE1MTWFwRWxlbWVudC5jcHAJKHJldmlzaW9uIDYwODY2KQorKysgV2ViQ29yZS9odG1sL0hU TUxNYXBFbGVtZW50LmNwcAkod29ya2luZyBjb3B5KQpAQCAtNTYsNyArNTYsNiBAQCBQYXNzUmVm UHRyPEhUTUxNYXBFbGVtZW50PiBIVE1MTWFwRWxlbWVuCiAKIEhUTUxNYXBFbGVtZW50Ojp+SFRN TE1hcEVsZW1lbnQoKQogewotICAgIGRvY3VtZW50KCktPnJlbW92ZUltYWdlTWFwKHRoaXMpOwog fQogCiBib29sIEhUTUxNYXBFbGVtZW50OjpjaGVja0RURChjb25zdCBOb2RlKiBuZXdDaGlsZCkK QEAgLTExNiwxMiArMTE1LDE0IEBAIHZvaWQgSFRNTE1hcEVsZW1lbnQ6OnBhcnNlTWFwcGVkQXR0 cmlidXQKICAgICAgICAgICAgIGlmIChkb2MtPmlzSFRNTERvY3VtZW50KCkpCiAgICAgICAgICAg ICAgICAgcmV0dXJuOwogICAgICAgICB9Ci0gICAgICAgIGRvYy0+cmVtb3ZlSW1hZ2VNYXAodGhp cyk7CisgICAgICAgIGlmIChpbkRvY3VtZW50KCkpCisgICAgICAgICAgICBkb2MtPnJlbW92ZUlt YWdlTWFwKHRoaXMpOwogICAgICAgICBTdHJpbmcgbWFwTmFtZSA9IGF0dHItPnZhbHVlKCk7CiAg ICAgICAgIGlmIChtYXBOYW1lWzBdID09ICcjJykKICAgICAgICAgICAgIG1hcE5hbWUgPSBtYXBO YW1lLnN1YnN0cmluZygxKTsKICAgICAgICAgbV9uYW1lID0gZG9jLT5pc0hUTUxEb2N1bWVudCgp ID8gbWFwTmFtZS5sb3dlcigpIDogbWFwTmFtZTsKLSAgICAgICAgZG9jLT5hZGRJbWFnZU1hcCh0 aGlzKTsKKyAgICAgICAgaWYgKGluRG9jdW1lbnQoKSkKKyAgICAgICAgICAgIGRvYy0+YWRkSW1h Z2VNYXAodGhpcyk7CiAgICAgfSBlbHNlCiAgICAgICAgIEhUTUxFbGVtZW50OjpwYXJzZU1hcHBl ZEF0dHJpYnV0ZShhdHRyKTsKIH0KQEAgLTE0MSw0ICsxNDIsMTYgQEAgdm9pZCBIVE1MTWFwRWxl bWVudDo6c2V0TmFtZShjb25zdCBTdHJpbgogICAgIHNldEF0dHJpYnV0ZShuYW1lQXR0ciwgdmFs dWUpOwogfQogCit2b2lkIEhUTUxNYXBFbGVtZW50OjppbnNlcnRlZEludG9Eb2N1bWVudCgpCit7 CisgICAgZG9jdW1lbnQoKS0+YWRkSW1hZ2VNYXAodGhpcyk7CisgICAgSFRNTEVsZW1lbnQ6Omlu c2VydGVkSW50b0RvY3VtZW50KCk7Cit9CisKK3ZvaWQgSFRNTE1hcEVsZW1lbnQ6OnJlbW92ZWRG cm9tRG9jdW1lbnQoKQoreworICAgIGRvY3VtZW50KCktPnJlbW92ZUltYWdlTWFwKHRoaXMpOwor ICAgIEhUTUxFbGVtZW50OjpyZW1vdmVkRnJvbURvY3VtZW50KCk7Cit9CisKIH0KSW5kZXg6IFdl YkNvcmUvaHRtbC9IVE1MTWFwRWxlbWVudC5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvaHRtbC9I VE1MTWFwRWxlbWVudC5oCShyZXZpc2lvbiA2MDg2NikKKysrIFdlYkNvcmUvaHRtbC9IVE1MTWFw RWxlbWVudC5oCSh3b3JraW5nIGNvcHkpCkBAIC01Niw2ICs1Niw5IEBAIHByaXZhdGU6CiAKICAg ICB2aXJ0dWFsIHZvaWQgcGFyc2VNYXBwZWRBdHRyaWJ1dGUoQXR0cmlidXRlKik7CiAKKyAgICB2 aXJ0dWFsIHZvaWQgaW5zZXJ0ZWRJbnRvRG9jdW1lbnQoKTsKKyAgICB2aXJ0dWFsIHZvaWQgcmVt b3ZlZEZyb21Eb2N1bWVudCgpOworCiAgICAgQXRvbWljU3RyaW5nIG1fbmFtZTsKIH07CiAKSW5k ZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFu Z2VMb2cJKHJldmlzaW9uIDYwOTIzKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5n IGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTAtMDYtMDkgIEFsZXhleSBQcm9za3VyeWFrb3Yg IDxhcEBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk1MDQKKyAg ICAgICAgaW1nIHRhZyByZXdyaXR0ZW4gYnkgaW5uZXJIVE1MIGRvZXNuJ3QgdXNlIGltYWdlIG1h cAorCisgICAgICAgICogZmFzdC9kb20vcmVwbGFjZWQtaW1hZ2UtbWFwLWV4cGVjdGVkLnR4dDog QWRkZWQuCisgICAgICAgICogZmFzdC9kb20vcmVwbGFjZWQtaW1hZ2UtbWFwLmh0bWw6IEFkZGVk LgorCiAyMDEwLTA2LTA5ICBBZGFtIEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAKICAgICAg ICAgUmV2aWV3ZWQgYnkgRXJpYyBTZWlkZWwuCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2RvbS9y ZXBsYWNlZC1pbWFnZS1tYXAtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3Rz L2Zhc3QvZG9tL3JlcGxhY2VkLWltYWdlLW1hcC1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisr KyBMYXlvdXRUZXN0cy9mYXN0L2RvbS9yZXBsYWNlZC1pbWFnZS1tYXAtZXhwZWN0ZWQudHh0CShy ZXZpc2lvbiAwKQpAQCAtMCwwICsxLDUgQEAKK1Rlc3QgZm9yIGJ1ZyA5NTA0OiBpbWcgdGFnIHJl d3JpdHRlbiBieSBpbm5lckhUTUwgZG9lc24ndCB1c2UgaW1hZ2UgbWFwLgorCitTVUNDRVNTCisK KwoKUHJvcGVydHkgY2hhbmdlcyBvbjogTGF5b3V0VGVzdHMvZmFzdC9kb20vcmVwbGFjZWQtaW1h Z2UtbWFwLWV4cGVjdGVkLnR4dApfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCk5hbWU6IHN2bjptaW1lLXR5cGUKICAgKyB0 ZXh0L3BsYWluCk5hbWU6IHN2bjplb2wtc3R5bGUKICAgKyBuYXRpdmUKCkluZGV4OiBMYXlvdXRU ZXN0cy9mYXN0L2RvbS9yZXBsYWNlZC1pbWFnZS1tYXAuaHRtbAo9PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlv dXRUZXN0cy9mYXN0L2RvbS9yZXBsYWNlZC1pbWFnZS1tYXAuaHRtbAkocmV2aXNpb24gMCkKKysr IExheW91dFRlc3RzL2Zhc3QvZG9tL3JlcGxhY2VkLWltYWdlLW1hcC5odG1sCShyZXZpc2lvbiAw KQpAQCAtMCwwICsxLDQ0IEBACis8IURPQ1RZUEUgaHRtbCBQVUJMSUMgIi0vL1czQy8vRFREIFhI VE1MIDEuMCBTdHJpY3QvL0VOIiAiaHR0cDovL3d3dy53My5vcmcvVFIveGh0bWwxL0RURC94aHRt bDEtc3RyaWN0LmR0ZCI+Cis8aHRtbCB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMTk5OS94aHRt bCI+Cis8aGVhZD4KKzx0aXRsZT5JbWFnZSBtYXAgYnVnPC90aXRsZT4KKzxzY3JpcHQgdHlwZT0i dGV4dC9qYXZhc2NyaXB0Ij4KK2lmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisgICAg bGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCgpOworCitmdW5jdGlvbiBjaGFuZ2VNYXAo KSB7CisJdmFyIGkgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnaW1nJyk7CisJdmFyIGggPSBk b2N1bWVudC5nZXRFbGVtZW50QnlJZCgnbWFwSG9sZGVyJyk7CisJaS51c2VNYXAgPSAnJzsKKwlo LmlubmVySFRNTCA9ICc8bWFwIG5hbWU9Im1hcCI+PGFyZWEgc2hhcGU9InJlY3QiIGNvb3Jkcz0i MCwwLDMwMCwyMDAiIGFsdD0iIiBocmVmPSJqYXZhc2NyaXB0OnN1Y2Nlc3MoKSI+PC9tYXA+JzsK KwlpLnVzZU1hcCA9ICcjbWFwJzsKK30KKworZnVuY3Rpb24gdGVzdCgpIHsKKyAgICBjaGFuZ2VN YXAoKTsKKyAgICBpZiAod2luZG93LmV2ZW50U2VuZGVyKSB7CisgICAgICAgIGV2ZW50U2VuZGVy Lm1vdXNlTW92ZVRvKDEwMCwgMTAwKTsKKyAgICAgICAgZXZlbnRTZW5kZXIubW91c2VEb3duKCk7 CisgICAgICAgIGV2ZW50U2VuZGVyLm1vdXNlVXAoKTsKKyAgICB9IGVsc2UKKyAgICAgICAgZG9j dW1lbnQuZ2V0RWxlbWVudEJ5SWQoImxvZyIpLmlubmVySFRNTCA9ICJQbGVhc2UgY2xpY2sgdGhl IGltYWdlIjsKK30KKworZnVuY3Rpb24gc3VjY2VzcygpIHsKKyAgICBkb2N1bWVudC5nZXRFbGVt ZW50QnlJZCgibG9nIikuaW5uZXJIVE1MID0gIlNVQ0NFU1MiOworfQorCitmdW5jdGlvbiBmYWls KCkgeworICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJsb2ciKS5pbm5lckhUTUwgPSAiRkFJ TCI7Cit9Cis8L3NjcmlwdD4KKzwvaGVhZD4KKzxib2R5IG9ubG9hZD0idGVzdCgpIj4KKzxwPlRl c3QgZm9yIDxhIGhyZWY9Imh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05 NTA0Ij5idWcgOTUwNDwvYT46IGltZyB0YWcgcmV3cml0dGVuIGJ5IGlubmVySFRNTCBkb2Vzbid0 IHVzZSBpbWFnZSBtYXAuPC9wPgorPGRpdiBpZD1sb2c+VGVzdGluZy4uLjwvZGl2PgorPGRpdj4K KzxpbWcgaWQ9ImltZyIgd2lkdGg9IjMwMCIgaGVpZ2h0PSIyMDAiIHVzZW1hcD0iI21hcCIgYWx0 PSIiIHNyYz0iZGF0YTppbWFnZS9wbmc7YmFzZTY0LGlWQk9SdzBLR2dvQUFBQU5TVWhFVWdBQUFB RUFBQUFCQ0FJQUFBQ1FkMVBlQUFBQUFYTlNSMElBcnM0YzZRQUFBQXhKUkVGVUNOZGpZS2huQUFB QkFnQ0FiVjd0WndBQUFBQkpSVTVFcmtKZ2dnPT0iLz4KKzwvZGl2PgorPGRpdiBpZD0ibWFwSG9s ZGVyIj48bWFwIG5hbWU9Im1hcCI+PGFyZWEgc2hhcGU9InJlY3QiIGNvb3Jkcz0iMCwwLDMwMCwy MDAiIGFsdD0iIiB0aXRsZT0iRmlyc3QiIGhyZWY9ImphdmFzY3JpcHQ6ZmFpbCgpIj48L21hcD48 L2Rpdj4KKzwvYm9keT4KKzwvaHRtbD4KClByb3BlcnR5IGNoYW5nZXMgb246IExheW91dFRlc3Rz L2Zhc3QvZG9tL3JlcGxhY2VkLWltYWdlLW1hcC5odG1sCl9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTmFtZTogc3ZuOm1p bWUtdHlwZQogICArIHRleHQvaHRtbAoK