RESOLVED FIXED 32399
Pan Scrolling Jumps out of frames if the initial location is in a frame that can't be scrolled
https://bugs.webkit.org/show_bug.cgi?id=32399
Summary Pan Scrolling Jumps out of frames if the initial location is in a frame that ...
Brian Weinstein
Reported 2009-12-10 16:33:03 PST
Pan Scrolling Jumps out of frames if the initial location is in a frame that can't be scrolled. Case: Empty iFrame that can't scroll inside of a page that can scroll Try to pan scroll - starting in the empty frame: IE: Doesn't scroll anything Firefox: Doesn't scroll anything Safari: Scrolls the page. We shouldn't scroll at all here.
Attachments
[PATCH] Fix (5.69 KB, patch)
2009-12-11 11:16 PST, Brian Weinstein
darin: review+
bweinstein: commit-queue-
Brian Weinstein
Comment 1 2009-12-10 16:34:41 PST
Additionally, we don't jump out of frames in the rest of the scrolling process, so it doesn't make sense for us to do it on the initial scroll.
Maxime Britto
Comment 2 2009-12-11 00:50:04 PST
The regular wheel scroll doesn't trigger in that case ?
Brian Weinstein
Comment 3 2009-12-11 10:58:02 PST
The scroll wheel doesn't jump out of iFrames either. The bug here is that: 1) There's an iFrame that can scroll. 2) There's an iFrame that can't scroll. If you start pan scrolling in (1), it will scroll the iFrame, and then NOT jump out to the main document. If you start pan scrolling in (2), it will jump out of the document. That behavior is inconsistent and a bug. It wasn't always like this, but was a regression from r52008. Before r52008, it would think it could pan scroll, and show the icon even if there was nothing for it to scroll inside the iFrame.
Adam Roben (:aroben)
Comment 4 2009-12-11 11:13:49 PST
(In reply to comment #3) > The scroll wheel doesn't jump out of iFrames either. Do you mean "pan scroll" instead of "scroll wheel"?
Brian Weinstein
Comment 5 2009-12-11 11:15:00 PST
(In reply to comment #4) > (In reply to comment #3) > > The scroll wheel doesn't jump out of iFrames either. > > Do you mean "pan scroll" instead of "scroll wheel"? No, Maxime asked about the scroll wheel, so I was saying that the scroll wheel doesn't jump out of frames/iFrames either.
Brian Weinstein
Comment 6 2009-12-11 11:16:56 PST
Created attachment 44698 [details] [PATCH] Fix
WebKit Review Bot
Comment 7 2009-12-11 11:19:49 PST
style-queue ran check-webkit-style on attachment 44698 [details] without any errors.
Adam Roben (:aroben)
Comment 8 2009-12-11 11:26:11 PST
(In reply to comment #5) > (In reply to comment #4) > > (In reply to comment #3) > > > The scroll wheel doesn't jump out of iFrames either. > > > > Do you mean "pan scroll" instead of "scroll wheel"? > > No, Maxime asked about the scroll wheel, so I was saying that the scroll wheel > doesn't jump out of frames/iFrames either. This does not match the behavior I see in ToT Safari. Scrolling the mouse wheel within an iframe scrolls that iframe until it can't be scrolled anymore, then scrolls the iframe's parent frame until that frame can't be scrolled, and so on.
Brian Weinstein
Comment 9 2009-12-11 11:28:01 PST
Oh, I was using a bad test case that called preventDefault on mousewheel. Sorry.
Darin Adler
Comment 10 2009-12-11 11:31:24 PST
Comment on attachment 44698 [details] [PATCH] Fix This looks OK and probably fixes this bug. But the existing logic doesn’t seem exactly right to me. A renderer can be an element with fixed positioning, in which case it doesn't make sense to advance to its parent when deciding what to scroll. r=me
Brian Weinstein
Comment 11 2009-12-11 11:59:30 PST
Landed in r52012.
Note You need to log in before you can comment on or make changes to this bug.