8449 2006-04-17 16:24:19 -0700 REGRESSION: Incomplete repaint of table cell that moved 2006-04-19 11:14:14 -0700 1 1 1 Unclassified WebKit Layout and Rendering 420+ Mac OS X 10.4 RESOLVED FIXED Regression P1 Normal --- 1 mitz webkit-unassigned hyatt oldest_to_newest 39566 0 mitz 2006-04-17 16:24:19 -0700 RenderTableSection::layoutRows() calls repaintDuringLayoutIfMoved() with an oldCellY that the cell lied about, but repaintDuringLayoutIfMoved compares it against the true m_y. Either yPos() should be used in both places or the caller should compensate for the lying. Attaching a test case. 39567 1 7785 mitz 2006-04-17 16:26:12 -0700 Created attachment 7785 Test case Press the Test button, then force a repaint to see what it should really look like. 39597 2 7796 mitz 2006-04-18 03:50:34 -0700 Created attachment 7796 Patch, including change log and manual test 39645 3 7796 hyatt 2006-04-18 14:55:02 -0700 Comment on attachment 7796 Patch, including change log and manual test r=me 39845 4 hyatt 2006-04-19 11:10:06 -0700 This bug fixes the repainting regression with textfields that occurs on google.com. 39846 5 hyatt 2006-04-19 11:14:14 -0700 Fixed. 7785 2006-04-17 16:26:12 -0700 2006-04-17 16:26:12 -0700 Test case 8449 test case.html text/html 498 mitz PGJ1dHRvbiBvbmNsaWNrPSJ0LnN0eWxlLmhlaWdodD0nNDknOyI+VGVzdDwvYnV0dG9uPgo8dGFi bGUgc3R5bGU9ImJhY2tncm91bmQ6IGJsdWU7IiBjZWxsc3BhY2luZz0iMCI+CiAgICA8dHI+CiAg ICAgICAgPHRkPjwvdGQ+CiAgICAgICAgPHRkIHN0eWxlPSJoZWlnaHQ6IDEwcHg7IiBpZD0idCI+ PC90ZD4KICAgIDwvdHI+CiAgICA8dHI+CiAgICAgICAgPHRkIHN0eWxlPSJ3aWR0aDogMTUwcHg7 IGJhY2tncm91bmQ6IHNpbHZlcjsiPgogICAgICAgICAgICA8ZGl2IHN0eWxlPSJiYWNrZ3JvdW5k OiByZWQ7IGhlaWdodDogMjBweDsiPjwvZGl2PgogICAgICAgIDwvdGQ+CiAgICAgICAgPHRkIHN0 eWxlPSJoZWlnaHQ6IDEwMHB4OyI+PC90ZD4KICAgIDwvdHI+CiAgICA8dHI+CiAgICAgICAgPHRk IHN0eWxlPSJiYWNrZ3JvdW5kOiBsaWdodGdyZWVuOyI+PC90ZD4KICAgICAgICA8dGQgc3R5bGU9 ImhlaWdodDogMTAwcHg7Ij48L3RkPgogICAgPC90cj4KPC90YWJsZT4K 7796 2006-04-18 03:50:34 -0700 2006-04-18 14:55:02 -0700 Patch, including change log and manual test 8449_r1.patch text/plain 3001 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAxMzk1MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTQgQEAKKzIwMDYtIyMtIyMgIE1pdHogUGV0dGVsICA8b3BlbmRhcndpbi5vcmdA bWl0enBldHRlbC5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK KyAgICAgICAgLSBmaXggaHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNn aT9pZD04NDQ5CisgICAgICAgICAgUkVHUkVTU0lPTjogSW5jb21wbGV0ZSByZXBhaW50IG9mIHRh YmxlIGNlbGwgdGhhdCBtb3ZlZAorCisgICAgICAgICogbWFudWFsLXRlc3RzL3RhYmxlLWNlbGwt bW92ZS5odG1sOiBBZGRlZC4KKyAgICAgICAgKiByZW5kZXJpbmcvUmVuZGVyVGFibGVTZWN0aW9u LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlJlbmRlclRhYmxlU2VjdGlvbjo6bGF5b3V0Um93cyk6 IFBhc3MgdGhlIHRydWUgb2xkIHkgY29vcmRpbmF0ZSB0byByZXBhaW50RHVyaW5nTGF5b3V0SWZN b3ZlZC4KKwogMjAwNi0wNC0xNyAgQWxleGV5IFByb3NrdXJ5YWtvdiAgPGFwQG55cG9wLmNvbT4K IAogICAgICAgICBSZXZpZXdlZCBieSBEYXJpbi4KSW5kZXg6IFdlYkNvcmUvbWFudWFsLXRlc3Rz L3RhYmxlLWNlbGwtbW92ZS5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvbWFudWFsLXRlc3Rz L3RhYmxlLWNlbGwtbW92ZS5odG1sCShyZXZpc2lvbiAwKQorKysgV2ViQ29yZS9tYW51YWwtdGVz dHMvdGFibGUtY2VsbC1tb3ZlLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsNDYgQEAKKzwh RE9DVFlQRSBIVE1MIFBVQkxJQyAiLS8vVzNDLy9EVEQgSFRNTCA0LjAxLy9FTiIKKyAgICAgICAg Imh0dHA6Ly93d3cudzMub3JnL1RSL2h0bWw0L3N0cmljdC5kdGQiPgorPGh0bWw+Cis8aGVhZD4K KzxzY3JpcHQ+CisgICAgZnVuY3Rpb24gdGVzdCgpCisgICAgeworICAgICAgICBkb2N1bWVudC5n ZXRFbGVtZW50QnlJZCgidCIpLnN0eWxlLmhlaWdodD0nNjBweCc7CisgICAgICAgIGRvY3VtZW50 LmdldEVsZW1lbnRCeUlkKCJzIikuc3R5bGUuaGVpZ2h0PScwJzsKKyAgICB9Cis8L3NjcmlwdD4K KzwvaGVhZD4KKzxib2R5IG9ubG9hZD0ic2V0VGltZW91dCh0ZXN0LCAwKTsiPgorPHRhYmxlIGNl bGxzcGFjaW5nPSIwIiBjZWxscGFkZGluZz0iMCI+CisgICAgPHRyPgorICAgICAgICA8dGQ+PC90 ZD4KKyAgICAgICAgPHRkIHN0eWxlPSJoZWlnaHQ6IDA7IiBpZD0idCI+PC90ZD4KKyAgICA8L3Ry PgorICAgIDx0cj4KKyAgICAgICAgPHRkIHN0eWxlPSJ3aWR0aDogNjBweDsiPgorICAgICAgICAg ICAgPGRpdiBzdHlsZT0iYmFja2dyb3VuZDogZ3JlZW47IGhlaWdodDogNjBweDsiPjwvZGl2Pgor ICAgICAgICA8L3RkPgorICAgICAgICA8dGQgc3R5bGU9ImhlaWdodDogMTgwcHg7Ij48L3RkPgor ICAgIDwvdHI+CisgICAgPHRyPgorICAgICAgICA8dGQgc3R5bGU9ImJhY2tncm91bmQ6IHJlZDsi PjwvdGQ+CisgICAgICAgIDx0ZCBzdHlsZT0iaGVpZ2h0OiA2MHB4OyIgaWQ9InMiPjwvdGQ+Cisg ICAgPC90cj4KKzwvdGFibGU+Cis8aHI+Cis8cD48Yj5CVUcgSUQ6PC9iPiA8YSBocmVmPSJodHRw Oi8vYnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lkPTg0NDkiPkJ1Z3ppbGxh IGJ1ZyA4NDQ5PC9hPgorUkVHUkVTU0lPTjogSW5jb21wbGV0ZSByZXBhaW50IG9mIHRhYmxlIGNl bGwgdGhhdCBtb3ZlZAorPHAgaWQ9InRlc3QiIHN0eWxlPSJiYWNrZ3JvdW5kLWNvbG9yOnNreWJs dWU7IHBhZGRpbmc6M3B4OyI+PGI+U1RFUFMgVE8gVEVTVDo8L2I+IAorICAgIERvIG5vdCByZXNp emUgdGhlIHdpbmRvdywgaGlkZSBpdCwgc2Nyb2xsLCBvciBzd2l0Y2ggdG8gYSBkaWZmZXJlbnQg dGFiLiBJZiB5b3UgZG8sIHJlbG9hZAorICAgIHRoZSB0ZXN0IGJlZm9yZSBwcm9jZWVkaW5nLgor PC9wPgorPHAgaWQ9InN1Y2Nlc3MiIHN0eWxlPSJiYWNrZ3JvdW5kLWNvbG9yOnBhbGVncmVlbjsg cGFkZGluZzozcHg7Ij48Yj5URVNUIFBBU1M6PC9iPiAKK1RoZXJlIHNob3VsZCBiZSBvbmx5IGEg Z3JlZW4gc3F1YXJlIGFib3ZlIHRoZSBsaW5lLCBhbmQgbm8gcmVkLgorPC9wPgorCis8cCBpZD0i ZmFpbHVyZSIgc3R5bGU9ImJhY2tncm91bmQtY29sb3I6I0ZGMzMwMDsgcGFkZGluZzozcHg7Ij48 Yj5URVNUIEZBSUw6PC9iPiAgCitUaGVyZSB3aWxsIGJlIGEgcmVkIHNxdWFyZSBhbmQgYSBncmVl biByZWN0YW5nbGUuCis8L3A+CisKKzwvYm9keT4KKzwvaHRtbD4KSW5kZXg6IFdlYkNvcmUvcmVu ZGVyaW5nL1JlbmRlclRhYmxlU2VjdGlvbi5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9yZW5k ZXJpbmcvUmVuZGVyVGFibGVTZWN0aW9uLmNwcAkocmV2aXNpb24gMTM5NTEpCisrKyBXZWJDb3Jl L3JlbmRlcmluZy9SZW5kZXJUYWJsZVNlY3Rpb24uY3BwCSh3b3JraW5nIGNvcHkpCkBAIC01Mjgs NyArNTI4LDcgQEAgaW50IFJlbmRlclRhYmxlU2VjdGlvbjo6bGF5b3V0Um93cyhpbnQgdAogICAg ICAgICAgICAgfQogICAgICAgICAgICAgCiAgICAgICAgICAgICBpbnQgb2xkQ2VsbFggPSBjZWxs LT54UG9zKCk7Ci0gICAgICAgICAgICBpbnQgb2xkQ2VsbFkgPSBjZWxsLT55UG9zKCk7CisgICAg ICAgICAgICBpbnQgb2xkQ2VsbFkgPSBjZWxsLT55UG9zKCkgLSBjZWxsLT5ib3JkZXJUb3BFeHRy YSgpOwogICAgICAgICAKICAgICAgICAgICAgIGlmIChzdHlsZSgpLT5kaXJlY3Rpb24oKSA9PSBS VEwpIHsKICAgICAgICAgICAgICAgICBjZWxsLT5zZXRQb3MoCg==