16339 2007-12-07 03:44:21 -0800 Frame scrollbar positions are not updated on page scroll 2008-03-22 15:22:09 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) PC Linux RESOLVED FIXED http://www.sharemanija.net Gtk P1 Normal --- 1 sandshrew marco.barisione alp nilesh.patil pierre-luc.beaudoin sandshrew oldest_to_newest 63578 0 sandshrew 2007-12-07 03:44:21 -0800 The chatbox scrollbar is detached from its position when scrolling down the page. I am currently using webkit r28388. I set priority to P1, because I don't remember having this issue with webkit, which I had downloaded from svn at 2007.10.21 (sorry, can't say for sure, which version it was). Webkit Qt doesn't have this problem. Here is a screenshot, where the bug is clearly visible: http://img100.imageshack.us/img100/6501/shdd7.png 64861 1 alp 2007-12-20 13:40:27 -0800 This is a very old bug that nobody's got round to fixing yet. It's due to the slightly messy code in ScrollViewGtk and/or the platform scrollbar impl. 71136 2 19208 pierre-luc.beaudoin 2008-02-19 07:29:01 -0800 Created attachment 19208 Test case This bug only affects iframe's scrollbars. This demo file has the two types of scrollbar we could encounter in a page (that I know). Only the iframe is affected. To see the bug with this file, your browser needs to be less than 500px high. 74253 3 19864 marco.barisione 2008-03-18 08:21:31 -0700 Created attachment 19864 Add a missing const so that ScrollViewScrollbar::geometryChanged() is called instead of Widget::geometryChanged() 74254 4 19864 darin 2008-03-18 08:31:48 -0700 Comment on attachment 19864 Add a missing const so that ScrollViewScrollbar::geometryChanged() is called instead of Widget::geometryChanged() r=me 74255 5 pierre-luc.beaudoin 2008-03-18 09:09:12 -0700 This patch has a side effect: While the scrollbar is correctly moved when scrolling, the iframe's content is not correctly repainted on http://www.sharemanija.net . In details, the content is not redrawn, but if you select the content of the iframe you seen that the content has moved... I suggest waiting a little before committing it. 74436 6 pierre-luc.beaudoin 2008-03-19 08:00:24 -0700 (In reply to comment #5) > This patch has a side effect: Ok, this is totally unrelated to the fix, I can reproduce the issue with and without the patch. See Bug #17949. 74773 7 mjs 2008-03-22 00:49:46 -0700 *** Bug 17342 has been marked as a duplicate of this bug. *** 74826 8 mrowe 2008-03-22 15:22:09 -0700 Landed in r31235. 19208 2008-02-19 07:29:01 -0800 2008-02-19 07:29:01 -0800 Test case iframe.html text/html 1414 pierre-luc.beaudoin PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMCBUcmFuc2l0aW9uYWwv L0VOIiAiaHR0cDovL3d3dy53My5vcmcvVFIvaHRtbDQvbG9vc2UuZHRkIj4NCjxodG1sIGxhbmc9 ImVuIj4NCjxoZWFkPg0KPHRpdGxlPlBvc2l0aW9uaW5nIGFuZCBTaXppbmcgdGhlIElmcmFtZTwv dGl0bGU+DQo8bWV0YSBodHRwLWVxdWl2PSJDb250ZW50LVR5cGUiIGNvbnRlbnQ9InRleHQvaHRt bDsgY2hhcnNldD1pc28tODg1OS0xIiAvPg0KPG1ldGEgbmFtZT0icm9ib3RzIiBjb250ZW50PSJu b2luZGV4LG5vZm9sbG93Ij4NCjxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyI+CmJvZHkgeyBoZWlnaHQ6 IDUwMHB4OyB9DQpkaXYuaWZyYW1lIHsgdGV4dC1hbGlnbjpjZW50ZXI7IH0NCmlmcmFtZSNpZnJt IHsgd2lkdGg6ODAlOyB9Ci5zY3JvbGxiYXJzIHsgbWFyZ2luOiBhdXRvOyBoZWlnaHQ6IDEwMHB4 OyBvdmVyZmxvdzpzY3JvbGw7IHdpZHRoOiA1MDBweH0NCjwvc3R5bGU+DQo8c2NyaXB0IHNyYz0i L2pzL2R3X3ZpZXdwb3J0LmpzIiB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPjwvc2NyaXB0Pg0KPC9o ZWFkPg0KPGJvZHk+DQo8ZGl2IGNsYXNzPSJpZnJhbWUiPg0KPGlmcmFtZSBuYW1lPSJpZnJtIiBp ZD0iaWZybSIgc3JjPSJodHRwOi8vd3d3Lmdvb2dsZS5jb20iIGZyYW1lYm9yZGVyPSIwIj4NClNv cnJ5LCB5b3VyIGJyb3dzZXIgZG9lc24ndCBzdXBwb3J0IGlmcmFtZXMuIEEgZGVtb25zdHJhdGlv biBvZiBhIDxhIGhyZWY9ImZsdWlkMS5odG1sIj5jZW50ZXJlZCBmbHVpZCBpZnJhbWU8L2E+IHdv dWxkIGJlIHZpc2libGUgaGVyZSBpZiB5b3Ugd2VyZSB1c2luZyBhIGNhcGFibGUgYnJvd3Nlci4N CjwvaWZyYW1lPg0KPC9kaXY+DQo8YnI+PGJyPg0KPGRpdiBjbGFzcz0ic2Nyb2xsYmFycyI+Ckxv bmcgY29udGVudCA8YnI+TG9uZyBjb250ZW50IDxicj5Mb25nIGNvbnRlbnQgPGJyPkxvbmcgY29u dGVudCA8YnI+TG9uZyBjb250ZW50IDxicj5Mb25nIGNvbnRlbnQgPGJyPkxvbmcgY29udGVudCA8 YnI+TG9uZyBjb250ZW50IDxicj5Mb25nIGNvbnRlbnQgPGJyPkxvbmcgY29udGVudCA8YnI+TG9u ZyBjb250ZW50IDxicj5Mb25nIGNvbnRlbnQgPGJyPkxvbmcgY29udGVudCA8YnI+TG9uZyBjb250 ZW50IDxicj5Mb25nIGNvbnRlbnQgPGJyPkxvbmcgY29udGVudCA8YnI+TG9uZyBjb250ZW50IDxi cj5Mb25nIGNvbnRlbnQgPGJyPkxvbmcgY29udGVudCA8YnI+TG9uZyBjb250ZW50IDxicj5Mb25n IGNvbnRlbnQgPGJyPkxvbmcgY29udGVudCA8YnI+TG9uZyBjb250ZW50IDxicj5Mb25nIGNvbnRl bnQgPGJyPkxvbmcgY29udGVudCA8YnI+TG9uZyBjb250ZW50IDxicj5Mb25nIGNvbnRlbnQgPGJy PkxvbmcgY29udGVudCA8YnI+DQo8L2Rpdj4NCjwvYm9keT4NCjwvaHRtbD4NCg== 19864 2008-03-18 08:21:31 -0700 2008-03-18 08:31:48 -0700 Add a missing const so that ScrollViewScrollbar::geometryChanged() is called instead of Widget::geometryChanged() iframe-scroll.patch text/plain 1702 marco.barisione SW5kZXg6IFdlYkNvcmUvcGxhdGZvcm0vZ3RrL1Njcm9sbFZpZXdHdGsuY3BwCj09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K LS0tIFdlYkNvcmUvcGxhdGZvcm0vZ3RrL1Njcm9sbFZpZXdHdGsuY3BwCShyZXZpc2lvbmUgMzEx MjUpCisrKyBXZWJDb3JlL3BsYXRmb3JtL2d0ay9TY3JvbGxWaWV3R3RrLmNwcAkoY29waWEgbG9j YWxlKQpAQCAtMiw2ICsyLDcgQEAKICAqIENvcHlyaWdodCAoQykgMjAwNiwgMjAwNyBBcHBsZSBD b21wdXRlciwgSW5jLiAgQWxsIHJpZ2h0cyByZXNlcnZlZC4KICAqIENvcHlyaWdodCAoQykgMjAw NiBNaWNoYWVsIEVtbWVsIG1pa2UuZW1tZWxAZ21haWwuY29tCiAgKiBDb3B5cmlnaHQgKEMpIDIw MDcgSG9sZ2VyIEhhbnMgUGV0ZXIgRnJleXRoZXIKKyAqIENvcHlyaWdodCAoQykgMjAwOCBDb2xs YWJvcmEgTHRkLgogICoKICAqIEFsbCByaWdodHMgcmVzZXJ2ZWQuCiAgKgpAQCAtNTQsNyArNTUs NyBAQAogICAgIFNjcm9sbFZpZXdTY3JvbGxiYXIoU2Nyb2xsYmFyQ2xpZW50KiwgU2Nyb2xsYmFy T3JpZW50YXRpb24sIFNjcm9sbGJhckNvbnRyb2xTaXplKTsKIAogcHJvdGVjdGVkOgotICAgIHZv aWQgZ2VvbWV0cnlDaGFuZ2VkKCk7CisgICAgdm9pZCBnZW9tZXRyeUNoYW5nZWQoKSBjb25zdDsK IH07CiAKIGNsYXNzIFNjcm9sbFZpZXc6OlNjcm9sbFZpZXdQcml2YXRlIDogcHVibGljIFNjcm9s bGJhckNsaWVudApAQCAtMTE5LDcgKzEyMCw3IEBACiB7CiB9CiAKLXZvaWQgU2Nyb2xsVmlld1Nj cm9sbGJhcjo6Z2VvbWV0cnlDaGFuZ2VkKCkKK3ZvaWQgU2Nyb2xsVmlld1Njcm9sbGJhcjo6Z2Vv bWV0cnlDaGFuZ2VkKCkgY29uc3QKIHsKICAgICBpZiAoIXBhcmVudCgpKQogICAgICAgICByZXR1 cm47CkluZGV4OiBXZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL0NoYW5n ZUxvZwkocmV2aXNpb25lIDMxMTI1KQorKysgV2ViQ29yZS9DaGFuZ2VMb2cJKGNvcGlhIGxvY2Fs ZSkKQEAgLTEsMyArMSwxNyBAQAorMjAwOC0wMy0xOCAgTWFyY28gQmFyaXNpb25lICA8bWFyY28u YmFyaXNpb25lQGNvbGxhYm9yYS5jby51az4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg KE9PUFMhKS4KKworICAgICAgICBodHRwOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD0xNjMzOQorICAgICAgICBGcmFtZSBzY3JvbGxiYXIgcG9zaXRpb25zIGFyZSBub3QgdXBkYXRl ZCBvbiBwYWdlIHNjcm9sbAorCisgICAgICAgIFdpZGdldDo6Z2VvbWV0cnlDaGFuZ2VkKCkgd2Fz IGNhbGxlZCBpbnN0ZWFkIG9mCisgICAgICAgIFNjcm9sbFZpZXdTY3JvbGxiYXI6Omdlb21ldHJ5 Q2hhbmdlZCgpIGJlY2F1c2Ugb2YgYSBtaXNzaW5nIGNvbnN0LgorCisgICAgICAgICogcGxhdGZv cm0vZ3RrL1Njcm9sbFZpZXdHdGsuY3BwOgorICAgICAgICAoV2ViQ29yZTo6U2Nyb2xsVmlld1Nj cm9sbGJhcjo6Z2VvbWV0cnlDaGFuZ2VkKTogYWRkIHRoZSBtaXNzaW5nCisgICAgICAgIGNvbnN0 LgorCiAyMDA4LTAzLTE4ICBSb2RuZXkgRGF3ZXMgIDxkb2JleUB3YXlvZnRoZW1vbmtleS5jb20+ CiAKICAgICAgICAgRml4IE1hYyBidWlsZCBmcm9tIGNvbW1pdCBvZiByMzExMjMuCg==