Bug 36674 - [Qt] qtwebkit_webframe_scrollRecursively scrolls when body.style.overflow="hidden"
Summary: [Qt] qtwebkit_webframe_scrollRecursively scrolls when body.style.overflow="hi...
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: S60 Hardware All
: P3 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks: 35784
  Show dependency tree / graph
 
Reported: 2010-03-26 14:06 PDT by Joseph Ligman
Modified: 2010-04-16 16:59 PDT (History)
4 users (show)

See Also:

Attachments
Proposed patch. (4.04 KB, patch)
2010-03-29 09:19 PDT, Joseph Ligman 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Ligman 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
Comment 1 Joseph Ligman 2010-03-29 09:19:50 PDT 
Created attachment 51920 [details]
Proposed patch.
Comment 2 WebKit Commit Bot 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>
Comment 3 WebKit Commit Bot 2010-04-08 02:14:06 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 4 Simon Hausmann 2010-04-08 03:40:46 PDT 
Revision r57266 cherry-picked into qtwebkit-2.0 with commit ea5bda76dbcd88545b7552630e4ff9015c3bfacf
Comment 5 Simon Hausmann 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