Bug 16339

Summary:

Frame scrollbar positions are not updated on page scroll

Product:

WebKit

Reporter:

Tomas Rimkus <sandshrew>

Component:

WebKitGTK

Assignee:

Marco Barisione <marco.barisione>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

alp, nilesh.patil, pierre-luc.beaudoin, sandshrew

Priority:

Keywords:

Gtk

Version:

528+ (Nightly build)

Hardware:

OS:

Linux

URL:

http://www.sharemanija.net

Attachments:

Description	Flags
Test case	none
Add a missing const so that ScrollViewScrollbar::geometryChanged() is called instead of Widget::geometryChanged()	darin: review+

Description Tomas Rimkus 2007-12-07 03:44:21 PST

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

Comment 1 Alp Toker 2007-12-20 13:40:27 PST

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.

Comment 2 Pierre-Luc Beaudoin 2008-02-19 07:29:01 PST

Created attachment 19208 [details]
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.

Comment 3 Marco Barisione 2008-03-18 08:21:31 PDT

Created attachment 19864 [details]
Add a missing const so that ScrollViewScrollbar::geometryChanged() is called instead of Widget::geometryChanged()

Comment 4 Darin Adler 2008-03-18 08:31:48 PDT

Comment on attachment 19864 [details]
Add a missing const so that ScrollViewScrollbar::geometryChanged() is called instead of Widget::geometryChanged()

r=me

Comment 5 Pierre-Luc Beaudoin 2008-03-18 09:09:12 PDT

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.

Comment 6 Pierre-Luc Beaudoin 2008-03-19 08:00:24 PDT

(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.

Comment 7 Maciej Stachowiak 2008-03-22 00:49:46 PDT

*** Bug 17342 has been marked as a duplicate of this bug. ***

Comment 8 Mark Rowe (bdash) 2008-03-22 15:22:09 PDT

Landed in r31235.