14693 2007-07-20 11:22:41 -0700 document.width/height doesn't force layout 2007-09-16 17:17:27 -0700 1 1 1 Unclassified WebKit Layout and Rendering 523.x (Safari 3) All All RESOLVED FIXED http://jjc.edu HasReduction, InRadar P2 Normal --- 1 dev+webkit webkit-unassigned mitz webkit oldest_to_newest 4184 0 dev+webkit 2007-07-20 11:22:41 -0700 The menu code at <http://jjc.edu/> calculates its positioning based on document.width, which is returning 0 by the time the position is calculated. However, this doesn't happen all the time. For me, if I load the URL in the address bar and hit enter, I can reproduce the bug consistently. If I click the reload button in the toolbar though, it works properly. The attached testcase shows the bug either way, but if I add in an alert(), it works OK after it is dismissed. 4181 1 15604 dev+webkit 2007-07-20 11:23:02 -0700 Created attachment 15604 Reduction 4135 2 dev+webkit 2007-07-20 17:37:41 -0700 Should mention this happens with Safari 2 up to ToT. 4136 3 hyatt 2007-07-20 17:42:39 -0700 Sounds easy to fix. Just need to add an updateLayoutIgnoringPendingStylesheets. 963 4 mitz 2007-09-03 23:28:35 -0700 *** Bug 15137 has been marked as a duplicate of this bug. *** 913 5 16202 sam 2007-09-04 21:45:25 -0700 Created attachment 16202 patch 503 6 16202 mjs 2007-09-13 19:38:46 -0700 Comment on attachment 16202 patch r=me 504 7 mrowe 2007-09-13 19:40:45 -0700 <rdar://problem/5481392> 426 8 sam 2007-09-16 17:17:27 -0700 Landed in r25591. 15604 2007-07-20 11:23:02 -0700 2007-07-20 11:23:02 -0700 Reduction jjc.html text/html 927 dev+webkit PGh0bWw+Cgo8aGVhZD4KICAgIDxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyIgbWVkaWE9InNjcmVlbiI+ CiAgICAgICAgI3BvcE1haW4gewogICAgICAgICAgICBwb3NpdGlvbjogYWJzb2x1dGU7CiAgICAg ICAgICAgIGxlZnQ6IC0xOwogICAgICAgICAgICB0b3A6IDk3OwogICAgICAgICAgICBiYWNrZ3Jv dW5kLWNvbG9yOiAjMDAwMDAwOwogICAgICAgICAgICB3aWR0aDogNjIycHg7CiAgICAgICAgICAg IGhlaWdodDogMjVweDsKICAgICAgICB9CiAgICA8L3N0eWxlPgogICAgPHNjcmlwdCB0eXBlPSJ0 ZXh0L2phdmFzY3JpcHQiIGNoYXJzZXQ9InV0Zi04Ij4KICAgICAgICB2YXIgcG9wbWNoZWlnaHQg PSAyNTsKICAgICAgICB2YXIgcG9wbWN3aWR0aCA9IDYyMjsKICAgICAgICB2YXIgcG9YID0gLTE7 CgogICAgICAgIGZ1bmN0aW9uIHBvcG1jcmVhdGUoKQogICAgICAgIHsKICAgICAgICAgICAgY29u c29sZS5sb2coIkJFRk9SRSAtIGRvY3VtZW50LndpZHRoID0gIiArIGRvY3VtZW50LndpZHRoKTsK CiAgICAgICAgICAgIC8vIFRocm93aW5nIGFuIGFsZXJ0IGNhdXNlcyBTYWZhcmkgdG8gbGF5b3V0 IHByb3Blcmx5CiAgICAgICAgICAgIGFsZXJ0KCIiKTsKICAgICAgICAgICAgcG9wTWFpbiA9IGRv Y3VtZW50LmdldEVsZW1lbnRCeUlkKCJwb3BNYWluIik7CiAgICAgICAgICAgIHBvcE1haW4uc3R5 bGUubGVmdCA9ICgoZG9jdW1lbnQud2lkdGggKyAxNiAtIHBvcG1jd2lkdGgpIC8gMikgKyBwb1g7 CgogICAgICAgICAgICBjb25zb2xlLmxvZygiQUZURVIgLSBkb2N1bWVudC53aWR0aCA9ICIgKyBk b2N1bWVudC53aWR0aCk7CiAgICAgICAgfQogICAgPC9zY3JpcHQ+CjwvaGVhZD4KCjxib2R5IG9u bG9hZD0icG9wbWNyZWF0ZSgpIj4KICAgIDxkaXYgaWQ9InBvcE1haW4iPiZuYnNwOzwvZGl2Pgo8 L2JvZHk+Cgo8L2h0bWw+ 16202 2007-09-04 21:45:25 -0700 2007-09-13 19:38:46 -0700 patch forceDocWidthHeight.diff text/plain 3991 sam SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAyNTM2NCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTcgQEAKKzIwMDctMDktMDQgIFNhbSBXZWluaWcgIDxzYW1Ad2Via2l0Lm9yZz4K KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGaXggZm9y IGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0NjkzCisgICAgICAgIGRv Y3VtZW50LndpZHRoL2hlaWdodCBkb2Vzbid0IGZvcmNlIGxheW91dAorCisgICAgICAgIFRlc3Q6 IGZhc3QvZG9tL2RvY3VtZW50LXdpZHRoLWhlaWdodC1mb3JjZS1sYXlvdXQuaHRtbAorCisgICAg ICAgICogaHRtbC9IVE1MRG9jdW1lbnQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6SFRNTERvY3Vt ZW50Ojp3aWR0aCk6IEZvcmNlIGxheW91dC4KKyAgICAgICAgKFdlYkNvcmU6OkhUTUxEb2N1bWVu dDo6aGVpZ2h0KTogZGl0dG8uCisgICAgICAgICogaHRtbC9IVE1MRG9jdW1lbnQuaDoKKwogMjAw Ny0wOS0wNCAgRGF2aWQgSHlhdHQgIDxoeWF0dEBhcHBsZS5jb20+CiAKICAgICAgICAgRml4IGZv ciA8cmRhcjovL3Byb2JsZW0vNTI3MTIxMz4sIHJlc2l6aW5nIGlDaGF0IHdpbmRvdyBpcyBzbG93 ZXIgdGhhbiBpbgpJbmRleDogV2ViQ29yZS9odG1sL0hUTUxEb2N1bWVudC5jcHAKPT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PQotLS0gV2ViQ29yZS9odG1sL0hUTUxEb2N1bWVudC5jcHAJKHJldmlzaW9uIDI1MzY0KQorKysg V2ViQ29yZS9odG1sL0hUTUxEb2N1bWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEyMCwxNCAr MTIwLDE2IEBAIHZvaWQgSFRNTERvY3VtZW50OjpzZXRCb2R5KEhUTUxFbGVtZW50KiAKICAgICAg ICAgZG9jdW1lbnRFbGVtZW50KCktPnJlcGxhY2VDaGlsZChuZXdCb2R5LCBiLCBlYyk7CiB9CiAK LWludCBIVE1MRG9jdW1lbnQ6OndpZHRoKCkgY29uc3QKK2ludCBIVE1MRG9jdW1lbnQ6OndpZHRo KCkKIHsKKyAgICB1cGRhdGVMYXlvdXRJZ25vcmVQZW5kaW5nU3R5bGVzaGVldHMoKTsKICAgICBG cmFtZVZpZXcqIGZyYW1lVmlldyA9IHZpZXcoKTsKICAgICByZXR1cm4gZnJhbWVWaWV3ID8gZnJh bWVWaWV3LT5jb250ZW50c1dpZHRoKCkgOiAwOwogfQogCi1pbnQgSFRNTERvY3VtZW50OjpoZWln aHQoKSBjb25zdAoraW50IEhUTUxEb2N1bWVudDo6aGVpZ2h0KCkKIHsKKyAgICB1cGRhdGVMYXlv dXRJZ25vcmVQZW5kaW5nU3R5bGVzaGVldHMoKTsKICAgICBGcmFtZVZpZXcqIGZyYW1lVmlldyA9 IHZpZXcoKTsKICAgICByZXR1cm4gZnJhbWVWaWV3ID8gZnJhbWVWaWV3LT5jb250ZW50c0hlaWdo dCgpIDogMDsKIH0KSW5kZXg6IFdlYkNvcmUvaHRtbC9IVE1MRG9jdW1lbnQuaAo9PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 Ci0tLSBXZWJDb3JlL2h0bWwvSFRNTERvY3VtZW50LmgJKHJldmlzaW9uIDI1MzY0KQorKysgV2Vi Q29yZS9odG1sL0hUTUxEb2N1bWVudC5oCSh3b3JraW5nIGNvcHkpCkBAIC00NCw4ICs0NCw4IEBA IHB1YmxpYzoKIAogICAgIHZvaWQgc2V0Qm9keShIVE1MRWxlbWVudCosIEV4Y2VwdGlvbkNvZGUm KTsKIAotICAgIGludCB3aWR0aCgpIGNvbnN0OwotICAgIGludCBoZWlnaHQoKSBjb25zdDsKKyAg ICBpbnQgd2lkdGgoKTsKKyAgICBpbnQgaGVpZ2h0KCk7CiAKICAgICBTdHJpbmcgZGlyKCk7CiAg ICAgdm9pZCBzZXREaXIoY29uc3QgU3RyaW5nJik7CkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VM b2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lvbiAyNTM2NCkK KysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBA CisyMDA3LTA5LTA0ICBTYW0gV2VpbmlnICA8c2FtQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2 aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGVzdCBmb3IgaHR0cDovL2J1Z3Mu d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTQ2OTMKKworICAgICAgICAqIGZhc3QvZG9tL2Rv Y3VtZW50LXdpZHRoLWhlaWdodC1mb3JjZS1sYXlvdXQtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAg ICAgICAgKiBmYXN0L2RvbS9kb2N1bWVudC13aWR0aC1oZWlnaHQtZm9yY2UtbGF5b3V0Lmh0bWw6 IEFkZGVkLgorCiAyMDA3LTA5LTA0ICBTYW0gV2VpbmlnICA8c2FtQHdlYmtpdC5vcmc+CiAKICAg ICAgICAgUmV2aWV3ZWQgYnkgQWRhbS4KSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvZG9tL2RvY3Vt ZW50LXdpZHRoLWhlaWdodC1mb3JjZS1sYXlvdXQtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t IExheW91dFRlc3RzL2Zhc3QvZG9tL2RvY3VtZW50LXdpZHRoLWhlaWdodC1mb3JjZS1sYXlvdXQt ZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvZmFzdC9kb20vZG9jdW1l bnQtd2lkdGgtaGVpZ2h0LWZvcmNlLWxheW91dC1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBA IC0wLDAgKzEsNSBAQAorVGhpcyB0ZXN0cyB0aGF0IGRvY3VtZW50LndpZHRoIGFuZCBkb2N1bWVu dC5oZWlnaHQgY2F1c2UgYSBsYXlvdXQgb24gdXNlLiBUaGUgdGVzdCBoYXMgcGFzc2VkIGlmIHRo ZXkgaGF2ZSBwb3NpdGl2ZSB2YWx1ZXMuCisKK2RvY3VtZW50LndpZHRoID0gODAwCitkb2N1bWVu dC5oZWlnaHQgPSA2MDAKKwpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9kb20vZG9jdW1lbnQtd2lk dGgtaGVpZ2h0LWZvcmNlLWxheW91dC5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zh c3QvZG9tL2RvY3VtZW50LXdpZHRoLWhlaWdodC1mb3JjZS1sYXlvdXQuaHRtbAkocmV2aXNpb24g MCkKKysrIExheW91dFRlc3RzL2Zhc3QvZG9tL2RvY3VtZW50LXdpZHRoLWhlaWdodC1mb3JjZS1s YXlvdXQuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwyMyBAQAorPGh0bWw+Cis8aGVhZD4K KyAgICA8c2NyaXB0PgorICAgICAgICBmdW5jdGlvbiBsb2cobXNnKQorICAgICAgICB7CisgICAg ICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnY29uc29sZScpLmFwcGVuZENoaWxkKGRv Y3VtZW50LmNyZWF0ZVRleHROb2RlKG1zZyArICJcbiIpKTsKKyAgICAgICAgfQorCisgICAgICAg IGZ1bmN0aW9uIHRlc3QoKQorICAgICAgICB7CisgICAgICAgICAgICBpZiAod2luZG93LmxheW91 dFRlc3RDb250cm9sbGVyKQorICAgICAgICAgICAgICAgIGxheW91dFRlc3RDb250cm9sbGVyLmR1 bXBBc1RleHQoKTsKKworICAgICAgICAgICAgbG9nKCJkb2N1bWVudC53aWR0aCA9ICIgKyBkb2N1 bWVudC53aWR0aCk7CisgICAgICAgICAgICBsb2coImRvY3VtZW50LmhlaWdodCA9ICIgKyBkb2N1 bWVudC5oZWlnaHQpOworICAgICAgICB9CisgICAgPC9zY3JpcHQ+Cis8L2hlYWQ+Cis8Ym9keSBv bmxvYWQ9InRlc3QoKSI+CisgICAgPHA+VGhpcyB0ZXN0cyB0aGF0IGRvY3VtZW50LndpZHRoIGFu ZCBkb2N1bWVudC5oZWlnaHQgY2F1c2UgYSBsYXlvdXQgb24gdXNlLiAgVGhlIHRlc3QgaGFzIHBh c3NlZCBpZiB0aGV5IGhhdmUgcG9zaXRpdmUgdmFsdWVzLjwvcD4KKyAgICA8cHJlIGlkPSJjb25z b2xlIj48L3ByZT4KKzwvYm9keT4KKzwvaHRtbD4KXCBObyBuZXdsaW5lIGF0IGVuZCBvZiBmaWxl Cg==