[Webkit-unassigned] [Bug 138083] [EFL] Apply adjustScrollPositionWithinRange() to scroll offset for ScrollingCoordinatorCoordinatedGraphics.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 9 19:24:51 PDT 2015


Ryuan Choi <ryuan.choi at navercorp.com> changed:

           What    |Removed                     |Added
                 CC|                            |ryuan.choi at navercorp.com

--- Comment #10 from Ryuan Choi <ryuan.choi at navercorp.com> ---
(In reply to comment #4)
> (In reply to comment #3)
> > Comment on attachment 240465 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=240465&action=review
> > 
> > >> Source/WebCore/ChangeLog:12
> > >> +        No new tests because of no behavior change.
> > > 
> > > I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test.
> > 
> > Can't you add new test in order to test this change ? It looks we need to
> > add a test which can handle when the offset value is not within the
> > scrollable range.
> Ok, let me check the WKViewScrollTo.cpp.
> However, this is a just bug fix for wheel scroll using mouse.
> Of course, you can reproduce issue easily using webkit.org.
> Just go to www.webkit.org and do the wheel scroll to the upper direction 10
> or more times, then do the wheel scroll to the down direction. you can
> notice that it will not scroll down for 10 times even though there is no
> webview changed.

Why don't you make the test case like you mentioned?

In my two cents,
we can make the test case that scroll more than two times to invalid position and scroll few pixel (but valid).

I checked some use cases and reproduced this issue with very simple test cases that use windows.scrollTo and/or with fixed elements.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150610/5f486069/attachment.html>

More information about the webkit-unassigned mailing list