16164 2007-11-27 16:18:37 -0800 SVGs with width or height of 100% hang when used as a background-image 2007-11-28 12:20:56 -0800 1 1 1 Unclassified WebKit SVG 528+ (Nightly build) Mac OS X 10.4 RESOLVED FIXED InRadar P2 Normal --- 15373 1 bdakin bdakin bdakin eric oldest_to_newest 62517 0 bdakin 2007-11-27 16:18:37 -0800 <rdar://problem/5616052> 62518 1 17562 bdakin 2007-11-27 16:19:24 -0800 Created attachment 17562 Test case 62519 2 17563 bdakin 2007-11-27 16:19:51 -0800 Created attachment 17563 Sample SVG 62537 3 17566 bdakin 2007-11-27 22:23:23 -0800 Created attachment 17566 Patch This fixes the hang appropriately. There is still a bug that the actual SVG does not render, but that is an existing problem. (I say it is "existing" because using this SVG in an <img> does NOT cause a hang, but it does cause the rendering issue, so this patch strictly addresses the hang.) The rendering issue is filed in radar, and I will file it shortly in Bugzilla as well. 62541 4 17566 oliver 2007-11-27 22:29:34 -0800 Comment on attachment 17566 Patch Woo! beth is a l33t svg hacker now :D 62542 5 bdakin 2007-11-27 22:29:51 -0800 Rendering issue filed as http://bugs.webkit.org/show_bug.cgi?id=16167 62543 6 bdakin 2007-11-27 22:32:04 -0800 Yay! Fixed with r28102. 62591 7 eric 2007-11-28 12:18:45 -0800 FYI: This was also in bugzilla as bug 15638 and I think bug 16167 was the reason why the patch on bug 15638 never landed. 62595 8 eric 2007-11-28 12:20:56 -0800 *** Bug 15638 has been marked as a duplicate of this bug. *** 17562 2007-11-27 16:19:24 -0800 2007-11-27 16:19:24 -0800 Test case svg-as-background.html text/html 231 bdakin PGh0bWw+CjxoZWFkPgo8c3R5bGU+CmRpdiB7CiAgYmFja2dyb3VuZDogcmVkOwogIGJhY2tncm91 bmQtcmVwZWF0OiBuby1yZXBlYXQ7CiAgYmFja2dyb3VuZC1pbWFnZTogdXJsKGV5ZXNDb3B5LnN2 Zyk7CiAgaGVpZ2h0OiA2MDBweDsKICB3aWR0aDogODAwcHg7Cn0KPC9zdHlsZT4KPC9oZWFkPgo8 Ym9keSBzdHlsZT0nbWFyZ2luOiAwcHgnPgo8ZGl2PiZuYnNwOzwvZGl2Pgo8L2JvZHk+CjwvaHRt bD4K 17563 2007-11-27 16:19:51 -0800 2007-11-27 16:19:51 -0800 Sample SVG eyesCopy.svg image/svg+xml 124 bdakin PHN2ZyB3aWR0aD0iMTAwJSIgaGVpZ2h0PSIxMDAlIgoJeG1sbnM9Imh0dHA6Ly93d3cudzMub3Jn LzIwMDAvc3ZnIgogICAgIHhtbG5zOnhsaW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5r Ij4KPC9zdmc+Cg== 17566 2007-11-27 22:23:23 -0800 2007-11-27 22:29:34 -0800 Patch sending.txt text/plain 5284 bdakin SW5kZXg6IFdlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3Mvc3ZnL1NWR0ltYWdlLmNwcAo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBXZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3N2Zy9TVkdJbWFnZS5jcHAJKHJldmlz aW9uIDI4MDk5KQorKysgV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9zdmcvU1ZHSW1hZ2UuY3Bw CSh3b3JraW5nIGNvcHkpCkBAIC04MSwxMSArODEsMTEgQEAKICAgICAKICAgICBJbnRTaXplIHN2 Z1NpemU7CiAgICAgaWYgKHdpZHRoLnVuaXRUeXBlKCkgPT0gTGVuZ3RoVHlwZVBlcmNlbnRhZ2Up Ci0gICAgICAgIHN2Z1NpemUuc2V0V2lkdGgoc3RhdGljX2Nhc3Q8aW50Pih3aWR0aC52YWx1ZUlu U3BlY2lmaWVkVW5pdHMoKSAqIGRlc3RTaXplLndpZHRoKCkpKTsKKyAgICAgICAgc3ZnU2l6ZS5z ZXRXaWR0aChzdGF0aWNfY2FzdDxpbnQ+KHdpZHRoLnZhbHVlQXNQZXJjZW50YWdlKCkgKiBkZXN0 U2l6ZS53aWR0aCgpKSk7CiAgICAgZWxzZQogICAgICAgICBzdmdTaXplLnNldFdpZHRoKHN0YXRp Y19jYXN0PGludD4od2lkdGgudmFsdWUoKSkpOwogICAgIGlmIChoZWlnaHQudW5pdFR5cGUoKSA9 PSBMZW5ndGhUeXBlUGVyY2VudGFnZSkKLSAgICAgICAgc3ZnU2l6ZS5zZXRIZWlnaHQoc3RhdGlj X2Nhc3Q8aW50PihoZWlnaHQudmFsdWVJblNwZWNpZmllZFVuaXRzKCkgKiBkZXN0U2l6ZS5oZWln aHQoKSkpOworICAgICAgICBzdmdTaXplLnNldEhlaWdodChzdGF0aWNfY2FzdDxpbnQ+KGhlaWdo dC52YWx1ZUFzUGVyY2VudGFnZSgpICogZGVzdFNpemUuaGVpZ2h0KCkpKTsKICAgICBlbHNlCiAg ICAgICAgIHN2Z1NpemUuc2V0SGVpZ2h0KHN0YXRpY19jYXN0PGludD4oaGVpZ2h0LnZhbHVlKCkp KTsKICAgICAKSW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUv Q2hhbmdlTG9nCShyZXZpc2lvbiAyODEwMSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5n IGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMDctMTEtMjcgIEJldGggRGFraW4gIDxiZGFraW5A YXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg ICAgIEZpeCBmb3IgPHJkYXI6Ly9wcm9ibGVtLzU2MTYwNTI+IFNWR3Mgd2l0aCB3aWR0aCBvciBo ZWlnaHQgb2YgMTAwJSAKKyAgICAgICAgaGFuZyB3aGVuIHVzZWQgYXMgYSBiYWNrZ3JvdW5kLWlt YWdlICgxNjE2NCkKKworICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL3N2Zy9TVkdJbWFnZS5j cHA6CisgICAgICAgIChXZWJDb3JlOjpTVkdJbWFnZTo6c2l6ZSk6IFVzZSB2YWx1ZUFzUGVyY2Vu dGFnZSgpIGZvciBwZXJjZW50cyAKKyAgICAgICAgaW5zdGVhZCBvZiB2YWx1ZUluU3BlY2lmaWVk VW5pdHMoKS4gdmFsdWVJblNwZWNpZmllZFVuaXRzKCkgCisgICAgICAgIHJldHVybnMsIGZvciBl eGFtcGxlLCA1MCBmb3IgNTAlLCBzbyBtdWx0aXBseWluZyBwZXJjZW50YWdlcyBieSB0aGUgCisg ICAgICAgIGRlZmF1bHQgc2l6ZSBvZiAzMDAgeCAxNTAgbGVkIHRvIHJpZGljdWxvdXNseSBodWdl IGRpbWVuc2lvbnMgZm9yIAorICAgICAgICBTVkcgaW1hZ2VzLCBhbmQgdGhlIHN1YnNlcXVlbnQg aGFuZy4gCisKIDIwMDctMTEtMjcgIEpvbiBIb25leWN1dHQgIDxqaG9uZXljdXR0QGFwcGxlLmNv bT4KIAogICAgICAgICBSZXZpZXdlZCBieSBBZGFtLgpJbmRleDogTGF5b3V0VGVzdHMvcGxhdGZv cm0vbWFjL2Zhc3QvaW1hZ2VzL3N2Zy13aWR0aC0xMDBwLWFzLWJhY2tncm91bmQtZXhwZWN0ZWQu dHh0Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL3BsYXRmb3JtL21hYy9mYXN0L2ltYWdlcy9z dmctd2lkdGgtMTAwcC1hcy1iYWNrZ3JvdW5kLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKKysr IExheW91dFRlc3RzL3BsYXRmb3JtL21hYy9mYXN0L2ltYWdlcy9zdmctd2lkdGgtMTAwcC1hcy1i YWNrZ3JvdW5kLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSw4IEBACitsYXll ciBhdCAoMCwwKSBzaXplIDgwMHg2MDAKKyAgUmVuZGVyVmlldyBhdCAoMCwwKSBzaXplIDgwMHg2 MDAKK2xheWVyIGF0ICgwLDApIHNpemUgODAweDYwMAorICBSZW5kZXJCbG9jayB7SFRNTH0gYXQg KDAsMCkgc2l6ZSA4MDB4NjAwCisgICAgUmVuZGVyQm9keSB7Qk9EWX0gYXQgKDAsMCkgc2l6ZSA4 MDB4NjAwCisgICAgICBSZW5kZXJCbG9jayB7RElWfSBhdCAoMCwwKSBzaXplIDgwMHg2MDAgW2Jn Y29sb3I9I0ZGMDAwMF0KKyAgICAgICAgUmVuZGVyVGV4dCB7I3RleHR9IGF0ICgwLDApIHNpemUg NHgxOAorICAgICAgICAgIHRleHQgcnVuIGF0ICgwLDApIHdpZHRoIDQ6ICIgIgpJbmRleDogTGF5 b3V0VGVzdHMvcGxhdGZvcm0vbWFjL2Zhc3QvaW1hZ2VzL3N2Zy13aWR0aC0xMDBwLWFzLWJhY2tn cm91bmQtZXhwZWN0ZWQucG5nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KQ2Fubm90IGRpc3BsYXk6IGZpbGUgbWFya2Vk IGFzIGEgYmluYXJ5IHR5cGUuCnN2bjptaW1lLXR5cGUgPSBhcHBsaWNhdGlvbi9vY3RldC1zdHJl YW0KClByb3BlcnR5IGNoYW5nZXMgb246IExheW91dFRlc3RzL3BsYXRmb3JtL21hYy9mYXN0L2lt YWdlcy9zdmctd2lkdGgtMTAwcC1hcy1iYWNrZ3JvdW5kLWV4cGVjdGVkLnBuZwpfX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f Ck5hbWU6IHN2bjptaW1lLXR5cGUKICAgKyBhcHBsaWNhdGlvbi9vY3RldC1zdHJlYW0KCkluZGV4 OiBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvZmFzdC9pbWFnZXMvc3ZnLXdpZHRoLTEwMHAtYXMt YmFja2dyb3VuZC1leHBlY3RlZC5jaGVja3N1bQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9w bGF0Zm9ybS9tYWMvZmFzdC9pbWFnZXMvc3ZnLXdpZHRoLTEwMHAtYXMtYmFja2dyb3VuZC1leHBl Y3RlZC5jaGVja3N1bQkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL3BsYXRmb3JtL21hYy9m YXN0L2ltYWdlcy9zdmctd2lkdGgtMTAwcC1hcy1iYWNrZ3JvdW5kLWV4cGVjdGVkLmNoZWNrc3Vt CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxIEBACis0ODNhNDYzZjY5ZDJmZDQ1NDI0OWE0ODEzZDky NWM2YwpcIE5vIG5ld2xpbmUgYXQgZW5kIG9mIGZpbGUKSW5kZXg6IExheW91dFRlc3RzL0NoYW5n ZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDI4MTAx KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYg QEAKKzIwMDctMTEtMjcgIEJldGggRGFraW4gIDxiZGFraW5AYXBwbGUuY29tPgorCisgICAgICAg IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRlc3QgZm9yIDxyZGFyOi8v cHJvYmxlbS81NjE2MDUyPiBTVkdzIHdpdGggd2lkdGggb3IgaGVpZ2h0IG9mIDEwMCUgCisgICAg ICAgIGhhbmcgd2hlbiB1c2VkIGFzIGEgYmFja2dyb3VuZC1pbWFnZSAoMTYxNjQpCisKKyAgICAg ICAgKiBmYXN0L2ltYWdlcy9yZXNvdXJjZXMvd2lkdGgtMTAwLXBlcmNlbnQuc3ZnOiBBZGRlZC4K KyAgICAgICAgKiBmYXN0L2ltYWdlcy9zdmctd2lkdGgtMTAwcC1hcy1iYWNrZ3JvdW5kLmh0bWw6 IEFkZGVkLgorICAgICAgICAqIHBsYXRmb3JtL21hYy9mYXN0L2ltYWdlcy9zdmctd2lkdGgtMTAw cC1hcy1iYWNrZ3JvdW5kLWV4cGVjdGVkLmNoZWNrc3VtOiBBZGRlZC4KKyAgICAgICAgKiBwbGF0 Zm9ybS9tYWMvZmFzdC9pbWFnZXMvc3ZnLXdpZHRoLTEwMHAtYXMtYmFja2dyb3VuZC1leHBlY3Rl ZC5wbmc6IEFkZGVkLgorICAgICAgICAqIHBsYXRmb3JtL21hYy9mYXN0L2ltYWdlcy9zdmctd2lk dGgtMTAwcC1hcy1iYWNrZ3JvdW5kLWV4cGVjdGVkLnR4dDogQWRkZWQuCisKIDIwMDctMTEtMjcg IFRpbW90aHkgSGF0Y2hlciAgPHRpbW90aHlAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2Vk IGJ5IEtldmluIE1jQ3VsbG91Z2guCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2ltYWdlcy9yZXNv dXJjZXMvd2lkdGgtMTAwLXBlcmNlbnQuc3ZnCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zh c3QvaW1hZ2VzL3Jlc291cmNlcy93aWR0aC0xMDAtcGVyY2VudC5zdmcJKHJldmlzaW9uIDApCisr KyBMYXlvdXRUZXN0cy9mYXN0L2ltYWdlcy9yZXNvdXJjZXMvd2lkdGgtMTAwLXBlcmNlbnQuc3Zn CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDQgQEAKKzxzdmcgd2lkdGg9IjEwMCUiIGhlaWdodD0i MTAwJSIKKwl4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciCisgICAgIHhtbG5zOnhs aW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIj4KKzwvc3ZnPgpJbmRleDogTGF5b3V0 VGVzdHMvZmFzdC9pbWFnZXMvc3ZnLXdpZHRoLTEwMHAtYXMtYmFja2dyb3VuZC5odG1sCj09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvaW1hZ2VzL3N2Zy13aWR0aC0xMDBwLWFzLWJhY2tn cm91bmQuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvaW1hZ2VzL3N2Zy13 aWR0aC0xMDBwLWFzLWJhY2tncm91bmQuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwxNiBA QAorPGh0bWw+Cis8aGVhZD4KKzxzdHlsZT4KK2RpdiB7CisgIGJhY2tncm91bmQ6IHJlZDsKKyAg YmFja2dyb3VuZC1yZXBlYXQ6IG5vLXJlcGVhdDsKKyAgYmFja2dyb3VuZC1pbWFnZTogdXJsKHJl c291cmNlcy93aWR0aC0xMDAtcGVyY2VudC5zdmcpOworICBoZWlnaHQ6IDYwMHB4OworICB3aWR0 aDogODAwcHg7Cit9Cis8L3N0eWxlPgorPC9oZWFkPgorPGJvZHkgc3R5bGU9J21hcmdpbjogMHB4 Jz4KKzxkaXY+Jm5ic3A7PC9kaXY+Cis8L2JvZHk+Cis8L2h0bWw+Cg==