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

See Also:

Proposed patch. (4.04 KB, patch)
2010-03-29 09:19 PDT, Joseph Ligman
no flags Details | Formatted Diff | Diff

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