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

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=138083

--- Comment #11 from KwangHyuk <hyuki.kim at samsung.com> ---
(In reply to comment #10)
> (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.

Ok, let me try. :)

-- 
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/067a76dc/attachment-0001.html>