<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [EFL] Apply adjustScrollPositionWithinRange() to scroll offset for ScrollingCoordinatorCoordinatedGraphics."
href="https://bugs.webkit.org/show_bug.cgi?id=138083#c11">Comment # 11</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [EFL] Apply adjustScrollPositionWithinRange() to scroll offset for ScrollingCoordinatorCoordinatedGraphics."
href="https://bugs.webkit.org/show_bug.cgi?id=138083">bug 138083</a>
from <span class="vcard"><a class="email" href="mailto:hyuki.kim@samsung.com" title="KwangHyuk <hyuki.kim@samsung.com>"> <span class="fn">KwangHyuk</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=138083#c10">comment #10</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=138083#c4">comment #4</a>)
> > (In reply to <a href="show_bug.cgi?id=138083#c3">comment #3</a>)
> > > Comment on <span class="bz_obsolete"><a href="attachment.cgi?id=240465&action=diff" name="attach_240465" title="Patch">attachment 240465</a> <a href="attachment.cgi?id=240465&action=edit" title="Patch">[details]</a></span>
> > > Patch
> > >
> > > View in context:
> > > <a href="https://bugs.webkit.org/attachment.cgi?id=240465&action=review">https://bugs.webkit.org/attachment.cgi?id=240465&action=review</a>
> > >
> > > >> 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.</span >
Ok, let me try. :)</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>