9347 2006-06-07 15:19:42 -0700 Border on root (<html>) element causes elements with position: absolute to layout incorrectly 2006-10-17 02:48:27 -0700 1 1 1 Unclassified WebKit New Bugs 420+ Mac OS X 10.4 RESOLVED FIXED P2 Normal --- 1 sam webkit-unassigned ian mrowe oldest_to_newest 45094 0 sam 2006-06-07 15:19:42 -0700 Moz has a similar bug in their bugzilla https://bugzilla.mozilla.org/show_bug.cgi?id=243519 45095 1 8757 sam 2006-06-07 15:20:39 -0700 Created attachment 8757 test case in Strict mode 45096 2 8758 sam 2006-06-07 15:21:01 -0700 Created attachment 8758 test case in Quirks mode 55373 3 hyatt 2006-09-12 23:58:58 -0700 This is a stunning change from CSS2. I do not believe it is compatible with the existing Web either. We should think very carefully before fixing this. 52466 4 11109 hyatt 2006-10-16 01:17:29 -0700 Created attachment 11109 Patch that fixes the containing block for positioned elements to be the RenderView ("initial containing block" in CSS terms) Here we go. A followup improvement I can make after this goes in is to eliminate the RenderLayer for the root element. 52467 5 hyatt 2006-10-16 01:27:07 -0700 Disregard my dramatic comment. I didn't quite understand the 2.0 -> 2.1 spec change. I do now, and it's not that scary. :) 52468 6 11109 mjs 2006-10-16 01:30:15 -0700 Comment on attachment 11109 Patch that fixes the containing block for positioned elements to be the RenderView ("initial containing block" in CSS terms) r=me 52469 7 hyatt 2006-10-16 01:34:37 -0700 Fixed. 52401 8 ddkilzer 2006-10-17 02:48:27 -0700 (In reply to comment #7) > Fixed. Tests in r17067, fix in r17068. 8757 2006-06-07 15:20:39 -0700 2006-06-07 15:20:39 -0700 test case in Strict mode absolute-with-html-border-strict.html text/html 478 sam PCFET0NUWVBFIGh0bWw+DQo8aHRtbD4NCjxoZWFkPg0KICAgIDxzdHlsZSB0eXBlPSJ0ZXh0L2Nz cyI+DQogICAgICAgIGh0bWwgew0KICAgICAgICAgICAgYm9yZGVyOiAxMDBweCBzb2xpZCBibGFj azsNCiAgICAgICAgfQ0KICAgICAgICAjdGVzdCB7DQogICAgICAgICAgICBwb3NpdGlvbjogYWJz b2x1dGU7DQogICAgICAgICAgICBib3R0b206IDA7DQogICAgICAgICAgICByaWdodDogMDsNCiAg ICAgICAgICAgIGhlaWdodDogMjAwcHg7DQogICAgICAgICAgICB3aWR0aDogMjAwcHg7DQogICAg ICAgICAgICBiYWNrZ3JvdW5kOiBncmVlbjsNCiAgICAgICAgfQ0KICAgIDwvc3R5bGU+DQo8L2hl YWQ+DQo8Ym9keT4NCiAgPGRpdiBpZD0idGVzdCI+VGhpcyBzaG91bGQgYmUgaW4gdGhlIGJvdHRv bSByaWdodC1oYW5kIGNvcm5lciBvZiB0aGUgdmlld3BvcnQgZXZlbiB3aGVuIHJlc2l6aW5nLjwv ZGl2Pg0KPC9ib2R5Pg0KPC9odG1sPg== 8758 2006-06-07 15:21:01 -0700 2006-06-07 15:21:01 -0700 test case in Quirks mode absolute-with-html-border-quirks.html text/html 461 sam PGh0bWw+DQo8aGVhZD4NCiAgICA8c3R5bGUgdHlwZT0idGV4dC9jc3MiPg0KICAgICAgICBodG1s IHsNCiAgICAgICAgICAgIGJvcmRlcjogMTAwcHggc29saWQgYmxhY2s7DQogICAgICAgIH0NCiAg ICAgICAgI3Rlc3Qgew0KICAgICAgICAgICAgcG9zaXRpb246IGFic29sdXRlOw0KICAgICAgICAg ICAgYm90dG9tOiAwOw0KICAgICAgICAgICAgcmlnaHQ6IDA7DQogICAgICAgICAgICBoZWlnaHQ6 IDIwMHB4Ow0KICAgICAgICAgICAgd2lkdGg6IDIwMHB4Ow0KICAgICAgICAgICAgYmFja2dyb3Vu ZDogZ3JlZW47DQogICAgICAgIH0NCiAgICA8L3N0eWxlPg0KPC9oZWFkPg0KPGJvZHk+DQogIDxk aXYgaWQ9InRlc3QiPlRoaXMgc2hvdWxkIGJlIGluIHRoZSBib3R0b20gcmlnaHQtaGFuZCBjb3Ju ZXIgb2YgdGhlIHZpZXdwb3J0IGV2ZW4gd2hlbiByZXNpemluZy48L2Rpdj4NCjwvYm9keT4NCjwv aHRtbD4= 11109 2006-10-16 01:17:29 -0700 2006-10-16 01:30:15 -0700 Patch that fixes the containing block for positioned elements to be the RenderView ("initial containing block" in CSS terms) patch.txt text/plain 4744 hyatt SW5kZXg6IHJlbmRlcmluZy9SZW5kZXJPYmplY3QuY3BwCj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHJlbmRlcmlu Zy9SZW5kZXJPYmplY3QuY3BwCShyZXZpc2lvbiAxNzA2NikKKysrIHJlbmRlcmluZy9SZW5kZXJP YmplY3QuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC03NzgsOCArNzc4LDcgQEAKICAgICAgICAgICAg IG8gPSBvLT5wYXJlbnQoKTsKICAgICB9CiAgICAgZWxzZSBpZiAoIWlzVGV4dCgpICYmIG1fc3R5 bGUtPnBvc2l0aW9uKCkgPT0gQWJzb2x1dGVQb3NpdGlvbikgewotICAgICAgICB3aGlsZSAobyAm JiAoby0+c3R5bGUoKS0+cG9zaXRpb24oKSA9PSBTdGF0aWNQb3NpdGlvbiB8fCAoby0+aXNJbmxp bmUoKSAmJiAhby0+aXNSZXBsYWNlZCgpKSkKLSAgICAgICAgICAgICAgICYmICFvLT5pc1Jvb3Qo KSAmJiAhby0+aXNSZW5kZXJWaWV3KCkpIHsKKyAgICAgICAgd2hpbGUgKG8gJiYgKG8tPnN0eWxl KCktPnBvc2l0aW9uKCkgPT0gU3RhdGljUG9zaXRpb24gfHwgKG8tPmlzSW5saW5lKCkgJiYgIW8t PmlzUmVwbGFjZWQoKSkpICYmICFvLT5pc1JlbmRlclZpZXcoKSkgewogICAgICAgICAgICAgLy8g Rm9yIHJlbHBvc2l0aW9uZWQgaW5saW5lcywgd2UgcmV0dXJuIHRoZSBuZWFyZXN0IGVuY2xvc2lu ZyBibG9jay4gIFdlIGRvbid0IHRyeQogICAgICAgICAgICAgLy8gdG8gcmV0dXJuIHRoZSBpbmxp bmUgaXRzZWxmLiAgVGhpcyBhbGxvd3MgdXMgdG8gYXZvaWQgaGF2aW5nIGEgcG9zaXRpb25lZCBv YmplY3RzCiAgICAgICAgICAgICAvLyBsaXN0IGluIGFsbCBSZW5kZXJJbmxpbmVzIGFuZCBsZXRz IHVzIHJldHVybiBhIHN0cm9uZ2x5LXR5cGVkIFJlbmRlckJsb2NrKiByZXN1bHQKQEAgLTI0MDIs NyArMjQwMSw3IEBACiAgICAgICAgIC8vIHdlIG1heSBub3QgaGF2ZSBvbmUgaWYgd2UncmUgcGFy dCBvZiBhbiB1bmluc3RhbGxlZCBzdWJ0cmVlLiAgV2UnbGwKICAgICAgICAgLy8gY2xpbWIgYXMg aGlnaCBhcyB3ZSBjYW4gdGhvdWdoLgogICAgICAgICBvID0gcGFyZW50KCk7Ci0gICAgICAgIHdo aWxlIChvICYmIG8tPnN0eWxlKCktPnBvc2l0aW9uKCkgPT0gU3RhdGljUG9zaXRpb24gJiYgIW8t PmlzUm9vdCgpICYmICFvLT5pc1JlbmRlclZpZXcoKSkKKyAgICAgICAgd2hpbGUgKG8gJiYgby0+ c3R5bGUoKS0+cG9zaXRpb24oKSA9PSBTdGF0aWNQb3NpdGlvbiAmJiAhby0+aXNSZW5kZXJWaWV3 KCkpCiAgICAgICAgICAgICBvID0gby0+cGFyZW50KCk7CiAgICAgfQogICAgIGVsc2UKSW5kZXg6 IHJlbmRlcmluZy9SZW5kZXJCb3guY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHJlbmRlcmluZy9SZW5kZXJC b3guY3BwCShyZXZpc2lvbiAxNzA2NikKKysrIHJlbmRlcmluZy9SZW5kZXJCb3guY3BwCSh3b3Jr aW5nIGNvcHkpCkBAIC0xNDk2LDEwICsxNDk2LDYgQEAKIAogaW50IFJlbmRlckJveDo6Y29udGFp bmluZ0Jsb2NrSGVpZ2h0Rm9yUG9zaXRpb25lZChjb25zdCBSZW5kZXJPYmplY3QqIGNvbnRhaW5p bmdCbG9jaykgY29uc3QKIHsKLSAgICAvLyBFdmVuIGluIHN0cmljdCBtb2RlICh3aGVyZSB3ZSBk b24ndCBncm93IHRoZSByb290IHRvIGZpbGwgdGhlIHZpZXdwb3J0KSBvdGhlciBicm93c2Vycwot ICAgIC8vIHBvc2l0aW9uIGFzIHRob3VnaCB0aGUgcm9vdCBmaWxscyB0aGUgdmlld3BvcnQuCi0g ICAgaWYgKGNvbnRhaW5pbmdCbG9jay0+aXNSb290KCkpCi0gICAgICAgIHJldHVybiBjb250YWlu aW5nQmxvY2stPmF2YWlsYWJsZUhlaWdodCgpOwogICAgIHJldHVybiBjb250YWluaW5nQmxvY2st PmhlaWdodCgpIC0gY29udGFpbmluZ0Jsb2NrLT5ib3JkZXJUb3AoKSAtIGNvbnRhaW5pbmdCbG9j ay0+Ym9yZGVyQm90dG9tKCk7CiB9CiAKQEAgLTE5MjIsMTEgKzE5MTgsNiBAQAogCiAgICAgaGVp Z2h0ICs9IGJvcmRlcnNQbHVzUGFkZGluZzsKIAotICAgIC8vIElmIG91ciBuYXR1cmFsL2NvbnRl bnQgaGVpZ2h0IGV4Y2VlZHMgdGhlIG5ldyBoZWlnaHQgb25jZSB3ZSd2ZSBzZXQgaXQsIHRoZW4g d2UKLSAgICAvLyBuZWVkIHRvIG1ha2Ugc3VyZSB0byB1cGRhdGUgb3ZlcmZsb3cgdG8gdHJhY2sg dGhlIHNwaWxsb3V0LgotICAgIGlmIChtX2hlaWdodCA+IGhlaWdodCkKLSAgICAgICAgc2V0T3Zl cmZsb3dIZWlnaHQobV9oZWlnaHQpOwotCiAgICAgLy8gU2V0IGZpbmFsIGhlaWdodCB2YWx1ZS4K ICAgICBtX2hlaWdodCA9IGhlaWdodDsKIH0KSW5kZXg6IHJlbmRlcmluZy9SZW5kZXJCbG9jay5j cHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PQotLS0gcmVuZGVyaW5nL1JlbmRlckJsb2NrLmNwcAkocmV2aXNpb24gMTcw NjYpCisrKyByZW5kZXJpbmcvUmVuZGVyQmxvY2suY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yMTQ5 LDE0ICsyMTQ5LDE2IEBACiAgICAgICAgIH0KICAgICB9CiAKLSAgICAvLyBGaXhlZCBwb3NpdGlv bmVkIG9iamVjdHMgZG8gbm90IHNjcm9sbCBhbmQgdGh1cyBzaG91bGQgbm90IGNvbnN0aXR1dGUK LSAgICAvLyBwYXJ0IG9mIHRoZSBsb3dlc3QgcG9zaXRpb24uCi0gICAgaWYgKG1fcG9zaXRpb25l ZE9iamVjdHMgJiYgIWlzUmVuZGVyVmlldygpKSB7CisgICAgaWYgKG1fcG9zaXRpb25lZE9iamVj dHMpIHsKICAgICAgICAgUmVuZGVyT2JqZWN0KiByOwogICAgICAgICBEZXByZWNhdGVkUHRyTGlz dEl0ZXJhdG9yPFJlbmRlck9iamVjdD4gaXQoKm1fcG9zaXRpb25lZE9iamVjdHMpOwogICAgICAg ICBmb3IgKCA7IChyID0gaXQuY3VycmVudCgpKTsgKytpdCApIHsKLSAgICAgICAgICAgIGludCBs cCA9IHItPnlQb3MoKSArIHItPmxvd2VzdFBvc2l0aW9uKGZhbHNlKTsKLSAgICAgICAgICAgIGJv dHRvbSA9IG1heChib3R0b20sIGxwKTsKKyAgICAgICAgICAgIC8vIEZpeGVkIHBvc2l0aW9uZWQg b2JqZWN0cyBkbyBub3Qgc2Nyb2xsIGFuZCB0aHVzIHNob3VsZCBub3QgY29uc3RpdHV0ZQorICAg ICAgICAgICAgLy8gcGFydCBvZiB0aGUgbG93ZXN0IHBvc2l0aW9uLgorICAgICAgICAgICAgaWYg KHItPnN0eWxlKCktPnBvc2l0aW9uKCkgIT0gRml4ZWRQb3NpdGlvbikgeworICAgICAgICAgICAg ICAgIGludCBscCA9IHItPnlQb3MoKSArIHItPmxvd2VzdFBvc2l0aW9uKGZhbHNlKTsKKyAgICAg ICAgICAgICAgICBib3R0b20gPSBtYXgoYm90dG9tLCBscCk7CisgICAgICAgICAgICB9CiAgICAg ICAgIH0KICAgICB9CiAKQEAgLTIxODcsMTIgKzIxODksMTYgQEAKICAgICAgICAgfQogICAgIH0K IAotICAgIGlmIChtX3Bvc2l0aW9uZWRPYmplY3RzICYmICFpc1JlbmRlclZpZXcoKSkgeworICAg IGlmIChtX3Bvc2l0aW9uZWRPYmplY3RzKSB7CiAgICAgICAgIFJlbmRlck9iamVjdCogcjsKICAg ICAgICAgRGVwcmVjYXRlZFB0ckxpc3RJdGVyYXRvcjxSZW5kZXJPYmplY3Q+IGl0KCptX3Bvc2l0 aW9uZWRPYmplY3RzKTsKICAgICAgICAgZm9yICggOyAociA9IGl0LmN1cnJlbnQoKSk7ICsraXQg KSB7Ci0gICAgICAgICAgICBpbnQgcnAgPSByLT54UG9zKCkgKyByLT5yaWdodG1vc3RQb3NpdGlv bihmYWxzZSk7Ci0gICAgICAgICAgICByaWdodCA9IG1heChyaWdodCwgcnApOworICAgICAgICAg ICAgLy8gRml4ZWQgcG9zaXRpb25lZCBvYmplY3RzIGRvIG5vdCBzY3JvbGwgYW5kIHRodXMgc2hv dWxkIG5vdCBjb25zdGl0dXRlCisgICAgICAgICAgICAvLyBwYXJ0IG9mIHRoZSByaWdodG1vc3Qg cG9zaXRpb24uCisgICAgICAgICAgICBpZiAoci0+c3R5bGUoKS0+cG9zaXRpb24oKSAhPSBGaXhl ZFBvc2l0aW9uKSB7CisgICAgICAgICAgICAgICAgaW50IHJwID0gci0+eFBvcygpICsgci0+cmln aHRtb3N0UG9zaXRpb24oZmFsc2UpOworICAgICAgICAgICAgICAgIHJpZ2h0ID0gbWF4KHJpZ2h0 LCBycCk7CisgICAgICAgICAgICB9CiAgICAgICAgIH0KICAgICB9CiAKQEAgLTIyMjksMTIgKzIy MzUsMTYgQEAKICAgICAgICAgfQogICAgIH0KICAgICAKLSAgICBpZiAobV9wb3NpdGlvbmVkT2Jq ZWN0cyAmJiAhaXNSZW5kZXJWaWV3KCkpIHsKKyAgICBpZiAobV9wb3NpdGlvbmVkT2JqZWN0cykg ewogICAgICAgICBSZW5kZXJPYmplY3QqIHI7CiAgICAgICAgIERlcHJlY2F0ZWRQdHJMaXN0SXRl cmF0b3I8UmVuZGVyT2JqZWN0PiBpdCgqbV9wb3NpdGlvbmVkT2JqZWN0cyk7CiAgICAgICAgIGZv ciAoIDsgKHIgPSBpdC5jdXJyZW50KCkpOyArK2l0ICkgewotICAgICAgICAgICAgaW50IGxwID0g ci0+eFBvcygpICsgci0+bGVmdG1vc3RQb3NpdGlvbihmYWxzZSk7Ci0gICAgICAgICAgICBsZWZ0 ID0gbWluKGxlZnQsIGxwKTsKKyAgICAgICAgICAgIC8vIEZpeGVkIHBvc2l0aW9uZWQgb2JqZWN0 cyBkbyBub3Qgc2Nyb2xsIGFuZCB0aHVzIHNob3VsZCBub3QgY29uc3RpdHV0ZQorICAgICAgICAg ICAgLy8gcGFydCBvZiB0aGUgbGVmdG1vc3QgcG9zaXRpb24uCisgICAgICAgICAgICBpZiAoci0+ c3R5bGUoKS0+cG9zaXRpb24oKSAhPSBGaXhlZFBvc2l0aW9uKSB7CisgICAgICAgICAgICAgICAg aW50IGxwID0gci0+eFBvcygpICsgci0+bGVmdG1vc3RQb3NpdGlvbihmYWxzZSk7CisgICAgICAg ICAgICAgICAgbGVmdCA9IG1pbihsZWZ0LCBscCk7CisgICAgICAgICAgICB9CiAgICAgICAgIH0K ICAgICB9CiAgICAgCg==