5789 2005-11-21 02:03:43 -0800 iframe with height in % within table is not rendered. 2010-06-11 17:22:05 -0700 1 1 1 Unclassified WebKit Layout and Rendering 420+ Mac OS X 10.4 RESOLVED DUPLICATE 15359 http://www.akafoe.de HasReduction, InRadar P2 Major --- 1 roberto.avanzi webkit-unassigned alice.barraclough ap henk.kampman hyatt ian mozillabug321 pedro sam oldest_to_newest 24346 0 roberto.avanzi 2005-11-21 02:03:43 -0800 The webpage http://www.akafoe.de (anyway horribly built) is not rendered. In particular, I cannot see some "frames" that appear with Opera (9.0pr1), but "cut off" in the lower part, and that render fine with Firefox (1.5) or Camino (1.0b1). Most likely the cause is a duplicate bug, but cannot track it down. 24396 1 joost 2005-11-22 13:57:23 -0800 Confirmed, reduced and renamed, uploading testcase in a sec. 24397 2 4770 joost 2005-11-22 13:58:28 -0800 Created attachment 4770 file needed for testcase This file is needed for the testcase i'll upload in a sec. 24398 3 4771 joost 2005-11-22 13:59:53 -0800 Created attachment 4771 Testcase this is a minimal testcase for the bug, using the previously uploaded file. 26886 4 eric 2005-12-27 16:22:19 -0800 I swear I saw a bug just like this earlier going through bugs, but I can't find it now. 26925 5 joost 2005-12-27 23:57:32 -0800 *** Bug 5592 has been marked as a duplicate of this bug. *** 26927 6 joost 2005-12-27 23:59:34 -0800 Upping the priority on this one, since even 100% height iframe don't get rendered and this bug has been reported on more pages, will enhance the testcase a bit in a second. 28182 7 alice.barraclough 2006-01-10 14:03:49 -0800 <rdar://problem/3184552> 37205 8 sam 2006-03-21 15:02:44 -0800 This is actually a bug for all replaced elements (subclasses of RenderReplaced in the the code) in webcore, including images. It seems to stem from the fact that we do not correctly honor the scenerio when a percentage height of an object depends on its containing blocks height, which in turn can only be calculated using the content height of the inner object. My reading of the correct behavior is to have the percentage height compute to the value of 'auto' and then perceed as normal (in the case of an image where there is an intrinsic width and height, by using the specified width in conjuction with the intrinsic ratio (h/w) to calculate the height, or with an iframe, by setting the height to 150px), as can be seen in the section 10.5 of the CSS 2.1 spec. (http://www.w3.org/TR/CSS21/visudet.html#the-height-property). Sidenote: Firefoxes behavior here does not conform with what I am interpreting the spec says, but instead uses the percentage value given to height to give a computed value that is that percent of the size that it should be. I don't know what to make of that. 37206 9 7222 sam 2006-03-21 15:06:02 -0800 Created attachment 7222 image testcase test case with image instread of iframe 37209 10 sam 2006-03-21 15:11:33 -0800 Another thing I should mention is that since this only occurs when the height of the containing block is base on the height of its contents, setting a height for the <td> cell in which the image or iframe resides, causes the problem to disappear. 37372 11 hyatt 2006-03-22 15:54:05 -0800 This should not be p1. Putting in p2. 37373 12 hyatt 2006-03-22 15:54:43 -0800 P1 = crashers/regressions. :) 37706 13 7342 sam 2006-03-27 15:15:14 -0800 Created attachment 7342 patch This patch matches the horizontal behavior and in addition to the new test case in the patch, changes the following test cases: fast/replaced/width100percent.html tables/mozilla/bugs/bug137388-2.html tables/mozilla_expected_failures/bugs/bug137388-1.html The last two, however, still do not display correctly as they are missing the requisite files. 39577 14 7342 hyatt 2006-04-17 17:47:53 -0700 Comment on attachment 7342 patch Seems like this will do the wrong thing when a height of 0px is explicitly specified. <div style="height:0px"><img style="height:50%"></div> 40063 15 sam 2006-04-21 13:45:49 -0700 (In reply to comment #14) Hyatt, Your absolutly correct that a container height of 0px would result in the incorrect behavior. This was done to match the horizontal behavior which is also wrong. Just test the horizontal corollary to your example, <div style="width: 0px"><img style="height:50%"></div> and notice that the image is not 0 width. A better/correct behavior would be to check if the containing block has a non-auto height instead of a positive height. Do you think that width should be changed as well? 47545 16 ap 2006-11-02 10:58:15 -0800 *** Bug 10643 has been marked as a duplicate of this bug. *** 71532 17 ddkilzer 2008-02-22 16:47:05 -0800 Most test cases in this bug were fixed by Bug 15359. Sam's test case in Comment #15 doesn't render zero-width, but it does render the same in WebKit r30458 as in Opera 9.2x, Firefox 2 and Firefox 3. I suggest closing this as RESOLVED/FIXED (or marking a duplicate of Bug 15359). 237169 18 ap 2010-06-11 17:22:05 -0700 As David said. Please file separate bugs for any remaining issues, if any. *** This bug has been marked as a duplicate of bug 15359 *** 4770 2005-11-22 13:58:28 -0800 2005-11-22 13:58:28 -0800 file needed for testcase testcase2.html text/html 114 joost PGh0bWw+Cjxib2R5PgoJPGRpdiBzdHlsZT0id2lkdGg6IDQwMHB4OyBoZWlnaHQ6IDQwMHB4OyBi YWNrZ3JvdW5kLWNvbG9yOiAjRjAwOyI+CgkJJm5ic3A7Cgk8L2Rpdj4KPC9ib2R5Pgo8L2h0bWw+ 4771 2005-11-22 13:59:53 -0800 2005-11-22 13:59:53 -0800 Testcase www.akafoe.de.html text/html 315 joost PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEgVHJhbnNpdGlvbmFs Ly9FTiI+CjxodG1sPgo8aGVhZD4KPC9oZWFkPgo8Ym9keT4KICAgIDx0YWJsZT4KICAgICAgICA8 dHI+CiAgICAgICAgICAgIDx0ZD4KICAgICAgICAgICAgICAgIDxpZnJhbWUgaGVpZ2h0PSI5MCUi IHdpZHRoPSI2MDAiIHNyYz0iaHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL2F0dGFjaG1l bnQuY2dpP2lkPTQ3NzAiPjwvaWZyYW1lPgogICAgICAgICAgICA8L3RkPgogICAgICAgIDwvdHI+ CiAgICA8L3RhYmxlPgo8L2JvZHk+CjwvaHRtbD4K 7222 2006-03-21 15:06:02 -0800 2006-03-21 15:06:02 -0800 image testcase percent-height-in-table-cell.html text/html 297 sam PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEgVHJhbnNpdGlvbmFs Ly9FTiI+CjxodG1sPgo8aGVhZD4KPC9oZWFkPgo8Ym9keT4KICAgIDx0YWJsZT4KICAgICAgICA8 dHI+CiAgICAgICAgICAgIDx0ZD4KICAgICAgICAgICAgICAgIDxpbWcgaGVpZ2h0PSI5MCUiIHdp ZHRoPSI2MDBweCIgc3JjPSJodHRwOi8vd3d3Lmdvb2dsZS5jb20vaW50bC9lbi9pbWFnZXMvbG9n by5naWYiPgogICAgICAgICAgICA8L3RkPgogICAgICAgIDwvdHI+CiAgICA8L3RhYmxlPgo8L2Jv ZHk+CjwvaHRtbD4K 7342 2006-03-27 15:15:14 -0800 2010-06-10 15:29:17 -0700 patch real3-27.diff text/plain 2649 sam SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDEzNDk5KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAKKzIwMDYtMDMtMjcgIFNhbSBXZWluaWcgIDxzYW0u d2VpbmlnQGdtYWlsLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K KworICAgICAgICAtIHRlc3QgZm9yIDxodHRwOi8vYnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hv d19idWcuY2dpP2lkPTU3ODk+CisgICAgICAgIGlmcmFtZSB3aXRoIGhlaWdodCBpbiAlIHdpdGhp biB0YWJsZSBpcyBub3QgcmVuZGVyZWQuCisKKyAgICAgICAgKiBmYXN0L3JlcGxhY2VkL3BlcmNl bnQtaGVpZ2h0LWluLWNlbGwuaHRtbDogQWRkZWQuCisKIDIwMDYtMDMtMjcgIEFsZXhhbmRlciBL ZWxsZXR0ICA8bHlwYW5vdkBrZGUub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IGRhcmluCklu ZGV4OiBMYXlvdXRUZXN0cy9mYXN0L3JlcGxhY2VkL3BlcmNlbnQtaGVpZ2h0LWluLWNlbGwuaHRt bAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L3JlcGxhY2VkL3BlcmNlbnQtaGVpZ2h0 LWluLWNlbGwuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvcmVwbGFjZWQv cGVyY2VudC1oZWlnaHQtaW4tY2VsbC5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDE0IEBA Cis8IURPQ1RZUEUgSFRNTCBQVUJMSUMgIi0vL1czQy8vRFREIEhUTUwgNC4wMSBUcmFuc2l0aW9u YWwvL0VOIj4KKzxodG1sPgorPGhlYWQ+Cis8L2hlYWQ+Cis8Ym9keT4KKyAgICA8dGFibGU+Cisg ICAgICAgIDx0cj4KKyAgICAgICAgICAgIDx0ZD4KKyAgICAgICAgICAgICAgICA8aWZyYW1lIGhl aWdodD0iNTAlIiB3aWR0aD0iNjAwcHgiIHNyYz0icmVzb3VyY2VzLzAwNy1zdWIuaHRtbCI+PC9p ZnJhbWU+CisgICAgICAgICAgICA8L3RkPgorICAgICAgICA8L3RyPgorICAgIDwvdGFibGU+Cis8 L2JvZHk+Cis8L2h0bWw+CkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJCb3guY3BwCj09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT0KLS0tIFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJveC5jcHAJKHJldmlzaW9uIDEz NDk1KQorKysgV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyQm94LmNwcAkod29ya2luZyBjb3B5KQpA QCAtMTI5OSw4ICsxMjk5LDExIEBAIGludCBSZW5kZXJCb3g6OmNhbGNSZXBsYWNlZEhlaWdodFVz aW5nKEgKICAgICBzd2l0Y2ggKGgudHlwZSgpKSB7CiAgICAgICAgIGNhc2UgRml4ZWQ6CiAgICAg ICAgICAgICByZXR1cm4gY2FsY0NvbnRlbnRCb3hIZWlnaHQoaC52YWx1ZSgpKTsKLSAgICAgICAg Y2FzZSBQZXJjZW50OgotICAgICAgICAgICAgcmV0dXJuIGNhbGNDb250ZW50Qm94SGVpZ2h0KGgu Y2FsY1ZhbHVlKGNvbnRhaW5pbmdCbG9jaygpLT5hdmFpbGFibGVIZWlnaHQoKSkpOworICAgICAg ICBjYXNlIFBlcmNlbnQ6IHsKKyAgICAgICAgICAgIGNvbnN0IGludCBjb250YWluZXJIZWlnaHQg PSBjb250YWluaW5nQmxvY2soKS0+YXZhaWxhYmxlSGVpZ2h0KCk7CisgICAgICAgICAgICBpZiAo Y29udGFpbmVySGVpZ2h0ID4gMCkKKyAgICAgICAgICAgICAgICByZXR1cm4gY2FsY0NvbnRlbnRC b3hIZWlnaHQoaC5jYWxjVmFsdWUoY29udGFpbmVySGVpZ2h0KSk7CisgICAgICAgIH0KICAgICAg ICAgZGVmYXVsdDoKICAgICAgICAgICAgIHJldHVybiBpbnRyaW5zaWNIZWlnaHQoKTsKICAgICB9 CkluZGV4OiBXZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL0NoYW5nZUxv ZwkocmV2aXNpb24gMTM0OTgpCisrKyBXZWJDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpA QCAtMSwzICsxLDE0IEBACisyMDA2LTAzLTI3ICBTYW0gV2VpbmlnICA8c2FtLndlaW5pZ0BnbWFp bC5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg LSBmaXhlZCA8aHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD01 Nzg5PgorICAgICAgICBpZnJhbWUgd2l0aCBoZWlnaHQgaW4gJSB3aXRoaW4gdGFibGUgaXMgbm90 IHJlbmRlcmVkLgorCisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlckJveC5jcHA6CisgICAgICAg IChXZWJDb3JlOjpSZW5kZXJCb3g6OmNhbGNSZXBsYWNlZEhlaWdodFVzaW5nKTogQ2hlY2sgdGhh dCB0aGUgCisgICAgICAgIGNvbnRhaW5lckhlaWdodCBpcyBncmVhdGVyIHRoYW4gMCB3aGVuIGNv bXB1dGluZyBwZXJjZW50YWdlIGhlaWdodHMuCisKIDIwMDYtMDMtMTUgIERhcmluIEFkbGVyICA8 ZGFyaW5AYXBwbGUuY29tPgogCiAgICAgICAgIEF0IHRoZSByZXF1ZXN0IG9mIE1pY2hhZWwgRW1t ZWwgPG1pa2UuZW1tZWxAZ21haWwuY29tPi4K