6301 2005-12-30 04:14:56 -0800 REGRESSION: Border repaint glitch 2006-01-31 21:20:52 -0800 1 1 1 Unclassified WebKit Layout and Rendering 420+ Mac OS X 10.4 VERIFIED FIXED P1 Normal --- 1 mitz webkit-unassigned oldest_to_newest 27233 0 mitz 2005-12-30 04:14:56 -0800 Summary: fieldsets don't repaint completely when their size changes, so parts of their border are either missing or left behind. To reproduce: open the testcase in Safari. Click each Test button once. Expected: the bottom border to be painted correctly after resize. Actual: in the top case (shrinking), a segment is missing on each side of the bottom border; in the bottom case (growing), a segment from each side of the bottom border remains behind. 27234 1 5377 mitz 2005-12-30 04:15:30 -0800 Created attachment 5377 testcase 27235 2 mitz 2005-12-30 04:21:16 -0800 Actually, this is a problem with borders in general, a DIV with solid border works just as well in the testcase. 27239 3 5380 mitz 2005-12-30 05:37:01 -0800 Created attachment 5380 Repaint moving border Like with bug 5699, an automatic test is currently impossible. 27252 4 5380 darin 2005-12-30 11:30:39 -0800 Comment on attachment 5380 Repaint moving border What about borderLeft/Top? 27254 5 mitz 2005-12-30 11:44:16 -0800 (In reply to comment #4) > What about borderLeft/Top? The left/top borders don't move so there's no need to repaint them. You're in this case: // We didn't move, but we did change size. Invalidate the delta, which will consist of possibly // two rectangles (but typically only one). You only need to extend this to include the area that was/will be occupied by the border that moves. 27268 6 5380 darin 2005-12-30 13:30:40 -0800 Comment on attachment 5380 Repaint moving border r=me 29398 7 joost 2006-01-22 04:56:50 -0800 Removing keyword(s) since bug is fixed. 29422 8 joost 2006-01-22 05:01:12 -0800 Removing keyword(s) since bug is fixed. 30887 9 eric 2006-01-31 21:20:52 -0800 Removing Regression keyword from bugs already fixed. 5377 2005-12-30 04:15:30 -0800 2005-12-30 04:15:30 -0800 testcase fieldset-repaint.html text/html 880 mitz PCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEgVHJhbnNpdGlvbmFs Ly9FTiI+CjxodG1sPgo8aGVhZD4KICAgIDx0aXRsZT4mbHQ7RklFTERTRVQmZ3Q7IHJlcGFpbnQg Z2xpdGNoPC90aXRsZT4KICAgIDxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyI+CiAgICAgICAgZGl2LnRl c3QgeyB3aWR0aDogMzAwcHg7IGhlaWdodDogMjAwcHg7IGJvcmRlcjogMXB4IHNvbGlkIGJsdWU7 IHBhZGRpbmc6IDEwcHg7IH0KICAgIDwvc3R5bGU+CjwvaGVhZD4KPGJvZHk+CiAgICA8cD5GaWVs ZHNldCByZXBhaW50IGdsaXRjaDwvcD4KICAgIDxkaXYgY2xhc3M9InRlc3QiPgogICAgICAgIDxi dXR0b24gb25jbGljaz0iZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2EnKS5zdHlsZS5oZWlnaHQg PSA0MCI+VGVzdDwvYnV0dG9uPgogICAgICAgIDxmaWVsZHNldCBzdHlsZT0iYmFja2dyb3VuZDog dGVhbDsiPgogICAgICAgICAgICA8ZGl2IGlkPSJhIiBzdHlsZT0iaGVpZ2h0OiAxMDBweDsgbWFy Z2luOiAyMHB4OyBiYWNrZ3JvdW5kOnNpbHZlcjsiPjwvZGl2PgogICAgICAgIDwvZmllbGRzZXQ+ CiAgICA8L2Rpdj4KICAgIDxicj4KICAgIDxkaXYgY2xhc3M9InRlc3QiPgogICAgICAgIDxidXR0 b24gb25jbGljaz0iZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2InKS5zdHlsZS5oZWlnaHQgPSAx MDAiPlRlc3Q8L2J1dHRvbj4KICAgICAgICA8ZmllbGRzZXQgc3R5bGU9ImJhY2tncm91bmQ6IHRl YWw7Ij4KICAgICAgICAgICAgPGRpdiBpZD0iYiIgc3R5bGU9ImhlaWdodDogNDBweDsgbWFyZ2lu OiAyMHB4OyBiYWNrZ3JvdW5kOnNpbHZlcjsiPjwvZGl2PgogICAgICAgIDwvZmllbGRzZXQ+CiAg ICA8L2Rpdj4KPC9ib2R5Pgo8L2h0bWw+Cg== 5380 2005-12-30 05:37:01 -0800 2005-12-30 13:30:40 -0800 Repaint moving border 6301_r1.patch text/plain 4616 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZpbGU6IC9jdnMvcm9vdC9X ZWJDb3JlL0NoYW5nZUxvZyx2CnJldHJpZXZpbmcgcmV2aXNpb24gMS42NQpkaWZmIC1wIC11IC1y MS42NSBXZWJDb3JlL0NoYW5nZUxvZwotLS0gV2ViQ29yZS9DaGFuZ2VMb2cJMzAgRGVjIDIwMDUg MDk6NDg6NTUgLTAwMDAJMS42NQorKysgV2ViQ29yZS9DaGFuZ2VMb2cJMzAgRGVjIDIwMDUgMTM6 NDA6MTggLTAwMDAKQEAgLTEsMyArMSwxNSBAQAorMjAwNi0wMS0jIyAgTWl0eiBQZXR0ZWwgIDxv cGVuZGFyd2luLm9yZ0BtaXR6cGV0dGVsLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKyAgICAgICAgCisgICAgICAgIC0gZml4IGh0dHA6Ly9idWd6aWxsYS5vcGVu ZGFyd2luLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjMwMQorICAgICAgICAgIFJFR1JFU1NJT046IEJv cmRlciByZXBhaW50IGdsaXRjaAorCisgICAgICAgICoga2h0bWwvcmVuZGVyaW5nL3JlbmRlcl9v YmplY3QuY3BwOgorICAgICAgICAoUmVuZGVyT2JqZWN0OjpyZXBhaW50QWZ0ZXJMYXlvdXRJZk5l ZWRlZCk6IFJlcGFpbnQgcGFydHMgb2YgdGhlCisgICAgICAgIGJvcmRlciB0aGF0IG1vdmVkLgor ICAgICAgICAqIG1hbnVhbC10ZXN0cy9ib3JkZXItcmVwYWludC1nbGl0Y2guaHRtbDogQWRkZWQu CisKIDIwMDUtMTItMzAgIEVyaWMgU2VpZGVsICA8ZXNlaWRlbEBhcHBsZS5jb20+CiAKICAgICAg ICAgUmV2aWV3ZWQgYnkgbWpzLgpJbmRleDogV2ViQ29yZS9raHRtbC9yZW5kZXJpbmcvcmVuZGVy X29iamVjdC5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogL2N2cy9yb290L1dlYkNvcmUva2h0bWwv cmVuZGVyaW5nL3JlbmRlcl9vYmplY3QuY3BwLHYKcmV0cmlldmluZyByZXZpc2lvbiAxLjIzMgpk aWZmIC1wIC11IC1yMS4yMzIgV2ViQ29yZS9raHRtbC9yZW5kZXJpbmcvcmVuZGVyX29iamVjdC5j cHAKLS0tIFdlYkNvcmUva2h0bWwvcmVuZGVyaW5nL3JlbmRlcl9vYmplY3QuY3BwCTI3IERlYyAy MDA1IDE5OjEzOjQwIC0wMDAwCTEuMjMyCisrKyBXZWJDb3JlL2todG1sL3JlbmRlcmluZy9yZW5k ZXJfb2JqZWN0LmNwcAkzMCBEZWMgMjAwNSAxMzo0MDozMiAtMDAwMApAQCAtMTQ0MiwxNiArMTQ0 MiwxNiBAQCBib29sIFJlbmRlck9iamVjdDo6cmVwYWludEFmdGVyTGF5b3V0SWZOCiAgICAgICAg IC8vIHR3byByZWN0YW5nbGVzIChidXQgdHlwaWNhbGx5IG9ubHkgb25lKS4KICAgICAgICAgaW50 IHdpZHRoID0gYWJzKG5ld0JvdW5kcy53aWR0aCgpIC0gb2xkQm91bmRzLndpZHRoKCkpOwogICAg ICAgICBpZiAod2lkdGgpCi0gICAgICAgICAgICBjLT5yZXBhaW50Vmlld1JlY3RhbmdsZShRUmVj dChrTWluKG5ld0JvdW5kcy54KCkgKyBuZXdCb3VuZHMud2lkdGgoKSwgb2xkQm91bmRzLngoKSAr IG9sZEJvdW5kcy53aWR0aCgpKSwKKyAgICAgICAgICAgIGMtPnJlcGFpbnRWaWV3UmVjdGFuZ2xl KFFSZWN0KGtNaW4obmV3Qm91bmRzLngoKSArIG5ld0JvdW5kcy53aWR0aCgpLCBvbGRCb3VuZHMu eCgpICsgb2xkQm91bmRzLndpZHRoKCkpIC0gYm9yZGVyUmlnaHQoKSwKICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgIG5ld0JvdW5kcy55KCksIAotICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgd2lkdGgsCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICB3aWR0aCArIGJvcmRlclJpZ2h0KCksCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICBrTWF4KG5ld0JvdW5kcy5oZWlnaHQoKSwgb2xkQm91bmRzLmhlaWdodCgpKSkpOwog ICAgICAgICBpbnQgaGVpZ2h0ID0gYWJzKG5ld0JvdW5kcy5oZWlnaHQoKSAtIG9sZEJvdW5kcy5o ZWlnaHQoKSk7CiAgICAgICAgIGlmIChoZWlnaHQpCiAgICAgICAgICAgICBjLT5yZXBhaW50Vmll d1JlY3RhbmdsZShRUmVjdChuZXdCb3VuZHMueCgpLAotICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAga01pbihuZXdCb3VuZHMueSgpICsgbmV3Qm91bmRzLmhlaWdodCgp LCBvbGRCb3VuZHMueSgpICsgb2xkQm91bmRzLmhlaWdodCgpKSwKKyAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgIGtNaW4obmV3Qm91bmRzLnkoKSArIG5ld0JvdW5kcy5o ZWlnaHQoKSwgb2xkQm91bmRzLnkoKSArIG9sZEJvdW5kcy5oZWlnaHQoKSkgLSBib3JkZXJCb3R0 b20oKSwKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGtNYXgobmV3 Qm91bmRzLndpZHRoKCksIG9sZEJvdW5kcy53aWR0aCgpKSwKLSAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgIGhlaWdodCkpOworICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgaGVpZ2h0ICsgYm9yZGVyQm90dG9tKCkpKTsKICAgICAgICAgcmV0 dXJuIGZhbHNlOwogICAgIH0KICAgICByZXR1cm4gdHJ1ZTsKSW5kZXg6IFdlYkNvcmUvbWFudWFs LXRlc3RzL2JvcmRlci1yZXBhaW50LWdsaXRjaC5odG1sCj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KZGlmZiAtTnB1IFdl YkNvcmUvbWFudWFsLXRlc3RzL2JvcmRlci1yZXBhaW50LWdsaXRjaC5odG1sCi0tLSBXZWJDb3Jl L21hbnVhbC10ZXN0cy9ib3JkZXItcmVwYWludC1nbGl0Y2guaHRtbAkxIEphbiAxOTcwIDAwOjAw OjAwIC0wMDAwCisrKyBXZWJDb3JlL21hbnVhbC10ZXN0cy9ib3JkZXItcmVwYWludC1nbGl0Y2gu aHRtbAkzMCBEZWMgMjAwNSAxMzoyNDoyMSAtMDAwMApAQCAtMCwwICsxLDQyIEBACis8IURPQ1RZ UEUgSFRNTCBQVUJMSUMgIi0vL1czQy8vRFREIEhUTUwgNC4wMS8vRU4iCisgICAgICAgICJodHRw Oi8vd3d3LnczLm9yZy9UUi9odG1sNC9zdHJpY3QuZHRkIj4KKzxodG1sIGxhbmc9ImVuIj4KKzxo ZWFkPgorPHN0eWxlIHR5cGU9InRleHQvY3NzIj4KKyAgICBkaXYudGVzdCB7IHdpZHRoOiAyMDBw eDsgaGVpZ2h0OiAxMjBweDsgYm9yZGVyOiAxcHggc29saWQgYmx1ZTsgcGFkZGluZzogMTBweDsg fQorPC9zdHlsZT4KKzwvaGVhZD4KKzxib2R5PgorPHA+PGI+QlVHIElEOjwvYj4gPGEgaHJlZj0i aHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD02MzAxIj5CdWd6 aWxsYSBidWcgNjMwMTwvYT4gUkVHUkVTU0lPTjogQm9yZGVyIHJlcGFpbnQgZ2xpdGNoPC9wPgor Cis8cCBpZD0idGVzdCIgc3R5bGU9ImJhY2tncm91bmQtY29sb3I6c2t5Ymx1ZTsgcGFkZGluZzoz cHg7Ij48Yj5TVEVQUyBUTyBURVNUOjwvYj4gCitDbGljayBlYWNoIFRlc3QgYnV0dG9uIG9uY2Uu Cis8L3A+CisKKzxwIGlkPSJzdWNjZXNzIiBzdHlsZT0iYmFja2dyb3VuZC1jb2xvcjpwYWxlZ3Jl ZW47IHBhZGRpbmc6M3B4OyI+PGI+VEVTVCBQQVNTOjwvYj4gCitUaGUgYm94ZXMgd2lsbCByZXNp emUsIG1haW50YWluaW5nIHRoZWlyIHNvbGlkIHJlZCByZWN0YW5ndWxhciBib3JkZXJzLgorPC9w PgorCis8cCBpZD0iZmFpbHVyZSIgc3R5bGU9ImJhY2tncm91bmQtY29sb3I6I0ZGMzMwMDsgcGFk ZGluZzozcHg7Ij48Yj5URVNUIEZBSUw6PC9iPiAgCitUaGUgYm94ZXMgd2lsbCByZXNpemUsIGJ1 dCB0aGUgYm9yZGVycyB3aWxsIGJyZWFrLiBUaGUgZmlyc3QgYm94IHdpbGwgYmUgbWlzc2luZwor c21hbGwgc2VnbWVudHMgZnJvbSBlaXRoZXIgc2lkZSBvZiBpdHMgYm90dG9tIGJvcmRlci4gVGhl IHNlY29uZCBib3ggd2lsbCBoYXZlCitzbWFsbCByZWQgaG9yaXpvbnRhbCBzZWdtZW50cyBwcm90 cnVkaW5nIGlud2FyZHMgZnJvbSBib3RoIHNpZGVzIG9mIHRoZSBib3JkZXIsCit3aGVyZSB0aGUg Ym90dG9tIGJvcmRlciB3YXMgYmVmb3JlLgorPC9wPgorCis8ZGl2IGNsYXNzPSJ0ZXN0Ij4KKyAg ICA8YnV0dG9uIG9uY2xpY2s9ImRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdhJykuc3R5bGUuaGVp Z2h0ID0gJzQwcHgnIj5UZXN0PC9idXR0b24+CisgICAgPGRpdiBzdHlsZT0iYmFja2dyb3VuZDog dGVhbDsgcGFkZGluZzogMDsgYm9yZGVyOiBzb2xpZCByZWQgNHB4OyI+CisgICAgICAgIDxkaXYg aWQ9ImEiIHN0eWxlPSJoZWlnaHQ6IDYwcHg7IG1hcmdpbjogMTBweDsgYmFja2dyb3VuZDpzaWx2 ZXI7Ij48L2Rpdj4KKyAgICA8L2Rpdj4KKzwvZGl2PgorPGJyPgorPGRpdiBjbGFzcz0idGVzdCI+ CisgICAgPGJ1dHRvbiBvbmNsaWNrPSJkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnYicpLnN0eWxl LmhlaWdodCA9ICc2MHB4JyI+VGVzdDwvYnV0dG9uPgorICAgIDxkaXYgc3R5bGU9ImJhY2tncm91 bmQ6IHRlYWw7IHBhZGRpbmc6IDA7IGJvcmRlcjogc29saWQgcmVkIDRweDsiPgorICAgICAgICA8 ZGl2IGlkPSJiIiBzdHlsZT0iaGVpZ2h0OiA0MHB4OyBtYXJnaW46IDEwcHg7IGJhY2tncm91bmQ6 c2lsdmVyOyI+PC9kaXY+CisgICAgPC9kaXY+Cis8L2Rpdj4KKworPC9ib2R5PgorPC9odG1sPgo=