Bug 36674

Summary: [Qt] qtwebkit_webframe_scrollRecursively scrolls when body.style.overflow="hidden"
Product: WebKit Reporter: Joseph Ligman <joseph.ligman>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal CC: commit-queue, hausmann, laszlo.gombos, tonikitoo
Priority: P3 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: S60 Hardware   
OS: All   
Bug Depends on:    
Bug Blocks: 35784    
Attachments:
Description Flags
Proposed patch. none

Joseph Ligman
Reported 2010-03-26 14:06:28 PDT
The frame scrolling check is based off of the frame view's IntSize scrollOffset = frame->view()->scrollOffset(); IntPoint maxScrollOffset = frame->view()->maximumScrollPosition(); This doesn't not automatically account for the body.overflow property
Attachments
Proposed patch. (4.04 KB, patch)
2010-03-29 09:19 PDT, Joseph Ligman 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joseph Ligman
Comment 1 2010-03-29 09:19:50 PDT
Created attachment 51920 [details] Proposed patch.
WebKit Commit Bot
Comment 2 2010-04-08 02:14:00 PDT
Comment on attachment 51920 [details] Proposed patch. Clearing flags on attachment: 51920 Committed r57266: <http://trac.webkit.org/changeset/57266>
WebKit Commit Bot
Comment 3 2010-04-08 02:14:06 PDT
All reviewed patches have been landed. Closing bug.
Simon Hausmann
Comment 4 2010-04-08 03:40:46 PDT
Revision r57266 cherry-picked into qtwebkit-2.0 with commit ea5bda76dbcd88545b7552630e4ff9015c3bfacf
Simon Hausmann
Comment 5 2010-04-16 16:59:00 PDT
(In reply to comment #2) > (From update of attachment 51920 [details]) > Clearing flags on attachment: 51920 > > Committed r57266: <http://trac.webkit.org/changeset/57266> Cherry-picked into qtwebkit-4.6 with commit 6ed0b6197addffc7dacbdb3e49db711420a2c47a
Note You need to log in before you can comment on or make changes to this bug.